Hi SImon, 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... 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