All of lore.kernel.org
 help / color / mirror / Atom feed
From: Olof Johansson <olof@lixom.net>
To: David Brown <davidb@codeaurora.org>
Cc: arm@kernel.org, Arnd Bergmann <arnd@arndb.de>,
	Nicolas Pitre <nicolas.pitre@linaro.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	Russell King <rmk@arm.linux.org.uk>,
	linux-arm-msm@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: Re: [GIT PULL 3/3] msm clock for 3.11
Date: Fri, 14 Jun 2013 18:35:46 -0700	[thread overview]
Message-ID: <20130615013546.GV18124@quad.lixom.net> (raw)
In-Reply-To: <20130614195238.GA2381@codeaurora.org>

On Fri, Jun 14, 2013 at 12:52:55PM -0700, David Brown wrote:
> The following changes since commit f722406faae2d073cc1d01063d1123c35425939e:
> 
>  Linux 3.10-rc1 (2013-05-11 17:14:08 -0700)
> 
> are available in the git repository at:
> 
>  git://git.kernel.org/pub/scm/linux/kernel/git/davidb/linux-msm.git tags/msm-clock-for-3.11
> 
> for you to fetch changes up to f54c1c11053d8aaf318a096b8b332fd41a394a66:
> 
>  ARM: msm: Migrate to common clock framework (2013-06-12 14:43:31 -0700)
> 
> ----------------------------------------------------------------
> MSM clock updates for 3.11.
> 
> Per Stephen Boyd's coverletter:
> 
> This patchset moves the existing MSM clock code and affected drivers
> to the common clock framework. A prerequisite of moving to the common
> clock framework is to use clk_prepare() and clk_enable() so the first
> few patches migrate drivers to that call (clk_prepare() is a no-op on
> MSM right now). It also removes some custom clock APIs that MSM
> provides and finally moves the proc_comm clock code to the common
> struct clk.
> 
> This patch series will be used as the foundation of the MSM 8660/8960
> clock code that I plan to send out after this series.
> 
> ----------------------------------------------------------------
> Stephen Boyd (12):
>      msm_serial: Convert to clk_prepare/unprepare
>      msm_serial: Use devm_clk_get() and properly return errors
>      usb: otg: msm: Convert to clk_prepare/unprepare
>      msm_sdcc: Convert to clk_prepare/unprepare
>      msm: iommu: Convert to clk_prepare/unprepare
>      msm: iommu: Use clk_set_rate() instead of clk_set_min_rate()
>      ARM: msm: Remove custom clk_set_flags() API
>      ARM: msm: Remove custom clk_set_{max,min}_rate() API
>      ARM: msm: Remove clock-7x30.h include file
>      ARM: msm: Prepare clk_get() users in mach-msm for clock-pcom driver
>      ARM: msm: Make proc_comm clock control into a platform driver
>      ARM: msm: Migrate to common clock framework

I don't see a single ack from any of the driver subsystem maintainers for
serial, usb or iommu. Some of them have been acked though, for example Felipe
acked the USB one on the mailing list.

I also didn't see Greg cc:d on the serial patches, which would explain why
there was no ack for them.

Care to repost/collect/rebuild this branch, please? It's a great cleanup and
I'd be happy to take it even if it ends up coming in after -rc6.


-Olof

WARNING: multiple messages have this Message-ID (diff)
From: olof@lixom.net (Olof Johansson)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL 3/3] msm clock for 3.11
Date: Fri, 14 Jun 2013 18:35:46 -0700	[thread overview]
Message-ID: <20130615013546.GV18124@quad.lixom.net> (raw)
In-Reply-To: <20130614195238.GA2381@codeaurora.org>

On Fri, Jun 14, 2013 at 12:52:55PM -0700, David Brown wrote:
> The following changes since commit f722406faae2d073cc1d01063d1123c35425939e:
> 
>  Linux 3.10-rc1 (2013-05-11 17:14:08 -0700)
> 
> are available in the git repository at:
> 
>  git://git.kernel.org/pub/scm/linux/kernel/git/davidb/linux-msm.git tags/msm-clock-for-3.11
> 
> for you to fetch changes up to f54c1c11053d8aaf318a096b8b332fd41a394a66:
> 
>  ARM: msm: Migrate to common clock framework (2013-06-12 14:43:31 -0700)
> 
> ----------------------------------------------------------------
> MSM clock updates for 3.11.
> 
> Per Stephen Boyd's coverletter:
> 
> This patchset moves the existing MSM clock code and affected drivers
> to the common clock framework. A prerequisite of moving to the common
> clock framework is to use clk_prepare() and clk_enable() so the first
> few patches migrate drivers to that call (clk_prepare() is a no-op on
> MSM right now). It also removes some custom clock APIs that MSM
> provides and finally moves the proc_comm clock code to the common
> struct clk.
> 
> This patch series will be used as the foundation of the MSM 8660/8960
> clock code that I plan to send out after this series.
> 
> ----------------------------------------------------------------
> Stephen Boyd (12):
>      msm_serial: Convert to clk_prepare/unprepare
>      msm_serial: Use devm_clk_get() and properly return errors
>      usb: otg: msm: Convert to clk_prepare/unprepare
>      msm_sdcc: Convert to clk_prepare/unprepare
>      msm: iommu: Convert to clk_prepare/unprepare
>      msm: iommu: Use clk_set_rate() instead of clk_set_min_rate()
>      ARM: msm: Remove custom clk_set_flags() API
>      ARM: msm: Remove custom clk_set_{max,min}_rate() API
>      ARM: msm: Remove clock-7x30.h include file
>      ARM: msm: Prepare clk_get() users in mach-msm for clock-pcom driver
>      ARM: msm: Make proc_comm clock control into a platform driver
>      ARM: msm: Migrate to common clock framework

I don't see a single ack from any of the driver subsystem maintainers for
serial, usb or iommu. Some of them have been acked though, for example Felipe
acked the USB one on the mailing list.

I also didn't see Greg cc:d on the serial patches, which would explain why
there was no ack for them.

Care to repost/collect/rebuild this branch, please? It's a great cleanup and
I'd be happy to take it even if it ends up coming in after -rc6.


-Olof

  reply	other threads:[~2013-06-15  1:35 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-14 17:56 [GIT PULL 0/3] msm changes for 3.11 David Brown
2013-06-14 17:56 ` David Brown
2013-06-14 17:56 ` David Brown
2013-06-14 17:56 ` [GIT PULL 1/3] msm cleanups " David Brown
2013-06-14 17:56   ` David Brown
2013-06-15  1:21   ` Olof Johansson
2013-06-15  1:21     ` Olof Johansson
2013-06-14 17:56 ` [GIT PULL 2/3] msm fixes " David Brown
2013-06-14 17:56   ` David Brown
2013-06-14 17:56 ` David Brown
2013-06-14 17:56   ` David Brown
2013-06-15  1:26   ` Olof Johansson
2013-06-15  1:26     ` Olof Johansson
2013-06-17 20:26     ` David Brown
2013-06-17 20:26       ` David Brown
2013-06-17 20:31       ` Olof Johansson
2013-06-17 20:31         ` Olof Johansson
2013-06-14 19:52 ` [GIT PULL 3/3] msm clock " David Brown
2013-06-14 19:52   ` David Brown
2013-06-15  1:35   ` Olof Johansson [this message]
2013-06-15  1:35     ` Olof Johansson
2013-06-17 17:18     ` Stephen Boyd
2013-06-17 17:18       ` Stephen Boyd
2013-06-17 17:35       ` Olof Johansson
2013-06-17 17:35         ` Olof Johansson
2013-06-17 17:35         ` Olof Johansson
2013-06-24 21:00   ` [GIT PULL v2] " David Brown
2013-06-24 21:00     ` David Brown
2013-06-24 21:02     ` Chris Ball
2013-06-24 21:02       ` Chris Ball
2013-06-28  0:06     ` Olof Johansson
2013-06-28  0:06       ` Olof Johansson

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=20130615013546.GV18124@quad.lixom.net \
    --to=olof@lixom.net \
    --cc=arm@kernel.org \
    --cc=arnd@arndb.de \
    --cc=davidb@codeaurora.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nicolas.pitre@linaro.org \
    --cc=rmk@arm.linux.org.uk \
    --cc=tglx@linutronix.de \
    /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.