[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