[PATCH v5 01/12] gpio: pxa: identify ed mask reg with platform data

Haojian Zhuang haojian.zhuang at gmail.com
Sun Mar 3 19:52:52 EST 2013


On Sat, Mar 2, 2013 at 5:52 PM, Grant Likely <grant.likely at secretlab.ca> wrote:
> On Mon, 25 Feb 2013 11:49:30 +0800, Haojian Zhuang <haojian.zhuang at linaro.org> wrote:
>> Avoid to judge whether edge mask register exists by CPU. Use platform
>> data to identify it instead. The gpio edge mask register exists in MMP
>> series SoC.
>>
>> Signed-off-by: Haojian Zhuang <haojian.zhuang at linaro.org>
>
> Hi Haojian,
>
> I don't think patches like this are a good idea. The driver already
> knows the difference between different types of chips. Adding /more/
> platform data is making life worse, not better. The diffstat is evidence
> of that:
>
>>  10 files changed, 65 insertions(+), 1 deletion(-)
>
> If you're looking to get rid of the pxa_gpio_nums() function, then a
> much better way to do it is to use a platform_device_id table and
> differentiate the type by the name. You can attach data to that table
> with the parameters for each chip. For example:
>
> struct pxa_gpio_id_data pxa_gpio_id_data[] = {
>         { .count = 84, type = PXA25X_GPIO, },
>         { .count = 89, type = PXA26X_GPIO, },
>         { .count = 127, type = MMP_GPIO, },
> };
>
> static const struct platform_device_id pxa_gpio_id_table[] = {
>         { "pxa25x-gpio", (unsigned long) &pxa_gpio_id_data[0], },
>         { "pxa65x-gpio", (unsigned long) &pxa_gpio_id_data[1], },
>         { "mmp-gpio", (unsigned long) &pxa_gpio_id_data[2], },
> };
>
> static struct platform_driver pxa_gpio_driver = {
>         .probe = pxa_gpio_probe,
>         .driver = {
>                 .name = "pxa-gpio",
>                 .of_match_table = of_match_ptr(pxa_gpio_dt_ids),
>         },
>         .id_table = pxa_gpio_id_table,
> };
>
> And that same data table can be used in the of_match_table too.
>
> g.
>

OK. I'll update it.

Thanks
Haojian



More information about the linux-arm-kernel mailing list