Hi David On Wed, 18 Sept 2024 at 20:08, David Given <dg@xxxxxxxxxxx> wrote: > > On Wed, 18 Sept 2024 at 06:07, Ricardo Ribalda <ribalda@xxxxxxxxxxxx> wrote: > [...] > > ERROR: trailing whitespace > > 38#213: FILE: drivers/media/usb/uvc/uvc_driver.c:2431: > > 39+ * $ > > Fixed. > > > You can run checkpatch on your patches before sending them to test them locally. > > > > ./scripts/checkpatch --strict -g HEAD > > I actually had, and it said they were fine! Turns out I'd put the > Signed-off-by in the wrong place, and checkpatch clearly just looks > for its presence while the linter on the CI server is stricter. (I'm > not a git user, and am having to do a lot of this stuff manually.) > > The last time I touched the Linux kernel was in the late 1990s. There > wasn't any process back then. np :) thanks for the new version. It can be annoying I know :S -- Ricardo Ribalda
