| Message ID | 20220713145340.10599-1-zajec5@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | arm64: bcmbca: include full family name in Kconfig | expand |
On 7/13/22 07:53, Rafał Miłecki wrote: > From: Rafał Miłecki <rafal@milecki.pl> > > It helps to understand that "magic" BCMBCA acronym. The full name was > found in Florian's pull request for 5.20. > > Signed-off-by: Rafał Miłecki <rafal@milecki.pl> This looks fine to me, can we get an arch/arm/mach-bcm/Kconfig equivalent as well?
On 7/13/22 07:53, Rafał Miłecki wrote: > From: Rafał Miłecki <rafal@milecki.pl> > > It helps to understand that "magic" BCMBCA acronym. The full name was > found in Florian's pull request for 5.20. > > Signed-off-by: Rafał Miłecki <rafal@milecki.pl> > --- > arch/arm64/Kconfig.platforms | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms > index 83e3943f7a55..2259d3732cc4 100644 > --- a/arch/arm64/Kconfig.platforms > +++ b/arch/arm64/Kconfig.platforms > @@ -64,7 +64,7 @@ config ARCH_BCM_IPROC > This enables support for Broadcom iProc based SoCs > > config ARCH_BCMBCA > - bool "Broadcom Broadband SoC" > + bool "Broadcom Broadband Carrier Access (BCA) SoC" > help > Say Y if you intend to run the kernel on a Broadcom Broadband ARM-based > BCA chipset. It is bit complicated about the history and its chips of the BCA group.. Now it is part of Broadband Video Group as known as BVG. WLAN chip is no longer belong to BCA or BVG while we still support its software release. So I would rather keep it in magic term ;) But if we really want to spell out the full name, we suggest to use "Broadcom Broadband Carrier Access (BCA) origin SoC". Same apply to Kconfig for arm32 platform.
On 7/13/22 13:47, William Zhang wrote: > > > On 7/13/22 07:53, Rafał Miłecki wrote: >> From: Rafał Miłecki <rafal@milecki.pl> >> >> It helps to understand that "magic" BCMBCA acronym. The full name was >> found in Florian's pull request for 5.20. >> >> Signed-off-by: Rafał Miłecki <rafal@milecki.pl> >> --- >> arch/arm64/Kconfig.platforms | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms >> index 83e3943f7a55..2259d3732cc4 100644 >> --- a/arch/arm64/Kconfig.platforms >> +++ b/arch/arm64/Kconfig.platforms >> @@ -64,7 +64,7 @@ config ARCH_BCM_IPROC >> This enables support for Broadcom iProc based SoCs >> config ARCH_BCMBCA >> - bool "Broadcom Broadband SoC" >> + bool "Broadcom Broadband Carrier Access (BCA) SoC" >> help >> Say Y if you intend to run the kernel on a Broadcom Broadband >> ARM-based >> BCA chipset. > It is bit complicated about the history and its chips of the BCA group.. > Now it is part of Broadband Video Group as known as BVG. WLAN chip is no > longer belong to BCA or BVG while we still support its software release. > So I would rather keep it in magic term ;) But if we really want to > spell out the full name, we suggest to use "Broadcom Broadband Carrier > Access (BCA) origin SoC". Same apply to Kconfig for arm32 platform. OK, I can use that and amend the patch accordingly.
On 2022-07-14 22:37, Florian Fainelli wrote: > On 7/13/22 13:47, William Zhang wrote: >> >> >> On 7/13/22 07:53, Rafał Miłecki wrote: >>> From: Rafał Miłecki <rafal@milecki.pl> >>> >>> It helps to understand that "magic" BCMBCA acronym. The full name was >>> found in Florian's pull request for 5.20. >>> >>> Signed-off-by: Rafał Miłecki <rafal@milecki.pl> >>> --- >>> arch/arm64/Kconfig.platforms | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/arch/arm64/Kconfig.platforms >>> b/arch/arm64/Kconfig.platforms >>> index 83e3943f7a55..2259d3732cc4 100644 >>> --- a/arch/arm64/Kconfig.platforms >>> +++ b/arch/arm64/Kconfig.platforms >>> @@ -64,7 +64,7 @@ config ARCH_BCM_IPROC >>> This enables support for Broadcom iProc based SoCs >>> config ARCH_BCMBCA >>> - bool "Broadcom Broadband SoC" >>> + bool "Broadcom Broadband Carrier Access (BCA) SoC" >>> help >>> Say Y if you intend to run the kernel on a Broadcom Broadband >>> ARM-based >>> BCA chipset. >> It is bit complicated about the history and its chips of the BCA >> group.. Now it is part of Broadband Video Group as known as BVG. WLAN >> chip is no longer belong to BCA or BVG while we still support its >> software release. So I would rather keep it in magic term ;) But if >> we really want to spell out the full name, we suggest to use >> "Broadcom Broadband Carrier Access (BCA) origin SoC". Same apply to >> Kconfig for arm32 platform. > > OK, I can use that and amend the patch accordingly. I'm leaving it up to you whether this name should be updated then.
On Wed, 13 Jul 2022 16:53:40 +0200, Rafał Miłecki <zajec5@gmail.com> wrote: > From: Rafał Miłecki <rafal@milecki.pl> > > It helps to understand that "magic" BCMBCA acronym. The full name was > found in Florian's pull request for 5.20. > > Signed-off-by: Rafał Miłecki <rafal@milecki.pl> > --- Applied to https://github.com/Broadcom/stblinux/commits/devicetree-arm64/next, thanks! -- Florian
diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms index 83e3943f7a55..2259d3732cc4 100644 --- a/arch/arm64/Kconfig.platforms +++ b/arch/arm64/Kconfig.platforms @@ -64,7 +64,7 @@ config ARCH_BCM_IPROC This enables support for Broadcom iProc based SoCs config ARCH_BCMBCA - bool "Broadcom Broadband SoC" + bool "Broadcom Broadband Carrier Access (BCA) SoC" help Say Y if you intend to run the kernel on a Broadcom Broadband ARM-based BCA chipset.