[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