Hi Simon, On Fri, Apr 28, 2017 at 9:32 AM, Simon Horman <horms@xxxxxxxxxxxx> wrote: > On Fri, Apr 28, 2017 at 09:11:36AM +0200, Geert Uytterhoeven wrote: >> On Fri, Apr 28, 2017 at 9:04 AM, Simon Horman <horms@xxxxxxxxxxxx> wrote: >> > On Thu, Apr 27, 2017 at 03:32:49PM +0200, Geert Uytterhoeven wrote: >> >> On Wed, Apr 26, 2017 at 10:11 AM, Geert Uytterhoeven >> >> <geert@xxxxxxxxxxxxxx> wrote: >> >> > CC Vladimir (which I forgot to CC initially, sorry for that) >> >> > >> >> > On Wed, Apr 26, 2017 at 10:06 AM, Simon Horman <horms@xxxxxxxxxxxx> wrote: >> >> >> On Fri, Apr 21, 2017 at 02:55:16PM +0200, Geert Uytterhoeven wrote: >> >> >>> The Renesas Salvator-X and ULCB development board can be equipped with >> >> >>> either an R-Car H3 or M3-W SiP, which are pin-compatible. All boards >> >> >>> use separate DTBs, but currently there's no sharing of board-specific >> >> >>> devices in DTS. >> >> >>> >> >> >>> This series reduces duplication by extracting common board support into >> >> >>> their own .dtsi files. As the level of support varies across boards and >> >> >>> SoCs, this requires the addition of a few external clocks and >> >> >>> placeholder devices on R-Car M3-W, so the common board support DTS can >> >> >>> refer to them. >> >> >>> >> >> >>> - Patches 1 and 2 add the external audio and PCIe bus clocks on R-Car >> >> >>> M3-W, which are present in r8a7795.dtsi, and used in >> >> >>> r8a7795-salvator-x.dts, >> >> >>> - RFC patch 3 adds placeholders for devices that are not yet supported >> >> >>> and/or tested on R-Car M3-W, but used on R-Car H3, >> >> >>> - RFC patch 4 extracts common Salvator-X board support, >> >> >>> - RFC patch 5 extracts common ULCB board support. >> >> >>> >> >> >>> For R-Car H3 based boards, there are no functional changes. >> >> >>> For R-Car M3-W based boards, some new devices are now described in DT. >> >> >>> >> >> >>> Dependencies: >> >> >>> - renesas-devel-20170420-v4.11-rc7, >> >> >>> - Patches 1 and 2 can be applied as-is, >> >> >>> - Patches 4 and 5 depend on "[PATCH 0/8] arm64: dts: renesas: Break >> >> >>> out R-Car H3 and M3-W SiP" >> >> >>> (http://www.spinics.net/lists/devicetree/msg173820.html). >> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ >> >> >> >>> DTB changes have been inspected using scripts/dtc/dtx_diff. >> >> >>> This has been tested on Salvator-X (both H3 and M3-W). >> >> >>> This has not been tested on H3ULCB and M3ULCB due to lack of hardware. >> >> >>> >> >> >>> Thanks for your comments! >> >> >> >> >> >> Thanks for tackling this important problem. I have looked over the changes >> >> >> and they seem nice to me. I would, however, be more comfortable applying >> >> >> them if they were rested on the ULCB boards. >> >> > >> >> > tested? >> >> > >> >> > I've pushed a branch for testing to topic/rcar3-dtsi-sharing in >> >> > git://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git. >> >> >> >> I managed to test it on the new H3ULCB and M3ULCB baords in Magnus' farm. >> >> No issues detected. >> > >> > Great! Any objections to me queuing this up? >> >> The dependency above (no feedback about the SiP types yet). >> >> I can respin without that dependency, if that is preferred... > > It seems to me that it would be nice to get these in sooner than later - in > particular earlier rather than later in the (v4.13) development cycle. But > I defer to your judgement on what is best. Agreed. Stay tuned for v2... Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
