All of lore.kernel.org
 help / color / mirror / Atom feed
From: sboyd@codeaurora.org (Stephen Boyd)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v13 0/9] Per-user clock constraints
Date: Tue, 23 Sep 2014 13:59:50 -0700	[thread overview]
Message-ID: <5421DF46.2010107@codeaurora.org> (raw)
In-Reply-To: <1411497613-24049-1-git-send-email-tomeu.vizoso@collabora.com>

On 09/23/14 11:40, Tomeu Vizoso wrote:
> Hello,
>
> this version of the patchset addresses some issues that Russell pointed out
> yesterday:
>
> * Refactor the changes to clkdev.c to reduce the amount of ifdefs.
>
> * Properly release clocks when there isn't enough memory to create the per-user
> wrapper.
>
> * Add clk_provider_put(struct clk_core*) for clock implementations to call
> instead of clk_put(struct clk*) (instead of exposing __clk_put).
>
> As the previous versions, this is based on top of 3.17-rc4 and Mike's patch at
> [0].

Any thoughts on my comments on patch set #10[1]? It seems like we can
avoid having a flag day to support this.

[1] https://lkml.org/lkml/2014/9/9/960

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by The Linux Foundation

WARNING: multiple messages have this Message-ID (diff)
From: Stephen Boyd <sboyd@codeaurora.org>
To: Tomeu Vizoso <tomeu.vizoso@collabora.com>,
	Mike Turquette <mturquette@linaro.org>
Cc: Russell King <linux@arm.linux.org.uk>,
	Stephen Warren <swarren@wwwdotorg.org>,
	Peter De Schrijver <pdeschrijver@nvidia.com>,
	linux-kernel@vger.kernel.org, tomasz.figa@gmail.com,
	rabin@rab.in, Thierry Reding <thierry.reding@gmail.com>,
	Javier Martinez Canillas <javier.martinez@collabora.co.uk>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v13 0/9] Per-user clock constraints
Date: Tue, 23 Sep 2014 13:59:50 -0700	[thread overview]
Message-ID: <5421DF46.2010107@codeaurora.org> (raw)
In-Reply-To: <1411497613-24049-1-git-send-email-tomeu.vizoso@collabora.com>

On 09/23/14 11:40, Tomeu Vizoso wrote:
> Hello,
>
> this version of the patchset addresses some issues that Russell pointed out
> yesterday:
>
> * Refactor the changes to clkdev.c to reduce the amount of ifdefs.
>
> * Properly release clocks when there isn't enough memory to create the per-user
> wrapper.
>
> * Add clk_provider_put(struct clk_core*) for clock implementations to call
> instead of clk_put(struct clk*) (instead of exposing __clk_put).
>
> As the previous versions, this is based on top of 3.17-rc4 and Mike's patch at
> [0].

Any thoughts on my comments on patch set #10[1]? It seems like we can
avoid having a flag day to support this.

[1] https://lkml.org/lkml/2014/9/9/960

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by The Linux Foundation


  parent reply	other threads:[~2014-09-23 20:59 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-23 18:40 [PATCH v13 0/9] Per-user clock constraints Tomeu Vizoso
2014-09-23 18:40 ` Tomeu Vizoso
2014-09-23 18:40 ` [PATCH v13 1/9] clk: Add temporary mapping to the existing API Tomeu Vizoso
2014-09-23 18:40   ` Tomeu Vizoso
2014-09-23 18:42 ` [PATCH v13 2/9] clk: Move all drivers to use internal API Tomeu Vizoso
2014-09-23 18:42   ` Tomeu Vizoso
2014-09-23 18:44 ` [PATCH v13 3/9] clk: use struct clk only for external API Tomeu Vizoso
2014-09-23 18:44   ` Tomeu Vizoso
2014-09-23 18:44   ` [PATCH v13 4/9] clk: per-user clock accounting for debug Tomeu Vizoso
2014-09-23 18:44     ` Tomeu Vizoso
2014-09-23 18:44   ` [PATCH v13 5/9] clk: Add floor and ceiling constraints to clock rates Tomeu Vizoso
2014-09-23 18:44     ` Tomeu Vizoso
2014-09-23 18:44   ` [PATCH v13 6/9] clk: Warn of unbalanced clk_prepare() calls Tomeu Vizoso
2014-09-23 18:44     ` Tomeu Vizoso
2014-09-23 18:44   ` [PATCH v13 7/9] clk: Take the prepare lock when updating the list of per-user clks Tomeu Vizoso
2014-09-23 18:44     ` Tomeu Vizoso
2014-09-23 18:44   ` [PATCH v13 8/9] clk: Take the prepare lock when updating the per-user constraints Tomeu Vizoso
2014-09-23 18:44     ` Tomeu Vizoso
2014-09-23 18:44   ` [PATCH v13 9/9] clk: Add docs about calling clk_put after clk_get_parent Tomeu Vizoso
2014-09-23 18:44     ` Tomeu Vizoso
2014-09-23 20:59 ` Stephen Boyd [this message]
2014-09-23 20:59   ` [PATCH v13 0/9] Per-user clock constraints Stephen Boyd
2014-09-24  8:27   ` Tomeu Vizoso
2014-09-24  8:27     ` Tomeu Vizoso
2014-09-26  1:29     ` Stephen Boyd
2014-09-26  1:29       ` Stephen Boyd
2014-09-26  8:09       ` Tomeu Vizoso
2014-09-26  8:09         ` Tomeu Vizoso
2014-09-26 23:20         ` Mike Turquette
2014-09-26 23:20           ` Mike Turquette
2014-09-27  0:15           ` Stephen Boyd
2014-09-27  0:15             ` Stephen Boyd
2014-09-29 18:17           ` [RFC] clk: Make clk API return per-user struct clk instances Tomeu Vizoso
2014-09-30  1:40             ` Stephen Boyd
2014-09-30  6:54               ` Mike Turquette
2014-09-30  7:41                 ` Tero Kristo
2014-09-30 18:16                   ` Tony Lindgren
2014-10-03 14:13                     ` Tero Kristo
2014-09-30  9:14             ` Russell King - ARM Linux
2014-09-30 14:28           ` [RFC v2] " Tomeu Vizoso
2014-09-24  9:14 ` [PATCH v13 0/9] Per-user clock constraints Tomeu Vizoso
2014-09-24  9:14   ` Tomeu Vizoso

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5421DF46.2010107@codeaurora.org \
    --to=sboyd@codeaurora.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.