[PATCH linux] ARM: dts: aspeed: quanta: add i2cmux for frus

Patrick Venture venture at google.com
Sat Dec 9 02:49:02 AEDT 2017


Thanks.  I added the @1 from reading another device-tree that used
this not super commonly used node.  But your explanation makes perfect
sense.

Patrick

On Thu, Dec 7, 2017 at 10:32 PM, Joel Stanley <joel at jms.id.au> wrote:
> On Tue, Nov 28, 2017 at 6:43 AM, Patrick Venture <venture at google.com> wrote:
>> The memory riser frus on the board are controlled
>> by a gpio mux.
>>
>> Signed-off-by: Patrick Venture <venture at google.com>
>> ---
>>  arch/arm/boot/dts/aspeed-bmc-quanta-q71l.dts | 93 +++++++++++++++++++++-------
>>  1 file changed, 69 insertions(+), 24 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/aspeed-bmc-quanta-q71l.dts b/arch/arm/boot/dts/aspeed-bmc-quanta-q71l.dts
>> index 42b1d8007316..b93b29191960 100644
>> --- a/arch/arm/boot/dts/aspeed-bmc-quanta-q71l.dts
>> +++ b/arch/arm/boot/dts/aspeed-bmc-quanta-q71l.dts
>> @@ -119,6 +119,75 @@
>>                         <&adc 4>, <&adc 5>, <&adc 6>, <&adc 7>,
>>                         <&adc 8>, <&adc 9>, <&adc 10>, <&adc 11>;
>>         };
>> +
>> +       i2cmux at 1 {
>
> When you stick a unit name on a node, it is supposed to have a
> matching reg property. We don't have a reg property here, so we
> shouldn't have the number. This matches what the i2c-mux-gpio bindings
> describe.
>
> I dropped the '@1' for you when applying. It's in dev-4.10 now.
>
> Cheers,
>
> Joel
>
>> +               compatible = "i2c-mux-gpio";
>> +               #address-cells = <1>;
>> +               #size-cells = <0>;


More information about the openbmc mailing list