[PATCH 6/6] pinctrl: sirf: enable the driver support new SiRFmarco SoC
Barry Song
21cnbao at gmail.com
Fri Sep 28 10:29:23 EDT 2012
Hi linus,
thanks very much!
2012/9/28 Linus Walleij <linus.walleij at linaro.org>:
> On Thu, Sep 27, 2012 at 11:56 AM, Barry Song <Barry.Song at csr.com> wrote:
>
>> From: Barry Song <Baohua.Song at csr.com>
>>
>> The driver supports old up SiRFprimaII SoCs, this patch makes it support
>> the new SiRFmarco as well.
>> SiRFmarco, as a SMP SoC, adds new SIRFSOC_GPIO_PAD_EN_CLR registers, to
>> disable GPIO pad, we should write 1 to the corresponding bit in the new
>> CLEAR register instead of writing 0 to SIRFSOC_GPIO_PAD_EN.
>>
>> Signed-off-by: Barry Song <Baohua.Song at csr.com>
>
> This doesn't apply to my tree, you must have some other changes that
> are not included in the patch set...
>
> Can you pls try to generate and test the patch against the pinctrl tree?
>
> I might apply it as per the "new driver" rule if I can convince myself that it
> doesn't destabilize the current driver.
this one actually depends on:
http://git.kernel.org/?p=linux/kernel/git/arm/arm-soc.git;a=commitdiff;h=056876f6c73406c06d530d16d020177f5ec4a0bd
which was applied to arm soc tree.
the only conflict comes from:
static const struct of_device_id pinmux_ids[] __devinitconst = {
{ .compatible = "sirf,prima2-pinctrl" },
+ { .compatible = "sirf,marco-pinctrl" },
{}
};
as commit 056876f6c73406c0 renamed the old
static const struct of_device_id pinmux_ids[] __devinitconst = {
{ .compatible = "sirf,prima2-gpio-pinmux" },
{}
};
to
static const struct of_device_id pinmux_ids[] __devinitconst = {
{ .compatible = "sirf,sirf,prima2-pinctrl" },
{}
};
so i think the best way is we hold on this patch for the moment, and
wait for 3.7-rc1.
all other 5 are critical fixes and have no merge conflict with arm-soc
tree as i have tried.
> Yours,
> Linus Walleij
Thanks
barry
More information about the linux-arm-kernel
mailing list