Em 18-01-2011 17:23, Igor M. Liplianin escreveu: > Ð ÑÐÐÐÑÐÐÐÐ ÐÑ 16 ÑÐÐÐÑÑ 2011 19:52:38 ÐÐÑÐÑ Mauro Carvalho Chehab ÐÐÐÐÑÐÐ: >> Em 02-01-2011 10:01, Igor M. Liplianin escreveu: >>> An Altera FPGA CI module for NetUP Dual DVB-T/C RF CI card. >>> >>> Signed-off-by: Igor M. Liplianin <liplianin@xxxxxxxx> >> >> Igor, >> >> There's something wrong with this patch. I got lots of error after applying >> it: >> >> drivers/media/video/cx23885/altera-ci.o: In function >> `netup_ci_read_attribute_mem': >> /home/v4l/v4l/patchwork/drivers/media/video/cx23885/altera-ci.c:304: >> multiple definition of `netup_ci_read_attribute_mem' >> drivers/media/video/cx23885/cx23885.o:/home/v4l/v4l/patchwork/drivers/medi >> a/video/cx23885/cimax2.c:241: first defined here >> drivers/media/video/cx23885/altera-ci.o: In function `netup_ci_op_cam': >> /home/v4l/v4l/patchwork/drivers/media/video/cx23885/altera-ci.c:269: >> multiple definition of `netup_ci_op_cam' >> drivers/media/video/cx23885/cx23885.o:/home/v4l/v4l/patchwork/drivers/medi >> a/video/cx23885/cimax2.c:171: first defined here >> drivers/media/video/cx23885/altera-ci.o: In function >> `netup_ci_slot_shutdown': >> /home/v4l/v4l/patchwork/drivers/media/video/cx23885/altera-ci.c:364: >> multiple definition of `netup_ci_slot_shutdown' >> drivers/media/video/cx23885/cx23885.o:/home/v4l/v4l/patchwork/drivers/medi >> a/video/cx23885/cimax2.c:293: first defined here >> drivers/media/video/cx23885/altera-ci.o: In function >> `netup_ci_slot_ts_ctl': >> /home/v4l/v4l/patchwork/drivers/media/video/cx23885/altera-ci.c:370: >> multiple definition of `netup_ci_slot_ts_ctl' >> drivers/media/video/cx23885/cx23885.o:/home/v4l/v4l/patchwork/drivers/medi >> a/video/cx23885/cimax2.c:320: first defined here >> drivers/media/video/cx23885/altera-ci.o: In function >> `netup_ci_write_cam_ctl': >> /home/v4l/v4l/patchwork/drivers/media/video/cx23885/altera-ci.c:322: >> multiple definition of `netup_ci_write_cam_ctl' >> drivers/media/video/cx23885/cx23885.o:/home/v4l/v4l/patchwork/drivers/medi >> a/video/cx23885/cimax2.c:259: first defined here >> drivers/media/video/cx23885/altera-ci.o: In function >> `netup_ci_read_cam_ctl': >> /home/v4l/v4l/patchwork/drivers/media/video/cx23885/altera-ci.c:315: >> multiple definition of `netup_ci_read_cam_ctl' >> drivers/media/video/cx23885/cx23885.o:/home/v4l/v4l/patchwork/drivers/medi >> a/video/cx23885/cimax2.c:252: first defined here >> drivers/media/video/cx23885/altera-ci.o: In function >> `netup_ci_write_attribute_mem': >> /home/v4l/v4l/patchwork/drivers/media/video/cx23885/altera-ci.c:310: >> multiple definition of `netup_ci_write_attribute_mem' >> drivers/media/video/cx23885/cx23885.o:/home/v4l/v4l/patchwork/drivers/medi >> a/video/cx23885/cimax2.c:247: first defined here >> drivers/media/video/cx23885/altera-ci.o: In function >> `netup_poll_ci_slot_status': >> /home/v4l/v4l/patchwork/drivers/media/video/cx23885/altera-ci.c:448: >> multiple definition of `netup_poll_ci_slot_status' >> drivers/media/video/cx23885/cx23885.o:/home/v4l/v4l/patchwork/drivers/medi >> a/video/cx23885/cimax2.c:403: first defined here >> drivers/media/video/cx23885/altera-ci.o: In function >> `netup_ci_slot_reset': >> /home/v4l/v4l/patchwork/drivers/media/video/cx23885/altera-ci.c:327: >> multiple definition of `netup_ci_slot_reset' >> drivers/media/video/cx23885/cx23885.o:/home/v4l/v4l/patchwork/drivers/medi >> a/video/cx23885/cimax2.c:264: first defined here >> >> >> Please fix it and submit a new version. The better is to replace all those >> new symbols to start with altera_ci. > I see, allyesconfig... Yes. >> >> While here, please, on the first patch, move the Altera FPGA driver to >> staging, to give people some time do discuss about it. > It means that cx23885 driver will depend on staging. Is it OK? The idea is to enable the Altera-dependent part of cx23885 only if compiled. There's no need to make the entire cx2388x dependent of Altera driver. > >> >> PS.: there are a few CodingStyle errors on this patch. Please always review >> your patches with ./scripts/checkpatch.pl. > I'll do it. Ok, thanks! >> >> Thanks! >> Mauro > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
