[PATCH] powerpc/85xx: Don't report SRAM to L2 cache fallback as error

Scott Wood scott.wood at nxp.com
Wed May 25 02:31:01 AEST 2016


On 05/24/2016 10:07 AM, Claudiu Manoil wrote:
> If the SRAM region parameters are missing the SRAM driver
> probing exits and the L2 region is configured as L2 cache
> entirely.  This is the expected default behaviour, so it
> makes no sense to report it as an error.
> 
> Signed-off-by: Claudiu Manoil <claudiu.manoil at nxp.com>
> ---
>  arch/powerpc/sysdev/fsl_85xx_l2ctlr.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/powerpc/sysdev/fsl_85xx_l2ctlr.c b/arch/powerpc/sysdev/fsl_85xx_l2ctlr.c
> index 861cebf..15bc867 100644
> --- a/arch/powerpc/sysdev/fsl_85xx_l2ctlr.c
> +++ b/arch/powerpc/sysdev/fsl_85xx_l2ctlr.c
> @@ -91,9 +91,8 @@ static int mpc85xx_l2ctlr_of_probe(struct platform_device *dev)
>  	l2cache_size = *prop;
>  
>  	if (get_cache_sram_params(&sram_params)) {
> -		dev_err(&dev->dev,
> -			"Entire L2 as cache, provide valid sram offset and size\n");
> -		return -EINVAL;
> +		dev_info(&dev->dev, "Configure entire L2 as cache\n");
> +		return 0;
>  	}

There's no need to report it at all.  Imagine the mess that the boot log
would be (even more than it already is) if every driver built into the
kernel reported "no, didn't find this one".

Plus, the message reads as a command to the user rather than a statement
of fact.

-Scott



More information about the Linuxppc-dev mailing list