Re: [PATCH 2/2] i2c: busses with dynamic ids should start after fixed ids for DT

From: Wolfram Sang
Date: Wed Mar 18 2015 - 08:46:36 EST


On Thu, Mar 12, 2015 at 05:17:59PM +0100, Wolfram Sang wrote:
> Make sure dynamic ids do not interfere with fixed ones and let them
> start after the highest fixed id. This patch might cause different
> bus-numbers for dynamic ids, however it fixes a bug. Assume:
>
> - fixed id0 defers probe
> - fixed id1 succeeds and registers a muxed bus with dynamic id
> - muxed bus gets id0
> - fixed id0 wants to probe again, but its fixed id is gone now
> - fixed id0 probe fails
>
> With this patch, the fixed ids are always reserved in the DT case.
> For legacy board init, we already have a mechanism like this in
> i2c_register_board_info().
>
> Reported-by: Bob Feretich <bob.feretich@xxxxxxxxxxxxxxx>
> Signed-off-by: Wolfram Sang <wsa@xxxxxxxxxxxxx>

Fixed the checkpatch warning and applied to for-next, thanks!

Attachment: signature.asc
Description: Digital signature