Hello Peter, On 04/13/2017 05:43 PM, Peter Rosin wrote: > On 2017-04-13 20:28, Javier Martinez Canillas wrote: [snip] >> >> eeprom@50 { >> - compatible = "nxp,24c02"; >> + compatible = "nxp,24c02","atmel,24c02"; >> reg = <0x50>; >> pagesize = <16>; >> }; >> > > Sure, in principle. But not having a space after the commas is a real OCD > trigger for me. My spine is rotating... :-) > Right, Geert already pointed out. It's something I missed in my sed script so I got it wrong in all the DTS :( I'll fix my script and send a v2. > Anyway, with those whitespace fixes (and without them too, but please): > Acked-by: Peter Rosin <peda@xxxxxxxxxx> > Thanks for the review! > Cheers, > peda > > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America -- 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