On Wed, Nov 23, 2016 at 10:57:18AM +0100, Simon Horman wrote: > On Mon, Nov 21, 2016 at 11:53:18AM +0100, Simon Horman wrote: > > On Fri, Nov 18, 2016 at 05:49:29PM -0800, Olof Johansson wrote: > > > On Thu, Nov 17, 2016 at 03:11:45PM +0100, Simon Horman wrote: > > > > Hi Olof, Hi Kevin, Hi Arnd, > > > > > > > > Please consider these second round of Renesas ARM based SoC DT updates for v4.10. > > > > > > > > This pull request is based on a merge of: > > > > > > > > * The previous round of such requests, tagged as renesas-dt-for-v4.10, > > > > which I have already sent a pull-request for. > > > > * The rzg-clock-defs tag of Geert Uytterhoeven's renesas-driver's tree. > > > > This is to provide dependencies for adding the r8a7743 and r8a7745 SoCs. > > > > * The "Second Round of Renesas ARM Based SoC Drivers Updates for v4.10", > > > > tagged as renesas-drivers2-for-v4.10, which I have also sent a pull > > > > request for. This is included to provide dependencies for adding device > > > > nodes for PRR, and adding the r8a7743 and r8a7745 SoCs. > > > > > > Again, nack. And again, I don't understand why you create dependencies that > > > aren't needed. Please fix. > > > > Hi Olof, > > > > I agree that calling out PRR above was incorrect. Please disregard that. > > > > However, there are dependencies for adding r8a7743 and r8a7745 SoCs > > in the form of header files: > > > > * The rzg-clock-defs tag provides dt-bindings/clock/r8a774[35]-cpg-mssr.h > > * The renesas-drivers2-for-v4.10 tag provides > > dt-bindings/power/r8a774[35]-sysc.h > > > > The drivers branches are usually pretty light-weight. But this time it is a > > bit heavy and you rightly raised some questions about it. After some > > discussion with Geert we'd like to suggest that for future releases > > we provide a "driver-defs" branch which both driver code and DT can > > depend on. Thus avoiding pulling (non essential) driver changes into the DT > > branch. > > > > Unfortunately its a bit late to do that for v4.10 as the r8a7743 sysc > > driver and its defines were already accepted accepted together > > (renesas-drivers-for-v4.10 tag). So for this release we would be grateful > > if you could re-consider the renesas-drivers2-for-v4.10 tag given the > > feedback which Geert has provided. And in turn re-consider this pull > > request. > > Hi again, > > while the above remains my preferred option I would like to put another one > on the table in case it would help in any way for v4.10. > > I could split this pull-request up as follows: > 1. The patches that add the r8a774[35] SoCs: > - r8a7743 depends on renesas-drivers-for-v4.10 and rzg-clock-defs > - r8a7745 depends on renesas-drivers2-for-v4.10 and rzg-clock-defs > 2. The patches rest of the patches > - I believe these have no special dependencies After some discussion on IRC with Arnd I think it makes sense to simplify the dependencies pulled in to this pull request to the extent of only pulling in patches which provide headers for macros used in DT. I am working on making this so.
