On Mon, Sep 23, 2013 at 10:05 AM, Lee Jones <lee.jo...@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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/