All of lore.kernel.org
 help / color / mirror / Atom feed
From: viresh.kumar@st.com (Viresh Kumar)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH V3 Resend 00/12] clk: Add non CONFIG_HAVE_CLK routines
Date: Tue, 8 May 2012 09:22:27 +0530	[thread overview]
Message-ID: <cover.1336448639.git.viresh.kumar@st.com> (raw)

Hi,

I am resending this patchset as Andrew didn't get few patches of this set. Also
i have updated patches with Acked-by or Reviewed by of people who replied with
these strings.

Last time there were few discussion over this patchset which can be found at

http://lkml.org/lkml/2012/4/24/154

Many drivers are shared between architectures that may or may not have HAVE_CLK
selected for them. To remove compilation errors for them we enclose clk_*()
calls in these drivers within #ifdef CONFIG_HAVE_CLK, #endif.

This patchset removes the need of these CONFIG_HAVE_CLK statements, by
introducing dummy routines when HAVE_CLK is not selected by platforms. So,
definition of these routines will always be available. These calls will return
error for platforms that don't select HAVE_CLK.

V2->V3:
- Dummy routines now return NULL or 0.
- All user drivers must fail if clk_get returned error other than NULL.
- All user drivers don't need to validate their clk pointer before every call to
  clk_*() routines.
- Patches dropped earlier are again taken back, as they were following similar
  approach to what is implemented now.

Viresh Kumar (12):
  clk: Add non CONFIG_HAVE_CLK routines
  clk: Remove redundant depends on from drivers/Kconfig
  i2c/i2c-pxa: Remove conditional compilation of clk code
  usb/marvell: Remove conditional compilation of clk code
  usb/musb: Remove conditional compilation of clk code
  ata/pata_arasan: Remove conditional compilation of clk code
  ata/sata_mv: Remove conditional compilation of clk code
  net/c_can: Remove conditional compilation of clk code
  net/stmmac: Remove conditional compilation of clk code
  gadget/m66592: Remove conditional compilation of clk code
  gadget/r8a66597: Remove conditional compilation of clk code
  usb/host/r8a66597: Remove conditional compilation of clk code

 drivers/ata/pata_arasan_cf.c                      |   14 +--
 drivers/ata/sata_mv.c                             |   10 --
 drivers/clk/Kconfig                               |    2 -
 drivers/i2c/busses/i2c-pxa.c                      |    7 -
 drivers/net/can/c_can/c_can_platform.c            |    8 -
 drivers/net/ethernet/stmicro/stmmac/stmmac.h      |   41 -----
 drivers/net/ethernet/stmicro/stmmac/stmmac_main.c |   33 ++--
 drivers/usb/gadget/m66592-udc.c                   |    9 +-
 drivers/usb/gadget/m66592-udc.h                   |    5 -
 drivers/usb/gadget/r8a66597-udc.c                 |   11 +-
 drivers/usb/gadget/r8a66597-udc.h                 |    5 -
 drivers/usb/host/r8a66597-hcd.c                   |   12 --
 drivers/usb/host/r8a66597.h                       |    5 -
 drivers/usb/musb/musb_core.h                      |    8 -
 include/linux/clk.h                               |  168 +++++++++++++-------
 include/linux/platform_data/mv_usb.h              |    9 -
 16 files changed, 131 insertions(+), 216 deletions(-)

-- 
1.7.9

WARNING: multiple messages have this Message-ID (diff)
From: Viresh Kumar <viresh.kumar@st.com>
To: <akpm@linux-foundation.org>
Cc: <spear-devel@list.st.com>, <viresh.linux@gmail.com>,
	<linux-kernel@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>, <mturquette@linaro.org>,
	<sshtylyov@mvista.com>, <jgarzik@redhat.com>,
	<linux@arm.linux.org.uk>, <w.sang@pengutronix.de>,
	<LW@KARO-electronics.de>, <andrew@lunn.ch>,
	Viresh Kumar <viresh.kumar@st.com>
Subject: [PATCH V3 Resend 00/12] clk: Add non CONFIG_HAVE_CLK routines
Date: Tue, 8 May 2012 09:22:27 +0530	[thread overview]
Message-ID: <cover.1336448639.git.viresh.kumar@st.com> (raw)

Hi,

I am resending this patchset as Andrew didn't get few patches of this set. Also
i have updated patches with Acked-by or Reviewed by of people who replied with
these strings.

Last time there were few discussion over this patchset which can be found at

http://lkml.org/lkml/2012/4/24/154

Many drivers are shared between architectures that may or may not have HAVE_CLK
selected for them. To remove compilation errors for them we enclose clk_*()
calls in these drivers within #ifdef CONFIG_HAVE_CLK, #endif.

This patchset removes the need of these CONFIG_HAVE_CLK statements, by
introducing dummy routines when HAVE_CLK is not selected by platforms. So,
definition of these routines will always be available. These calls will return
error for platforms that don't select HAVE_CLK.

V2->V3:
- Dummy routines now return NULL or 0.
- All user drivers must fail if clk_get returned error other than NULL.
- All user drivers don't need to validate their clk pointer before every call to
  clk_*() routines.
- Patches dropped earlier are again taken back, as they were following similar
  approach to what is implemented now.

Viresh Kumar (12):
  clk: Add non CONFIG_HAVE_CLK routines
  clk: Remove redundant depends on from drivers/Kconfig
  i2c/i2c-pxa: Remove conditional compilation of clk code
  usb/marvell: Remove conditional compilation of clk code
  usb/musb: Remove conditional compilation of clk code
  ata/pata_arasan: Remove conditional compilation of clk code
  ata/sata_mv: Remove conditional compilation of clk code
  net/c_can: Remove conditional compilation of clk code
  net/stmmac: Remove conditional compilation of clk code
  gadget/m66592: Remove conditional compilation of clk code
  gadget/r8a66597: Remove conditional compilation of clk code
  usb/host/r8a66597: Remove conditional compilation of clk code

 drivers/ata/pata_arasan_cf.c                      |   14 +--
 drivers/ata/sata_mv.c                             |   10 --
 drivers/clk/Kconfig                               |    2 -
 drivers/i2c/busses/i2c-pxa.c                      |    7 -
 drivers/net/can/c_can/c_can_platform.c            |    8 -
 drivers/net/ethernet/stmicro/stmmac/stmmac.h      |   41 -----
 drivers/net/ethernet/stmicro/stmmac/stmmac_main.c |   33 ++--
 drivers/usb/gadget/m66592-udc.c                   |    9 +-
 drivers/usb/gadget/m66592-udc.h                   |    5 -
 drivers/usb/gadget/r8a66597-udc.c                 |   11 +-
 drivers/usb/gadget/r8a66597-udc.h                 |    5 -
 drivers/usb/host/r8a66597-hcd.c                   |   12 --
 drivers/usb/host/r8a66597.h                       |    5 -
 drivers/usb/musb/musb_core.h                      |    8 -
 include/linux/clk.h                               |  168 +++++++++++++-------
 include/linux/platform_data/mv_usb.h              |    9 -
 16 files changed, 131 insertions(+), 216 deletions(-)

-- 
1.7.9


             reply	other threads:[~2012-05-08  3:52 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-08  3:52 Viresh Kumar [this message]
2012-05-08  3:52 ` [PATCH V3 Resend 00/12] clk: Add non CONFIG_HAVE_CLK routines Viresh Kumar
2012-05-08  3:52 ` [PATCH V3 Resend 01/12] " Viresh Kumar
2012-05-08  3:52   ` Viresh Kumar
2012-05-08  3:52 ` [PATCH V3 Resend 02/12] clk: Remove redundant depends on from drivers/Kconfig Viresh Kumar
2012-05-08  3:52   ` Viresh Kumar
2012-05-08  3:52 ` [PATCH V3 Resend 03/12] i2c/i2c-pxa: Remove conditional compilation of clk code Viresh Kumar
2012-05-08  3:52   ` Viresh Kumar
2012-05-08  3:52   ` Viresh Kumar
2012-05-08  3:52 ` [PATCH V3 Resend 04/12] usb/marvell: " Viresh Kumar
2012-05-08  3:52   ` Viresh Kumar
2012-05-08  3:52 ` [PATCH V3 Resend 05/12] usb/musb: " Viresh Kumar
2012-05-08  3:52   ` Viresh Kumar
2012-05-08  3:52 ` [PATCH V3 Resend 06/12] ata/pata_arasan: " Viresh Kumar
2012-05-08  3:52   ` Viresh Kumar
2012-05-08  3:52   ` Viresh Kumar
2012-05-08  3:52 ` [PATCH V3 Resend 07/12] ata/sata_mv: " Viresh Kumar
2012-05-08  3:52   ` Viresh Kumar
2012-05-08  3:52   ` Viresh Kumar
2012-05-17 20:42   ` Andrew Morton
2012-05-17 20:42     ` Andrew Morton
2012-05-17 20:42     ` Andrew Morton
2012-05-08  3:52 ` [PATCH V3 Resend 08/12] net/c_can: " Viresh Kumar
2012-05-08  3:52   ` Viresh Kumar
2012-05-10  1:22   ` David Miller
2012-05-10  1:22     ` David Miller
2012-05-08  3:52 ` [PATCH V3 Resend 09/12] net/stmmac: " Viresh Kumar
2012-05-08  3:52   ` Viresh Kumar
2012-05-10  1:22   ` David Miller
2012-05-10  1:22     ` David Miller
2012-05-10  1:22     ` David Miller
2012-05-08  3:52 ` [PATCH V3 Resend 10/12] gadget/m66592: " Viresh Kumar
2012-05-08  3:52   ` Viresh Kumar
2012-05-08  3:52 ` [PATCH V3 Resend 11/12] gadget/r8a66597: " Viresh Kumar
2012-05-08  3:52   ` Viresh Kumar
2012-05-08  3:52 ` [PATCH V3 Resend 12/12] usb/host/r8a66597: " Viresh Kumar
2012-05-08  3:52   ` Viresh Kumar

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=cover.1336448639.git.viresh.kumar@st.com \
    --to=viresh.kumar@st.com \
    --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.