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

2016-09-13 Thread Neil Armstrong
On 09/13/2016 08:14 AM, Carlo Caione wrote: > On Mon, Sep 12, 2016 at 11:43 PM, Andreas Färber wrote: > > [cut] >> I'm not arguing over the file name, where it perfectly makes sense to >> have a meson-gxl- prefix (already discussed), just about the compatible >> string where we

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

2016-09-13 Thread Neil Armstrong
On 09/13/2016 08:14 AM, Carlo Caione wrote: > On Mon, Sep 12, 2016 at 11:43 PM, Andreas Färber wrote: > > [cut] >> I'm not arguing over the file name, where it perfectly makes sense to >> have a meson-gxl- prefix (already discussed), just about the compatible >> string where we don't have

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

2016-09-13 Thread Carlo Caione
On Mon, Sep 12, 2016 at 11:43 PM, Andreas Färber wrote: [cut] > I'm not arguing over the file name, where it perfectly makes sense to > have a meson-gxl- prefix (already discussed), just about the compatible > string where we don't have "amlogic,meson-gxl-s905x-p231" either

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

2016-09-13 Thread Carlo Caione
On Mon, Sep 12, 2016 at 11:43 PM, Andreas Färber wrote: [cut] > I'm not arguing over the file name, where it perfectly makes sense to > have a meson-gxl- prefix (already discussed), just about the compatible > string where we don't have "amlogic,meson-gxl-s905x-p231" either because > it is

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

2016-09-12 Thread Kevin Hilman
On Mon, Sep 12, 2016 at 2:43 PM, Andreas Färber wrote: > Am 12.09.2016 um 22:43 schrieb Kevin Hilman: >> Carlo Caione writes: >>> On Mon, Sep 12, 2016 at 6:28 PM, Andreas Färber wrote: >>> > +Boards with the Amlogic Meson GXL SoC shall

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

2016-09-12 Thread Kevin Hilman
On Mon, Sep 12, 2016 at 2:43 PM, Andreas Färber wrote: > Am 12.09.2016 um 22:43 schrieb Kevin Hilman: >> Carlo Caione writes: >>> On Mon, Sep 12, 2016 at 6:28 PM, Andreas Färber wrote: >>> > +Boards with the Amlogic Meson GXL SoC shall have the following > properties: > + Required

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

2016-09-12 Thread Andreas Färber
Am 12.09.2016 um 22:43 schrieb Kevin Hilman: > Carlo Caione writes: >> On Mon, Sep 12, 2016 at 6:28 PM, Andreas Färber wrote: >> +Boards with the Amlogic Meson GXL SoC shall have the following properties: + Required root node property: +

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

2016-09-12 Thread Andreas Färber
Am 12.09.2016 um 22:43 schrieb Kevin Hilman: > Carlo Caione writes: >> On Mon, Sep 12, 2016 at 6:28 PM, Andreas Färber wrote: >> +Boards with the Amlogic Meson GXL SoC shall have the following properties: + Required root node property: +compatible: "amlogic,meson-gxl-s905x",

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

2016-09-12 Thread Kevin Hilman
Carlo Caione writes: > On Mon, Sep 12, 2016 at 6:28 PM, Andreas Färber wrote: > >>> +Boards with the Amlogic Meson GXL SoC shall have the following properties: >>> + Required root node property: >>> +compatible: "amlogic,meson-gxl-s905x",

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

2016-09-12 Thread Kevin Hilman
Carlo Caione writes: > On Mon, Sep 12, 2016 at 6:28 PM, Andreas Färber 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

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

2016-09-12 Thread Carlo Caione
On Mon, Sep 12, 2016 at 6:28 PM, Andreas Färber 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",

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

2016-09-12 Thread Carlo Caione
On Mon, Sep 12, 2016 at 6:28 PM, Andreas Färber 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"?

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

2016-09-12 Thread Andreas Färber
Hi, Am 03.09.2016 um 10:22 schrieb Neil Armstrong: > From: Carlo Caione > > This patch introduces the basic support for the Amlogic S905X (Meson > GXL) and for the Amlogic evaluation board P212. > No documentation has been released yet for this SoC, so for now only the >

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

2016-09-12 Thread Andreas Färber
Hi, Am 03.09.2016 um 10:22 schrieb Neil Armstrong: > From: Carlo Caione > > This patch introduces the basic support for the Amlogic S905X (Meson > GXL) and for the Amlogic evaluation board P212. > No documentation has been released yet for this SoC, so for now only the > bare minimum has been

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

2016-09-03 Thread Neil Armstrong
From: Carlo Caione This patch introduces the basic support for the Amlogic S905X (Meson GXL) and for the Amlogic evaluation board P212. No documentation has been released yet for this SoC, so for now only the bare minimum has been added in the DT. Acked-by: Rob Herring

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

2016-09-03 Thread Neil Armstrong
From: Carlo Caione This patch introduces the basic support for the Amlogic S905X (Meson GXL) and for the Amlogic evaluation board P212. No documentation has been released yet for this SoC, so for now only the bare minimum has been added in the DT. Acked-by: Rob Herring Reviewed-by: Andreas