Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <fced7f47-985b-7ef8-431b-051353a56d30@gmail.com>
Date:   Wed, 13 Jul 2022 22:19:14 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:96.0) Gecko/20100101
 Thunderbird/96.0
Subject: Re: [RESEND PATCH 1/8] ARM: debug: bcmbca: Replace ARCH_BCM_63XX with
 ARCH_BCMBCA
To:     Florian Fainelli <f.fainelli@gmail.com>,
        William Zhang <william.zhang@broadcom.com>,
        Linux ARM List <linux-arm-kernel@lists.infradead.org>,
        Broadcom Kernel List <bcm-kernel-feedback-list@broadcom.com>
Cc:     anand.gore@broadcom.com, dan.beygelman@broadcom.com,
        kursad.oney@broadcom.com, joel.peshkin@broadcom.com,
        Ard Biesheuvel <ardb@kernel.org>,
        Arnd Bergmann <arnd@arndb.de>,
        Eugen Hristev <eugen.hristev@microchip.com>,
        Kavyasree Kotagiri <kavyasree.kotagiri@microchip.com>,
        Lukas Bulwahn <lukas.bulwahn@gmail.com>,
        Masahiro Yamada <masahiroy@kernel.org>,
        Nathan Chancellor <nathan@kernel.org>,
        Nick Desaulniers <ndesaulniers@google.com>,
        Nicolas Ferre <nicolas.ferre@microchip.com>,
        Russell King <linux@armlinux.org.uk>,
        Shawn Guo <shawnguo@kernel.org>, linux-kernel@vger.kernel.org
References: <20220707065800.261269-1-william.zhang@broadcom.com>
 <86444849-e80a-6e28-c060-cecb9f07ae75@gmail.com>
 <5392c58e-d479-94c9-f025-099f99ae17ba@gmail.com>
From:   =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= <zajec5@gmail.com>
In-Reply-To: <5392c58e-d479-94c9-f025-099f99ae17ba@gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk

On 13.07.2022 18:10, Florian Fainelli wrote:
> On 7/13/22 08:39, Rafał Miłecki wrote:
>> On 7.07.2022 08:57, William Zhang wrote:
>>> Prepare for the BCM63138 ARCH_BCM_63XX migration to ARCH_BCMBCA. Make
>>> DEBUG_BCM63XX_UART depending on ARCH_BCMBCA.
>>>
>>> Signed-off-by: William Zhang <william.zhang@broadcom.com>
>>> ---
>>>
>>>   arch/arm/Kconfig.debug | 2 +-
>>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
>>> index 9b0aa4822d69..792796a348c3 100644
>>> --- a/arch/arm/Kconfig.debug
>>> +++ b/arch/arm/Kconfig.debug
>>> @@ -271,7 +271,7 @@ choice
>>>       config DEBUG_BCM63XX_UART
>>>           bool "Kernel low-level debugging on BCM63XX UART"
>>> -        depends on ARCH_BCM_63XX
>>> +        depends on ARCH_BCMBCA
>>>       config DEBUG_BERLIN_UART
>>>           bool "Marvell Berlin SoC Debug UART"
>>
>> There is no such config symbol (ARCH_BCMBCA) in Linus's tree.
>>
>> This patchset is going to break builds until code it depends on hits Linus's tree.
>>
>> All those patches should be hold until that.
> 
> There is, ARCH_BCMBCA was added to arch/arm/mach-bcm/Kconfig with b32c613b3fda3 ("arm: bcmbca: add arch bcmbca machine entry") which is in v5.19-rc1 and newer:

Oops, my tree updating script failed me. Sorry!

OK then, maybe this migration wasn't handled flawlessly but it's half
done now and remaining things should be fixed with all pending work.
Looks OK!