Hi, Marc Configuring tegra i2c slave address does not put it into slave mode (as opposed to master mode). Tegra can act as an i2c master as well as i2c slave. And 0xFC is not a special address. It's just an i2c address which is not being used by any other i2c slave devices. Thanks Wei. -----Original Message----- From: Marc Dietrich [mailto:marvin24@xxxxxx] Sent: Friday, May 13, 2011 1:39 PM To: Wei Ni Cc: Stephen Warren; Bharat Nihalani; linux-i2c@xxxxxxxxxxxxxxx; linux-tegra@xxxxxxxxxxxxxxx Subject: Re: Tegra I2C slave address patches Hi Wei, Am Fri, 13 May 2011 12:28:27 +0800 schrieb Wei Ni <wni@xxxxxxxxxx>: > Hi, Stephen, Marc > These i2c fix are come from the android-tegra-2.6.36, they will > Fix some i2c transfer bugs. > > Tegra2 has an improved new i2c slave controller. This should be > used instead of the old i2c slave controller. With old i2c slave > controller, occasionally it generates spurious slave interrupts > causing disruptions in master i2c transfers. > And new i2c slave should be configured with a non-zero i2c address. > Configure i2c slave with an i2c address assigned by platform > data for an i2c dev. until now I was unter the impression that programming an i2c slave addresss puts the contoller into slave mode (opposed to master mode). Sorry if this sounds like a dumb question, but is 0xfc some kind of special address or is the master/slave operation programmed in some different way? Thanks, Marc > Marc Dietrich wrote at Thursday, May 12, 2011 1:09 PM: > > Hi Stephen, > > > > Am Donnerstag 12 Mai 2011, 20:15:08 schrieb Stephen Warren: > > > Wei, > > > > > > I'm trying to consolidate the two Tegra I2C slave mode patches we > > > posted to the mailing lists: > > > > > > Yours: http://www.spinics.net/lists/linux-i2c/msg05437.html > > > Mine: http://www.spinics.net/lists/linux-i2c/msg05464.html > > > > > > I had some questions: > > > > > > 1) Why does your patch enable I2C_SL_CNFG_NACK. According to the > > > Tegra documentation, this prevents the slave I2C controller from > > > ACKing any transfers. Doesn't this prevent the slave > > > functionality from working at all? > > > > > > 2) Your patch sets up the slave_addr registers based on platform > > > data. However, I don't see any code in i2c-tegra.c to actually > > > act as a slave device. Hence, it seems pointless to configure the > > > slave address. > > > > > > Are those two things bug workarounds or something? > > > > > > I know that Marc said he'd like to see the slave address > > > configuration code merged, since the AC100 uses it. However, I'm > > > having a hard time seeing how it'll make any difference to the > > > driver right now; it seems if/when slave mode is actually > > > implemented, the slave_addr setup should be part of that patch. > > > > you are right, slave implementation requires at least an additional > > ISR. As we don't have any tegra documentation, it will probably > > take a little longer to implement it correctly. Maybe I hoped that > > there are some other devices using it. Because there seem to be > > none, it's ok from my side to drop that part. If we ever get > > something hacked together, I'm going resubmit it. > > > > Btw, I was a little bit "shocked" that seaboard addedd the > > slave_addr (0xfc) to the board file, which looks like all ports > > (except dvc) became slaves now. Is this a special kind of address, > > because it is the last possible one? > > Interesting. For reference, you're talking about the following in > http://git.chromium.org/chromiumos/third_party/kernel.git > chromeos-2.6.38 > > 1eb243925afed4ef70a04f632f5288e320607ac0 > CHROMIUM: tegra: seaboard: i2c address for new i2c slaves > New i2c slave should be configured with a non-zero i2c address. > Configure i2c slave with an i2c address assigned by platform > data for an i2c dev. > Assign 0xFC as i2c slave address for all new i2c slaves. > > I'll have to let Wei comment on that; he wrote the change. It > certainly sounds like it's setting an unused slave address as some > kind of bug workaround. > -- 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
