Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1481650

Re: [PATCH 2/3] ARM64: dts: amlogic: Add basic support for Amlogic S905X

Show all headers | View raw

On Mon, Sep 12, 2016 at 6:28 PM, Andreas Färber <afaerber@suse.de> wrote:

>> +Boards with the Amlogic Meson GXL SoC shall have the following properties:
>> +  Required root node property:
>> +    compatible: "amlogic,meson-gxl-s905x", "amlogic,meson-gxl";
>
> Can we please use "amlogic,s905x", "amlogic,meson-gxl"? No need to
> complicate the name. Also affects .dtsi and .dts below.

gxl != s905x.

AFAWK to the GXL family belong several different SoCs, like S905X,
S905D, etc... (see patch 3/3)
This is why we use meson-gxl-s905x, meson-gxl-s905d, etc...

We could s/meson-gxl-s905x/meson-s905x/ and
s/meson-gxl-s905d/meson-s905d/ but I honestly prefer this way because
we can clearly see which family the SoC belongs to (the Amlogic naming
convention is already messy enough).
I mean, yes it's longer, but it's for the sake of documentation IMO.


[cut]
>> +     compatible = "amlogic,p212", "amlogic,meson-gxl-s905x", "amlogic,meson-gxl";
>> +     model = "Amlogic Meson GXL (S905X) P212 Development Board";
>
> Is that its official name? No objection, just wondering whether we need
> both GXL and S905X in the name, and then again the SoC name at all if
> the P212 is unique. Haven't compared the previous Pxxx ones.

P212 is the official name for the S905X Amlogic dev board. It follows
the name we used for the other P20x boards.


[cut]
>> +
>> +/* This UART is brought out to the DB9 connector */
>> +&uart_AO {
>> +     status = "okay";
>> +};
>> +
>
> Trailing white line - please watch out for that, git-am will complain.

Right.

[cut]
>> +#include "meson-gxl.dtsi"
>> +
>> +/ {
>> +     compatible = "amlogic,meson-gxl", "amlogic,meson-gxl-s905x";
>
> This needs to be reversed.

Agree.

Cheers,

-- 
Carlo Caione

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH 2/3] ARM64: dts: amlogic: Add basic support for Amlogic  S905X Andreas Färber <afaerber@suse.de> - 2016-09-12 18:30 +0200
  Re: [PATCH 2/3] ARM64: dts: amlogic: Add basic support for Amlogic S905X Carlo Caione <carlo@caione.org> - 2016-09-12 19:20 +0200
    Re: [PATCH 2/3] ARM64: dts: amlogic: Add basic support for Amlogic S905X Kevin Hilman <khilman@baylibre.com> - 2016-09-12 22:50 +0200
      Re: [PATCH 2/3] ARM64: dts: amlogic: Add basic support for Amlogic  S905X Andreas Färber <afaerber@suse.de> - 2016-09-12 23:50 +0200
        Re: [PATCH 2/3] ARM64: dts: amlogic: Add basic support for Amlogic S905X Kevin Hilman <khilman@baylibre.com> - 2016-09-13 06:50 +0200
        Re: [PATCH 2/3] ARM64: dts: amlogic: Add basic support for Amlogic S905X Carlo Caione <carlo@caione.org> - 2016-09-13 08:20 +0200
          Re: [PATCH 2/3] ARM64: dts: amlogic: Add basic support for Amlogic  S905X Neil Armstrong <narmstrong@baylibre.com> - 2016-09-13 09:40 +0200

csiph-web