On Fri, 2015-01-23 at 14:48 +0200, Mika Westerberg wrote:
> On Fri, Jan 23, 2015 at 01:41:55PM +0100, Wolfram Sang wrote:
> > On Fri, Jan 23, 2015 at 01:54:03PM +0200, Andy Shevchenko wrote:
> > > There is no need to duplicate same data for each controller. If we need
> > > specific stuff for a certain controller in the future we may add it later. The
> > > patch leaves one controller per platform.
> > >
> > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> > > ---
> > > drivers/i2c/busses/i2c-designware-pcidrv.c | 63 ++++++------------------------
> > > 1 file changed, 11 insertions(+), 52 deletions(-)
> > >
> > > diff --git a/drivers/i2c/busses/i2c-designware-pcidrv.c b/drivers/i2c/busses/i2c-designware-pcidrv.c
> > > index 5c6fca7..435a8ec 100644
> > > --- a/drivers/i2c/busses/i2c-designware-pcidrv.c
> > > +++ b/drivers/i2c/busses/i2c-designware-pcidrv.c
> > > @@ -40,13 +40,7 @@
> > > #define DRIVER_NAME "i2c-designware-pci"
> > >
> > > enum dw_pci_ctl_id_t {
> > > - medfield_0,
> > > - medfield_1,
> > > - medfield_2,
> > > - medfield_3,
> > > - medfield_4,
> > > - medfield_5,
> > > -
> > > + medfield,
> > > baytrail,
> > > haswell,
> > > };
> > > @@ -98,47 +92,12 @@ static struct dw_scl_sda_cfg hsw_config = {
> > > };
> > >
> > > static struct dw_pci_controller dw_pci_controllers[] = {
> > > - [medfield_0] = {
> > > - .bus_num = 0,
> >
> > Wasn't that bus_num used to ensure stable bus numbers? Adding Mika.
>
> That's right.
>
> However, I don't think anybody really uses Medfield outside Intel so in
> that sense this patch should not break anything.
I think better to ask David who is closer to that Intel MID area.
AFAIU, technically due to fake PCI bus on Medfield even with automatic
bus number you will get stable assignment, though it might be changed
once from the original version.
--
Andy Shevchenko <andriy.shevchenko@xxxxxxxxx>
Intel Finland Oy
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
