Re: [PATCH v4] mtd: parsers: bcm63xx: simplify CFE detection

From: Miquel Raynal
Date: Mon Sep 28 2020 - 10:16:25 EST


Hello,

Guenter Roeck <linux@xxxxxxxxxxxx> wrote on Mon, 21 Sep 2020 20:26:19
-0700:

> On 9/21/20 8:18 PM, Naresh Kamboju wrote:
> > On Fri, 14 Aug 2020 at 14:26, Guenter Roeck <linux@xxxxxxxxxxxx> wrote:
> >>
> >> On Mon, Jun 15, 2020 at 11:17:40AM +0200, Álvaro Fernández Rojas wrote:
> >>> Instead of trying to parse CFE version string, which is customized by some
> >>> vendors, let's just check that "CFE1" was passed on argument 3.
> >>>
> >>> Signed-off-by: Álvaro Fernández Rojas <noltari@xxxxxxxxx>
> >>> Signed-off-by: Jonas Gorski <jonas.gorski@xxxxxxxxx>
> >>> Reviewed-by: Florian Fainelli <f.fainelli@xxxxxxxxx>
> >>
> >
> > We still see mips: allmodconfig build failure on Linus tree
> >
>
> Yes, same here.

Perhaps this check should be done by an exported helper so that we do
not blindly export the variable?

Alvaro, Jonas, can one of you try to address this issue please?

Thanks,
Miquèl