[U-Boot] [PATCH 03/12] 83xx, kmeter1: add I2C, dtt, eeprom support
Heiko Schocher
hs at denx.de
Tue Feb 17 07:03:23 CET 2009
Hello Kim,
Kim Phillips wrote:
> On Wed, 11 Feb 2009 19:25:49 +0100
> Heiko Schocher <hs at denx.de> wrote:
>
>> diff --git a/drivers/hwmon/lm75.c b/drivers/hwmon/lm75.c
>> index 8119821..ecfc345 100644
>> --- a/drivers/hwmon/lm75.c
>> +++ b/drivers/hwmon/lm75.c
>> @@ -42,6 +42,8 @@
>> #define DTT_TEMP_HYST 0x2
>> #define DTT_TEMP_SET 0x3
>>
>> +DECLARE_GLOBAL_DATA_PTR;
>> +
>> int dtt_read(int sensor, int reg)
>> {
>> int dlen;
>> @@ -157,7 +159,17 @@ int dtt_init (void)
>>
>> /* switch to correct I2C bus */
>> old_bus = I2C_GET_BUS();
>> - I2C_SET_BUS(CONFIG_SYS_DTT_BUS_NUM);
>> + if ((gd->flags & GD_FLG_RELOC) != GD_FLG_RELOC) {
>> + uchar *tmp = (uchar *)getenv("dtt_bus");
>
> sorry Heiko, I should have asked you before: this is a hardcoded
> environment variable in driver code, does the dtt bus value change
> between boots? If not, what's the reason the value isn't specified as
> a CONFIG_SYS_DTT_BUS_NUM?
It changes not between boots, but between different hardware.
>> + if (tmp == NULL)
>> + printf("FAILED: dtt_bus not defined\n");
>> + else {
>> + if (i2c_mux_ident_muxstring_f(tmp))
>> + printf("FAILED: couldnt switch to bus\n");
>> + }
>
> btw, these braces unnecessary.
Yep, I change this.
thanks
Heiko
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
More information about the U-Boot
mailing list