Re: [CORRECTED] I2C driver supporting Moorestown and Medfield platform

[Alan Cox <alan-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>]

> hardware/software resources, but IMO, MRST is not easy to imagine
> that it represents "Moorstone".  But this is not such a big deal.

mrst is used all over the place - I'm not familiar with how that came
about originally.

> > + * Copyright (c) 2009 Intel Corporation.
> > + * Copyright (c) 2009 Synopsys. Inc.
> 
> Having a quick glance of the patch, the driver is similar to the
> existing I2C_DESIGNWARE driver.  Presumably Intel people have already
> checkekd the i2c-designware.c and made a decision that it would be
> reasonable to create a new I2C driver to incorporate its requirements
> (such as PCI resource handlings, power management code, etc.), rather
> than to extend the existing one.  Is my understanding correct?
> 
> I'm not objecting to the patch, but would like to know the background
> of this driver, as one of the DW users.

There are various little differences along with the PCI, power
management, timing and configuration side. I've also looked at the two
side by side and agreed with the original Intel authors that it seemed
best to keep it as two drivers. Otherwise you drag a lot of PCI and PCI
pm logic into a simpler driver that is used on non PCI platforms.

Alan