Hi Andrzej, On 23 May 2014 13:17, Andrzej Hajda <a.hajda at samsung.com> 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 at linaro.org> >> --- >> 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
