* [PATCH 2/3] ARM: dts: aspeed: amd-ethanolx: Enable KCS channel 3
2020-10-27 12:37 [PATCH 1/3] ARM: dts: aspeed: amd-ethanolx: Update KCS nodes to use v2 binding Konstantin Aladyshev
@ 2020-10-27 12:37 ` Konstantin Aladyshev
2020-10-28 5:28 ` Joel Stanley
2020-10-27 12:37 ` [PATCH 3/3] ARM: dts: aspeed: amd-ethanolx: Enable devices for the iKVM functionality Konstantin Aladyshev
` (2 subsequent siblings)
3 siblings, 1 reply; 12+ messages in thread
From: Konstantin Aladyshev @ 2020-10-27 12:37 UTC (permalink / raw)
Cc: supreeth.venkatesh, aladyshev22, Rob Herring, Joel Stanley,
Andrew Jeffery, devicetree, linux-arm-kernel, linux-aspeed,
linux-kernel
The KCS interface on the LPC channel 3 in the controller
is used for the in-band BMC<->BIOS IPMI communication.
0xCA2 is a default host CPU LPC IO address for this
interface.
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
---
arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
index 89ddc3847222..2a86bda8afd8 100644
--- a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
@@ -147,6 +147,11 @@
aspeed,lpc-io-reg = <0x62>;
};
+&kcs3 {
+ status = "okay";
+ aspeed,lpc-io-reg = <0xCA2>;
+};
+
&kcs4 {
status = "okay";
aspeed,lpc-io-reg = <0x97DE>;
--
2.17.1
^ permalink raw reply related [flat|nested] 12+ messages in thread* Re: [PATCH 2/3] ARM: dts: aspeed: amd-ethanolx: Enable KCS channel 3
2020-10-27 12:37 ` [PATCH 2/3] ARM: dts: aspeed: amd-ethanolx: Enable KCS channel 3 Konstantin Aladyshev
@ 2020-10-28 5:28 ` Joel Stanley
2020-10-28 15:59 ` Supreeth Venkatesh
0 siblings, 1 reply; 12+ messages in thread
From: Joel Stanley @ 2020-10-28 5:28 UTC (permalink / raw)
To: Konstantin Aladyshev
Cc: Supreeth Venkatesh, Rob Herring, Andrew Jeffery, devicetree,
Linux ARM, linux-aspeed, Linux Kernel Mailing List
On Tue, 27 Oct 2020 at 12:41, Konstantin Aladyshev
<aladyshev22@gmail.com> wrote:
>
> The KCS interface on the LPC channel 3 in the controller
> is used for the in-band BMC<->BIOS IPMI communication.
> 0xCA2 is a default host CPU LPC IO address for this
> interface.
>
> Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
I don't have any docs on the platform so I'll wait for a review from
Supreeth before applying this one.
It's a correct use of the bindings:
Reviewed-by: Joel Stanley <joel@jms.id.au>
> ---
> arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> index 89ddc3847222..2a86bda8afd8 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> @@ -147,6 +147,11 @@
> aspeed,lpc-io-reg = <0x62>;
> };
>
> +&kcs3 {
> + status = "okay";
> + aspeed,lpc-io-reg = <0xCA2>;
> +};
> +
> &kcs4 {
> status = "okay";
> aspeed,lpc-io-reg = <0x97DE>;
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PATCH 2/3] ARM: dts: aspeed: amd-ethanolx: Enable KCS channel 3
2020-10-28 5:28 ` Joel Stanley
@ 2020-10-28 15:59 ` Supreeth Venkatesh
0 siblings, 0 replies; 12+ messages in thread
From: Supreeth Venkatesh @ 2020-10-28 15:59 UTC (permalink / raw)
To: Joel Stanley, Konstantin Aladyshev
Cc: Rob Herring, Andrew Jeffery, devicetree, Linux ARM, linux-aspeed,
Linux Kernel Mailing List
Thanks Konstantin for this patch.
Thanks Joel for reviewing this.
On 10/28/20 12:28 AM, Joel Stanley wrote:
> [CAUTION: External Email]
>
> On Tue, 27 Oct 2020 at 12:41, Konstantin Aladyshev
> <aladyshev22@gmail.com> wrote:
>>
>> The KCS interface on the LPC channel 3 in the controller
>> is used for the in-band BMC<->BIOS IPMI communication.
>> 0xCA2 is a default host CPU LPC IO address for this
>> interface.
>>
>> Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
>
> I don't have any docs on the platform so I'll wait for a review from
> Supreeth before applying this one.
Done.
>
> It's a correct use of the bindings:
>
> Reviewed-by: Joel Stanley <joel@jms.id.au>
Reviewed-by: Supreeth Venkatesh <supreeth.venkatesh@amd.com>
>
>> ---
>> arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts | 5 +++++
>> 1 file changed, 5 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
>> index 89ddc3847222..2a86bda8afd8 100644
>> --- a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
>> +++ b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
>> @@ -147,6 +147,11 @@
>> aspeed,lpc-io-reg = <0x62>;
>> };
>>
>> +&kcs3 {
>> + status = "okay";
>> + aspeed,lpc-io-reg = <0xCA2>;
>> +};
>> +
>> &kcs4 {
>> status = "okay";
>> aspeed,lpc-io-reg = <0x97DE>;
>> --
>> 2.17.1
>>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 3/3] ARM: dts: aspeed: amd-ethanolx: Enable devices for the iKVM functionality
2020-10-27 12:37 [PATCH 1/3] ARM: dts: aspeed: amd-ethanolx: Update KCS nodes to use v2 binding Konstantin Aladyshev
2020-10-27 12:37 ` [PATCH 2/3] ARM: dts: aspeed: amd-ethanolx: Enable KCS channel 3 Konstantin Aladyshev
@ 2020-10-27 12:37 ` Konstantin Aladyshev
2020-10-27 14:02 ` Supreeth Venkatesh
2020-10-28 5:28 ` Joel Stanley
2020-10-27 14:09 ` [PATCH 1/3] ARM: dts: aspeed: amd-ethanolx: Update KCS nodes to use v2 binding Supreeth Venkatesh
2020-10-28 5:25 ` Joel Stanley
3 siblings, 2 replies; 12+ messages in thread
From: Konstantin Aladyshev @ 2020-10-27 12:37 UTC (permalink / raw)
Cc: supreeth.venkatesh, aladyshev22, Rob Herring, Joel Stanley,
Andrew Jeffery, devicetree, linux-arm-kernel, linux-aspeed,
linux-kernel
Enable the USB 2.0 Virtual Hub Controller and
the Video Engine with it's reserved memory region for the implementation
of the iKVM functionality in the BMC.
Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
---
arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts | 22 +++++++++++++++++++
1 file changed, 22 insertions(+)
diff --git a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
index 2a86bda8afd8..b93ed44eba0c 100644
--- a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
@@ -13,6 +13,21 @@
memory@80000000 {
reg = <0x80000000 0x20000000>;
};
+
+ reserved-memory {
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges;
+
+ video_engine_memory: jpegbuffer {
+ size = <0x02000000>; /* 32M */
+ alignment = <0x01000000>;
+ compatible = "shared-dma-pool";
+ reusable;
+ };
+ };
+
+
aliases {
serial0 = &uart1;
serial4 = &uart5;
@@ -220,5 +235,12 @@
};
};
+&video {
+ status = "okay";
+ memory-region = <&video_engine_memory>;
+};
+&vhub {
+ status = "okay";
+};
--
2.17.1
^ permalink raw reply related [flat|nested] 12+ messages in thread* Re: [PATCH 3/3] ARM: dts: aspeed: amd-ethanolx: Enable devices for the iKVM functionality
2020-10-27 12:37 ` [PATCH 3/3] ARM: dts: aspeed: amd-ethanolx: Enable devices for the iKVM functionality Konstantin Aladyshev
@ 2020-10-27 14:02 ` Supreeth Venkatesh
2020-10-28 5:28 ` Joel Stanley
1 sibling, 0 replies; 12+ messages in thread
From: Supreeth Venkatesh @ 2020-10-27 14:02 UTC (permalink / raw)
To: Konstantin Aladyshev
Cc: Rob Herring, Joel Stanley, Andrew Jeffery, devicetree,
linux-arm-kernel, linux-aspeed, linux-kernel
On 10/27/20 7:37 AM, Konstantin Aladyshev wrote:
> [CAUTION: External Email]
>
> Enable the USB 2.0 Virtual Hub Controller and
> the Video Engine with it's reserved memory region for the implementation
> of the iKVM functionality in the BMC.
>
> Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
Reviewed-by: Supreeth Venkatesh <supreeth.venkatesh@amd.com>
> ---
> arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts | 22 +++++++++++++++++++
> 1 file changed, 22 insertions(+)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> index 2a86bda8afd8..b93ed44eba0c 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> @@ -13,6 +13,21 @@
> memory@80000000 {
> reg = <0x80000000 0x20000000>;
> };
> +
> + reserved-memory {
> + #address-cells = <1>;
> + #size-cells = <1>;
> + ranges;
> +
> + video_engine_memory: jpegbuffer {
> + size = <0x02000000>; /* 32M */
> + alignment = <0x01000000>;
> + compatible = "shared-dma-pool";
> + reusable;
> + };
> + };
> +
> +
> aliases {
> serial0 = &uart1;
> serial4 = &uart5;
> @@ -220,5 +235,12 @@
> };
> };
>
> +&video {
> + status = "okay";
> + memory-region = <&video_engine_memory>;
> +};
>
> +&vhub {
> + status = "okay";
> +};
>
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PATCH 3/3] ARM: dts: aspeed: amd-ethanolx: Enable devices for the iKVM functionality
2020-10-27 12:37 ` [PATCH 3/3] ARM: dts: aspeed: amd-ethanolx: Enable devices for the iKVM functionality Konstantin Aladyshev
2020-10-27 14:02 ` Supreeth Venkatesh
@ 2020-10-28 5:28 ` Joel Stanley
1 sibling, 0 replies; 12+ messages in thread
From: Joel Stanley @ 2020-10-28 5:28 UTC (permalink / raw)
To: Konstantin Aladyshev
Cc: Supreeth Venkatesh, Rob Herring, Andrew Jeffery, devicetree,
Linux ARM, linux-aspeed, Linux Kernel Mailing List
On Tue, 27 Oct 2020 at 12:41, Konstantin Aladyshev
<aladyshev22@gmail.com> wrote:
>
> Enable the USB 2.0 Virtual Hub Controller and
> the Video Engine with it's reserved memory region for the implementation
> of the iKVM functionality in the BMC.
>
> Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
Reviewed-by: Joel Stanley <joel@jms.id.au>
> ---
> arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts | 22 +++++++++++++++++++
> 1 file changed, 22 insertions(+)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> index 2a86bda8afd8..b93ed44eba0c 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> @@ -13,6 +13,21 @@
> memory@80000000 {
> reg = <0x80000000 0x20000000>;
> };
> +
> + reserved-memory {
> + #address-cells = <1>;
> + #size-cells = <1>;
> + ranges;
> +
> + video_engine_memory: jpegbuffer {
> + size = <0x02000000>; /* 32M */
> + alignment = <0x01000000>;
> + compatible = "shared-dma-pool";
> + reusable;
> + };
> + };
> +
> +
> aliases {
> serial0 = &uart1;
> serial4 = &uart5;
> @@ -220,5 +235,12 @@
> };
> };
>
> +&video {
> + status = "okay";
> + memory-region = <&video_engine_memory>;
> +};
>
> +&vhub {
> + status = "okay";
> +};
>
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/3] ARM: dts: aspeed: amd-ethanolx: Update KCS nodes to use v2 binding
2020-10-27 12:37 [PATCH 1/3] ARM: dts: aspeed: amd-ethanolx: Update KCS nodes to use v2 binding Konstantin Aladyshev
2020-10-27 12:37 ` [PATCH 2/3] ARM: dts: aspeed: amd-ethanolx: Enable KCS channel 3 Konstantin Aladyshev
2020-10-27 12:37 ` [PATCH 3/3] ARM: dts: aspeed: amd-ethanolx: Enable devices for the iKVM functionality Konstantin Aladyshev
@ 2020-10-27 14:09 ` Supreeth Venkatesh
2020-10-27 21:00 ` Konstantin Aladyshev
2020-10-28 9:50 ` Joel Stanley
2020-10-28 5:25 ` Joel Stanley
3 siblings, 2 replies; 12+ messages in thread
From: Supreeth Venkatesh @ 2020-10-27 14:09 UTC (permalink / raw)
To: Konstantin Aladyshev
Cc: Rob Herring, Joel Stanley, Andrew Jeffery, devicetree,
linux-arm-kernel, linux-aspeed, linux-kernel
These changes are already merged in https://github.com/openbmc/linux/blob/dev-5.8/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts by Joel.
Please abandon these changes.
Thanks,
Supreeth
On 10/27/20 7:37 AM, Konstantin Aladyshev wrote:
> [CAUTION: External Email]
>
> KCS nodes compatible property in the 'aspeed-g5.dtsi' file was
> changed to use v2 binding in the commit fa4c8ec6feaa
> ("ARM: dts: aspeed: Change KCS nodes to v2 binding").
> For the proper initialization of /dev/ipmi-kcs* devices
> KCS node variables also need to be changed to use v2 binding.
>
> Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
> ---
> arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> index 60ba86f3e5bc..89ddc3847222 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> @@ -139,17 +139,17 @@
>
> &kcs1 {
> status = "okay";
> - kcs_addr = <0x60>;
> + aspeed,lpc-io-reg = <0x60>;
> };
>
> &kcs2 {
> status = "okay";
> - kcs_addr = <0x62>;
> + aspeed,lpc-io-reg = <0x62>;
> };
>
> &kcs4 {
> status = "okay";
> - kcs_addr = <0x97DE>;
> + aspeed,lpc-io-reg = <0x97DE>;
> };
>
> &lpc_snoop {
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PATCH 1/3] ARM: dts: aspeed: amd-ethanolx: Update KCS nodes to use v2 binding
2020-10-27 14:09 ` [PATCH 1/3] ARM: dts: aspeed: amd-ethanolx: Update KCS nodes to use v2 binding Supreeth Venkatesh
@ 2020-10-27 21:00 ` Konstantin Aladyshev
2020-10-28 9:50 ` Joel Stanley
1 sibling, 0 replies; 12+ messages in thread
From: Konstantin Aladyshev @ 2020-10-27 21:00 UTC (permalink / raw)
To: Supreeth Venkatesh
Cc: Rob Herring, Joel Stanley, Andrew Jeffery, devicetree,
linux-arm-kernel, linux-aspeed, linux-kernel
These changes got picked up from my backport patch to the 'dev-5.8'
branch of the OpenBMC Linux repository.
This patch also needed to be added to the main ASPEED Linux kernel, so
it could get into subsequent branches of the OpenBMC Linux repository.
Best regards,
Konstantin Aladyshev
On Tue, Oct 27, 2020 at 5:10 PM Supreeth Venkatesh
<supreeth.venkatesh@amd.com> wrote:
>
> These changes are already merged in https://github.com/openbmc/linux/blob/dev-5.8/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts by Joel.
> Please abandon these changes.
>
> Thanks,
> Supreeth
>
> On 10/27/20 7:37 AM, Konstantin Aladyshev wrote:
> > [CAUTION: External Email]
> >
> > KCS nodes compatible property in the 'aspeed-g5.dtsi' file was
> > changed to use v2 binding in the commit fa4c8ec6feaa
> > ("ARM: dts: aspeed: Change KCS nodes to v2 binding").
> > For the proper initialization of /dev/ipmi-kcs* devices
> > KCS node variables also need to be changed to use v2 binding.
> >
> > Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
> > ---
> > arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> > index 60ba86f3e5bc..89ddc3847222 100644
> > --- a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> > +++ b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> > @@ -139,17 +139,17 @@
> >
> > &kcs1 {
> > status = "okay";
> > - kcs_addr = <0x60>;
> > + aspeed,lpc-io-reg = <0x60>;
> > };
> >
> > &kcs2 {
> > status = "okay";
> > - kcs_addr = <0x62>;
> > + aspeed,lpc-io-reg = <0x62>;
> > };
> >
> > &kcs4 {
> > status = "okay";
> > - kcs_addr = <0x97DE>;
> > + aspeed,lpc-io-reg = <0x97DE>;
> > };
> >
> > &lpc_snoop {
> > --
> > 2.17.1
> >
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PATCH 1/3] ARM: dts: aspeed: amd-ethanolx: Update KCS nodes to use v2 binding
2020-10-27 14:09 ` [PATCH 1/3] ARM: dts: aspeed: amd-ethanolx: Update KCS nodes to use v2 binding Supreeth Venkatesh
2020-10-27 21:00 ` Konstantin Aladyshev
@ 2020-10-28 9:50 ` Joel Stanley
1 sibling, 0 replies; 12+ messages in thread
From: Joel Stanley @ 2020-10-28 9:50 UTC (permalink / raw)
To: Supreeth Venkatesh
Cc: Konstantin Aladyshev, Rob Herring, Andrew Jeffery, devicetree,
Linux ARM, linux-aspeed, Linux Kernel Mailing List
On Wed, 28 Oct 2020 at 09:00, Supreeth Venkatesh
<supreeth.venkatesh@amd.com> wrote:
>
> These changes are already merged in https://github.com/openbmc/linux/blob/dev-5.8/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts by Joel.
> Please abandon these changes.
That is the openbmc kernel tree, where we stage patches on their way
to mainline. This submission is for mainline, where this patch has not
been merged.
Cheers,
Joel
>
> Thanks,
> Supreeth
>
> On 10/27/20 7:37 AM, Konstantin Aladyshev wrote:
> > [CAUTION: External Email]
> >
> > KCS nodes compatible property in the 'aspeed-g5.dtsi' file was
> > changed to use v2 binding in the commit fa4c8ec6feaa
> > ("ARM: dts: aspeed: Change KCS nodes to v2 binding").
> > For the proper initialization of /dev/ipmi-kcs* devices
> > KCS node variables also need to be changed to use v2 binding.
> >
> > Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
> > ---
> > arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> > index 60ba86f3e5bc..89ddc3847222 100644
> > --- a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> > +++ b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> > @@ -139,17 +139,17 @@
> >
> > &kcs1 {
> > status = "okay";
> > - kcs_addr = <0x60>;
> > + aspeed,lpc-io-reg = <0x60>;
> > };
> >
> > &kcs2 {
> > status = "okay";
> > - kcs_addr = <0x62>;
> > + aspeed,lpc-io-reg = <0x62>;
> > };
> >
> > &kcs4 {
> > status = "okay";
> > - kcs_addr = <0x97DE>;
> > + aspeed,lpc-io-reg = <0x97DE>;
> > };
> >
> > &lpc_snoop {
> > --
> > 2.17.1
> >
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/3] ARM: dts: aspeed: amd-ethanolx: Update KCS nodes to use v2 binding
2020-10-27 12:37 [PATCH 1/3] ARM: dts: aspeed: amd-ethanolx: Update KCS nodes to use v2 binding Konstantin Aladyshev
` (2 preceding siblings ...)
2020-10-27 14:09 ` [PATCH 1/3] ARM: dts: aspeed: amd-ethanolx: Update KCS nodes to use v2 binding Supreeth Venkatesh
@ 2020-10-28 5:25 ` Joel Stanley
2020-10-28 5:32 ` Joel Stanley
3 siblings, 1 reply; 12+ messages in thread
From: Joel Stanley @ 2020-10-28 5:25 UTC (permalink / raw)
To: Konstantin Aladyshev
Cc: Supreeth Venkatesh, Rob Herring, Andrew Jeffery, devicetree,
Linux ARM, linux-aspeed, Linux Kernel Mailing List
On Tue, 27 Oct 2020 at 12:38, Konstantin Aladyshev
<aladyshev22@gmail.com> wrote:
>
> KCS nodes compatible property in the 'aspeed-g5.dtsi' file was
> changed to use v2 binding in the commit fa4c8ec6feaa
> ("ARM: dts: aspeed: Change KCS nodes to v2 binding").
> For the proper initialization of /dev/ipmi-kcs* devices
> KCS node variables also need to be changed to use v2 binding.
>
> Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
Fixes: 09f5f680707e ("ipmi: kcs: aspeed: Implement v2 bindings")
Reviewed-by: Joel Stanley <joel@jms.id.au>
> ---
> arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> index 60ba86f3e5bc..89ddc3847222 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> @@ -139,17 +139,17 @@
>
> &kcs1 {
> status = "okay";
> - kcs_addr = <0x60>;
> + aspeed,lpc-io-reg = <0x60>;
> };
>
> &kcs2 {
> status = "okay";
> - kcs_addr = <0x62>;
> + aspeed,lpc-io-reg = <0x62>;
> };
>
> &kcs4 {
> status = "okay";
> - kcs_addr = <0x97DE>;
> + aspeed,lpc-io-reg = <0x97DE>;
> };
>
> &lpc_snoop {
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 12+ messages in thread* Re: [PATCH 1/3] ARM: dts: aspeed: amd-ethanolx: Update KCS nodes to use v2 binding
2020-10-28 5:25 ` Joel Stanley
@ 2020-10-28 5:32 ` Joel Stanley
0 siblings, 0 replies; 12+ messages in thread
From: Joel Stanley @ 2020-10-28 5:32 UTC (permalink / raw)
To: Konstantin Aladyshev
Cc: Supreeth Venkatesh, Rob Herring, Andrew Jeffery, devicetree,
Linux ARM, linux-aspeed, Linux Kernel Mailing List
On Wed, 28 Oct 2020 at 05:25, Joel Stanley <joel@jms.id.au> wrote:
>
> On Tue, 27 Oct 2020 at 12:38, Konstantin Aladyshev
> <aladyshev22@gmail.com> wrote:
> >
> > KCS nodes compatible property in the 'aspeed-g5.dtsi' file was
> > changed to use v2 binding in the commit fa4c8ec6feaa
> > ("ARM: dts: aspeed: Change KCS nodes to v2 binding").
> > For the proper initialization of /dev/ipmi-kcs* devices
> > KCS node variables also need to be changed to use v2 binding.
> >
> > Signed-off-by: Konstantin Aladyshev <aladyshev22@gmail.com>
>
> Fixes: 09f5f680707e ("ipmi: kcs: aspeed: Implement v2 bindings")
Actually, this is probably better:
Fixes: fa4c8ec6feaa ("ARM: dts: aspeed: Change KCS nodes to v2 binding")
I slightly reworked your commit message when applying.
> Reviewed-by: Joel Stanley <joel@jms.id.au>
>
> > ---
> > arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> > index 60ba86f3e5bc..89ddc3847222 100644
> > --- a/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> > +++ b/arch/arm/boot/dts/aspeed-bmc-amd-ethanolx.dts
> > @@ -139,17 +139,17 @@
> >
> > &kcs1 {
> > status = "okay";
> > - kcs_addr = <0x60>;
> > + aspeed,lpc-io-reg = <0x60>;
> > };
> >
> > &kcs2 {
> > status = "okay";
> > - kcs_addr = <0x62>;
> > + aspeed,lpc-io-reg = <0x62>;
> > };
> >
> > &kcs4 {
> > status = "okay";
> > - kcs_addr = <0x97DE>;
> > + aspeed,lpc-io-reg = <0x97DE>;
> > };
> >
> > &lpc_snoop {
> > --
> > 2.17.1
> >
^ permalink raw reply [flat|nested] 12+ messages in thread