Re: [PATCH 3/4] arm64: dts: add support for A4 based Amlogic BA400

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

 



Hi Martin,
   Thanks for your review.

On 2024/3/13 01:29, Martin Blumenstingl wrote:
[ EXTERNAL EMAIL ]

On Tue, Mar 12, 2024 at 10:19 AM Xianwei Zhao via B4 Relay
<devnull+xianwei.zhao.amlogic.com@xxxxxxxxxx> wrote:
[...]
+               apb@fe000000 {
Node names need to be generic - since this is a bug it needs to be:
   bus@fe000000 {
Or if you want to make it clear how this bus is called then you can use:
   apb: bus@fe000000 {

Will fix it.
The same comment applies to the amlogic-a5.dtsi patch (4/4).
And while here, I fully agree with Jerome: having a bit more details
would be great so we can judge on whether a common .dtsi makes sense.
For this it would be helpful to know how many IP blocks those two SoCs
have in common and how many are different.

They are mostly the same, but the follow-on series is unclear, and I would like to wait for the follow-on chips to come out before considering a merger with common dtsi file.

Best regards,
Martin




[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