Hi Thomas, On 15/07/2023 20:51, Thomas Zimmermann wrote: > The flag FBINFO_FLAG_DEFAULT is 0 and has no effect, as struct > fbinfo.flags has been allocated to zero by kzalloc(). So do not > set it. > > Flags should signal differences from the default values. After cleaning > up all occurrences of FBINFO_DEFAULT, the token will be removed. > > v2: > * fix commit message (Miguel) > > Signed-off-by: Thomas Zimmermann <tzimmermann@xxxxxxx> > Acked-by: Sam Ravnborg <sam@xxxxxxxxxxxx> > Cc: Andy Walls <awalls@xxxxxxxxxxxxxxxx> > Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> > Cc: Hans Verkuil <hverkuil@xxxxxxxxx> > --- > drivers/media/pci/ivtv/ivtvfb.c | 1 - > drivers/media/test-drivers/vivid/vivid-osd.c | 1 - > 2 files changed, 2 deletions(-) I can take this patches for 6.6, unless you prefer to have this whole series merged in one go? In that case you can use my: Reviewed-by: Hans Verkuil <hverkuil-cisco@xxxxxxxxx> Regards, Hans > > diff --git a/drivers/media/pci/ivtv/ivtvfb.c b/drivers/media/pci/ivtv/ivtvfb.c > index 0aeb9daaee4c..23c8c094e791 100644 > --- a/drivers/media/pci/ivtv/ivtvfb.c > +++ b/drivers/media/pci/ivtv/ivtvfb.c > @@ -1048,7 +1048,6 @@ static int ivtvfb_init_vidmode(struct ivtv *itv) > /* Generate valid fb_info */ > > oi->ivtvfb_info.node = -1; > - oi->ivtvfb_info.flags = FBINFO_FLAG_DEFAULT; > oi->ivtvfb_info.par = itv; > oi->ivtvfb_info.var = oi->ivtvfb_defined; > oi->ivtvfb_info.fix = oi->ivtvfb_fix; > diff --git a/drivers/media/test-drivers/vivid/vivid-osd.c b/drivers/media/test-drivers/vivid/vivid-osd.c > index ec25edc679b3..051f1805a16d 100644 > --- a/drivers/media/test-drivers/vivid/vivid-osd.c > +++ b/drivers/media/test-drivers/vivid/vivid-osd.c > @@ -310,7 +310,6 @@ static int vivid_fb_init_vidmode(struct vivid_dev *dev) > /* Generate valid fb_info */ > > dev->fb_info.node = -1; > - dev->fb_info.flags = FBINFO_FLAG_DEFAULT; > dev->fb_info.par = dev; > dev->fb_info.var = dev->fb_defined; > dev->fb_info.fix = dev->fb_fix;
