Hi Sebastian, Thank you for the patch. CC'ing Sakari. Sakari, could you please pick this one up ? On Fri, Nov 13, 2020 at 03:28:31PM +0100, Sebastian Andrzej Siewior wrote: > From: Thomas Gleixner <tglx@xxxxxxxxxxxxx> > > in_interrupt() covers hard and soft interrupt servicing and bottom half > disabled contexts, which is semantically undefined and useless. > > The comment for __ccdc_lsc_configure() "Context: in_interrupt()" is > therefore as useful as "Context: unknown'. Remove it. > > Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx> > Signed-off-by: Sebastian Andrzej Siewior <bigeasy@xxxxxxxxxxxxx> > Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> > Cc: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > Cc: linux-media@xxxxxxxxxxxxxxx > --- > Repost of https://lore.kernel.org/linux-media/20201013143731.704783731@xxxxxxxxxxxxx/ > The other patches in this thread were applied. > > drivers/media/platform/omap3isp/ispccdc.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/media/platform/omap3isp/ispccdc.c b/drivers/media/platform/omap3isp/ispccdc.c > index 0fbb2aa6dd2c0..4e8905ef362f2 100644 > --- a/drivers/media/platform/omap3isp/ispccdc.c > +++ b/drivers/media/platform/omap3isp/ispccdc.c > @@ -299,11 +299,10 @@ static int ccdc_lsc_busy(struct isp_ccdc_device *ccdc) > ISPCCDC_LSC_BUSY; > } > > -/* __ccdc_lsc_configure - Apply a new configuration to the LSC engine > +/* > + * __ccdc_lsc_configure - Apply a new configuration to the LSC engine > * @ccdc: Pointer to ISP CCDC device > * @req: New configuration request > - * > - * context: in_interrupt() > */ > static int __ccdc_lsc_configure(struct isp_ccdc_device *ccdc, > struct ispccdc_lsc_config_req *req) -- Regards, Laurent Pinchart
