Messages in this thread | | | Date | Mon, 23 Sep 2013 10:09:53 +0200 | Subject | Re: [PATCH v3] mfd: add STw481x driver | From | Linus Walleij <> |
| |
On Mon, Sep 23, 2013 at 10:05 AM, Lee Jones <lee.jones@linaro.org> 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
| |