Re: [U-Boot] [PATCH v3 3/5] video: add anx6345 DM driver

2017-10-27 Thread Anatolij Gustschin
On Thu, 26 Oct 2017 18:29:39 +0200
Anatolij Gustschin ag...@denx.de wrote:

> From: Vasily Khoruzhick 
> 
> This is a eDP bridge similar to ANX9804, it allows to connect eDP panels
> to the chips that can output only parallel signal
> 
> Signed-off-by: Vasily Khoruzhick 
> Signed-off-by: Anatolij Gustschin 
> ---
> v3: - fixed checkpatch errors/warnings
> v2: - no changes
> 
>  drivers/video/bridge/Kconfig   |   8 +
>  drivers/video/bridge/Makefile  |   1 +
>  drivers/video/bridge/anx6345.c | 426 
> +
>  3 files changed, 435 insertions(+)
>  create mode 100644 drivers/video/bridge/anx6345.c

Applied to u-boot-video/master, thanks!

--
Anatolij
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[U-Boot] [PATCH v3 3/5] video: add anx6345 DM driver

2017-10-26 Thread Anatolij Gustschin
From: Vasily Khoruzhick 

This is a eDP bridge similar to ANX9804, it allows to connect eDP panels
to the chips that can output only parallel signal

Signed-off-by: Vasily Khoruzhick 
Signed-off-by: Anatolij Gustschin 
---
v3: - fixed checkpatch errors/warnings
v2: - no changes

 drivers/video/bridge/Kconfig   |   8 +
 drivers/video/bridge/Makefile  |   1 +
 drivers/video/bridge/anx6345.c | 426 +
 3 files changed, 435 insertions(+)
 create mode 100644 drivers/video/bridge/anx6345.c

diff --git a/drivers/video/bridge/Kconfig b/drivers/video/bridge/Kconfig
index 2a3b6c4bee..765f7380b8 100644
--- a/drivers/video/bridge/Kconfig
+++ b/drivers/video/bridge/Kconfig
@@ -25,3 +25,11 @@ config VIDEO_BRIDGE_NXP_PTN3460
  signalling) converter. It enables an LVDS LCD panel to be connected
  to an eDP output device such as an SoC that lacks LVDS capability,
  or where LVDS requires too many signals to route on the PCB.
+
+config VIDEO_BRIDGE_ANALOGIX_ANX6345
+   bool "Support Analogix ANX6345 RGB->DP bridge"
+   depends on VIDEO_BRIDGE
+   select DM_I2C
+   help
+The Analogix ANX6345 is RGB-to-DP converter. It enables an eDP LCD
+panel to be connected to an parallel LCD interface.
diff --git a/drivers/video/bridge/Makefile b/drivers/video/bridge/Makefile
index ce731fa4ca..2a746c6f8b 100644
--- a/drivers/video/bridge/Makefile
+++ b/drivers/video/bridge/Makefile
@@ -7,3 +7,4 @@
 obj-$(CONFIG_VIDEO_BRIDGE) += video-bridge-uclass.o
 obj-$(CONFIG_VIDEO_BRIDGE_PARADE_PS862X) += ps862x.o
 obj-$(CONFIG_VIDEO_BRIDGE_NXP_PTN3460) += ptn3460.o
+obj-$(CONFIG_VIDEO_BRIDGE_ANALOGIX_ANX6345) += anx6345.o
diff --git a/drivers/video/bridge/anx6345.c b/drivers/video/bridge/anx6345.c
new file mode 100644
index 00..0a94affb9f
--- /dev/null
+++ b/drivers/video/bridge/anx6345.c
@@ -0,0 +1,426 @@
+/*
+ * Copyright (C) 2017 Vasily Khoruzhick 
+ *
+ * SPDX-License-Identifier:GPL-2.0+
+ */
+
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include "../anx98xx-edp.h"
+
+#define DP_MAX_LINK_RATE   0x001
+#define DP_MAX_LANE_COUNT  0x002
+#define DP_MAX_LANE_COUNT_MASK 0x1f
+
+DECLARE_GLOBAL_DATA_PTR;
+
+struct anx6345_priv {
+   u8 edid[EDID_SIZE];
+};
+
+static int anx6345_write(struct udevice *dev, unsigned int addr_off,
+unsigned char reg_addr, unsigned char value)
+{
+   uint8_t buf[2];
+   struct i2c_msg msg;
+   int ret;
+
+   msg.addr = addr_off;
+   msg.flags = 0;
+   buf[0] = reg_addr;
+   buf[1] = value;
+   msg.buf = buf;
+   msg.len = 2;
+   ret = dm_i2c_xfer(dev, &msg, 1);
+   if (ret) {
+   debug("%s: write failed, reg=%#x, value=%#x, ret=%d\n",
+ __func__, reg_addr, value, ret);
+   return ret;
+   }
+
+   return 0;
+}
+
+static int anx6345_read(struct udevice *dev, unsigned int addr_off,
+   unsigned char reg_addr, unsigned char *value)
+{
+   uint8_t addr, val;
+   struct i2c_msg msg[2];
+   int ret;
+
+   msg[0].addr = addr_off;
+   msg[0].flags = 0;
+   addr = reg_addr;
+   msg[0].buf = &addr;
+   msg[0].len = 1;
+   msg[1].addr = addr_off;
+   msg[1].flags = I2C_M_RD;
+   msg[1].buf = &val;
+   msg[1].len = 1;
+   ret = dm_i2c_xfer(dev, msg, 2);
+   if (ret) {
+   debug("%s: read failed, reg=%.2x, value=%p, ret=%d\n",
+ __func__, (int)reg_addr, value, ret);
+   return ret;
+   }
+   *value = val;
+
+   return 0;
+}
+
+static int anx6345_write_r0(struct udevice *dev, unsigned char reg_addr,
+   unsigned char value)
+{
+   struct dm_i2c_chip *chip = dev_get_parent_platdata(dev);
+
+   return anx6345_write(dev, chip->chip_addr, reg_addr, value);
+}
+
+static int anx6345_read_r0(struct udevice *dev, unsigned char reg_addr,
+  unsigned char *value)
+{
+   struct dm_i2c_chip *chip = dev_get_parent_platdata(dev);
+
+   return anx6345_read(dev, chip->chip_addr, reg_addr, value);
+}
+
+static int anx6345_write_r1(struct udevice *dev, unsigned char reg_addr,
+   unsigned char value)
+{
+   struct dm_i2c_chip *chip = dev_get_parent_platdata(dev);
+
+   return anx6345_write(dev, chip->chip_addr + 1, reg_addr, value);
+}
+
+static int anx6345_read_r1(struct udevice *dev, unsigned char reg_addr,
+  unsigned char *value)
+{
+   struct dm_i2c_chip *chip = dev_get_parent_platdata(dev);
+
+   return anx6345_read(dev, chip->chip_addr + 1, reg_addr, value);
+}
+
+static int anx6345_set_backlight(struct udevice *dev, int percent)
+{
+   return -ENOSYS;
+}
+
+static int anx6345_aux_wait(struct udevice *dev)
+{
+   int ret = -ETIMEDOUT;
+   u8 v;
+   int retries = 1000;
+
+   do {
+