[PATCH/RFC 0/5] arm64: dts: renesas: Break out common board support

[Simon Horman]

On Thu, Apr 27, 2017 at 03:32:49PM +0200, Geert Uytterhoeven wrote:
> Hi Simon,
> 
> On Wed, Apr 26, 2017 at 10:11 AM, Geert Uytterhoeven
> <geert at linux-m68k.org> wrote:
> > CC Vladimir (which I forgot to CC initially, sorry for that)
> >
> > On Wed, Apr 26, 2017 at 10:06 AM, Simon Horman <horms at verge.net.au> 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?