Hello Kieran, On Fri, Aug 17, 2018 at 02:44:25PM +0100, Kieran Bingham wrote: > Hi Eugeniu > > Thank you for the patch. > > On 12/08/18 14:31, Eugeniu Rosca wrote: > > Document the support for rcar_can on R8A77965 SoC devices. > > Add R8A77965 to the list of SoCs which require the "assigned-clocks" and > > "assigned-clock-rates" properties (thanks, Sergei). Rewrap text. > > I don't think you needed to say you rewrapped the text in the commit log > - but it's fine :)
IMHO "Rewrap text" is pretty much from the same category as "no functional change was intended". As a reviewer, I would take these details in the commit description any day (and sometimes I would NAK a patch which lacks these details), since they precisely express the goals set by the author and make reviewer's life easier. But, of course, preferences vary and therefore I won't elaborate on that too much. > > > Signed-off-by: Eugeniu Rosca <ero...@de.adit-jv.com> > > Reviewed-by: Kieran Bingham <kieran.bingham+rene...@ideasonboard.com> > Best regards, Eugeniu.
