lkml.org 
[lkml]   [2017]   [Nov]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v7 07/13] slimbus: Add support for 'clock-pause' feature
From
Date
Thanks for the Review,

On 23/11/17 07:28, Charles Keepax wrote:
> On Wed, Nov 15, 2017 at 02:10:37PM +0000, srinivas.kandagatla@linaro.org wrote:
>> From: Sagar Dharia <sdharia@codeaurora.org>
>>
>> Per slimbus specification, a reconfiguration sequence known as
>> 'clock pause' needs to be broadcast over the bus while entering low-
>> power mode. Clock-pause is initiated by the controller driver.
>> To exit clock-pause, controller typically wakes up the framer device.
>> Since wakeup precedure is controller-specific, framework calls it via
>> controller's function pointer to invoke it.
>>
>> Signed-off-by: Sagar Dharia <sdharia@codeaurora.org>
>> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
>> ---
>> +/**
>> + * struct slim_sched: Framework uses this structure internally for scheduling.
>
> Missing kernel doc for clkgear here.
Will add that in next version.

>
>> + * @clk_state: Controller's clock state from enum slim_clk_state
>> + * @pause_comp: Signals completion of clock pause sequence. This is useful when
>> + * client tries to call slimbus transaction when controller is entering
>> + * clock pause.
>> + * @m_reconf: This mutex is held until current reconfiguration (data channel
>> + * scheduling, message bandwidth reservation) is done. Message APIs can
>> + * use the bus concurrently when this mutex is held since elemental access
>> + * messages can be sent on the bus when reconfiguration is in progress.
>> + */
>> +struct slim_sched {
>> + int clkgear;
>> + enum slim_clk_state clk_state;
>> + struct completion pause_comp;
>> + struct mutex m_reconf;
>> +};
>
> Thanks,
> Charles
>

\
 
 \ /
  Last update: 2017-11-24 15:41    [W:0.103 / U:0.104 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site