2022-07-08 20:13:37

by Florian Fainelli

[permalink] [raw]
Subject: Re: [PATCH v2 1/2] arm64: dts: broadcom: bcm4908: Fix timer node for BCM4906 SoC

On Fri, 8 Jul 2022 11:25:06 -0700, William Zhang <[email protected]> wrote:
> The cpu mask value in interrupt property inherits from bcm4908.dtsi
> which sets to four cpus. Correct the value to two cpus for dual core
> BCM4906 SoC.
>
> Fixes: c8b404fb05dc ("arm64: dts: broadcom: bcm4908: add BCM4906
> Netgear R8000P DTS files")
>
> Signed-off-by: William Zhang <[email protected]>
> Reviewed-by: Florian Fainelli <[email protected]>
>
> ---

Applied to https://github.com/Broadcom/stblinux/commits/devicetree-arm64/fixes, thanks!
--
Florian

2022-07-13 07:22:00

by Rafał Miłecki

[permalink] [raw]
Subject: Re: [PATCH v2 1/2] arm64: dts: broadcom: bcm4908: Fix timer node for BCM4906 SoC

On 2022-07-08 20:25, William Zhang wrote:
> The cpu mask value in interrupt property inherits from bcm4908.dtsi
> which sets to four cpus. Correct the value to two cpus for dual core
> BCM4906 SoC.
>
> Fixes: c8b404fb05dc ("arm64: dts: broadcom: bcm4908: add BCM4906
> Netgear R8000P DTS files")

Well, one note. You shouldn't line break Fixes: line.


> Signed-off-by: William Zhang <[email protected]>
> Reviewed-by: Florian Fainelli <[email protected]>

2022-07-13 07:24:00

by Rafał Miłecki

[permalink] [raw]
Subject: Re: [PATCH v2 1/2] arm64: dts: broadcom: bcm4908: Fix timer node for BCM4906 SoC

On 2022-07-08 20:25, William Zhang wrote:
> The cpu mask value in interrupt property inherits from bcm4908.dtsi
> which sets to four cpus. Correct the value to two cpus for dual core
> BCM4906 SoC.
>
> Fixes: c8b404fb05dc ("arm64: dts: broadcom: bcm4908: add BCM4906
> Netgear R8000P DTS files")
>
> Signed-off-by: William Zhang <[email protected]>
> Reviewed-by: Florian Fainelli <[email protected]>

Both look good, thanks!

2022-07-13 16:41:09

by Florian Fainelli

[permalink] [raw]
Subject: Re: [PATCH v2 1/2] arm64: dts: broadcom: bcm4908: Fix timer node for BCM4906 SoC

On 7/12/22 23:49, Rafał Miłecki wrote:
> On 2022-07-08 20:25, William Zhang wrote:
>> The cpu mask value in interrupt property inherits from bcm4908.dtsi
>> which sets to four cpus. Correct the value to two cpus for dual core
>> BCM4906 SoC.
>>
>> Fixes: c8b404fb05dc ("arm64: dts: broadcom: bcm4908: add BCM4906
>> Netgear R8000P DTS files")
>
> Well, one note. You shouldn't line break Fixes: line.

Took care of that before applying the two patches:

https://github.com/torvalds/linux/commit/b4a544e415e9be33b37d9bfa9d9f9f4d13f553d6

https://github.com/torvalds/linux/commit/8bd582ae9a71d7f14c4e0c735b2eacaf7516d626
--
Florian