Re: [PATCH 03/11] arm64: dts: intel: socfpga: override clocks by label

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/02/2021 18:18, Krzysztof Kozlowski wrote:
Using full paths to extend or override a device tree node is error
prone.  If there was a typo error, a new node will be created instead of
extending the existing node.  This will lead to run-time errors that
could be hard to detect.

A mistyped label on the other hand, will cause a dtc compile error
(during build time).

Signed-off-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
---
  arch/arm64/boot/dts/intel/socfpga_agilex_socdk.dts   | 12 ++++--------
  .../boot/dts/intel/socfpga_agilex_socdk_nand.dts     | 12 ++++--------
  arch/arm64/boot/dts/intel/socfpga_n5x_socdk.dts      | 12 ++++--------
  3 files changed, 12 insertions(+), 24 deletions(-)

Hi Arnd and Olof,

There was no response from Intel SoCFPGA maintainers for the DTS patches, so maybe you could take it directly? I can provide you pull request with these, if that's preferred. They apply cleanly.

You could also take the dt-bindings 2/11 or it could go via Rob.

About the defconfig patch (1/11) I will work on it more and resend it with your comments (you gave me review).

Best regards,
Krzysztof



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux