diff --git a/device-types/Huawei/S1720-28GWR-PWR-4TP-E.yaml b/device-types/Huawei/S1720-28GWR-PWR-4TP-E.yaml
new file mode 100644
index 00000000..3009096f
--- /dev/null
+++ b/device-types/Huawei/S1720-28GWR-PWR-4TP-E.yaml
@@ -0,0 +1,143 @@
+---
+manufacturer: Huawei
+model: S1720-28GWR-PWR-4TP-E
+slug: s1720-28gwr-pwr-4tp-e
+part_number: "98010752"
+u_height: 1
+is_full_depth: false
+airflow: passive
+power-ports:
+  - name: PWR
+    label: "~100/240V; 50/60Hz: 3A"
+    type: iec-60320-c14
+    maximum_draw: 166
+    allocated_draw: 19
+interfaces:
+  - name: GigabitEthernet0/0/1
+    label: "1"
+    type: 1000base-t
+    mgmt_only: false
+    poe_mode: pd
+    poe_type: type2-ieee802.3at
+  - name: GigabitEthernet0/0/2
+    label: "2"
+    type: 1000base-t
+    mgmt_only: false
+    poe_mode: pd
+    poe_type: type2-ieee802.3at
+  - name: GigabitEthernet0/0/3
+    label: "3"
+    type: 1000base-t
+    mgmt_only: false
+    poe_mode: pd
+    poe_type: type2-ieee802.3at
+  - name: GigabitEthernet0/0/4
+    label: "4"
+    type: 1000base-t
+    mgmt_only: false
+    poe_mode: pd
+    poe_type: type2-ieee802.3at
+  - name: GigabitEthernet0/0/5
+    label: "5"
+    type: 1000base-t
+    mgmt_only: false
+    poe_mode: pd
+    poe_type: type2-ieee802.3at
+  - name: GigabitEthernet0/0/6
+    label: "6"
+    type: 1000base-t
+    mgmt_only: false
+    poe_mode: pd
+    poe_type: type2-ieee802.3at
+  - name: GigabitEthernet0/0/7
+    label: "7"
+    type: 1000base-t
+    mgmt_only: false
+    poe_mode: pd
+    poe_type: type2-ieee802.3at
+  - name: GigabitEthernet0/0/8
+    label: "8"
+    type: 1000base-t
+    mgmt_only: false
+    poe_mode: pd
+    poe_type: type2-ieee802.3at
+  - name: GigabitEthernet0/0/9
+    label: "9"
+    type: 1000base-t
+    mgmt_only: false
+  - name: GigabitEthernet0/0/10
+    label: "10"
+    type: 1000base-t
+    mgmt_only: false
+  - name: GigabitEthernet0/0/11
+    label: "11"
+    type: 1000base-t
+    mgmt_only: false
+  - name: GigabitEthernet0/0/12
+    label: "12"
+    type: 1000base-t
+    mgmt_only: false
+  - name: GigabitEthernet0/0/13
+    label: "13"
+    type: 1000base-t
+    mgmt_only: false
+  - name: GigabitEthernet0/0/14
+    label: "14"
+    type: 1000base-t
+    mgmt_only: false
+  - name: GigabitEthernet0/0/15
+    label: "15"
+    type: 1000base-t
+    mgmt_only: false
+  - name: GigabitEthernet0/0/16
+    label: "16"
+    type: 1000base-t
+    mgmt_only: false
+  - name: GigabitEthernet0/0/17
+    label: "17"
+    type: 1000base-t
+    mgmt_only: false
+  - name: GigabitEthernet0/0/18
+    label: "18"
+    type: 1000base-t
+    mgmt_only: false
+  - name: GigabitEthernet0/0/19
+    label: "19"
+    type: 1000base-t
+    mgmt_only: false
+  - name: GigabitEthernet0/0/20
+    label: "20"
+    type: 1000base-t
+    mgmt_only: false
+  - name: GigabitEthernet0/0/21
+    label: "21"
+    type: 1000base-t
+    mgmt_only: false
+  - name: GigabitEthernet0/0/22
+    label: "22"
+    type: 1000base-t
+    mgmt_only: false
+  - name: GigabitEthernet0/0/23
+    label: "23"
+    type: 1000base-t
+    mgmt_only: false
+  - name: GigabitEthernet0/0/24
+    label: "24"
+    type: 1000base-t
+    mgmt_only: false
+  - name: GigabitEthernet0/0/25
+    label: "25"
+    type: 1000base-x-sfp
+    mgmt_only: false
+  - name: GigabitEthernet0/0/26
+    label: "26"
+    type: 1000base-x-sfp
+    mgmt_only: false
+  - name: GigabitEthernet0/0/27
+    label: "27"
+    type: 1000base-x-sfp
+    mgmt_only: false
+  - name: GigabitEthernet0/0/28
+    label: "28"
+    type: 1000base-x-sfp
+    mgmt_only: false