> > --- a/arch/arm/mach-davinci/board-dm355-evm.c
> > +++ b/arch/arm/mach-davinci/board-dm355-evm.c
> > @@ -136,10 +136,66 @@ static void dm355evm_mmcsd_gpios(unsigned gpio)
> > dm355evm_mmc_gpios = gpio;
> > }
> >
> > -static struct tvp514x_platform_data tvp5146_pdata = {
> > - .clk_polarity = 0,
> > - .hs_polarity = 1,
> > - .vs_polarity = 1
Clearly this patch is against neither mainline nor the
current DaVinci GIT tree... I suggest reissuing against
mainline, now that it's got most DM355 stuff.
> > +/*
> > + * MSP430 supports RTC, card detection, input from IR remote, and
> > + * a bit more. It triggers interrupts on GPIO(7) from pressing
> > + * buttons on the IR remote, and for card detect switches.
> > + */
> > +static struct i2c_client *dm355evm_msp;
> > +
> > +static int dm355evm_msp_probe(struct i2c_client *client,
> > + const struct i2c_device_id *id)
> > +{
> > + dm355evm_msp = client;
> > + return 0;
> > +}
> > +
> > +static int dm355evm_msp_remove(struct i2c_client *client)
> > +{
> > + dm355evm_msp = NULL;
> > + return 0;
> > +}
> > +
> > +static const struct i2c_device_id dm355evm_msp_ids[] = {
> > + { "dm355evm_msp", 0, },
> > + { /* end of list */ },
> > +};
Needless to say: NAK on all this. There is already a
drivers/mfd/dm355evm_msp.c managing this device. You
shouldn't have video code crap all over it.
It currently sets up for TVP5146 based capture iff that
driver is configured (else the external imager); and
exports the NTSC/nPAL switch setting as a GPIO that's
also visible in sysfs.
I suggest the first revision of this VPFE stuff use
the existing setup. A later patch could add some
support for runtime reconfiguration.
- Dave
> > +
> > +static struct i2c_driver dm355evm_msp_driver = {
> > + .driver.name = "dm355evm_msp",
> > + .id_table = dm355evm_msp_ids,
> > + .probe = dm355evm_msp_probe,
> > + .remove = dm355evm_msp_remove,
> > +};
> > +
--
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
