Re: [PATCH 1/1 linux-next] hwrng: update Freescale i.MX RNGA Random Number Generator

[Fabian Frederick]

> On 15 August 2016 at 17:05 Arnd Bergmann <arnd@xxxxxxxx> wrote:
>
>
> On Friday 12 August 2016, Fabian Frederick wrote:
> > We can directly depend on SOC_IMX31 since
> > commit c9ee94965dce
> > ("ARM: imx: deconstruct mxc_rnga initialization")
> >
> > Signed-off-by: Fabian Frederick <fabf@xxxxxxxxx>
>
> Did I cause a regression with my patch, without this change? If so, this
> should be
> marked
>
> Cc: stable@xxxxxxxxxxxxxxx
> Fixes: c9ee94965dce ("ARM: imx: deconstruct mxc_rnga initialization")
>
> and the patch applied ASAP.

Thanks Arnd. Your patch brings the following regression:
CONFIG_HW_RANDOM_MXC_RNGA can't be switched on with an unknown symbol and
mxc-rnga.o will never build with
ARCH=arm make M=drivers/char/hw_random
Before your patch, HW_RANDOM_MXC_RNGA required ARCH_HAS_RNGA
which was based on IMX_HAVE_PLATFORM_MXC_RNGAÂ && ARCH_MXC
IMX_HAVE_PLATFORM_MXC_RNGAÂ was based on SOC_IMX31.
As I see SOC_IMX31 is selected by ARCH_MXC, my patch still seems ok and I can Cc
it for stable.

Regards,
Fabian

>
> > ---
> >Â drivers/char/hw_random/Kconfig | 2 +-
> >Â 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/char/hw_random/Kconfig b/drivers/char/hw_random/Kconfig
> > index 56ad5a59..8c0770b 100644
> > --- a/drivers/char/hw_random/Kconfig
> > +++ b/drivers/char/hw_random/Kconfig
> > @@ -244,7 +244,7 @@ config HW_RANDOM_TX4939
> >Â
> >Â config HW_RANDOM_MXC_RNGA
> >Â Â Âtristate "Freescale i.MX RNGA Random Number Generator"
> > -Â Âdepends on ARCH_HAS_RNGA
> > +Â Âdepends on SOC_IMX31
> >Â Â Âdefault HW_RANDOM
>
> It would be nice to add "|| COMPILE_TEST here", but that should probably be
> a separate patch, as we don't want to backport that or treat it as a bugfix.
>
>Â Â Â ÂArnd