Messages in this thread | | | Date | Mon, 10 Feb 2014 20:50:02 +0530 | Subject | Re: [PATCH v2] mfd: MAX6650/6651 support | From | Sachin Kamat <> |
| |
On 10 February 2014 17:51, Lee Jones <lee.jones@linaro.org> wrote: >> > +#include <linux/device.h> >> > +#include <linux/mfd/core.h> >> > +#include <linux/module.h> >> > +#include <linux/i2c.h> >> >> Please arrange these alphabetically. > > Why?
1. It makes it easier to avoid adding duplicate includes. 2. Code looks more ordered/organized. 3. Prevents further clean up patches arranging them so :)
-- With warm regards, Sachin
| |