On Fri, Dec 06, 2013 at 01:33:50PM -0700, Stephen Warren wrote: > From: Stephen Warren <swarren@xxxxxxxxxx> > > Add a missing break to the switch in tegra_init_fuse() which determines > which SoC the code is running on. This prevents the Tegra30+ fuse > handling code from running on Tegra20. > > Fixes: 3bd1ae57f7bb ("ARM: tegra: add fuses as device randomness") > Signed-off-by: Stephen Warren <swarren@xxxxxxxxxx> > --- > This is a fix for 3.13. > > arch/arm/mach-tegra/fuse.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/mach-tegra/fuse.c b/arch/arm/mach-tegra/fuse.c > index 9a4e910c3796..13995b0be9d9 100644 > --- a/arch/arm/mach-tegra/fuse.c > +++ b/arch/arm/mach-tegra/fuse.c > @@ -198,6 +198,7 @@ void __init tegra_init_fuse(void) > switch (tegra_chip_id) { > case TEGRA20: > tegra20_fuse_init_randomness(); > + break; > case TEGRA30: > case TEGRA114: > default: The code that follows here also doesn't have a break. While not strictly necessary it might be good to have one for consistency. Either way, though: Reviewed-by: Thierry Reding <treding@xxxxxxxxxx>
Attachment:
pgpohXDAInvpO.pgp
Description: PGP signature