All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stephen Boyd <sboyd@codeaurora.org>
To: Mike Turquette <mturquette@linaro.org>
Cc: David Brown <davidb@codeaurora.org>,
	Daniel Walker <dwalker@fifo99.com>,
	Bryan Huntsman <bryanh@codeaurora.org>,
	linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	Saravana Kannan <skannan@codeaurora.org>
Subject: Re: [PATCH 12/12] ARM: msm: Migrate to common clock framework
Date: Wed, 12 Jun 2013 09:54:57 -0700	[thread overview]
Message-ID: <51B8A7E1.1020708@codeaurora.org> (raw)
In-Reply-To: <20130611222832.8816.77120@quantum>

On 06/11/13 15:28, Mike Turquette wrote:
> Quoting Stephen Boyd (2013-06-04 12:25:38)
>> Move the existing clock code in mach-msm to the common clock
>> framework. We lose our capability to set the rate of and enable a
>> clock through debugfs. This is ok though because the debugfs
>> features are mainly used for testing and development of new clock
>> code.
>>
>> To maintain compatibility with the original MSM clock code we
>> make a wrapper for clk_reset() that calls the struct msm_clk
>> specific reset function. This is necessary for the usb and sdcc
>> devices on MSM until a better suited API is made available.
>>
>> Cc: Saravana Kannan <skannan@codeaurora.org>
>> Cc: Mike Turquette <mturquette@linaro.org>
>> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
> I just went through this quickly and nothing popped out at me.  Nice
> diffstat btw!  Do you have any plans to move this to drivers/clk/msm ?
>
> Acked-by: Mike Turquette <mturquette@linaro.org>

Thanks Mike.

I plan to add mmio support under drivers/clk/msm. Moving this proc_comm
based code to drivers/clk/msm is possible but it would need to depend on
the 3 architectures that use it so that it isn't available or compiled
on our DT based platforms. We can do that later I suppose.

-- 
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: sboyd@codeaurora.org (Stephen Boyd)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 12/12] ARM: msm: Migrate to common clock framework
Date: Wed, 12 Jun 2013 09:54:57 -0700	[thread overview]
Message-ID: <51B8A7E1.1020708@codeaurora.org> (raw)
In-Reply-To: <20130611222832.8816.77120@quantum>

On 06/11/13 15:28, Mike Turquette wrote:
> Quoting Stephen Boyd (2013-06-04 12:25:38)
>> Move the existing clock code in mach-msm to the common clock
>> framework. We lose our capability to set the rate of and enable a
>> clock through debugfs. This is ok though because the debugfs
>> features are mainly used for testing and development of new clock
>> code.
>>
>> To maintain compatibility with the original MSM clock code we
>> make a wrapper for clk_reset() that calls the struct msm_clk
>> specific reset function. This is necessary for the usb and sdcc
>> devices on MSM until a better suited API is made available.
>>
>> Cc: Saravana Kannan <skannan@codeaurora.org>
>> Cc: Mike Turquette <mturquette@linaro.org>
>> Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
> I just went through this quickly and nothing popped out at me.  Nice
> diffstat btw!  Do you have any plans to move this to drivers/clk/msm ?
>
> Acked-by: Mike Turquette <mturquette@linaro.org>

Thanks Mike.

I plan to add mmio support under drivers/clk/msm. Moving this proc_comm
based code to drivers/clk/msm is possible but it would need to depend on
the 3 architectures that use it so that it isn't available or compiled
on our DT based platforms. We can do that later I suppose.

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

  reply	other threads:[~2013-06-12 16:54 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-04 19:25 [PATCH 00/12] Convert MSM to common clock framework Stephen Boyd
2013-06-04 19:25 ` Stephen Boyd
2013-06-04 19:25 ` [PATCH 01/12] msm_serial: Convert to clk_prepare/unprepare Stephen Boyd
2013-06-04 19:25   ` Stephen Boyd
2013-06-04 19:25 ` [PATCH 02/12] msm_serial: Use devm_clk_get() and properly return errors Stephen Boyd
2013-06-04 19:25   ` Stephen Boyd
2013-06-04 19:25 ` [PATCH 03/12] usb: otg: msm: Convert to clk_prepare/unprepare Stephen Boyd
2013-06-04 19:25   ` Stephen Boyd
2013-06-04 20:21   ` Felipe Balbi
2013-06-04 20:21     ` Felipe Balbi
2013-06-04 20:21     ` Felipe Balbi
2013-06-04 19:25 ` [PATCH 04/12] msm_sdcc: " Stephen Boyd
2013-06-04 19:25   ` Stephen Boyd
2013-06-04 19:25   ` Stephen Boyd
2013-06-04 19:25 ` [PATCH 05/12] msm: iommu: " Stephen Boyd
2013-06-04 19:25   ` Stephen Boyd
2013-06-04 19:25 ` [PATCH 06/12] msm: iommu: Use clk_set_rate() instead of clk_set_min_rate() Stephen Boyd
2013-06-04 19:25   ` Stephen Boyd
2013-06-04 19:25 ` [PATCH 07/12] ARM: msm: Remove custom clk_set_flags() API Stephen Boyd
2013-06-04 19:25   ` Stephen Boyd
2013-06-04 19:25 ` [PATCH 08/12] ARM: msm: Remove custom clk_set_{max,min}_rate() API Stephen Boyd
2013-06-04 19:25   ` Stephen Boyd
2013-06-04 19:25 ` [PATCH 09/12] ARM: msm: Remove clock-7x30.h include file Stephen Boyd
2013-06-04 19:25   ` Stephen Boyd
2013-06-04 19:25 ` [PATCH 10/12] ARM: msm: Prepare clk_get() users in mach-msm for clock-pcom driver Stephen Boyd
2013-06-04 19:25   ` Stephen Boyd
2013-06-04 19:25 ` [PATCH 11/12] ARM: msm: Make proc_comm clock control into a platform driver Stephen Boyd
2013-06-04 19:25   ` Stephen Boyd
2013-06-04 19:25 ` [PATCH 12/12] ARM: msm: Migrate to common clock framework Stephen Boyd
2013-06-04 19:25   ` Stephen Boyd
2013-06-11 22:28   ` Mike Turquette
2013-06-11 22:28     ` Mike Turquette
2013-06-12 16:54     ` Stephen Boyd [this message]
2013-06-12 16:54       ` Stephen Boyd

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=51B8A7E1.1020708@codeaurora.org \
    --to=sboyd@codeaurora.org \
    --cc=bryanh@codeaurora.org \
    --cc=davidb@codeaurora.org \
    --cc=dwalker@fifo99.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mturquette@linaro.org \
    --cc=skannan@codeaurora.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.