Em Fri, 28 Aug 2015 11:19:29 +0900 Junghak Sung <jh1009.s...@samsung.com> escreveu:
> Dear Mauro, > > I'm sorry but patch 3 is very incomplete. > At this round, I could not modify all device drivers, > because there are too many files to be changed > and I should change them by hands. > > I just want to be reviewed the modification pattern only before > I start to modify whole related device drivers. > I mean that .. most of drivers will be changed with a similar pattern > ,even though, detailed will be a little different. > > Could you verify whether my modification in this patch is correct? It is hard to tell without being able to compile ;) Provided that you don't add uneeded typecases, the compiler do a way better job checking those things than humans. Of course, we still do manual check in order to check border cases. Regards, Mauro > > > On 08/27/2015 07:33 PM, Mauro Carvalho Chehab wrote: > > Em Wed, 26 Aug 2015 20:59:30 +0900 > > Junghak Sung <jh1009.s...@samsung.com> escreveu: > > > >> Modify all device drivers related with previous change that restructures > >> vb2_buffer for common use. > >> Actually, not all device drivers, yet. So, it required to modifiy more file > >> to complete this patch. > > > > I was expecting to be able to compile everything after applying both > > patches 2 and 3, but compilation failed: > > > > In file included from drivers/media/platform/am437x/am437x-vpfe.c:41:0: > > drivers/media/platform/am437x/am437x-vpfe.h:107:25: error: field 'vb' has > > incomplete type > > struct vb2_v4l2_buffer vb; > > ^ > > drivers/media/platform/am437x/am437x-vpfe.c: In function 'to_vpfe_buffer': > > drivers/media/platform/am437x/am437x-vpfe.c:312:72: warning: initialization > > from incompatible pointer type [-Wincompatible-pointer-types] > > drivers/media/platform/am437x/am437x-vpfe.c: In function > > 'vpfe_buffer_prepare': > > drivers/media/platform/am437x/am437x-vpfe.c:1952:4: error: 'struct > > vb2_buffer' has no member named 'v4l2_buf' > > vb->v4l2_buf.field = vpfe->fmt.fmt.pix.field; > > ^ > > drivers/media/platform/am437x/am437x-vpfe.c: In function 'to_vpfe_buffer': > > drivers/media/platform/am437x/am437x-vpfe.c:313:1: warning: control reaches > > end of non-void function [-Wreturn-type] > > } > > ^ > > scripts/Makefile.build:258: recipe for target > > 'drivers/media/platform/am437x/am437x-vpfe.o' failed > > make[3]: *** [drivers/media/platform/am437x/am437x-vpfe.o] Error 1 > > scripts/Makefile.build:403: recipe for target > > 'drivers/media/platform/am437x' failed > > make[2]: *** [drivers/media/platform/am437x] Error 2 > > scripts/Makefile.build:403: recipe for target 'drivers/media/platform' > > failed > > make[1]: *** [drivers/media/platform] Error 2 > > make[1]: *** Waiting for unfinished jobs.... > > In file included from drivers/media/pci/cobalt/cobalt-driver.c:30:0: > > drivers/media/pci/cobalt/cobalt-driver.h:209:25: error: field 'vb' has > > incomplete type > > struct vb2_v4l2_buffer vb; > > ^ > > drivers/media/pci/cobalt/cobalt-driver.h: In function 'to_cobalt_buffer': > > drivers/media/pci/cobalt/cobalt-driver.h:215:70: warning: initialization > > from incompatible pointer type [-Wincompatible-pointer-types] > > scripts/Makefile.build:258: recipe for target > > 'drivers/media/pci/cobalt/cobalt-driver.o' failed > > make[3]: *** [drivers/media/pci/cobalt/cobalt-driver.o] Error 1 > > scripts/Makefile.build:403: recipe for target 'drivers/media/pci/cobalt' > > failed > > make[2]: *** [drivers/media/pci/cobalt] Error 2 > > scripts/Makefile.build:403: recipe for target 'drivers/media/pci' failed > > make[1]: *** [drivers/media/pci] Error 2 > > drivers/media/dvb-frontends/rtl2832_sdr.c:110:25: error: field 'vb' has > > incomplete type > > struct vb2_v4l2_buffer vb; /* common v4l buffer stuff -- must be first > > */ > > ^ > > drivers/media/dvb-frontends/rtl2832_sdr.c: In function > > 'rtl2832_sdr_buf_queue': > > drivers/media/dvb-frontends/rtl2832_sdr.c:523:73: warning: initialization > > from incompatible pointer type [-Wincompatible-pointer-types] > > drivers/media/dvb-frontends/rtl2832_sdr.c:523:73: note: (near > > initialization for 'buf') > > scripts/Makefile.build:258: recipe for target > > 'drivers/media/dvb-frontends/rtl2832_sdr.o' failed > > make[2]: *** [drivers/media/dvb-frontends/rtl2832_sdr.o] Error 1 > > scripts/Makefile.build:403: recipe for target 'drivers/media/dvb-frontends' > > failed > > make[1]: *** [drivers/media/dvb-frontends] Error 2 > > Makefile:1380: recipe for target '_module_drivers/media' failed > > make: *** [_module_drivers/media] Error 2 > > > > > > So, obviously there's something wrong there. Please fix. > > > > I'll skip this patch from my review. > > > > Thanks! > > Mauro > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-media" in > > the body of a message to majord...@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
