On Mon, Nov 21, 2022 at 09:04:29AM +0100, Krzysztof Kozlowski wrote: > On 21/11/2022 06:09, Paul Elder wrote: > > On Sun, Nov 20, 2022 at 11:36:31AM +0100, Krzysztof Kozlowski wrote: > >> On 19/11/2022 07:55, Paul Elder wrote: > >>> On Fri, Nov 18, 2022 at 02:06:14PM +0100, Krzysztof Kozlowski wrote: > >>>> On 18/11/2022 10:39, Paul Elder wrote: > >>>>> From: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > >>>>> > >>>>> Add an example to the rockchip-isp1 DT binding that showcases usage of > >>>>> the parallel input of the ISP, connected to the CSI-2 receiver internal > >>>>> to the i.MX8MP. > >>>>> > >>>>> Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > >>>> > >>>> Missing SoB. > >>> > >>> I don't quite understand. I see an SoB right there. > >> > >> Laurent did not sent it. Did you run checkpatch before sending? > > > > That's why he's on the "From:" in the beginning. checkpatch says it's > > fine. > > Ah, indeed, checkpatch misses that feature (it's part of Greg's > verify_signedoff.sh). Anyway, your SoB is missing, as Laurent did not > send the patch. I thought adding an SoB was only required either when making changes or when pushing commits through git, not when forwarding patches on mailing lists ? -- Regards, Laurent Pinchart