Re: [v4l-dvb-maintainer] [PATCH] saa7134: Don't support I2C_M_NOSTART
- Date: Tue, 12 Feb 2008 11:24:45 -0800
- From: "Trent Piepho" <xyzzy@xxxxxxxxxxxxx>
- Subject: Re: [v4l-dvb-maintainer] [PATCH] saa7134: Don't support I2C_M_NOSTART
On Feb 12, 2008 1:51 AM, Jean Delvare <khali@xxxxxxxxxxxx> wrote:
> I2C adapters should only support I2C_M_NOSTART if they really have to
> (i.e. if they are connected to a broken I2C device which needs this
> deviation from the standard I2C protocol.) As no media chip driver
> uses I2C_M_NOSTART, I don't think that the saa7134 driver needs to
> support it.I2C_M_NOSTART,
There is a chip used on some Osprey bt878 cards, X9221 POT, that does need
I2C_M_NOSTART.
I don't know of any saa7134 cards using that part, but the problem with
removing this code is that the datasheets for saa713x chips isn't
available. In the m920x case, it's all from reverse engineering. Suppose
someone does want to support a device that needs this, how will they add
support for it at that time if they don't know how the I2C controller works?
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@xxxxxxxxxx?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list