ARM: BCM5301X: Fix chip typo in ARCH_BCM_5301X help message
diff mbox

Message ID 1431501645-3887-1-git-send-email-zajec5@gmail.com
State New
Headers show

Commit Message

Rafał Miłecki May 13, 2015, 7:20 a.m. UTC
Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
---
 arch/arm/mach-bcm/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Florian Fainelli May 13, 2015, 5:31 p.m. UTC | #1
On 13/05/15 00:20, Rafał Miłecki wrote:
> Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
> ---
>  arch/arm/mach-bcm/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/mach-bcm/Kconfig b/arch/arm/mach-bcm/Kconfig
> index 8b11f44..7fec7e8 100644
> --- a/arch/arm/mach-bcm/Kconfig
> +++ b/arch/arm/mach-bcm/Kconfig
> @@ -47,7 +47,7 @@ config ARCH_BCM_5301X
>  	  BCM53014, BCM53015, BCM53016, BCM53017, BCM53018, BCM4707,
>  	  BCM4708 and BCM4709.
>  
> -	  Do not confuse this with the BCM4760 which is a totally
> +	  Do not confuse this with the BCM4706 which is a totally
>  	  different SoC or with the older BCM47XX and BCM53XX based
>  	  network SoC using a MIPS CPU, they are supported by arch/mips/bcm47xx

This is not a real typo per-se, there is a 4760 chip (GPS), do you want
to resubmit to include both?
Hauke Mehrtens May 13, 2015, 5:37 p.m. UTC | #2
On 05/13/2015 09:20 AM, Rafał Miłecki wrote:
> Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
> ---
>  arch/arm/mach-bcm/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/mach-bcm/Kconfig b/arch/arm/mach-bcm/Kconfig
> index 8b11f44..7fec7e8 100644
> --- a/arch/arm/mach-bcm/Kconfig
> +++ b/arch/arm/mach-bcm/Kconfig
> @@ -47,7 +47,7 @@ config ARCH_BCM_5301X
>  	  BCM53014, BCM53015, BCM53016, BCM53017, BCM53018, BCM4707,
>  	  BCM4708 and BCM4709.
>  
> -	  Do not confuse this with the BCM4760 which is a totally
> +	  Do not confuse this with the BCM4706 which is a totally
>  	  different SoC or with the older BCM47XX and BCM53XX based
>  	  network SoC using a MIPS CPU, they are supported by arch/mips/bcm47xx
>  
> 
Nack

Like Florian said this is not a typo. There is a BCM4760 SoC and it was
or is supported by Linux.

See this for BCM4760: https://lwn.net/Articles/563957/

I forgot about this at first too. Nice naming Broadcom. (the other
companies are not better).  ;-)

Hauke
Rafał Miłecki May 13, 2015, 6:16 p.m. UTC | #3
On 13 May 2015 at 19:31, Florian Fainelli <f.fainelli@gmail.com> wrote:
> On 13/05/15 00:20, Rafał Miłecki wrote:
>> Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
>> ---
>>  arch/arm/mach-bcm/Kconfig | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/mach-bcm/Kconfig b/arch/arm/mach-bcm/Kconfig
>> index 8b11f44..7fec7e8 100644
>> --- a/arch/arm/mach-bcm/Kconfig
>> +++ b/arch/arm/mach-bcm/Kconfig
>> @@ -47,7 +47,7 @@ config ARCH_BCM_5301X
>>         BCM53014, BCM53015, BCM53016, BCM53017, BCM53018, BCM4707,
>>         BCM4708 and BCM4709.
>>
>> -       Do not confuse this with the BCM4760 which is a totally
>> +       Do not confuse this with the BCM4706 which is a totally
>>         different SoC or with the older BCM47XX and BCM53XX based
>>         network SoC using a MIPS CPU, they are supported by arch/mips/bcm47xx
>
> This is not a real typo per-se, there is a 4760 chip (GPS), do you want
> to resubmit to include both?

Oops, drop it please.

Patch
diff mbox

diff --git a/arch/arm/mach-bcm/Kconfig b/arch/arm/mach-bcm/Kconfig
index 8b11f44..7fec7e8 100644
--- a/arch/arm/mach-bcm/Kconfig
+++ b/arch/arm/mach-bcm/Kconfig
@@ -47,7 +47,7 @@  config ARCH_BCM_5301X
 	  BCM53014, BCM53015, BCM53016, BCM53017, BCM53018, BCM4707,
 	  BCM4708 and BCM4709.
 
-	  Do not confuse this with the BCM4760 which is a totally
+	  Do not confuse this with the BCM4706 which is a totally
 	  different SoC or with the older BCM47XX and BCM53XX based
 	  network SoC using a MIPS CPU, they are supported by arch/mips/bcm47xx