Re: [PATCH v3] mfd: add STw481x driver

From: Linus Walleij
Date: Mon Sep 23 2013 - 04:10:00 EST


On Mon, Sep 23, 2013 at 10:05 AM, Lee Jones <lee.jones@xxxxxxxxxx> wrote:

> Nit:
>
>> +static const struct i2c_device_id stw481x_id[] = {
>> + { "stw481x", 0 },
>> + { }
>
> Where we have '{ }'.
>
>> +};
>> +
>> +static const struct of_device_id stw481x_match[] = {
>> + { .compatible = "st,stw4810", },
>> + { .compatible = "st,stw4811", },
>> + {},
>
> And just down here the same function is provided with '{},'.
>
> I'm happy to change it myself, rather that asking you to resubmit.
>
> Which to you prefer?

Pls fix it if you have time!

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/