[PATCH linux 2/2] ARM: dts: aspeed: zaius: Add I2C master selector and hotswap controller

Joel Stanley joel at jms.id.au
Fri Nov 18 13:51:40 AEDT 2016


On Fri, Nov 18, 2016 at 8:23 AM, Xo Wang <xow at google.com> wrote:
> Hi Joel,
>
> On Wed, Nov 16, 2016 at 8:59 PM, Joel Stanley <joel at jms.id.au> wrote:
>> Hi Xo,
>>
>> On Thu, Nov 17, 2016 at 9:59 AM, Xo Wang <xow at google.com> wrote:
>>> Add device-tree bindings to instantiate the PCA9541A I2C master
>>> selector. This driver allows mastering downstream devices after sending
>>> an arbitration request to the PCA9541A. Downstream devices go into the
>>> 'i2c-arb' child node.
>>
>> I took a look at the bindings document for i2c-mux[1] and it suggests
>> something different.
>>
>> Is there a reason why you didn't follow what it says?
>>
>> [1] https://www.kernel.org/doc/Documentation/devicetree/bindings/i2c/i2c-mux.txt
>>
>
> I was following the I2C arbitration bus doc, which has its i2c slave
> device nodes in an "i2c-arb" child node. I'm not sure why it was
> written differently from "i2c-mux" bindings.
>
> To be clear, I haven't 100% tested the LM5066I instantiation since the
> motherboard I have actually has an ADI ADM1272 at address 0x10
> downstream of the NXP PCA9541A.
>
> https://www.kernel.org/doc/Documentation/devicetree/bindings/i2c/i2c-arb.txt
>

Okay. Thanks for the explanation. I have merged both patches into dev-4.7.

Cheers,

Joel


More information about the openbmc mailing list