Re: [PATCH v1] soc/tegra: pmc: Drop locking from tegra_powergate_is_powered()

2018-10-25 Thread Thierry Reding
On Sun, Oct 21, 2018 at 09:36:14PM +0300, Dmitry Osipenko wrote: > This fixes splats like the one below if CONFIG_DEBUG_ATOMIC_SLEEP=y > and machine (Tegra30) booted with SMP=n or all secondary CPU's are put > offline. Locking isn't needed because it protects atomic operation. > > # echo 0 | tee /

Re: [PATCH v1] soc/tegra: pmc: Drop locking from tegra_powergate_is_powered()

2018-10-24 Thread Jon Hunter
On 24/10/2018 11:44, Dmitry Osipenko wrote: > On 10/24/18 1:20 PM, Jon Hunter wrote: >> >> On 21/10/2018 19:36, Dmitry Osipenko wrote: >>> This fixes splats like the one below if CONFIG_DEBUG_ATOMIC_SLEEP=y >>> and machine (Tegra30) booted with SMP=n or all secondary CPU's are put >>> offline. Lo

Re: [PATCH v1] soc/tegra: pmc: Drop locking from tegra_powergate_is_powered()

2018-10-24 Thread Dmitry Osipenko
On 10/24/18 1:20 PM, Jon Hunter wrote: > > On 21/10/2018 19:36, Dmitry Osipenko wrote: >> This fixes splats like the one below if CONFIG_DEBUG_ATOMIC_SLEEP=y >> and machine (Tegra30) booted with SMP=n or all secondary CPU's are put >> offline. Locking isn't needed because it protects atomic operat

Re: [PATCH v1] soc/tegra: pmc: Drop locking from tegra_powergate_is_powered()

2018-10-24 Thread Jon Hunter
On 21/10/2018 19:36, Dmitry Osipenko wrote: > This fixes splats like the one below if CONFIG_DEBUG_ATOMIC_SLEEP=y > and machine (Tegra30) booted with SMP=n or all secondary CPU's are put > offline. Locking isn't needed because it protects atomic operation. > > # echo 0 | tee /sys/devices/system/

[PATCH v1] soc/tegra: pmc: Drop locking from tegra_powergate_is_powered()

2018-10-21 Thread Dmitry Osipenko
This fixes splats like the one below if CONFIG_DEBUG_ATOMIC_SLEEP=y and machine (Tegra30) booted with SMP=n or all secondary CPU's are put offline. Locking isn't needed because it protects atomic operation. # echo 0 | tee /sys/devices/system/cpu/cpu[1-3]/online BUG: sleeping function called from