public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Rabin Vincent <rabin@rab.in>
To: Amit Kucheria <amit.kucheria@canonical.com>
Cc: List Linux Kernel <linux-kernel@vger.kernel.org>,
	linux@arm.linux.org.uk, Dinh.Nguyen@freescale.com,
	s.hauer@pengutronix.de, grant.likely@secretlab.ca,
	r.herring@freescale.com, linux-arm-kernel@lists.infradead.org,
	daniel@caiaq.de, bryan.wu@canonical.com,
	valentin.longchamp@epfl.ch
Subject: Re: [PATCHv2 05/11] mxc: Core support for i.MX5 series of processors from Freescale
Date: Wed, 3 Feb 2010 21:38:36 +0530	[thread overview]
Message-ID: <20100203160835.GA1928@debian> (raw)
In-Reply-To: <f10f79e7313ecdeeb59ab881795baedf2430c636.1265173480.git.amit.kucheria@canonical.com>

On Tue, Feb 02, 2010 at 09:16:27PM -0800, Amit Kucheria wrote:
> diff --git a/arch/arm/mach-mx5/clock.c b/arch/arm/mach-mx5/clock.c
[...]
> +static struct clk_lookup lookups[] __initdata = {
> +	_REGISTER_CLOCK("imx-uart.0", NULL, uart1_clk)
> +	_REGISTER_CLOCK("imx-uart.1", NULL, uart2_clk)
> +	_REGISTER_CLOCK("imx-uart.2", NULL, uart3_clk)
> +	_REGISTER_CLOCK(NULL, "gpt", gpt_clk)
> +	_REGISTER_CLOCK("fec.0", NULL, fec_clk)
> +};

This shouldn't be __initdata.  clk_get() uses this, and modules may use
clk_get().

Rabin

  parent reply	other threads:[~2010-02-03 16:07 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-03  5:16 [PATCHv2 00/11] Base support for Freescale i.MX51 SoC platform Amit Kucheria
     [not found] ` <cover.1265173480.git.amit.kucheria@canonical.com>
2010-02-03  5:16   ` [PATCHv2 01/11] arm: mxc: TrustZone interrupt controller (TZIC) for i.MX5 family Amit Kucheria
2010-02-03  6:23     ` Eric Miao
2010-02-03  9:45       ` Sascha Hauer
2010-02-03 13:24       ` Amit Kucheria
2010-02-03 15:09         ` Eric Miao
2010-02-04  0:54           ` Eric Miao
2010-02-04 17:09         ` Nguyen Dinh-R00091
2010-02-03  5:16   ` [PATCHv2 02/11] mxc timer: refactor timer code to use timer versions Amit Kucheria
2010-02-03 16:23     ` Grant Likely
2010-02-03  5:16   ` [PATCHv2 03/11] mxc: Fix Drive Strength Field in the IOMUX controller Amit Kucheria
2010-02-03  6:29     ` Eric Miao
2010-02-03  9:40       ` Sascha Hauer
2010-02-03 16:27     ` Grant Likely
2010-02-04  0:25       ` Amit Kucheria
2010-02-03  5:16   ` [PATCHv2 04/11] mxc: changes to common plat-mxc code to add support for i.MX5 Amit Kucheria
2010-02-03  6:43     ` Eric Miao
2010-02-03  9:49       ` Sascha Hauer
2010-02-03 13:38         ` Amit Kucheria
2010-02-03 15:16           ` Eric Miao
2010-02-03 16:35     ` Grant Likely
2010-02-03  5:16   ` [PATCHv2 05/11] mxc: Core support for i.MX5 series of processors from Freescale Amit Kucheria
2010-02-03  7:03     ` Eric Miao
2010-02-03 14:20       ` Amit Kucheria
2010-02-03  9:24     ` Russell King - ARM Linux
2010-02-03 11:04     ` Sascha Hauer
2010-02-03 20:07       ` Amit Kucheria
2010-02-03 16:08     ` Rabin Vincent [this message]
2010-02-03  5:16   ` [PATCHv2 06/11] mxc: enable support for Freescale i.MX5 series of processors Amit Kucheria
2010-02-03  5:16   ` [PATCHv2 07/11] mxc: Add support for the Babbage board Amit Kucheria
2010-02-03 11:10     ` Sascha Hauer
2010-02-03  5:16   ` [PATCHv2 08/11] fec: fix uninitialized rx buffer usage Amit Kucheria
2010-02-03 16:46     ` Grant Likely
2010-02-03 18:33       ` Amit Kucheria
2010-02-03 18:38         ` Grant Likely
2010-02-03 20:23           ` Grant Likely
2010-02-03  5:16   ` [PATCHv2 09/11] fec: Add LAN8700 phy support Amit Kucheria
2010-02-03  5:16   ` [PATCHv2 10/11] fec: Add ARCH_MX5 as a dependency Amit Kucheria
2010-02-03  5:16   ` [PATCHv2 11/11] mxc: Add imx51_defconfig Amit Kucheria
2010-02-05  6:48     ` Sascha Hauer

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=20100203160835.GA1928@debian \
    --to=rabin@rab.in \
    --cc=Dinh.Nguyen@freescale.com \
    --cc=amit.kucheria@canonical.com \
    --cc=bryan.wu@canonical.com \
    --cc=daniel@caiaq.de \
    --cc=grant.likely@secretlab.ca \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=r.herring@freescale.com \
    --cc=s.hauer@pengutronix.de \
    --cc=valentin.longchamp@epfl.ch \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox