-
Notifications
You must be signed in to change notification settings - Fork 105
Support corepvt, phy leds on titan and update dts #1416
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
fe4db96
4d54526
5161934
714d624
e820da8
1d21f28
c85276f
1a5d525
0c93dcb
5cfaf46
d73c9da
d1858eb
4db6bf6
180152c
9b049d8
9879762
3c77cb3
f394f9b
653dc15
75c6560
f6e0921
548ae7d
83451b6
ba9d9c5
4bd026d
f0d47ba
00330fe
9d4c767
372f584
3536d53
7187b72
d58ab77
c513210
597509f
e860567
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -88,6 +88,8 @@ Description: | |
| speed of 10MBps of the named network device. | ||
| Setting this value also immediately changes the LED state. | ||
|
|
||
| Present only if the named network device supports 10Mbps link speed. | ||
|
|
||
| What: /sys/class/leds/<led>/link_100 | ||
| Date: Jun 2023 | ||
| KernelVersion: 6.5 | ||
|
|
@@ -101,6 +103,8 @@ Description: | |
| speed of 100Mbps of the named network device. | ||
| Setting this value also immediately changes the LED state. | ||
|
|
||
| Present only if the named network device supports 100Mbps link speed. | ||
|
|
||
| What: /sys/class/leds/<led>/link_1000 | ||
| Date: Jun 2023 | ||
| KernelVersion: 6.5 | ||
|
|
@@ -114,6 +118,53 @@ Description: | |
| speed of 1000Mbps of the named network device. | ||
| Setting this value also immediately changes the LED state. | ||
|
|
||
| Present only if the named network device supports 1000Mbps link speed. | ||
|
|
||
| What: /sys/class/leds/<led>/link_2500 | ||
| Date: Nov 2023 | ||
| KernelVersion: 6.8 | ||
| Contact: [email protected] | ||
| Description: | ||
| Signal the link speed state of 2500Mbps of the named network device. | ||
|
|
||
| If set to 0 (default), the LED's normal state is off. | ||
|
|
||
| If set to 1, the LED's normal state reflects the link state | ||
| speed of 2500Mbps of the named network device. | ||
| Setting this value also immediately changes the LED state. | ||
|
|
||
| Present only if the named network device supports 2500Mbps link speed. | ||
|
|
||
| What: /sys/class/leds/<led>/link_5000 | ||
| Date: Nov 2023 | ||
| KernelVersion: 6.8 | ||
| Contact: [email protected] | ||
| Description: | ||
| Signal the link speed state of 5000Mbps of the named network device. | ||
|
|
||
| If set to 0 (default), the LED's normal state is off. | ||
|
|
||
| If set to 1, the LED's normal state reflects the link state | ||
| speed of 5000Mbps of the named network device. | ||
| Setting this value also immediately changes the LED state. | ||
|
|
||
| Present only if the named network device supports 5000Mbps link speed. | ||
|
|
||
| What: /sys/class/leds/<led>/link_10000 | ||
| Date: Nov 2023 | ||
| KernelVersion: 6.8 | ||
| Contact: [email protected] | ||
| Description: | ||
| Signal the link speed state of 10000Mbps of the named network device. | ||
|
|
||
| If set to 0 (default), the LED's normal state is off. | ||
|
|
||
| If set to 1, the LED's normal state reflects the link state | ||
| speed of 10000Mbps of the named network device. | ||
| Setting this value also immediately changes the LED state. | ||
|
|
||
| Present only if the named network device supports 10000Mbps link speed. | ||
|
|
||
| What: /sys/class/leds/<led>/half_duplex | ||
| Date: Jun 2023 | ||
| KernelVersion: 6.5 | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,90 @@ | ||
| # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) | ||
| %YAML 1.2 | ||
| --- | ||
| $id: http://devicetree.org/schemas/hwmon/ultrarisc,dp1000-pvt.yaml# | ||
| $schema: http://devicetree.org/meta-schemas/core.yaml# | ||
|
|
||
| title: UltraRISC DP1000 Core PVT Sensor | ||
|
|
||
| maintainers: | ||
| - Jia Wang <[email protected]> | ||
|
|
||
| description: | | ||
| UltraRISC DP1000 SoC provides a voltage and temperature (PVT) sensor to | ||
| monitor the internal SoC environment including chip temperature and supply voltage. | ||
| properties: | ||
| compatible: | ||
| const: ultrarisc,dp1000-pvt | ||
|
|
||
| reg: | ||
| maxItems: 1 | ||
| description: Register space of the PVT controller | ||
|
|
||
| interrupts: | ||
| maxItems: 1 | ||
| description: Optional interrupt number for the PVT controller. This property is optional and can be omitted if not used. | ||
|
|
||
| clock-frequency: | ||
| description: Clock frequency of the PVT controller in Hz | ||
|
|
||
| channels: | ||
| description: Total number of PVT channels supported | ||
|
|
||
| patternProperties: | ||
| "^channel@[0-9a-f]+$": | ||
| type: object | ||
| description: Child node describing a PVT channel | ||
| properties: | ||
| reg: | ||
| description: Channel index | ||
| minimum: 0 | ||
| maximum: 63 | ||
|
|
||
| label: | ||
| description: Name for this channel, typically indicating its purpose | ||
|
|
||
| trim: | ||
| description: Trim value for calibration | ||
| default: 7 | ||
| minimum: 0 | ||
| maximum: 15 | ||
|
|
||
| required: | ||
| - reg | ||
| - label | ||
|
|
||
| required: | ||
| - compatible | ||
| - reg | ||
| - clock-frequency | ||
| - channels | ||
|
|
||
| additionalProperties: false | ||
|
|
||
| examples: | ||
| - | | ||
| core_pvt: pvt@20008000 { | ||
| compatible = "ultrarisc,dp1000-pvt"; | ||
| reg = <0x20008000 0x1000>; | ||
| clock-frequency = <4000000>; | ||
| channels = <13>; | ||
| channel@0 { | ||
| reg = <0>; | ||
| label = "Core temp0"; | ||
| trim = <7>; | ||
| }; | ||
| channel@1 { | ||
| reg = <1>; | ||
| label = "Core temp1"; | ||
| trim = <7>; | ||
| }; | ||
| channel@2 { | ||
| reg = <2>; | ||
| label = "Core temp2"; | ||
| trim = <7>; | ||
| }; | ||
| }; | ||
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -67,6 +67,9 @@ properties: | |||||||||
| - allwinner,sun20i-d1-plic | ||||||||||
| - thead,th1520-plic | ||||||||||
| - const: thead,c900-plic | ||||||||||
| - items: | ||||||||||
| - const: ultrarisc,dp1000-plic | ||||||||||
| - const: ultrarisc,cp100-plic | ||||||||||
|
Comment on lines
+71
to
+72
|
||||||||||
| - const: ultrarisc,dp1000-plic | |
| - const: ultrarisc,cp100-plic | |
| - const: ultrarisc,cp100-plic | |
| - const: ultrarisc,dp1000-plic |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,4 @@ | ||
| # SPDX-License-Identifier: GPL-2.0 | ||
| dtb-$(CONFIG_ARCH_ULTRARISC) += dp1000-evb-v1.dtb | ||
| dtb-$(CONFIG_ARCH_ULTRARISC) += dp1000-mo-v1.dtb | ||
| dtb-$(CONFIG_ARCH_ULTRARISC) += dp1000-m0-v1.dtb | ||
| dtb-$(CONFIG_ARCH_ULTRARISC) += dp1000-titan-v1.dtb |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The interrupts property documentation states it is optional, but the 'required' section at line 57 does not list it, which is correct. However, the description on line 26 should more clearly indicate it's optional by starting with "Optional:" or similar wording for clarity.