On Fri, Sep 27, 2013 at 10:02 AM, Sachin Kamat <sachin.kamat@xxxxxxxxxx> wrote: > On 27 September 2013 22:23, Bryan Wu <cooloney@xxxxxxxxx> wrote: >> On Fri, Sep 27, 2013 at 4:36 AM, Sachin Kamat <sachin.kamat@xxxxxxxxxx> wrote: >>> 'of_match_ptr' is defined in linux/of.h. Include it explicitly. >>> >> >> Why we need this? I think it is include from of_gpio.h or >> of_platform.h. And I tried to build these driver files you patched, >> there is no error or warning for this. >> >> Is there any error or warning you met? > > No, this is only a coding style best practice to include the header > files explicitly as is widely followed across the kernel. This will > avoid breakage in the future if the indirect inclusion is moved or > removed . > OK, I assumed some building failures. Please help me to fold all you 6 patches together and we can apply them in one patch. Thanks, -Bryan -- To unsubscribe from this list: send the line "unsubscribe linux-leds" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html