All of lore.kernel.org
 help / color / mirror / Atom feed
* Locking in the clk API
@ 2011-01-11  2:16 ` Jeremy Kerr
  0 siblings, 0 replies; 248+ messages in thread
From: Jeremy Kerr @ 2011-01-11  2:16 UTC (permalink / raw)
  To: linux-arm-kernel

Hi all,

I'd like to define some locking semantics for the clk API, so that device 
driver can have some assurance about when it is safe to call various clk_ 
functions from atomic contexts.

Vincent - you mentioned in a conversation that your code had some specific 
requirements for clock enable/disable at interrupt time - could you reply with 
an outline of this? Also, Sascha and Uwe: do you have any thoughts from your 
work with the common struct clk?

= Requirements =

To get things started, here are some basic requirements from the external API 
side:

1) clk_enable: the clock should be outputting a valid clock signal before
   returning from this function.

    - drivers may require valid clock signal to be present for subsequent
      device interactions.

3) clk_disable: may be called from atomic context

    - Vincent: this is what I recall from our conversation, is it still true?

4) clk_set_rate: clock should change to the new rate before this returns

5) clk_get_rate: may be called from atomic context

6) in general, drivers shouldn't require details about the clock
   implementation

And from the clock implementation side:

7) interactions with clock hardware may require sleeping (eg, clocks on an i2c
   bus)

8) clk_enable() may require enabling a parent, which may also require
   sleeping. Ideally, we shouldn't have to care about the parent's
   implementation.

I'm sure there are others, please feel free to add to this list.

= Implementation =

At present, we can satisfy these with:

* clk_enable: may sleep

* clk_disable: may not sleep, but it's possible to make the global
  clk_disable() atomic and defer the actual disable (clk->ops.disable()) to a
  non-atomic context.

* clk_get_rate: may not sleep

* clk_set_rate: may sleep

As we build up our requirements, we can adjust as suitable.

I'm excluding clk_{get,set}_parent at present, as I'm not sure we want these 
as part of the device-driver API (ie, they require knowledge of the platform 
clock infrastructure).

Thanks,


Jeremy

^ permalink raw reply	[flat|nested] 248+ messages in thread

end of thread, other threads:[~2011-01-28  3:29 UTC | newest]

Thread overview: 248+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-11  2:16 Locking in the clk API Jeremy Kerr
2011-01-11  2:16 ` Jeremy Kerr
2011-01-11  2:16 ` Jeremy Kerr
2011-01-11  3:15 ` Paul Mundt
2011-01-11  3:15   ` Paul Mundt
2011-01-11  3:15   ` Paul Mundt
2011-01-11  4:11   ` Jeremy Kerr
2011-01-11  4:11     ` Jeremy Kerr
2011-01-11  4:11     ` Jeremy Kerr
2011-01-11  4:54     ` Paul Mundt
2011-01-11  4:54       ` Paul Mundt
2011-01-11  4:54       ` Paul Mundt
2011-01-20 16:32       ` Ben Dooks
2011-01-20 16:32         ` Ben Dooks
2011-01-20 16:32         ` Ben Dooks
2011-01-20 18:57         ` Russell King - ARM Linux
2011-01-20 18:57           ` Russell King - ARM Linux
2011-01-20 18:57           ` Russell King - ARM Linux
2011-01-21  3:43           ` Saravana Kannan
2011-01-21  3:43             ` Saravana Kannan
2011-01-21  3:43             ` Saravana Kannan
2011-01-21  9:31             ` Russell King - ARM Linux
2011-01-21  9:31               ` Russell King - ARM Linux
2011-01-21  9:31               ` Russell King - ARM Linux
2011-01-11  9:03     ` Sascha Hauer
2011-01-11  9:03       ` Sascha Hauer
2011-01-11  9:03       ` Sascha Hauer
2011-01-11  9:28       ` Russell King - ARM Linux
2011-01-11  9:28         ` Russell King - ARM Linux
2011-01-11  9:28         ` Russell King - ARM Linux
2011-01-11 14:34         ` Pavel Machek
2011-01-11 14:34           ` Pavel Machek
2011-01-11 14:34           ` Pavel Machek
2011-01-20 16:29   ` Ben Dooks
2011-01-20 16:29     ` Ben Dooks
2011-01-20 16:29     ` Ben Dooks
2011-01-20 18:56     ` Russell King - ARM Linux
2011-01-20 18:56       ` Russell King - ARM Linux
2011-01-20 18:56       ` Russell King - ARM Linux
2011-01-20 21:30       ` Nicolas Pitre
2011-01-20 21:30         ` Nicolas Pitre
2011-01-20 21:30         ` Nicolas Pitre
2011-01-21  2:06         ` Dima Zavin
2011-01-21  2:06           ` Dima Zavin
2011-01-21  2:06           ` Dima Zavin
2011-01-21  4:12           ` Saravana Kannan
2011-01-21  4:12             ` Saravana Kannan
2011-01-21  4:12             ` Saravana Kannan
2011-01-21  9:32             ` Russell King - ARM Linux
2011-01-21  9:32               ` Russell King - ARM Linux
2011-01-21  9:32               ` Russell King - ARM Linux
2011-01-22  1:53               ` Saravana Kannan
2011-01-22  2:24                 ` Colin Cross
2011-01-22  2:56                   ` Saravana Kannan
2011-01-22  9:15                 ` Russell King - ARM Linux
2011-01-24 19:31                   ` Saravana Kannan
2011-01-21 21:03             ` Dima Zavin
2011-01-21 21:03               ` Dima Zavin
2011-01-21 21:03               ` Dima Zavin
2011-01-21 21:53               ` Nicolas Pitre
2011-01-21 21:53                 ` Nicolas Pitre
2011-01-21 21:53                 ` Nicolas Pitre
2011-01-21 22:02                 ` Russell King - ARM Linux
2011-01-21 22:02                   ` Russell King - ARM Linux
2011-01-21 22:02                   ` Russell King - ARM Linux
2011-01-21 22:28                   ` Colin Cross
2011-01-21 22:28                     ` Colin Cross
2011-01-21 22:28                     ` Colin Cross
2011-01-21 23:21                     ` Benjamin Herrenschmidt
2011-01-21 23:21                       ` Benjamin Herrenschmidt
2011-01-21 23:21                       ` Benjamin Herrenschmidt
2011-01-21 23:50                     ` Nicolas Pitre
2011-01-21 23:50                       ` Nicolas Pitre
2011-01-21 23:50                       ` Nicolas Pitre
2011-01-22  1:35                     ` Saravana Kannan
2011-01-22  1:35                       ` Saravana Kannan
2011-01-22  1:35                       ` Saravana Kannan
2011-01-22  2:22                       ` Colin Cross
2011-01-22  2:22                         ` Colin Cross
2011-01-22  2:22                         ` Colin Cross
2011-01-21 22:29                   ` Nicolas Pitre
2011-01-21 22:29                     ` Nicolas Pitre
2011-01-21 22:29                     ` Nicolas Pitre
2011-01-21 23:28                 ` Bryan Huntsman
2011-01-21 23:28                   ` Bryan Huntsman
2011-01-21 23:28                   ` Bryan Huntsman
2011-01-11  9:16 ` Russell King - ARM Linux
2011-01-11  9:16   ` Russell King - ARM Linux
2011-01-11  9:16   ` Russell King - ARM Linux
2011-01-11  9:44   ` Jeremy Kerr
2011-01-11  9:44     ` Jeremy Kerr
2011-01-11  9:44     ` Jeremy Kerr
2011-01-11 10:13     ` Paul Mundt
2011-01-11 10:13       ` Paul Mundt
2011-01-11 10:13       ` Paul Mundt
2011-01-11 10:30       ` Jeremy Kerr
2011-01-11 10:30         ` Jeremy Kerr
2011-01-11 10:30         ` Jeremy Kerr
2011-01-11 12:18         ` Paul Mundt
2011-01-11 12:18           ` Paul Mundt
2011-01-11 12:18           ` Paul Mundt
2011-01-11 13:52           ` 
2011-01-11 13:52             ` Uwe Kleine-König
2011-01-11 13:52             ` Uwe Kleine-König
2011-01-11 14:35           ` Jeremy Kerr
2011-01-11 14:35             ` Jeremy Kerr
2011-01-11 14:35             ` Jeremy Kerr
2011-01-12  3:25             ` Saravana Kannan
2011-01-12  3:25               ` Saravana Kannan
2011-01-12  3:25               ` Saravana Kannan
2011-01-12  7:40               ` 
2011-01-12  7:40                 ` Uwe Kleine-König
2011-01-12  7:40                 ` Uwe Kleine-König
2011-01-12  1:54           ` Saravana Kannan
2011-01-12  1:54             ` Saravana Kannan
2011-01-12  1:54             ` Saravana Kannan
2011-01-12  2:25             ` Paul Mundt
2011-01-12  2:25               ` Paul Mundt
2011-01-12  2:25               ` Paul Mundt
2011-01-20 16:57               ` Ben Dooks
2011-01-20 16:57                 ` Ben Dooks
2011-01-20 16:57                 ` Ben Dooks
2011-01-20 16:53           ` Ben Dooks
2011-01-20 16:53             ` Ben Dooks
2011-01-20 16:53             ` Ben Dooks
2011-01-20 16:40       ` Ben Dooks
2011-01-20 16:40         ` Ben Dooks
2011-01-20 16:40         ` Ben Dooks
2011-01-11 10:39     ` 
2011-01-11 10:39       ` Uwe Kleine-König
2011-01-11 10:39       ` Uwe Kleine-König
2011-01-11 10:47       ` Russell King - ARM Linux
2011-01-11 10:47         ` Russell King - ARM Linux
2011-01-11 10:47         ` Russell King - ARM Linux
2011-01-11 10:56         ` 
2011-01-11 10:56           ` Uwe Kleine-König
2011-01-11 10:56           ` Uwe Kleine-König
2011-01-11 11:15       ` Richard Zhao
2011-01-11 11:15         ` Richard Zhao
2011-01-11 11:15         ` Richard Zhao
2011-01-20 17:02         ` Ben Dooks
2011-01-20 17:02           ` Ben Dooks
2011-01-20 17:02           ` Ben Dooks
2011-01-20 19:08           ` Russell King - ARM Linux
2011-01-20 19:08             ` Russell King - ARM Linux
2011-01-20 19:08             ` Russell King - ARM Linux
2011-01-21  0:09             ` Jassi Brar
2011-01-21  0:09               ` Jassi Brar
2011-01-21  0:09               ` Jassi Brar
2011-01-21  4:47               ` Jassi Brar
2011-01-21  4:47                 ` Jassi Brar
2011-01-21  4:47                 ` Jassi Brar
2011-01-21  9:39                 ` Russell King - ARM Linux
2011-01-21  9:39                   ` Russell King - ARM Linux
2011-01-21  9:39                   ` Russell King - ARM Linux
2011-01-21 10:11                   ` Jassi Brar
2011-01-21 10:11                     ` Jassi Brar
2011-01-21 10:11                     ` Jassi Brar
2011-01-22  4:08                 ` Richard Zhao
2011-01-22  4:08                   ` Richard Zhao
2011-01-22  4:08                   ` Richard Zhao
2011-01-22  5:30                   ` Jassi Brar
2011-01-22  5:30                     ` Jassi Brar
2011-01-22  5:30                     ` Jassi Brar
2011-01-21  7:16             ` Saravana Kannan
2011-01-21  7:16               ` Saravana Kannan
2011-01-21  7:16               ` Saravana Kannan
2011-01-21  9:40               ` Russell King - ARM Linux
2011-01-21  9:40                 ` Russell King - ARM Linux
2011-01-21  9:40                 ` Russell King - ARM Linux
2011-01-22  1:47                 ` Saravana Kannan
2011-01-27  4:34                 ` Saravana Kannan
2011-01-27  4:34                   ` Saravana Kannan
2011-01-27  4:34                   ` Saravana Kannan
2011-01-27  8:54                   ` Russell King - ARM Linux
2011-01-27  8:54                     ` Russell King - ARM Linux
2011-01-27  8:54                     ` Russell King - ARM Linux
2011-01-27 20:30                     ` Saravana Kannan
2011-01-27 20:30                       ` Saravana Kannan
2011-01-27 20:30                       ` Saravana Kannan
2011-01-27 20:43                       ` Russell King - ARM Linux
2011-01-27 20:43                         ` Russell King - ARM Linux
2011-01-27 20:43                         ` Russell King - ARM Linux
2011-01-27 21:07                         ` Alan Cox
2011-01-27 21:07                           ` Alan Cox
2011-01-27 21:07                           ` Alan Cox
2011-01-27 21:11                           ` Russell King - ARM Linux
2011-01-27 21:11                             ` Russell King - ARM Linux
2011-01-27 21:11                             ` Russell King - ARM Linux
2011-01-27 21:15                           ` Russell King - ARM Linux
2011-01-27 21:15                             ` Russell King - ARM Linux
2011-01-27 21:15                             ` Russell King - ARM Linux
2011-01-28  3:29                           ` Saravana Kannan
2011-01-28  3:29                             ` Saravana Kannan
2011-01-28  3:29                             ` Saravana Kannan
2011-01-28  3:27                         ` Saravana Kannan
2011-01-28  3:27                           ` Saravana Kannan
2011-01-28  3:27                           ` Saravana Kannan
2011-01-11 12:11   ` Jassi Brar
2011-01-11 12:23     ` Jassi Brar
2011-01-11 12:11     ` Jassi Brar
2011-01-12  2:56   ` Saravana Kannan
2011-01-12  2:56     ` Saravana Kannan
2011-01-12  2:56     ` Saravana Kannan
2011-01-12  9:03     ` Russell King - ARM Linux
2011-01-12  9:03       ` Russell King - ARM Linux
2011-01-12  9:03       ` Russell King - ARM Linux
2011-01-15 14:02       ` Christer Weinigel
2011-01-15 14:02         ` Christer Weinigel
2011-01-15 14:02         ` Christer Weinigel
2011-01-15 14:53         ` Russell King - ARM Linux
2011-01-15 14:53           ` Russell King - ARM Linux
2011-01-15 14:53           ` Russell King - ARM Linux
2011-01-15 15:03           ` 
2011-01-15 15:03             ` Uwe Kleine-König
2011-01-15 15:03             ` Uwe Kleine-König
2011-01-15 15:15             ` Russell King - ARM Linux
2011-01-15 15:15               ` Russell King - ARM Linux
2011-01-15 15:15               ` Russell King - ARM Linux
2011-01-15 16:03               ` 
2011-01-15 16:03                 ` Uwe Kleine-König
2011-01-15 16:03                 ` Uwe Kleine-König
2011-01-15 16:21                 ` Russell King - ARM Linux
2011-01-15 16:21                   ` Russell King - ARM Linux
2011-01-15 16:21                   ` Russell King - ARM Linux
2011-01-15 16:31                   ` 
2011-01-15 16:31                     ` Uwe Kleine-König
2011-01-15 16:31                     ` Uwe Kleine-König
2011-01-16  6:59               ` Grant Likely
2011-01-16  6:59                 ` Grant Likely
2011-01-16  6:59                 ` Grant Likely
2011-01-15 17:07           ` Christer Weinigel
2011-01-15 17:07             ` Christer Weinigel
2011-01-15 17:07             ` Christer Weinigel
2011-01-15 17:20             ` Russell King - ARM Linux
2011-01-15 17:20               ` Russell King - ARM Linux
2011-01-15 17:20               ` Russell King - ARM Linux
2011-01-15 17:44               ` Christer Weinigel
2011-01-15 17:44                 ` Christer Weinigel
2011-01-15 17:44                 ` Christer Weinigel
2011-01-15 20:30                 ` Russell King - ARM Linux
2011-01-15 20:30                   ` Russell King - ARM Linux
2011-01-15 20:30                   ` Russell King - ARM Linux
2011-01-17  1:19 ` Jeremy Kerr
2011-01-17  1:19   ` Jeremy Kerr
2011-01-17  1:19   ` Jeremy Kerr
2011-01-17  1:27 ` Jeremy Kerr
2011-01-17  1:27   ` Jeremy Kerr

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.