Message ID | 20240731103353.39245-1-kartik.koolks@gmail.com (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | [v2] staging: atomisp: bnr: fix trailing statement | expand |
Hi, On 7/31/24 12:33 PM, Kartik Kulkarni wrote: > Fix checkpatch error trailing statements should be on next line in > ia_css_bnr.host.c:48 > > Signed-off-by: Kartik Kulkarni <kartik.koolks@gmail.com> Thank you for your patch(es). I have merged this/these in my media-atomisp branch: https://git.kernel.org/pub/scm/linux/kernel/git/hansg/linux.git/log/?h=media-atomisp And this/these will be included in my next pull-request to Mauro (to media subsystem maintainer) Regards, Hans > --- > v2: Move Signed-off-by line above the exclusion block > --- > .../atomisp/pci/isp/kernels/bnr/bnr_1.0/ia_css_bnr.host.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/bnr/bnr_1.0/ia_css_bnr.host.c b/drivers/staging/media/atomisp/pci/isp/kernels/bnr/bnr_1.0/ia_css_bnr.host.c > index 457a004e1..b75cfd309 100644 > --- a/drivers/staging/media/atomisp/pci/isp/kernels/bnr/bnr_1.0/ia_css_bnr.host.c > +++ b/drivers/staging/media/atomisp/pci/isp/kernels/bnr/bnr_1.0/ia_css_bnr.host.c > @@ -45,7 +45,8 @@ ia_css_bnr_dump( > const struct sh_css_isp_bnr_params *bnr, > unsigned int level) > { > - if (!bnr) return; > + if (!bnr) > + return; > ia_css_debug_dtrace(level, "Bayer Noise Reduction:\n"); > ia_css_debug_dtrace(level, "\t%-32s = %d\n", > "bnr_gain_all", bnr->gain_all);
diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/bnr/bnr_1.0/ia_css_bnr.host.c b/drivers/staging/media/atomisp/pci/isp/kernels/bnr/bnr_1.0/ia_css_bnr.host.c index 457a004e1..b75cfd309 100644 --- a/drivers/staging/media/atomisp/pci/isp/kernels/bnr/bnr_1.0/ia_css_bnr.host.c +++ b/drivers/staging/media/atomisp/pci/isp/kernels/bnr/bnr_1.0/ia_css_bnr.host.c @@ -45,7 +45,8 @@ ia_css_bnr_dump( const struct sh_css_isp_bnr_params *bnr, unsigned int level) { - if (!bnr) return; + if (!bnr) + return; ia_css_debug_dtrace(level, "Bayer Noise Reduction:\n"); ia_css_debug_dtrace(level, "\t%-32s = %d\n", "bnr_gain_all", bnr->gain_all);
Fix checkpatch error trailing statements should be on next line in ia_css_bnr.host.c:48 Signed-off-by: Kartik Kulkarni <kartik.koolks@gmail.com> --- v2: Move Signed-off-by line above the exclusion block --- .../atomisp/pci/isp/kernels/bnr/bnr_1.0/ia_css_bnr.host.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)