[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <9F1430C9-1AC7-4955-8785-1E321BCD6E88@weidenauer.cc>
Date: Mon, 14 Jul 2025 15:51:19 +0200
From: Martin Weidenauer <martin@...denauer.cc>
To: Greg KH <gregkh@...uxfoundation.org>
CC: hansg@...nel.org, mchehab@...nel.org, sakari.ailus@...ux.intel.com,
andy@...nel.org, linux-media@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-staging@...ts.linux.dev, ~lkcamp/patches@...ts.sr.ht, koike@...lia.com
Subject: Re: [PATCH] staging: atomisp: isp: fix open brace on new line
If you mean the indentation in the file, I think it's unfortunately just a messy driver in general.
I mean I could fix it in another commit in general if you want.
Be well,
Martin
On 14 July 2025 15:24:57 CEST, Greg KH <gregkh@...uxfoundation.org> wrote:
>On Mon, Jul 14, 2025 at 03:19:53PM +0200, Martin Weidenauer wrote:
>> Fix checkpatch error "ERROR: that open brace { should be on the previous line"
>> in ia_css_dvs.host.c:277.
>>
>> Signed-off-by: Martin Weidenauer <martin@...denauer.cc>
>>
>> ---
>>
>> Hey, this is my first patch, I appreciate any feedback. Greetings from DebConf25!
>
>Odd indentation, how did that happen?
>
>thanks,
>
>greg k-h
Powered by blists - more mailing lists