[i2c] [PATCH 2/3] i2c: Let bus drivers add SPD to their class

[Jon Smirl]

On 6/3/08, David Brownell <david-b at pacbell.net> wrote:
> On Tuesday 03 June 2008, Jean Delvare wrote:
>  >
>  > Note that I took a conservative approach here, adding I2C_CLASS_SPD to
>  > all drivers that had I2C_CLASS_HWMON before. This is to make sure that
>  > the eeprom driver doesn't stop probing buses where SPD EEPROMs live.
>  > But the truth is that, for most of these, I simply have no idea whether
>  > they can host SPD EEPROMs or not.
>
>
> Few embedded platforms use discrete sticks of DRAM.
>  My two cents:  use the opposite default in those cases.
>
>  I'll highlight a few below, where I happen to have more
>  specific knowledge.
>
>
>
>  > So, bus driver maintainers and users
>  > should feel free to remove the SPD class from drivers those buses never
>  > have SPD EEPROMs or they don't want the eeprom driver to bind to them.
>  > Likewise, feel free to add the SPD class to any bus driver I might have
>  > missed.
>  >
>  > Signed-off-by: Jean Delvare <khali at linux-fr.org>
>  > ---
>  >  drivers/i2c/busses/i2c-ali1535.c       |    2 +-
>  >  drivers/i2c/busses/i2c-ali1563.c       |    2 +-
>  >  drivers/i2c/busses/i2c-ali15x3.c       |    2 +-
>  >  drivers/i2c/busses/i2c-amd756.c        |    2 +-
>  >  drivers/i2c/busses/i2c-amd8111.c       |    2 +-
>  >  drivers/i2c/busses/i2c-at91.c          |    2 +-
>
>
> I've never heard of an AT91 board using DRAM sticks ...
>
>
>  >  drivers/i2c/busses/i2c-cpm.c           |    2 +-
>  >  drivers/i2c/busses/i2c-davinci.c       |    2 +-
>
>
> ... or a DaVinci one ...
>
>
>
>  >  drivers/i2c/busses/i2c-elektor.c       |    2 +-
>  >  drivers/i2c/busses/i2c-gpio.c          |    2 +-
>  >  drivers/i2c/busses/i2c-i801.c          |    2 +-
>  >  drivers/i2c/busses/i2c-ibm_iic.c       |    4 ++--
>  >  drivers/i2c/busses/i2c-iop3xx.c        |    2 +-
>  >  drivers/i2c/busses/i2c-isch.c          |    2 +-
>  >  drivers/i2c/busses/i2c-mpc.c           |    2 +-

Freescale embedded, no sticks

>  >  drivers/i2c/busses/i2c-mv64xxx.c       |    2 +-
>  >  drivers/i2c/busses/i2c-nforce2.c       |    2 +-
>  >  drivers/i2c/busses/i2c-ocores.c        |    2 +-
>  >  drivers/i2c/busses/i2c-omap.c          |    2 +-
>
>
> ... or an OMAP one ...
>
>
>  >  drivers/i2c/busses/i2c-parport-light.c |    2 +-
>  >  drivers/i2c/busses/i2c-parport.c       |    2 +-
>  >  drivers/i2c/busses/i2c-pasemi.c        |    2 +-
>  >  drivers/i2c/busses/i2c-piix4.c         |    2 +-
>  >  drivers/i2c/busses/i2c-pmcmsp.c        |    2 +-
>  >  drivers/i2c/busses/i2c-s3c2410.c       |    2 +-
>  >  drivers/i2c/busses/i2c-sibyte.c        |    4 ++--
>  >  drivers/i2c/busses/i2c-sis5595.c       |    2 +-
>  >  drivers/i2c/busses/i2c-sis630.c        |    2 +-
>  >  drivers/i2c/busses/i2c-sis96x.c        |    2 +-
>  >  drivers/i2c/busses/i2c-stub.c          |    2 +-
>  >  drivers/i2c/busses/i2c-tiny-usb.c      |    2 +-
>
>
> ... DRAM-over-USB would be bizarre too ...
>
>
>  >  drivers/i2c/busses/i2c-via.c           |    2 +-
>  >  drivers/i2c/busses/i2c-viapro.c        |    2 +-
>  >  drivers/i2c/busses/scx200_acb.c        |    2 +-
>  >  include/linux/i2c.h                    |    1 +
>  >  35 files changed, 37 insertions(+), 36 deletions(-)
>
>
>
>
> _______________________________________________
>  i2c mailing list
>  i2c at lm-sensors.org
>  http://lists.lm-sensors.org/mailman/listinfo/i2c
>


-- 
Jon Smirl
jonsmirl at gmail.com