Kenneth Aafløy wrote:
>
> I've attached an archive with patches for the not yet converted frontends and
> some minor modifications to the existing.
>
> Frontends converted to kernel i2c here is:
> cx24110, dst, dvb_dummy_fe, grundig*, mt*, nxt6000, ves1820
>
> For the rest of the frontends, I've changed the names to dvbfe_* and moved
> I2C_DRIVERID_DVBFE* to dvb_i2c.h.
>
> I'm unsure if they will actually work since I have no way of testing them,
> but I'm most concerned about dst and mt* which apparently is using the bttv
> driver.
>
> Is it okay to apply these to CVS?
I have no time (and not the hardware) for testing, but the patches
look good so far. Please commit, if something breaks we can fix later.
I believe this is the only way to get this stuff tested.
One thing:
./include/linux/i2c.h:#define I2C_DEVNAME(str) .name = str
I believe that this is some legacy crap that we should get rid off.
It isn't used anywhere in drivers/i2c/.
E.g.:
static struct i2c_client client_template = {
- I2C_DEVNAME("dvbfe_tdlb7"),
+ .name = "dvbfe_tdlb7",
.flags = I2C_CLIENT_ALLOW_USE,
Looks much saner to me.
Thanks a lot for doing all this work!
Johannes
