[U-Boot] [PATCH v3 8/9] tegra: i2c: Enable new CONFIG_SYS_I2Cframework
Heiko Schocher
hs at denx.de
Wed May 8 06:11:33 CEST 2013
Hello Stephen,
Am 07.05.2013 16:55, schrieb Stephen Warren:
> On 05/07/2013 02:01 AM, Marc Dietrich wrote:
>> Hi,
>>
>> Am Montag, 6. Mai 2013, 13:08:31 schrieb Stephen Warren:
>>> On 05/04/2013 06:01 AM, Heiko Schocher wrote:
>>>> From: Simon Glass <sjg at chromium.org>
>>>>
>>>> This enables CONFIG_SYS_I2C on Tegra, updating existing boards and the
>>>> Tegra i2c driver to support this.
>>>>
>>>> include/configs/beaver.h | 5 ++-
>>>> include/configs/cardhu.h | 3 +-
>>>> include/configs/dalmore.h | 3 +-
>>>> include/configs/seaboard.h | 5 ++-
>>>> include/configs/trimslice.h | 5 ++-
>>>> include/configs/whistler.h | 5 ++-
>>>
>>> There are a lot more Tegra boards than just those. Shouldn't they all be
>>> updated? You also didn't Cc the Tegra maintainer - I have done on this mail.
>>
>> not all boards use I2C up to now, to the patch seems to change only the boards
>> which do. So this is ok for now. The question is if CONFIG_SYS_I2C could be
>> moved to tegra_common or so because the device-tree disables I2C be default.
>
> Yes, we should do that for all the devices that are now
> instantiated/configured using device tree. But, we can do that as a
> separate patch series after this.
Yes, that makes sense.
bye,
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