Hi Laurent,
On Tue, Jan 10, 2017 at 8:58 PM, Laurent Pinchart
<laurent.pinchart@xxxxxxxxxxxxxxxx> wrote:
> On Tuesday 10 Jan 2017 16:07:01 Geert Uytterhoeven wrote:
>> On Mon, Jan 9, 2017 at 8:31 PM, Jacopo Mondi wrote:
>> > From: Magnus Damm <damm@xxxxxxxxxxxxx>
>> >
>> > This is a squash of several commits, adding peripherals groups
>> > configuration to r7s72100 device tree, and enabling some of them on
>> > Genmai evaluation board
>> >
>> > Signed-off-by: Jacopo Mondi <jacopo+renesas@xxxxxxxxxx>
>>
>> Thanks for the rework!
>>
>> > arch/arm/boot/dts/r7s72100-genmai.dts | 51 ++++++++++++
>> > arch/arm/boot/dts/r7s72100.dtsi | 151 +++++++++++++++++++++++++++++
>>
>> This path should be split in multiple parts:
>> - Add the pfc node to r7s72100.dtsi,
>> - Add the gpio nodes to r7s72100.dtsi,
>> - 4 patches for r7s72100-genmai.dts, adding support for LEDs, SCIF,
>> Ethernet, and SPI.
>
> I can agree about the .dtsi/.dts split, but isn't this going a bit overboard ?
Let's find out what Simon's opinion is...
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
