On Mon, Aug 24, 2020 at 12:23:04PM +0300, Sakari Ailus wrote: > On Sun, Aug 23, 2020 at 08:02:57AM +0300, Laurent Pinchart wrote: > > When compiled on 32-bit ARM, the CAL driver fails with the FIELD_PREP() > > macro complaining that the mask is not constant. While all callers of > > the inline cal_write_field() function pass a constant mask, the mask > > parameter itself is a variable, which likely doesn't please the > > compiler. > > > > Fix it by replacing FIELD_PREP() with a manual implementation. > > > > Fixes: 50797fb30b95 ("media: ti-vpe: cal: Turn reg_(read|write)_field() into inline functions") > > Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > > Thanks! > > Reviewed-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> And I forgot to add Reported-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> Tomi, will you handle this patch ? Could you add the tags ? -- Regards, Laurent Pinchart