On Tuesday, November 18, 2014 8:50 PM, Sylwester Nawrocki Wrote: > On 18/11/14 11:47, Pankaj Dubey wrote: > > diff --git a/arch/arm/mach-exynos/pmu.c b/arch/arm/mach-exynos/pmu.c > > index 6c8a76d..e4c3512 100644 > > --- a/arch/arm/mach-exynos/pmu.c > > +++ b/arch/arm/mach-exynos/pmu.c > > @@ -11,8 +11,11 @@ > > > @@ -716,6 +719,13 @@ static void exynos5420_pmu_init(void) > > pr_info("EXYNOS5420 PMU initialized\n"); } > > > > +static int pmu_restart_notify(struct notifier_block *this, > > + unsigned long code, void *unused) > > +{ > > + pmu_raw_writel(0x1, EXYNOS_SWRESET); > > Is it also valid on exynos5440 ? Doesn't it cause any harm there ? > Perhaps we could add a comment saying it has not effect on that SoC ? > Seeing original code from exynos.c (restart hook) I can say it's not valid for exynos5440. But it should not cause any harm as of now, because for exynos54440 PMU will not be probed. If in future support for exynos5440 PMU gets added, that time one may need to consider this point. This will be applicable for any SoC which does not have same register for restart. Thanks, Pankaj Dubey > > + return NOTIFY_DONE; > > +} > > -- > Thanks, > Sylwester -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html