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

[Heiko Schocher]

Hello Stephen,

Am 31.07.2013 21:41, schrieb Stephen Warren:
> 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...

http://lists.denx.de/pipermail/u-boot/2013-July/159853.html
(you were also on Cc ...)

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany