From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomasz Figa Subject: Re: [PATCH v4 14/34] drm/exynos: hdmi: remove the i2c drivers and use devtree Date: Sat, 08 Feb 2014 03:52:49 +0100 Message-ID: <52F59C01.3000001@gmail.com> References: <1391116773-28471-1-git-send-email-seanpaul@chromium.org> <1391116773-28471-15-git-send-email-seanpaul@chromium.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ee0-f49.google.com (mail-ee0-f49.google.com [74.125.83.49]) by gabe.freedesktop.org (Postfix) with ESMTP id 17155FB246 for ; Fri, 7 Feb 2014 18:52:54 -0800 (PST) Received: by mail-ee0-f49.google.com with SMTP id d17so1857752eek.8 for ; Fri, 07 Feb 2014 18:52:54 -0800 (PST) In-Reply-To: <1391116773-28471-15-git-send-email-seanpaul@chromium.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces@lists.freedesktop.org Errors-To: dri-devel-bounces@lists.freedesktop.org To: Sean Paul , dri-devel@lists.freedesktop.org, inki.dae@samsung.com Cc: marcheu@chromium.org List-Id: dri-devel@lists.freedesktop.org Hi, On 30.01.2014 22:19, Sean Paul wrote: > From: Daniel Kurtz > > The i2c client was previously being passed into the hdmi driver via a > dedicated i2c driver, and then a global variable. This patch removes all > of that and just uses the device tree to get the i2c_client. This patch > also properly references the client so we don't lose it before we're > done with it. > > Signed-off-by: Daniel Kurtz > [seanpaul changed to phandle lookup instead of using of node name] > Signed-off-by: Sean Paul > --- > > Changes in v2: > - Change include to linux/i2c.h instead of linux/of_i2c.h > Changes in v3: None > Changes in v4: > - Changed to find phy via phandle instead of by name > > .../devicetree/bindings/video/exynos_hdmi.txt | 5 ++ > drivers/gpu/drm/exynos/Makefile | 1 - > drivers/gpu/drm/exynos/exynos_ddc.c | 63 --------------------- > drivers/gpu/drm/exynos/exynos_hdmi.c | 59 +++++++++----------- > drivers/gpu/drm/exynos/exynos_hdmi.h | 23 -------- > drivers/gpu/drm/exynos/exynos_hdmiphy.c | 65 ---------------------- > 6 files changed, 32 insertions(+), 184 deletions(-) > delete mode 100644 drivers/gpu/drm/exynos/exynos_ddc.c > delete mode 100644 drivers/gpu/drm/exynos/exynos_hdmi.h > delete mode 100644 drivers/gpu/drm/exynos/exynos_hdmiphy.c > > diff --git a/Documentation/devicetree/bindings/video/exynos_hdmi.txt b/Documentation/devicetree/bindings/video/exynos_hdmi.txt > index 50decf8..f9187a2 100644 > --- a/Documentation/devicetree/bindings/video/exynos_hdmi.txt > +++ b/Documentation/devicetree/bindings/video/exynos_hdmi.txt > @@ -25,6 +25,9 @@ Required properties: > sclk_pixel. > - clock-names: aliases as per driver requirements for above clock IDs: > "hdmi", "sclk_hdmi", "sclk_pixel", "sclk_hdmiphy" and "mout_hdmi". > +- ddc: phandle to the hdmi ddc node > +- phy: phandle to the hdmi phy node > + Adding new required properties to an already defined binding is breaking backwards compatibility, which is supposed to be preserved, since DT is an ABI. Now, I'm not really a big fan of DT stability, but if we decide to maintain it for other Exynos drivers as well (e.g. USB), then we probably should do the same here... Best regards, Tomasz