Hans, I have already changed v4l2_i2c_new_probed_subdev() to v4l2_i2c_new_subdev_board() in my latest patch set for adding vpif capture driver for DM6467 that you had reviewed. I think this change is not needed once that patch is applied. Murali Karicheri Software Design Engineer Texas Instruments Inc. Germantown, MD 20874 new phone: 301-407-9583 Old Phone : 301-515-3736 (will be deprecated) email: m-karicheri2@xxxxxx >-----Original Message----- >From: linux-media-owner@xxxxxxxxxxxxxxx [mailto:linux-media- >owner@xxxxxxxxxxxxxxx] On Behalf Of Hans Verkuil >Sent: Monday, August 10, 2009 2:07 AM >To: Subrahmanya, Chaithrika >Cc: linux-media@xxxxxxxxxxxxxxx >Subject: vpif_display.c bug > >Hi Chaithrika, > >This code in vpif_display.c is not correct: > > for (i = 0; i < subdev_count; i++) { > vpif_obj.sd[i] = >v4l2_i2c_new_probed_subdev(&vpif_obj.v4l2_dev, > i2c_adap, >subdevdata[i].name, > subdevdata[i].name, > &subdevdata[i].addr); > if (!vpif_obj.sd[i]) { > vpif_err("Error registering v4l2 subdevice\n"); > goto probe_subdev_out; > } > > if (vpif_obj.sd[i]) > vpif_obj.sd[i]->grp_id = 1 << i; > } > >This: '&subdevdata[i].addr' should be: I2C_ADDRS(subdevdata[i].addr). > >The list of probe addresses must be terminated by I2C_CLIENT_END (= -1) and >that isn't the case here. > >An alternative solution is to use v4l2_i2c_new_subdev, but then no probing >will take place. But I think that you don't want probing at all since this >address information comes from the platform data, so one can assume that >that data is correct. > >Even better is to copy the implementation from vpfe_capture.c and to use >v4l2_i2c_new_subdev_board(). > >Regards, > > Hans > >-- >Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom >-- >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 ��.n��������+%������w��{.n�����{��g����^n�r������&��z�ޗ�zf���h���~����������_��+v���)ߣ�m
