Hi Andrzej, On 23 May 2014 13:17, Andrzej Hajda <a.hajda@xxxxxxxxxxx> wrote: > Hi Sachin, > > On 05/22/2014 07:02 AM, Sachin Kamat wrote: >> i2c.h was included twice. >> >> Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx> >> --- >> drivers/gpu/drm/exynos/exynos_hdmi.c | 1 - >> 1 file changed, 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c b/drivers/gpu/drm/exynos/exynos_hdmi.c >> index ed6176e..d3f9684 100644 >> --- a/drivers/gpu/drm/exynos/exynos_hdmi.c >> +++ b/drivers/gpu/drm/exynos/exynos_hdmi.c >> @@ -34,7 +34,6 @@ >> #include <linux/io.h> >> #include <linux/of.h> >> #include <linux/of_address.h> >> -#include <linux/i2c.h> >> #include <linux/of_gpio.h> >> #include <linux/hdmi.h> >> #include <linux/component.h> >> > > It would be also good to order headers alphabetically. That is outside the scope of this patch and can be taken up separately if needed. -- With warm regards, Sachin _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel