Re: [PATCH] ARM: sti: stih410-clocks: Add PROC_STFE as a critical clock

2016-10-26 Thread Lee Jones
> > > > > If the clock is enabled when Linux boots, the Linux clock framework > > > > > *needs* > > > > > to assume the hardware may have been used in previous boot stages, > > > > > and it should > > > > > not attempt to disable the clock. > > > > > > > > None of the boot loaders we use do this

Re: [PATCH] ARM: sti: stih410-clocks: Add PROC_STFE as a critical clock

2016-10-26 Thread Peter Griffin
Hi Lee, On Wed, 26 Oct 2016, Lee Jones wrote: > > > > If the clock is enabled when Linux boots, the Linux clock framework > > > > *needs* > > > > to assume the hardware may have been used in previous boot stages, and > > > > it should > > > > not attempt to disable the clock. > > > > > > None

Re: [PATCH] ARM: sti: stih410-clocks: Add PROC_STFE as a critical clock

2016-10-26 Thread Lee Jones
> > > If the clock is enabled when Linux boots, the Linux clock framework > > > *needs* > > > to assume the hardware may have been used in previous boot stages, and it > > > should > > > not attempt to disable the clock. > > > > None of the boot loaders we use do this. > > But the Linux kernel

Re: [PATCH] ARM: sti: stih410-clocks: Add PROC_STFE as a critical clock

2016-10-25 Thread Peter Griffin
Hi Lee, On Tue, 25 Oct 2016, Lee Jones wrote: > On Tue, 25 Oct 2016, Peter Griffin wrote: > > > Hi Lee, > > > > On Tue, 25 Oct 2016, Lee Jones wrote: > > > > > On Mon, 24 Oct 2016, Peter Griffin wrote: > > > > > > > Hi Lee, > > > > > > > > On Mon, 24 Oct 2016, Lee Jones wrote: > > > > > On T

Re: [PATCH] ARM: sti: stih410-clocks: Add PROC_STFE as a critical clock

2016-10-25 Thread Lee Jones
On Tue, 25 Oct 2016, Peter Griffin wrote: > Hi Lee, > > On Tue, 25 Oct 2016, Lee Jones wrote: > > > On Mon, 24 Oct 2016, Peter Griffin wrote: > > > > > Hi Lee, > > > > > > On Mon, 24 Oct 2016, Lee Jones wrote: > > > > On Tue, 18 Oct 2016, Peter Griffin wrote: > > > > > > > > > Once the ST fro

Re: [PATCH] ARM: sti: stih410-clocks: Add PROC_STFE as a critical clock

2016-10-25 Thread Peter Griffin
Hi Lee, On Tue, 25 Oct 2016, Lee Jones wrote: > On Mon, 24 Oct 2016, Peter Griffin wrote: > > > Hi Lee, > > > > On Mon, 24 Oct 2016, Lee Jones wrote: > > > On Tue, 18 Oct 2016, Peter Griffin wrote: > > > > > > > Once the ST frontend demux HW IP has been enabled, the clock can't > > > > be disa

Re: [PATCH] ARM: sti: stih410-clocks: Add PROC_STFE as a critical clock

2016-10-25 Thread Lee Jones
On Mon, 24 Oct 2016, Peter Griffin wrote: > Hi Lee, > > On Mon, 24 Oct 2016, Lee Jones wrote: > > On Tue, 18 Oct 2016, Peter Griffin wrote: > > > > > Once the ST frontend demux HW IP has been enabled, the clock can't > > > be disabled otherwise the system will hang and the board will > > > be un

Re: [PATCH] ARM: sti: stih410-clocks: Add PROC_STFE as a critical clock

2016-10-24 Thread Peter Griffin
Hi Lee, On Mon, 24 Oct 2016, Lee Jones wrote: > On Tue, 18 Oct 2016, Peter Griffin wrote: > > > Once the ST frontend demux HW IP has been enabled, the clock can't > > be disabled otherwise the system will hang and the board will > > be unserviceable. > > > > To allow balanced clock enable/disab

Re: [PATCH] ARM: sti: stih410-clocks: Add PROC_STFE as a critical clock

2016-10-24 Thread Lee Jones
On Tue, 18 Oct 2016, Peter Griffin wrote: > Once the ST frontend demux HW IP has been enabled, the clock can't > be disabled otherwise the system will hang and the board will > be unserviceable. > > To allow balanced clock enable/disable calls in the driver we use > the critical clock infrastruct

Re: [PATCH] ARM: sti: stih410-clocks: Add PROC_STFE as a critical clock

2016-10-20 Thread Patrice Chotard
On 10/18/2016 05:35 PM, Peter Griffin wrote: > Once the ST frontend demux HW IP has been enabled, the clock can't > be disabled otherwise the system will hang and the board will > be unserviceable. > > To allow balanced clock enable/disable calls in the driver we use > the critical clock infrast

[PATCH] ARM: sti: stih410-clocks: Add PROC_STFE as a critical clock

2016-10-18 Thread Peter Griffin
Once the ST frontend demux HW IP has been enabled, the clock can't be disabled otherwise the system will hang and the board will be unserviceable. To allow balanced clock enable/disable calls in the driver we use the critical clock infrastructure to take an extra reference on the clock so the cloc