On Wed, Aug 09, 2017 at 09:23:05AM +0000, Biju Das wrote: > > > > -----Original Message----- > > From: Simon Horman [mailto:horms@xxxxxxxxxxxx] > > Sent: 09 August 2017 09:53 > > To: Biju Das <biju.das@xxxxxxxxxxxxxx> > > Cc: Rob Herring <robh+dt@xxxxxxxxxx>; Mark Rutland > > <mark.rutland@xxxxxxx>; Wolfram Sang <wsa@xxxxxxxxxxxxx>; Magnus > > Damm <magnus.damm@xxxxxxxxx>; Russell King <linux@xxxxxxxxxxxxxxx>; > > Chris Paterson <Chris.Paterson2@xxxxxxxxxxx>; devicetree@xxxxxxxxxxxxxxx; > > linux-renesas-soc@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > > Subject: Re: [PATCH 2/2] ARM: dts: r8a7743: Add I2C DT support > > > > On Tue, Aug 08, 2017 at 12:24:09PM +0100, Biju Das wrote: > > > Add the I2C[0-5] devices to the r8a7743 device tree. > > > > > > Signed-off-by: Biju Das <biju.das@xxxxxxxxxxxxxx> > > > --- > > > This patch has been tested against Linux-next tag 20170727 and renesas-dev > > branch. > > > This patch depends on > > > https://www.mail-archive.com/linux-renesas-soc@xxxxxxxxxxxxxxx/msg1700 > > > 8.html > > > > As per the patch at the link above it seems that these patches are targeted at > > the renesas tree. The best practice in that case is to base patches on the latest > > devel branch. Please consider doing so in future. > > > > I would also slightly prefer if dependencies were referenced by name, in this > > case: > > > > * [PATCH 2/3] ARM: dts: r8a7743: Add APMU node and second CPU core > > > > or > > > > * [PATCH 0/3] Add SMP support > > > > Also including a link is fine by me. > > > > > > It would also be useful if you described the dependency as a conflict (merge- > > time), compile-time or run-time. > > > Thanks Simon. I will take care this next time. > > > As for the patch itself, it looks good to me. As I believe the dependencies are apply-time and seemed easy to resolve I have applied this patch for v4.14. -- 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
