* [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
@ 2023-09-25 9:06 ` Huqiang Qin
0 siblings, 0 replies; 18+ messages in thread
From: Huqiang Qin @ 2023-09-25 9:06 UTC (permalink / raw)
To: neil.armstrong, khilman, jbrunet, martin.blumenstingl, robh+dt,
krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel, Huqiang Qin
Add watchdog device.
Signed-off-by: Huqiang Qin <huqiang.qin@amlogic.com>
---
arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi b/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
index 998f5050795c..106f4dd09a1b 100644
--- a/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
+++ b/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
@@ -81,6 +81,12 @@ apb4: bus@fe000000 {
#size-cells = <2>;
ranges = <0x0 0x0 0x0 0xfe000000 0x0 0x480000>;
+ watchdog@2100 {
+ compatible = "amlogic,t7-wdt";
+ reg = <0x0 0x2100 0x0 0x10>;
+ clocks = <&xtal>;
+ };
+
periphs_pinctrl: pinctrl@4000 {
compatible = "amlogic,c3-periphs-pinctrl";
#address-cells = <2>;
base-commit: 8fff9184d1b5810dca5dd1a02726d4f844af88fc
--
2.42.0
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
@ 2023-09-25 9:06 ` Huqiang Qin
0 siblings, 0 replies; 18+ messages in thread
From: Huqiang Qin @ 2023-09-25 9:06 UTC (permalink / raw)
To: neil.armstrong, khilman, jbrunet, martin.blumenstingl, robh+dt,
krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel, Huqiang Qin
Add watchdog device.
Signed-off-by: Huqiang Qin <huqiang.qin@amlogic.com>
---
arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi b/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
index 998f5050795c..106f4dd09a1b 100644
--- a/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
+++ b/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
@@ -81,6 +81,12 @@ apb4: bus@fe000000 {
#size-cells = <2>;
ranges = <0x0 0x0 0x0 0xfe000000 0x0 0x480000>;
+ watchdog@2100 {
+ compatible = "amlogic,t7-wdt";
+ reg = <0x0 0x2100 0x0 0x10>;
+ clocks = <&xtal>;
+ };
+
periphs_pinctrl: pinctrl@4000 {
compatible = "amlogic,c3-periphs-pinctrl";
#address-cells = <2>;
base-commit: 8fff9184d1b5810dca5dd1a02726d4f844af88fc
--
2.42.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
@ 2023-09-25 9:06 ` Huqiang Qin
0 siblings, 0 replies; 18+ messages in thread
From: Huqiang Qin @ 2023-09-25 9:06 UTC (permalink / raw)
To: neil.armstrong, khilman, jbrunet, martin.blumenstingl, robh+dt,
krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel, Huqiang Qin
Add watchdog device.
Signed-off-by: Huqiang Qin <huqiang.qin@amlogic.com>
---
arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi b/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
index 998f5050795c..106f4dd09a1b 100644
--- a/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
+++ b/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
@@ -81,6 +81,12 @@ apb4: bus@fe000000 {
#size-cells = <2>;
ranges = <0x0 0x0 0x0 0xfe000000 0x0 0x480000>;
+ watchdog@2100 {
+ compatible = "amlogic,t7-wdt";
+ reg = <0x0 0x2100 0x0 0x10>;
+ clocks = <&xtal>;
+ };
+
periphs_pinctrl: pinctrl@4000 {
compatible = "amlogic,c3-periphs-pinctrl";
#address-cells = <2>;
base-commit: 8fff9184d1b5810dca5dd1a02726d4f844af88fc
--
2.42.0
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
2023-09-25 9:06 ` Huqiang Qin
(?)
@ 2023-09-25 9:33 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 18+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-25 9:33 UTC (permalink / raw)
To: Huqiang Qin, neil.armstrong, khilman, jbrunet,
martin.blumenstingl, robh+dt, krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel
On 25/09/2023 11:06, Huqiang Qin wrote:
> Add watchdog device.
>
> Signed-off-by: Huqiang Qin <huqiang.qin@amlogic.com>
> ---
> arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi b/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
> index 998f5050795c..106f4dd09a1b 100644
> --- a/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
> +++ b/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
> @@ -81,6 +81,12 @@ apb4: bus@fe000000 {
> #size-cells = <2>;
> ranges = <0x0 0x0 0x0 0xfe000000 0x0 0x480000>;
>
> + watchdog@2100 {
> + compatible = "amlogic,t7-wdt";
This is c3, not t7. Why do you use t7 compatible alone?
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
@ 2023-09-25 9:33 ` Krzysztof Kozlowski
0 siblings, 0 replies; 18+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-25 9:33 UTC (permalink / raw)
To: Huqiang Qin, neil.armstrong, khilman, jbrunet,
martin.blumenstingl, robh+dt, krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel
On 25/09/2023 11:06, Huqiang Qin wrote:
> Add watchdog device.
>
> Signed-off-by: Huqiang Qin <huqiang.qin@amlogic.com>
> ---
> arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi b/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
> index 998f5050795c..106f4dd09a1b 100644
> --- a/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
> +++ b/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
> @@ -81,6 +81,12 @@ apb4: bus@fe000000 {
> #size-cells = <2>;
> ranges = <0x0 0x0 0x0 0xfe000000 0x0 0x480000>;
>
> + watchdog@2100 {
> + compatible = "amlogic,t7-wdt";
This is c3, not t7. Why do you use t7 compatible alone?
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
@ 2023-09-25 9:33 ` Krzysztof Kozlowski
0 siblings, 0 replies; 18+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-25 9:33 UTC (permalink / raw)
To: Huqiang Qin, neil.armstrong, khilman, jbrunet,
martin.blumenstingl, robh+dt, krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel
On 25/09/2023 11:06, Huqiang Qin wrote:
> Add watchdog device.
>
> Signed-off-by: Huqiang Qin <huqiang.qin@amlogic.com>
> ---
> arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi b/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
> index 998f5050795c..106f4dd09a1b 100644
> --- a/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
> +++ b/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
> @@ -81,6 +81,12 @@ apb4: bus@fe000000 {
> #size-cells = <2>;
> ranges = <0x0 0x0 0x0 0xfe000000 0x0 0x480000>;
>
> + watchdog@2100 {
> + compatible = "amlogic,t7-wdt";
This is c3, not t7. Why do you use t7 compatible alone?
Best regards,
Krzysztof
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
2023-09-25 9:33 ` Krzysztof Kozlowski
(?)
@ 2023-09-25 10:42 ` Huqiang Qin
-1 siblings, 0 replies; 18+ messages in thread
From: Huqiang Qin @ 2023-09-25 10:42 UTC (permalink / raw)
To: Krzysztof Kozlowski, neil.armstrong, khilman, jbrunet,
martin.blumenstingl, robh+dt, krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel
Hi Krzysztof,
On 2023/9/25 17:33, Krzysztof Kozlowski wrote:
> This is c3, not t7. Why do you use t7 compatible alone?
The C3 and T7 use the same watchdog controller, so they are compatible.
Best regards,
Huqiang Qin
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
@ 2023-09-25 10:42 ` Huqiang Qin
0 siblings, 0 replies; 18+ messages in thread
From: Huqiang Qin @ 2023-09-25 10:42 UTC (permalink / raw)
To: Krzysztof Kozlowski, neil.armstrong, khilman, jbrunet,
martin.blumenstingl, robh+dt, krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel
Hi Krzysztof,
On 2023/9/25 17:33, Krzysztof Kozlowski wrote:
> This is c3, not t7. Why do you use t7 compatible alone?
The C3 and T7 use the same watchdog controller, so they are compatible.
Best regards,
Huqiang Qin
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
@ 2023-09-25 10:42 ` Huqiang Qin
0 siblings, 0 replies; 18+ messages in thread
From: Huqiang Qin @ 2023-09-25 10:42 UTC (permalink / raw)
To: Krzysztof Kozlowski, neil.armstrong, khilman, jbrunet,
martin.blumenstingl, robh+dt, krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel
Hi Krzysztof,
On 2023/9/25 17:33, Krzysztof Kozlowski wrote:
> This is c3, not t7. Why do you use t7 compatible alone?
The C3 and T7 use the same watchdog controller, so they are compatible.
Best regards,
Huqiang Qin
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
2023-09-25 10:42 ` Huqiang Qin
(?)
@ 2023-09-25 10:45 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 18+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-25 10:45 UTC (permalink / raw)
To: Huqiang Qin, neil.armstrong, khilman, jbrunet,
martin.blumenstingl, robh+dt, krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel
On 25/09/2023 12:42, Huqiang Qin wrote:
> Hi Krzysztof,
>
> On 2023/9/25 17:33, Krzysztof Kozlowski wrote:
>> This is c3, not t7. Why do you use t7 compatible alone?
>
> The C3 and T7 use the same watchdog controller, so they are compatible.
I am not saying that they are not compatible. I am saying that different
SoCs should use the same compatible. Just like for every other platform,
Amlogic is not special here.
https://elixir.bootlin.com/linux/v6.6-rc1/source/Documentation/devicetree/bindings/writing-bindings.rst
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
@ 2023-09-25 10:45 ` Krzysztof Kozlowski
0 siblings, 0 replies; 18+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-25 10:45 UTC (permalink / raw)
To: Huqiang Qin, neil.armstrong, khilman, jbrunet,
martin.blumenstingl, robh+dt, krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel
On 25/09/2023 12:42, Huqiang Qin wrote:
> Hi Krzysztof,
>
> On 2023/9/25 17:33, Krzysztof Kozlowski wrote:
>> This is c3, not t7. Why do you use t7 compatible alone?
>
> The C3 and T7 use the same watchdog controller, so they are compatible.
I am not saying that they are not compatible. I am saying that different
SoCs should use the same compatible. Just like for every other platform,
Amlogic is not special here.
https://elixir.bootlin.com/linux/v6.6-rc1/source/Documentation/devicetree/bindings/writing-bindings.rst
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
@ 2023-09-25 10:45 ` Krzysztof Kozlowski
0 siblings, 0 replies; 18+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-25 10:45 UTC (permalink / raw)
To: Huqiang Qin, neil.armstrong, khilman, jbrunet,
martin.blumenstingl, robh+dt, krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel
On 25/09/2023 12:42, Huqiang Qin wrote:
> Hi Krzysztof,
>
> On 2023/9/25 17:33, Krzysztof Kozlowski wrote:
>> This is c3, not t7. Why do you use t7 compatible alone?
>
> The C3 and T7 use the same watchdog controller, so they are compatible.
I am not saying that they are not compatible. I am saying that different
SoCs should use the same compatible. Just like for every other platform,
Amlogic is not special here.
https://elixir.bootlin.com/linux/v6.6-rc1/source/Documentation/devicetree/bindings/writing-bindings.rst
Best regards,
Krzysztof
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
2023-09-25 10:45 ` Krzysztof Kozlowski
(?)
@ 2023-09-25 10:58 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 18+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-25 10:58 UTC (permalink / raw)
To: Huqiang Qin, neil.armstrong, khilman, jbrunet,
martin.blumenstingl, robh+dt, krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel
On 25/09/2023 12:45, Krzysztof Kozlowski wrote:
> On 25/09/2023 12:42, Huqiang Qin wrote:
>> Hi Krzysztof,
>>
>> On 2023/9/25 17:33, Krzysztof Kozlowski wrote:
>>> This is c3, not t7. Why do you use t7 compatible alone?
>>
>> The C3 and T7 use the same watchdog controller, so they are compatible.
>
> I am not saying that they are not compatible. I am saying that different
> SoCs should use the same compatible. Just like for every other platform,
Eh, missing "not", so "should not use":
I am not saying that they are not compatible. I am saying that different
SoCs should not use the same compatible. Just like for every other
platform, Amlogic is not special here.
> Amlogic is not special here.
>
> https://elixir.bootlin.com/linux/v6.6-rc1/source/Documentation/devicetree/bindings/writing-bindings.rst
Please define compatibility list with t7 as fallback.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
@ 2023-09-25 10:58 ` Krzysztof Kozlowski
0 siblings, 0 replies; 18+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-25 10:58 UTC (permalink / raw)
To: Huqiang Qin, neil.armstrong, khilman, jbrunet,
martin.blumenstingl, robh+dt, krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel
On 25/09/2023 12:45, Krzysztof Kozlowski wrote:
> On 25/09/2023 12:42, Huqiang Qin wrote:
>> Hi Krzysztof,
>>
>> On 2023/9/25 17:33, Krzysztof Kozlowski wrote:
>>> This is c3, not t7. Why do you use t7 compatible alone?
>>
>> The C3 and T7 use the same watchdog controller, so they are compatible.
>
> I am not saying that they are not compatible. I am saying that different
> SoCs should use the same compatible. Just like for every other platform,
Eh, missing "not", so "should not use":
I am not saying that they are not compatible. I am saying that different
SoCs should not use the same compatible. Just like for every other
platform, Amlogic is not special here.
> Amlogic is not special here.
>
> https://elixir.bootlin.com/linux/v6.6-rc1/source/Documentation/devicetree/bindings/writing-bindings.rst
Please define compatibility list with t7 as fallback.
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
@ 2023-09-25 10:58 ` Krzysztof Kozlowski
0 siblings, 0 replies; 18+ messages in thread
From: Krzysztof Kozlowski @ 2023-09-25 10:58 UTC (permalink / raw)
To: Huqiang Qin, neil.armstrong, khilman, jbrunet,
martin.blumenstingl, robh+dt, krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel
On 25/09/2023 12:45, Krzysztof Kozlowski wrote:
> On 25/09/2023 12:42, Huqiang Qin wrote:
>> Hi Krzysztof,
>>
>> On 2023/9/25 17:33, Krzysztof Kozlowski wrote:
>>> This is c3, not t7. Why do you use t7 compatible alone?
>>
>> The C3 and T7 use the same watchdog controller, so they are compatible.
>
> I am not saying that they are not compatible. I am saying that different
> SoCs should use the same compatible. Just like for every other platform,
Eh, missing "not", so "should not use":
I am not saying that they are not compatible. I am saying that different
SoCs should not use the same compatible. Just like for every other
platform, Amlogic is not special here.
> Amlogic is not special here.
>
> https://elixir.bootlin.com/linux/v6.6-rc1/source/Documentation/devicetree/bindings/writing-bindings.rst
Please define compatibility list with t7 as fallback.
Best regards,
Krzysztof
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
2023-09-25 10:58 ` Krzysztof Kozlowski
(?)
@ 2023-09-25 11:07 ` Huqiang Qin
-1 siblings, 0 replies; 18+ messages in thread
From: Huqiang Qin @ 2023-09-25 11:07 UTC (permalink / raw)
To: Krzysztof Kozlowski, neil.armstrong, khilman, jbrunet,
martin.blumenstingl, robh+dt, krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel
On 2023/9/25 18:58, Krzysztof Kozlowski wrote:
> [ EXTERNAL EMAIL ]
>
> On 25/09/2023 12:45, Krzysztof Kozlowski wrote:
>> On 25/09/2023 12:42, Huqiang Qin wrote:
>>> Hi Krzysztof,
>>>
>>> On 2023/9/25 17:33, Krzysztof Kozlowski wrote:
>>>> This is c3, not t7. Why do you use t7 compatible alone?
>>>
>>> The C3 and T7 use the same watchdog controller, so they are compatible.
>>
>> I am not saying that they are not compatible. I am saying that different
>> SoCs should use the same compatible. Just like for every other platform,
>
> Eh, missing "not", so "should not use":
>
> I am not saying that they are not compatible. I am saying that different
> SoCs should not use the same compatible. Just like for every other
> platform, Amlogic is not special here.
>
>> Amlogic is not special here.
>>
>> https://elixir.bootlin.com/linux/v6.6-rc1/source/Documentation/devicetree/bindings/writing-bindings.rst
>
> Please define compatibility list with t7 as fallback.
Okay, got it, I will modify it in the next version.
Thanks
Best regards,
Huqiang Qin
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
@ 2023-09-25 11:07 ` Huqiang Qin
0 siblings, 0 replies; 18+ messages in thread
From: Huqiang Qin @ 2023-09-25 11:07 UTC (permalink / raw)
To: Krzysztof Kozlowski, neil.armstrong, khilman, jbrunet,
martin.blumenstingl, robh+dt, krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel
On 2023/9/25 18:58, Krzysztof Kozlowski wrote:
> [ EXTERNAL EMAIL ]
>
> On 25/09/2023 12:45, Krzysztof Kozlowski wrote:
>> On 25/09/2023 12:42, Huqiang Qin wrote:
>>> Hi Krzysztof,
>>>
>>> On 2023/9/25 17:33, Krzysztof Kozlowski wrote:
>>>> This is c3, not t7. Why do you use t7 compatible alone?
>>>
>>> The C3 and T7 use the same watchdog controller, so they are compatible.
>>
>> I am not saying that they are not compatible. I am saying that different
>> SoCs should use the same compatible. Just like for every other platform,
>
> Eh, missing "not", so "should not use":
>
> I am not saying that they are not compatible. I am saying that different
> SoCs should not use the same compatible. Just like for every other
> platform, Amlogic is not special here.
>
>> Amlogic is not special here.
>>
>> https://elixir.bootlin.com/linux/v6.6-rc1/source/Documentation/devicetree/bindings/writing-bindings.rst
>
> Please define compatibility list with t7 as fallback.
Okay, got it, I will modify it in the next version.
Thanks
Best regards,
Huqiang Qin
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs
@ 2023-09-25 11:07 ` Huqiang Qin
0 siblings, 0 replies; 18+ messages in thread
From: Huqiang Qin @ 2023-09-25 11:07 UTC (permalink / raw)
To: Krzysztof Kozlowski, neil.armstrong, khilman, jbrunet,
martin.blumenstingl, robh+dt, krzysztof.kozlowski+dt, conor+dt
Cc: linux-arm-kernel, linux-amlogic, devicetree, linux-kernel
On 2023/9/25 18:58, Krzysztof Kozlowski wrote:
> [ EXTERNAL EMAIL ]
>
> On 25/09/2023 12:45, Krzysztof Kozlowski wrote:
>> On 25/09/2023 12:42, Huqiang Qin wrote:
>>> Hi Krzysztof,
>>>
>>> On 2023/9/25 17:33, Krzysztof Kozlowski wrote:
>>>> This is c3, not t7. Why do you use t7 compatible alone?
>>>
>>> The C3 and T7 use the same watchdog controller, so they are compatible.
>>
>> I am not saying that they are not compatible. I am saying that different
>> SoCs should use the same compatible. Just like for every other platform,
>
> Eh, missing "not", so "should not use":
>
> I am not saying that they are not compatible. I am saying that different
> SoCs should not use the same compatible. Just like for every other
> platform, Amlogic is not special here.
>
>> Amlogic is not special here.
>>
>> https://elixir.bootlin.com/linux/v6.6-rc1/source/Documentation/devicetree/bindings/writing-bindings.rst
>
> Please define compatibility list with t7 as fallback.
Okay, got it, I will modify it in the next version.
Thanks
Best regards,
Huqiang Qin
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2023-09-25 11:07 UTC | newest]
Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-25 9:06 [PATCH] arm64: dts: Add watchdog node for Amlogic-C3 SoCs Huqiang Qin
2023-09-25 9:06 ` Huqiang Qin
2023-09-25 9:06 ` Huqiang Qin
2023-09-25 9:33 ` Krzysztof Kozlowski
2023-09-25 9:33 ` Krzysztof Kozlowski
2023-09-25 9:33 ` Krzysztof Kozlowski
2023-09-25 10:42 ` Huqiang Qin
2023-09-25 10:42 ` Huqiang Qin
2023-09-25 10:42 ` Huqiang Qin
2023-09-25 10:45 ` Krzysztof Kozlowski
2023-09-25 10:45 ` Krzysztof Kozlowski
2023-09-25 10:45 ` Krzysztof Kozlowski
2023-09-25 10:58 ` Krzysztof Kozlowski
2023-09-25 10:58 ` Krzysztof Kozlowski
2023-09-25 10:58 ` Krzysztof Kozlowski
2023-09-25 11:07 ` Huqiang Qin
2023-09-25 11:07 ` Huqiang Qin
2023-09-25 11:07 ` Huqiang Qin
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.