[U-Boot] [PATCH v3 19/26] dm: i2c: Move slave details to child platdata

Simon Glass sjg at chromium.org
Mon Jan 26 21:13:33 CET 2015


On 25 January 2015 at 19:05, Masahiro Yamada <yamada.m at jp.panasonic.com> wrote:
>
> On Sun, 25 Jan 2015 08:27:13 -0700
> Simon Glass <sjg at chromium.org> wrote:
>
>> At present we go through various contortions to store the I2C's chip
>> address in its private data. This only exists when the chip is active so
>> must be set up when it is probed. Until the device is probed we don't
>> actually record what address it will appear on.
>>
>> However, now that we can support per-child platform data, we can use that
>> instead. This allows us to set up the address when the child is bound,
>> and avoid the messy contortions.
>>
>> Unfortunately this is a fairly large change and it seems to be difficult to
>> break it down further.
>>
>> Signed-off-by: Simon Glass <sjg at chromium.org>
>> ---
>>
>> Changes in v3:
>> - Add missing 'static' to two functions
>> - Remove unnecessary per_child_auto_alloc_size value
>
>
> I tested this series on my board.
> I am OK with the change in the UniPhier I2C driver area.
>
>
> Reviewed-by: Masahiro Yamada <yamada.m at jp.panasonic.com>
>

Applied to -u-boot-dm


More information about the U-Boot mailing list