[PATCH v4 14/34] drm/exynos: hdmi: remove the i2c drivers and use devtree (original) (raw)

Tomasz Figa tomasz.figa at gmail.com
Fri Feb 7 18:52:49 PST 2014

Hi,

On 30.01.2014 22:19, Sean Paul wrote:

From: Daniel Kurtz <djkurtz at chromium.org>

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 i2cclient. This patch also properly references the client so we don't lose it before we're done with it. Signed-off-by: Daniel Kurtz <djkurtz at chromium.org> [seanpaul changed to phandle lookup instead of using of node name] Signed-off-by: Sean Paul <seanpaul at chromium.org> --- Changes in v2: - Change include to linux/i2c.h instead of linux/ofi2c.h Changes in v3: None Changes in v4: - Changed to find phy via phandle instead of by name .../devicetree/bindings/video/exynoshdmi.txt | 5 ++ drivers/gpu/drm/exynos/Makefile | 1 - drivers/gpu/drm/exynos/exynosddc.c | 63 --------------------- drivers/gpu/drm/exynos/exynoshdmi.c | 59 +++++++++----------- drivers/gpu/drm/exynos/exynoshdmi.h | 23 -------- drivers/gpu/drm/exynos/exynoshdmiphy.c | 65 ---------------------- 6 files changed, 32 insertions(+), 184 deletions(-) delete mode 100644 drivers/gpu/drm/exynos/exynosddc.c delete mode 100644 drivers/gpu/drm/exynos/exynoshdmi.h delete mode 100644 drivers/gpu/drm/exynos/exynoshdmiphy.c diff --git a/Documentation/devicetree/bindings/video/exynoshdmi.txt b/Documentation/devicetree/bindings/video/exynoshdmi.txt index 50decf8..f9187a2 100644 --- a/Documentation/devicetree/bindings/video/exynoshdmi.txt +++ b/Documentation/devicetree/bindings/video/exynoshdmi.txt @@ -25,6 +25,9 @@ Required properties: sclkpixel. - clock-names: aliases as per driver requirements for above clock IDs: "hdmi", "sclkhdmi", "sclkpixel", "sclkhdmiphy" and "mouthdmi". +- 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

More information about the dri-devel mailing list