Re: [PATCH] [media] pci: constify vb2_ops structures

[Julia Lawall]

Le 09.09.2016 17:17, Andrey Utkin a ÃcritÂ:
> On Fri, Sep 09, 2016 at 01:59:18AM +0200, Julia Lawall wrote:
> > Check for vb2_ops structures that are only stored in the ops field of 
> > a
> > vb2_queue structure.  That field is declared const, so vb2_ops 
> > structures
> > that have this property can be declared as const also.
>
> > Signed-off-by: Julia Lawall <Julia.Lawall@xxxxxxx>
> >
> > ---
> >  drivers/media/pci/cx23885/cx23885-417.c            |    2 +-
> >  drivers/media/pci/cx23885/cx23885-dvb.c            |    2 +-
> >  drivers/media/pci/cx23885/cx23885-video.c          |    2 +-
> >  drivers/media/pci/cx25821/cx25821-video.c          |    2 +-
> >  drivers/media/pci/cx88/cx88-blackbird.c            |    2 +-
> >  drivers/media/pci/cx88/cx88-dvb.c                  |    2 +-
> >  drivers/media/pci/cx88/cx88-video.c                |    2 +-
> >  drivers/media/pci/netup_unidvb/netup_unidvb_core.c |    2 +-
> >  drivers/media/pci/saa7134/saa7134-empress.c        |    2 +-
> >  drivers/media/pci/saa7134/saa7134-video.c          |    2 +-
> >  drivers/media/pci/solo6x10/solo6x10-v4l2-enc.c     |    2 +-
> >  drivers/media/pci/tw68/tw68-video.c                |    2 +-
> >  drivers/media/pci/tw686x/tw686x-video.c            |    2 +-
> >  13 files changed, 13 insertions(+), 13 deletions(-)
>
> Applies and compiles cleanly on media tree.
> But please regenerate this patch on git://linuxtv.org/media_tree.git -
> it has a new driver by me, drivers/media/pci/tw5864 , which is 
> affected.

Do you want the whole patch again, or would a patch on the new driver by 
itself be sufficient?  The changes in the different files are all 
independent.

julia

> Otherwise
>
> Acked-by: Andrey Utkin <andrey.utkin@xxxxxxxxxxxxxxxxxxx>
>
> (however this time I'm writing from another email.)
> --
> To unsubscribe from this list: send the line "unsubscribe 
> kernel-janitors" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html