[U-Boot] [PATCH v3 8/9] tegra: i2c: Enable new CONFIG_SYS_I2C framework

Stephen Warren swarren at wwwdotorg.org
Wed Jul 31 21:41:54 CEST 2013


On 07/31/2013 02:31 AM, Heiko Schocher wrote:
> Am 31.07.2013 10:16, schrieb Albert ARIBAUD:
>> On Wed, 31 Jul 2013 09:36:19 +0200, Heiko Schocher<hs at denx.de>  wrote:
...
>>>> In the present instance, I'd rather we either:
>>>>
>>>> - removed dependency on DT etc. by using "hard-coded" low level I2C
>>>>     reads for those boards that need it (I assume that for each of
>>>> these
>>>>     boards the I2C slave, offset, and length to read are constant)
>>>> in _f
>>>>     phase, or
>>>
>>> But DT is used for initializing the i2c driver in tegra ...
>>
>> Alright, out goes this proposal. Anyway, I didn't like it best. :)
> 
> Yes.
> 
>>>> - parsed the _f phase looking for offending functions or calls which
>>>>     write to .data or .bss and fix them, suffixing them with _f; in
>>>>     essence, that amounts to starting the implementation of my
>>>> suggestion
>>>>     above alongside fixing the issue at hand.
>>>>
>>>> The first approach is rather "let's bring the thing back up first", so
>>>> it does not have my preference, but I would understand the need to
>>>> quickly fix things.
>>>
>>> Yes.
>>>
>>>> The second approach seems to be going in the same direction as Heiko's
>>>> proposal of 07:52 +0200, which I thus second provided it is applicable
>>>> to all the boards Wolfgang had in mind.
>>>
>>> Lets do us this step as fixup ;-)
>>
>> Alright too. :)
> 
> Ok.
> 
> @Stephen, Simon: Could anyone from you do this 2 steps?

Sorry, I really can't tell which 2 steps you mean? I don't see any email
in any timezone with minutes==:52...


More information about the U-Boot mailing list