Re: [PATCH linux dev-4.10] ARM: dts: aspeed: Add FSI devices for Zaius

[Xo Wang <xow@google.com>]

On Wed, Jul 12, 2017 at 6:04 PM, Joel Stanley <joel@jms.id.au> wrote:
> On Thu, Jul 13, 2017 at 5:36 AM, Eddie James <eajames@linux.vnet.ibm.com> wrote:
>> From: "Edward A. James" <eajames@us.ibm.com>
>>
>> Missing label, properties, and include file.
>>
>> Signed-off-by: Edward A. James <eajames@us.ibm.com>
>
> Hey Xo, can I get an ack on this?
>
> Cheers,
>
> Joel
>

Ah, missed this.

Thanks.

Acked-by: Xo Wang <xow@google.com>

>> ---
>>  arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts | 8 +++++---
>>  1 file changed, 5 insertions(+), 3 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts b/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
>> index c620015..750ec12 100644
>> --- a/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
>> +++ b/arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts
>> @@ -71,10 +71,10 @@
>>                 };
>>         };
>>
>> -       gpio-fsi {
>> +       fsi: gpio-fsi {
>>                 compatible = "fsi-master-gpio", "fsi-master";
>> -
>> -               status = "okay";
>> +               #address-cells = <2>;
>> +               #size-cells = <0>;
>>
>>                 /*
>>                  * These 2 GPIOs are set per board revision in aspeed.c.
>> @@ -96,6 +96,8 @@
>>         };
>>  };
>>
>> +#include "ibm-power9-cfam.dtsi"
>> +
>>  &fmc {
>>         status = "okay";
>>         flash@0 {
>> --
>> 1.8.3.1
>>