On Thu, Sep 30, 2010 at 01:48:17PM +0300, Amit Kucheria wrote:
> Add'ed linaro-dev and linux-arm-kernel to CC.
>
> Thanks Yong, some feeback follows inline.
>
> On 10 Sep 29, Yong Shen wrote:
> > From: Yong Shen <yong.s...@linaro.org>
> >
> > ---
> > arch/arm/Kconfig | 6 +
> > arch/arm/mach-mx5/Kconfig | 1 +
> > arch/arm/mach-mx5/board-mx51_babbage.c | 32 ++++
> > arch/arm/mach-mx5/clock-mx51.c | 53 ++++++
> > arch/arm/plat-mxc/Makefile | 2 +
> > arch/arm/plat-mxc/cpufreq.c | 282
> > ++++++++++++++++++++++++++++++++
> > arch/arm/plat-mxc/include/mach/mxc.h | 20 +++
> > 7 files changed, 396 insertions(+), 0 deletions(-)
> > create mode 100644 arch/arm/plat-mxc/cpufreq.c
> >
> > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> > index 4db064e..64ebbc0 100644
> > --- a/arch/arm/Kconfig
> > +++ b/arch/arm/Kconfig
> > @@ -1517,6 +1517,12 @@ if ARCH_HAS_CPUFREQ
> >
> > source "drivers/cpufreq/Kconfig"
> >
> > +config CPU_FREQ_IMX
> > + tristate "CPUfreq driver for i.MX CPUs"
> > + depends on ARCH_MXC && CPU_FREQ
> > + help
> > + This enables the CPUfreq driver for i.MX CPUs.
> > +
> > config CPU_FREQ_SA1100
> > bool
> >
> > diff --git a/arch/arm/mach-mx5/Kconfig b/arch/arm/mach-mx5/Kconfig
> > index 1576d51..5956fee 100644
> > --- a/arch/arm/mach-mx5/Kconfig
> > +++ b/arch/arm/mach-mx5/Kconfig
> > @@ -5,6 +5,7 @@ config ARCH_MX51
> > default y
> > select MXC_TZIC
> > select ARCH_MXC_IOMUX_V3
> > + select ARCH_HAS_CPUFREQ
> >
> > comment "MX5 platforms:"
> >
> > diff --git a/arch/arm/mach-mx5/board-mx51_babbage.c
> > b/arch/arm/mach-mx5/board-mx51_babbage.c
> > index ed885f9..e449e0b 100644
> > --- a/arch/arm/mach-mx5/board-mx51_babbage.c
> > +++ b/arch/arm/mach-mx5/board-mx51_babbage.c
> > @@ -43,6 +43,31 @@
> > #define MX51_USB_PLL_DIV_19_2_MHZ 0x01
> > #define MX51_USB_PLL_DIV_24_MHZ 0x02
> >
> > +struct cpu_wp *(*get_cpu_wp)(int *wp);
This should be moved to a generic place, otherwise it breaks the build
when multiple boards are selected.
> > +static int num_cpu_wp = 2;
>
> use sizeof(array) instead of hard coding this.
>
> > +/* working point(wp): 0 - 800MHz; 1 - 166.25MHz; */
> > +static struct cpu_wp cpu_wp_auto[] = {
> > + {
> > + .pll_rate = 800000000,
> > + .cpu_rate = 800000000,
> > + .pdf = 0,
> > + .mfi = 8,
> > + .mfd = 2,
> > + .mfn = 1,
> > + .cpu_podf = 0,
> > + .cpu_voltage = 1100000,},
> > + {
> > + .pll_rate = 800000000,
> > + .cpu_rate = 160000000,
> > + .pdf = 4,
> > + .mfi = 8,
> > + .mfd = 2,
> > + .mfn = 1,
> > + .cpu_podf = 4,
> > + .cpu_voltage = 850000,},
> > +};
>
>
> This data should be moved out to a separate file (e.g. mx51_ratetable.h)
> since it will be useful to other boards too.
>
> If other boards can have different rate tables (and they can, depending on
> the revision of the silicon), then we can either 'assemble' the correct rate
> table based on a flag field or have explicit separate rate tables for each
> silicon revision.
>
> In any case, I suspect that there will be some core rates that will be common
> across silicon revisions.
>
> > static struct platform_device *devices[] __initdata = {
> > &mxc_fec_device,
> > };
> > @@ -87,6 +112,12 @@ static struct imxuart_platform_data uart_pdata = {
> > .flags = IMXUART_HAVE_RTSCTS,
> > };
> >
> > +struct cpu_wp *mx51_babbage_get_cpu_wp(int *wp)
static
> > +{
> > + *wp = num_cpu_wp;
> > + return cpu_wp_auto;
> > +}
> > +
> > static inline void mxc_init_imx_uart(void)
> > {
> > mxc_register_device(&mxc_uart_device0, &uart_pdata);
> > @@ -246,6 +277,7 @@ static void __init mxc_board_init(void)
> >
> > static void __init mx51_babbage_timer_init(void)
> > {
> > + get_cpu_wp = mx51_babbage_get_cpu_wp;
It looks strange to have a global function pointer which gets
overwritten by the boards. I would expect a
imx_add_cpu_workpoints(cpu_wp_auto, ARRAY_SIZE(cpu_wp_auto));
here.
> > mx51_clocks_init(32768, 24000000, 22579200, 0);
> > }
> >
> > diff --git a/arch/arm/mach-mx5/clock-mx51.c b/arch/arm/mach-mx5/clock-mx51.c
> > index d9f612d..f2488e6 100644
> > --- a/arch/arm/mach-mx5/clock-mx51.c
> > +++ b/arch/arm/mach-mx5/clock-mx51.c
> > @@ -14,6 +14,7 @@
> > #include <linux/delay.h>
> > #include <linux/clk.h>
> > #include <linux/io.h>
> > +#include <linux/time.h>
unneeded include
> >
> > #include <asm/clkdev.h>
> > #include <asm/div64.h>
> > @@ -28,6 +29,11 @@
> > static unsigned long external_high_reference, external_low_reference;
> > static unsigned long oscillator_reference, ckih2_reference;
> >
> > +extern struct cpu_wp *(*get_cpu_wp)(int *wp);
> > +static int cpu_wp_nr;
> > +static int cpu_curr_wp;
> > +static struct cpu_wp *cpu_wp_tbl;
> > +
> > static struct clk osc_clk;
> > static struct clk pll1_main_clk;
> > static struct clk pll1_sw_clk;
> > @@ -38,7 +44,9 @@ static struct clk periph_apm_clk;
> > static struct clk ahb_clk;
> > static struct clk ipg_clk;
> > static struct clk usboh3_clk;
> > +static void __iomem *pll1_base;
> >
> > +#define SPIN_DELAY 1000000 /* in nanoseconds */
unused
> > #define MAX_DPLL_WAIT_TRIES 1000 /* 1000 * udelay(1) = 1ms */
> >
> > static int _clk_ccgr_enable(struct clk *clk)
> > @@ -330,6 +338,32 @@ static int _clk_lp_apm_set_parent(struct clk *clk,
> > struct clk *parent)
> > return 0;
> > }
> >
> > +/*!
> > + * Setup cpu clock based on working point.
> > + * @param wp cpu freq working point
> > + * @return 0 on success or error code on failure.
> > + */
> > +int cpu_clk_set_wp(int wp)
static
> > +{
> > + struct cpu_wp *p;
> > + u32 reg;
> > +
> > + if (wp == cpu_curr_wp)
> > + return 0;
> > +
> > + p = &cpu_wp_tbl[wp];
> > +
> > + /*use post divider to change freq
> > + */
> > + reg = __raw_readl(MXC_CCM_CACRR);
> > + reg &= ~MXC_CCM_CACRR_ARM_PODF_MASK;
> > + reg |= cpu_wp_tbl[wp].cpu_podf << MXC_CCM_CACRR_ARM_PODF_OFFSET;
> > + __raw_writel(reg, MXC_CCM_CACRR);
We have a simple divider here. Why do we need a reference to struct
cpu_wp then? This code could look much simpler.
(side note: I'm aware that the original Freescale code is also able
to change the cpu frequency using the pll instead of the divider, but is
this really necessary?)
> > + cpu_curr_wp = wp;
> > +
> > + return 0;
> > +}
> > +
> > static unsigned long clk_arm_get_rate(struct clk *clk)
> > {
> > u32 cacrr, div;
> > @@ -342,6 +376,20 @@ static unsigned long clk_arm_get_rate(struct clk *clk)
> > return parent_rate / div;
> > }
> >
> > +int _clk_cpu_set_rate(struct clk *clk, unsigned long rate)
> > +{
> > + u32 i;
> > + for (i = 0; i < cpu_wp_nr; i++) {
> > + if (rate == cpu_wp_tbl[i].cpu_rate)
> > + break;
> > + }
> > + if (i >= cpu_wp_nr)
> > + return -EINVAL;
> > + cpu_clk_set_wp(i);
> > +
> > + return 0;
> > +}
> > +
> > static int _clk_periph_apm_set_parent(struct clk *clk, struct clk *parent)
> > {
> > u32 reg, mux;
> > @@ -694,6 +742,7 @@ static struct clk periph_apm_clk = {
> > static struct clk cpu_clk = {
> > .parent = &pll1_sw_clk,
> > .get_rate = clk_arm_get_rate,
> > + .set_rate = _clk_cpu_set_rate,
> > };
> >
> > static struct clk ahb_clk = {
> > @@ -821,6 +870,7 @@ static struct clk_lookup lookups[] = {
> > _REGISTER_CLOCK("mxc-ehci.1", "usb_ahb", ahb_clk)
> > _REGISTER_CLOCK("fsl-usb2-udc", "usb", usboh3_clk)
> > _REGISTER_CLOCK("fsl-usb2-udc", "usb_ahb", ahb_clk)
> > + _REGISTER_CLOCK(NULL, "cpu_clk", cpu_clk)
> > };
> >
> > static void clk_tree_init(void)
> > @@ -848,10 +898,13 @@ int __init mx51_clocks_init(unsigned long ckil,
> > unsigned long osc,
> > {
> > int i;
> >
> > + pll1_base = ioremap(MX51_PLL1_BASE_ADDR, SZ_4K);
> > +
this is unused.
> > external_low_reference = ckil;
> > external_high_reference = ckih1;
> > ckih2_reference = ckih2;
> > oscillator_reference = osc;
> > + cpu_wp_tbl = get_cpu_wp(&cpu_wp_nr);
> >
> > for (i = 0; i < ARRAY_SIZE(lookups); i++)
> > clkdev_add(&lookups[i]);
> > diff --git a/arch/arm/plat-mxc/Makefile b/arch/arm/plat-mxc/Makefile
> > index 895bc3c..c1bb400 100644
> > --- a/arch/arm/plat-mxc/Makefile
> > +++ b/arch/arm/plat-mxc/Makefile
> > @@ -17,6 +17,8 @@ obj-$(CONFIG_USB_EHCI_MXC) += ehci.o
> > obj-$(CONFIG_MXC_ULPI) += ulpi.o
> > obj-$(CONFIG_ARCH_MXC_AUDMUX_V1) += audmux-v1.o
> > obj-$(CONFIG_ARCH_MXC_AUDMUX_V2) += audmux-v2.o
> > +# CPU FREQ support
> > +obj-$(CONFIG_CPU_FREQ_IMX) += cpufreq.o
> > ifdef CONFIG_SND_IMX_SOC
> > obj-y += ssi-fiq.o
> > obj-y += ssi-fiq-ksym.o
> > diff --git a/arch/arm/plat-mxc/cpufreq.c b/arch/arm/plat-mxc/cpufreq.c
> > new file mode 100644
> > index 0000000..cae40f1
> > --- /dev/null
> > +++ b/arch/arm/plat-mxc/cpufreq.c
> > @@ -0,0 +1,282 @@
> > +/*
> > + * Copyright (C) 2010 Freescale Semiconductor, Inc. All Rights Reserved.
> > + */
> > +
> > +/*
> > + * The code contained herein is licensed under the GNU General Public
> > + * License. You may obtain a copy of the GNU General Public License
> > + * Version 2 or later at the following locations:
> > + *
> > + * http://www.opensource.org/licenses/gpl-license.html
> > + * http://www.gnu.org/copyleft/gpl.html
> > + */
> > +
> > +/*!
> > + * @file cpufreq.c
> > + *
> > + * @brief A driver for the Freescale Semiconductor i.MXC CPUfreq module.
> > + *
> > + * The CPUFREQ driver is for controling CPU frequency. It allows you to
> > change
> > + * the CPU clock speed on the fly.
> > + *
> > + * @ingroup PM
> > + */
>
> Fix these comments to follow the kernel commenting style.
>
> > +#include <linux/types.h>
> > +#include <linux/kernel.h>
> > +#include <linux/cpufreq.h>
> > +#include <linux/init.h>
> > +#include <linux/proc_fs.h>
> > +#include <linux/regulator/consumer.h>
> > +#include <linux/clk.h>
> > +#include <linux/delay.h>
> > +#include <linux/io.h>
> > +#include <mach/hardware.h>
> > +#include <asm/setup.h>
> > +#include <mach/clock.h>
> > +#include <asm/cacheflush.h>
> > +#include <linux/hrtimer.h>
> > +
> > +int cpu_freq_khz_min;
> > +int cpu_freq_khz_max;
> > +int arm_lpm_clk;
> > +int arm_normal_clk;
> > +int cpufreq_suspended;
> > +int cpufreq_trig_needed;
> > +
> > +static struct clk *cpu_clk;
> > +static struct cpufreq_frequency_table imx_freq_table[4];
Three frequencies should be enough for everyone, right? This should be
dynamically allocated like in other cpufreq drivers.
> > +
> > +extern int set_low_bus_freq(void);
> > +extern int set_high_bus_freq(int high_bus_speed);
> > +extern int low_freq_bus_used(void);
declared but never used.
> > +
> > +extern struct cpu_wp *(*get_cpu_wp)(int *wp);
> > +static int cpu_wp_nr;
> > +static struct cpu_wp *cpu_wp_tbl;
> > +
> > +int set_cpu_freq(int freq)
> > +{
> > + int ret = 0;
> > + int org_cpu_rate;
> > + int gp_volt = 0;
> > + int i;
> > +
> > + org_cpu_rate = clk_get_rate(cpu_clk);
> > + if (org_cpu_rate == freq)
> > + return ret;
> > +
> > + for (i = 0; i < cpu_wp_nr; i++) {
> > + if (freq == cpu_wp_tbl[i].cpu_rate)
> > + gp_volt = cpu_wp_tbl[i].cpu_voltage;
> > + }
> > +
> > + if (gp_volt == 0)
> > + return ret;
> > +
> > + ret = clk_set_rate(cpu_clk, freq);
> > + if (ret != 0) {
> > + printk(KERN_DEBUG "cannot set CPU clock rate\n");
> > + return ret;
> > + }
> > +
Please remove trailing whitespaces.
> > + return ret;
> > +}
> > +
> > +static int mxc_verify_speed(struct cpufreq_policy *policy)
> > +{
> > + if (policy->cpu != 0)
> > + return -EINVAL;
> > +
> > + return cpufreq_frequency_table_verify(policy, imx_freq_table);
> > +}
> > +
> > +static unsigned int mxc_get_speed(unsigned int cpu)
> > +{
> > + if (cpu)
> > + return 0;
> > +
> > + return clk_get_rate(cpu_clk) / 1000;
> > +}
> > +
> > +static int calc_frequency_khz(int target, unsigned int relation)
> > +{
> > + int i;
> > +
> > + if ((target * 1000) == clk_get_rate(cpu_clk))
> > + return target;
> > +
> > + if (relation == CPUFREQ_RELATION_H) {
> > + for (i = cpu_wp_nr - 1; i >= 0; i--) {
> > + if (imx_freq_table[i].frequency <= target)
> > + return imx_freq_table[i].frequency;
> > + }
> > + } else if (relation == CPUFREQ_RELATION_L) {
> > + for (i = 0; i < cpu_wp_nr; i++) {
> > + if (imx_freq_table[i].frequency >= target)
> > + return imx_freq_table[i].frequency;
> > + }
> > + }
It looks like cpufreq_frequency_table_target could help here.
> > + printk(KERN_ERR "Error: No valid cpufreq relation\n");
> > + return cpu_freq_khz_max;
> > +}
> > +
> > +static int mxc_set_target(struct cpufreq_policy *policy,
> > + unsigned int target_freq, unsigned int relation)
> > +{
> > + struct cpufreq_freqs freqs;
> > + int freq_Hz;
> > + int ret = 0;
> > +
> > + if (cpufreq_suspended) {
> > + target_freq = clk_get_rate(cpu_clk) / 1000;
> > + freq_Hz = calc_frequency_khz(target_freq, relation) * 1000;
> > + if (freq_Hz == arm_lpm_clk)
> > + freqs.old = cpu_wp_tbl[cpu_wp_nr - 2].cpu_rate / 1000;
> > + else
> > + freqs.old = arm_lpm_clk / 1000;
> > +
> > + freqs.new = freq_Hz / 1000;
> > + freqs.cpu = 0;
> > + freqs.flags = 0;
> > + cpufreq_notify_transition(&freqs, CPUFREQ_PRECHANGE);
> > + cpufreq_notify_transition(&freqs, CPUFREQ_POSTCHANGE);
> > + return ret;
> > + }
> > + /*
> > + * Some governors do not respects CPU and policy lower limits
> > + * which leads to bad things (division by zero etc), ensure
> > + * that such things do not happen.
> > + */
>
> Isn't that a bug in the governor? Can you explain a bit?
>
> > + if (target_freq < policy->cpuinfo.min_freq)
> > + target_freq = policy->cpuinfo.min_freq;
> > +
> > + if (target_freq < policy->min)
> > + target_freq = policy->min;
> > +
> > + freq_Hz = calc_frequency_khz(target_freq, relation) * 1000;
> > +
> > + freqs.old = clk_get_rate(cpu_clk) / 1000;
> > + freqs.new = freq_Hz / 1000;
> > + freqs.cpu = 0;
> > + freqs.flags = 0;
> > + cpufreq_notify_transition(&freqs, CPUFREQ_PRECHANGE);
> > +
> > + if (freqs.old != freqs.new)
> > + ret = set_cpu_freq(freq_Hz);
> > +
> > + cpufreq_notify_transition(&freqs, CPUFREQ_POSTCHANGE);
> > +
> > + return ret;
> > +}
> > +
> > +static int __init mxc_cpufreq_driver_init(struct cpufreq_policy *policy)
> > +{
> > + int ret;
> > + int i;
> > +
> > + printk(KERN_INFO "i.MXC CPU frequency driver\n");
> > +
> > + if (policy->cpu != 0)
> > + return -EINVAL;
> > +
> > + cpu_clk = clk_get(NULL, "cpu_clk");
> > + if (IS_ERR(cpu_clk)) {
> > + printk(KERN_ERR "%s: failed to get cpu clock\n", __func__);
> > + return PTR_ERR(cpu_clk);
> > + }
> > +
> > + /* Set the current working point. */
> > + cpu_wp_tbl = get_cpu_wp(&cpu_wp_nr);
> > +
> > + cpu_freq_khz_min = cpu_wp_tbl[0].cpu_rate / 1000;
> > + cpu_freq_khz_max = cpu_wp_tbl[0].cpu_rate / 1000;
> > +
> > + for (i = 0; i < cpu_wp_nr; i++) {
> > + imx_freq_table[cpu_wp_nr - 1 - i].index = cpu_wp_nr - i;
>
> cpu_wp_nr = 2 here
>
> 1st iteration of for loop:
> imx_freq_table[2 - 1 - 0].index = 2 - 0
> so, imx_freq_table[1].index = 2
>
> 2nd iteration:
> imx_freq_table[2 - 1 - 1].index = 2 - 1
> imx_freq_table[0].index = 1
>
> So you're trying to reverse the table order? Why not just sort the table date
> in the way you want and add a comment on the top to keep it sorted.
>
> > + imx_freq_table[cpu_wp_nr - 1 - i].frequency =
> > + cpu_wp_tbl[i].cpu_rate / 1000;
> > +
> > + if ((cpu_wp_tbl[i].cpu_rate / 1000) < cpu_freq_khz_min)
> > + cpu_freq_khz_min = cpu_wp_tbl[i].cpu_rate / 1000;
> > +
> > + if ((cpu_wp_tbl[i].cpu_rate / 1000) > cpu_freq_khz_max)
> > + cpu_freq_khz_max = cpu_wp_tbl[i].cpu_rate / 1000;
> > + }
> > +
> > + imx_freq_table[i].index = 0;
>
> imx_freq_table[i].index = i ?
>
> > + imx_freq_table[i].frequency = CPUFREQ_TABLE_END;
> > +
> > + policy->cur = clk_get_rate(cpu_clk) / 1000;
> > + policy->governor = CPUFREQ_DEFAULT_GOVERNOR;
> > + policy->min = policy->cpuinfo.min_freq = cpu_freq_khz_min;
> > + policy->max = policy->cpuinfo.max_freq = cpu_freq_khz_max;
> > +
> > + arm_lpm_clk = cpu_freq_khz_min * 1000;
> > + arm_normal_clk = cpu_freq_khz_max * 1000;
> > +
> > + /* Manual states, that PLL stabilizes in two CLK32 periods */
> > + policy->cpuinfo.transition_latency = 10;
> > +
> > + ret = cpufreq_frequency_table_cpuinfo(policy, imx_freq_table);
> > +
> > + if (ret < 0) {
> > + clk_put(cpu_clk);
> > + printk(KERN_ERR "%s: failed to register i.MXC CPUfreq\n",
> > + __func__);
> > + return ret;
> > + }
> > +
> > + cpufreq_frequency_table_get_attr(imx_freq_table, policy->cpu);
> > + return 0;
> > +}
> > +
> > +static int mxc_cpufreq_suspend(struct cpufreq_policy *policy,
> > + pm_message_t state)
> > +{
> > + return 0;
> > +}
>
> Get rid of these, since you don't use them.
>
> > +static int mxc_cpufreq_resume(struct cpufreq_policy *policy)
> > +{
> > + return 0;
> > +}
>
> Same here.
>
> > +static int mxc_cpufreq_driver_exit(struct cpufreq_policy *policy)
> > +{
> > + cpufreq_frequency_table_put_attr(policy->cpu);
> > +
> > + /* Reset CPU to 665MHz */
> > + set_cpu_freq(arm_normal_clk);
> > + clk_put(cpu_clk);
> > + return 0;
> > +}
> > +
> > +static struct cpufreq_driver mxc_driver = {
> > + .flags = CPUFREQ_STICKY,
> > + .verify = mxc_verify_speed,
> > + .target = mxc_set_target,
> > + .get = mxc_get_speed,
> > + .init = mxc_cpufreq_driver_init,
>
> mxc_cpufreq_init is ok. Lose the driver.
>
> > + .exit = mxc_cpufreq_driver_exit,
>
> same.
>
> > + .suspend = mxc_cpufreq_suspend,
> > + .resume = mxc_cpufreq_resume,
>
> Get rid of suspend/resume
>
> > + .name = "imx",
> > +};
> > +
> > +static int __devinit mxc_cpufreq_init(void)
> > +{
> > + return cpufreq_register_driver(&mxc_driver);
> > +}
> > +
> > +static void mxc_cpufreq_exit(void)
> > +{
> > + cpufreq_unregister_driver(&mxc_driver);
> > +}
> > +
> > +module_init(mxc_cpufreq_init);
> > +module_exit(mxc_cpufreq_exit);
> > +
> > +MODULE_AUTHOR("Freescale Semiconductor, Inc.");
> > +MODULE_DESCRIPTION("CPUfreq driver for i.MX");
> > +MODULE_LICENSE("GPL");
> > diff --git a/arch/arm/plat-mxc/include/mach/mxc.h
> > b/arch/arm/plat-mxc/include/mach/mxc.h
> > index a790bf2..14003b9 100644
> > --- a/arch/arm/plat-mxc/include/mach/mxc.h
> > +++ b/arch/arm/plat-mxc/include/mach/mxc.h
> > @@ -133,6 +133,26 @@ extern unsigned int __mxc_cpu_type;
> > # define cpu_is_mxc91231() (0)
> > #endif
> >
> > +#ifndef __ASSEMBLY__
> > +
> > +struct cpu_wp {
> > + u32 pll_reg;
> > + u32 pll_rate;
> > + u32 cpu_rate;
> > + u32 pdr0_reg;
> > + u32 pdf;
> > + u32 mfi;
> > + u32 mfd;
> > + u32 mfn;
> > + u32 cpu_voltage;
> > + u32 cpu_podf;
> > +};
> > +
> > +#ifndef CONFIG_ARCH_MX5
> > +struct cpu_wp *get_cpu_wp(int *wp);
> > +#endif
> > +#endif
> > +
> > #if defined(CONFIG_ARCH_MX3) || defined(CONFIG_ARCH_MX2)
> > /* These are deprecated, use mx[23][157]_setup_weimcs instead. */
> > #define CSCR_U(n) (IO_ADDRESS(WEIM_BASE_ADDR + n * 0x10))
> > --
> > 1.6.3.3
> >
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-ker...@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev
