Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1481950
| From | Kevin Hilman <khilman@baylibre.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH 2/3] ARM64: dts: amlogic: Add basic support for Amlogic S905X |
| Date | 2016-09-12 22:50 +0200 |
| Message-ID | <sgGkV-8vM-11@gated-at.bofh.it> (permalink) |
| References | <sdeuR-2Qj-5@gated-at.bofh.it> <sdeuS-2Qj-17@gated-at.bofh.it> <sgChj-5GQ-11@gated-at.bofh.it> <sgD3K-6fr-103@gated-at.bofh.it> |
| Organization | BayLibre |
Carlo Caione <carlo@caione.org> writes: > 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... Correct. > 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. +1 Kevin
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll 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