2017-04-19 17:01:48

by Florian Fainelli

Subject: [PATCH] soc: bcm: brcmstb: Correctly match 7435 SoC

Remove the duplicate brcm,bcm7425-sun-top-ctrl compatible string and
replace it with brcm,bcm7435-sun-top-ctrl which was intentend.

Fixes: bd0faf08dc7f ("soc: bcm: brcmstb: Match additional compatible strings")
Reported-by: Andreas Oberritter <[email protected]>
Signed-off-by: Florian Fainelli <[email protected]>
---
drivers/soc/bcm/brcmstb/common.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/soc/bcm/brcmstb/common.c b/drivers/soc/bcm/brcmstb/common.c
index b6195fdf0d00..22e98a90468c 100644
--- a/drivers/soc/bcm/brcmstb/common.c
+++ b/drivers/soc/bcm/brcmstb/common.c
@@ -49,7 +49,7 @@ static const struct of_device_id sun_top_ctrl_match[] = {
{ .compatible = "brcm,bcm7420-sun-top-ctrl", },
{ .compatible = "brcm,bcm7425-sun-top-ctrl", },
{ .compatible = "brcm,bcm7429-sun-top-ctrl", },
- { .compatible = "brcm,bcm7425-sun-top-ctrl", },
+ { .compatible = "brcm,bcm7435-sun-top-ctrl", },
{ .compatible = "brcm,brcmstb-sun-top-ctrl", },
{ }
};
--
2.9.3

2017-04-19 20:28:38

by Gregory Fong

Subject: Re: [PATCH] soc: bcm: brcmstb: Correctly match 7435 SoC

On Wed, Apr 19, 2017 at 10:01 AM, Florian Fainelli <[email protected]> wrote:
> Remove the duplicate brcm,bcm7425-sun-top-ctrl compatible string and
> replace it with brcm,bcm7435-sun-top-ctrl which was intentend.

s/intentend/intended/

>
> Fixes: bd0faf08dc7f ("soc: bcm: brcmstb: Match additional compatible strings")
> Reported-by: Andreas Oberritter <[email protected]>
> Signed-off-by: Florian Fainelli <[email protected]>

Acked-by: Gregory Fong <[email protected]>

2017-04-19 21:24:21

by Florian Fainelli

Subject: Re: [PATCH] soc: bcm: brcmstb: Correctly match 7435 SoC

On 04/19/2017 01:28 PM, Gregory Fong wrote:
> On Wed, Apr 19, 2017 at 10:01 AM, Florian Fainelli <[email protected]> wrote:
>> Remove the duplicate brcm,bcm7425-sun-top-ctrl compatible string and
>> replace it with brcm,bcm7435-sun-top-ctrl which was intentend.
>
> s/intentend/intended/
>
>>
>> Fixes: bd0faf08dc7f ("soc: bcm: brcmstb: Match additional compatible strings")
>> Reported-by: Andreas Oberritter <[email protected]>
>> Signed-off-by: Florian Fainelli <[email protected]>
>
> Acked-by: Gregory Fong <[email protected]>
>

Fixed the typo and applied with your Acked-by, thanks Gregory!
--
Florian