| Message ID | 1398119958-32005-9-git-send-email-ajaykumar.rs@samsung.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Tuesday, April 22, 2014 7:39 AM, Ajay Kumar wrote: > > This patch adds a drm_bridge driver for the PS8622 DisplayPort to LVDS > bridge chip. > > Signed-off-by: Andrew Bresticker <abrestic@chromium.org> > Signed-off-by: Sean Paul <seanpaul@chromium.org> > Signed-off-by: Rahul Sharma <rahul.sharma@samsung.com> > Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> > --- > Changes since V1: > Pushing V1 for this as V2 because this patch holds good in this series. > > drivers/gpu/drm/bridge/Kconfig | 7 + > drivers/gpu/drm/bridge/Makefile | 1 + > drivers/gpu/drm/bridge/ps8622.c | 566 +++++++++++++++++++++++++++++++++++++++ > include/drm/bridge/ps8622.h | 42 +++ > 4 files changed, 616 insertions(+) > create mode 100644 drivers/gpu/drm/bridge/ps8622.c > create mode 100644 include/drm/bridge/ps8622.h > > diff --git a/drivers/gpu/drm/bridge/Kconfig b/drivers/gpu/drm/bridge/Kconfig > index 3bc6845..aba21fc 100644 > --- a/drivers/gpu/drm/bridge/Kconfig > +++ b/drivers/gpu/drm/bridge/Kconfig > @@ -4,3 +4,10 @@ config DRM_PTN3460 > select DRM_KMS_HELPER > select DRM_PANEL > ---help--- > + > +config DRM_PS8622 > + tristate "Parade eDP/LVDS bridge" > + depends on DRM > + select DRM_KMS_HELPER > + select DRM_PANEL Please add the following select. + select BACKLIGHT_LCD_SUPPORT + select BACKLIGHT_CLASS_DEVICE Without this, the following build issues happen. drivers/gpu/drm/bridge/ps8622.c:353: undefined reference to `backlight_device_unregister' drivers/built-in.o: In function `ps8622_init': drivers/gpu/drm/bridge/ps8622.c:559: undefined reference to `backlight_device_unregister' drivers/gpu/drm/bridge/ps8622.c:507: undefined reference to `backlight_device_register' > + ---help--- > diff --git a/drivers/gpu/drm/bridge/Makefile b/drivers/gpu/drm/bridge/Makefile > index b4733e1..d1b5daa 100644 > --- a/drivers/gpu/drm/bridge/Makefile > +++ b/drivers/gpu/drm/bridge/Makefile > @@ -1,3 +1,4 @@ > ccflags-y := -Iinclude/drm > > obj-$(CONFIG_DRM_PTN3460) += ptn3460.o > +obj-$(CONFIG_DRM_PS8622) += ps8622.o > diff --git a/drivers/gpu/drm/bridge/ps8622.c b/drivers/gpu/drm/bridge/ps8622.c > new file mode 100644 > index 0000000..1e6b3ca > --- /dev/null > +++ b/drivers/gpu/drm/bridge/ps8622.c [.....] > +static int ps8622_send_config(struct ps8622_bridge *ps_bridge) > +{ > + struct i2c_client *cl = ps_bridge->client; > + int err = 0; > + > + /* wait 20ms after power ON */ > + usleep_range(20000, 30000); > + > + err |= ps8622_set(cl, 0x02, 0xa1, 0x01); /* HPD low */ > + /* SW setting */ > + err |= ps8622_set(cl, 0x04, 0x14, 0x01); /* [1:0] SW output 1.2V voltage > + * is lower to 96% */ The comment style looks awkward. Please choose one of two options. And change all comments in this file. 1. + /* SW setting */ + err |= ps8622_set(cl, 0x04, 0x14, 0x01); /* [1:0] SW output 1.2V voltage is lower to 96% */ 2. + /* SW setting */ + /* [1:0] SW output 1.2V voltage is lower to 96% */ + err |= ps8622_set(cl, 0x04, 0x14, 0x01); > + /* RCO SS setting */ > + err |= ps8622_set(cl, 0x04, 0xe3, 0x20); /* [5:4] = b01 0.5%, b10 1%, > + * b11 1.5% */ > + err |= ps8622_set(cl, 0x04, 0xe2, 0x80); /* [7] RCO SS enable */ > + /* RPHY Setting */ > + err |= ps8622_set(cl, 0x04, 0x8a, 0x0c); /* [3:2] CDR tune wait cycle > + * before measure for fine tune > + * b00: 1us b01: 0.5us b10:2us > + * b11: 4us */ > + err |= ps8622_set(cl, 0x04, 0x89, 0x08); /* [3] RFD always on */ > + err |= ps8622_set(cl, 0x04, 0x71, 0x2d); /* CTN lock in/out: > + * 20000ppm/80000ppm. > + * Lock out 2 times. */ > + /* 2.7G CDR settings */ > + err |= ps8622_set(cl, 0x04, 0x7d, 0x07); /* NOF=40LSB for HBR CDR > + * setting */ > + err |= ps8622_set(cl, 0x04, 0x7b, 0x00); /* [1:0] Fmin=+4bands */ > + err |= ps8622_set(cl, 0x04, 0x7a, 0xfd); /* [7:5] DCO_FTRNG=+-40% */ > + /* 1.62G CDR settings */ > + err |= ps8622_set(cl, 0x04, 0xc0, 0x12); /* [5:2]NOF=64LSB [1:0]DCO > + * scale is 2/5 */ > + err |= ps8622_set(cl, 0x04, 0xc1, 0x92); /* Gitune=-37% */ > + err |= ps8622_set(cl, 0x04, 0xc2, 0x1c); /* Fbstep=100% */ > + err |= ps8622_set(cl, 0x04, 0x32, 0x80); /* [7] LOS signal disable */ > + /* RPIO Setting */ > + err |= ps8622_set(cl, 0x04, 0x00, 0xb0); /* [7:4] LVDS driver bias > + * current : 75% (250mV swing) > + * */ > + err |= ps8622_set(cl, 0x04, 0x15, 0x40); /* [7:6] Right-bar GPIO output > + * strength is 8mA */ > + /* EQ Training State Machine Setting */ > + err |= ps8622_set(cl, 0x04, 0x54, 0x10); /* RCO calibration start */ > + /* Logic, needs more than 10 I2C command */ > + err |= ps8622_set(cl, 0x01, 0x02, 0x80 | ps_bridge->max_lane_count); > + /* [4:0] MAX_LANE_COUNT set to > + * max supported lanes */ > + err |= ps8622_set(cl, 0x01, 0x21, 0x80 | ps_bridge->lane_count); > + /* [4:0] LANE_COUNT_SET set to > + * chosen lane count */ > + err |= ps8622_set(cl, 0x00, 0x52, 0x20); > + err |= ps8622_set(cl, 0x00, 0xf1, 0x03); /* HPD CP toggle enable */ > + err |= ps8622_set(cl, 0x00, 0x62, 0x41); > + err |= ps8622_set(cl, 0x00, 0xf6, 0x01); /* Counter number, add 1ms > + * counter delay */ > + err |= ps8622_set(cl, 0x00, 0x77, 0x06); /* [6]PWM function control by > + * DPCD0040f[7], default is PWM > + * block always works. */ > + err |= ps8622_set(cl, 0x00, 0x4c, 0x04); /* 04h Adjust VTotal tolerance > + * to fix the 30Hz no display > + * issue */ > + err |= ps8622_set(cl, 0x01, 0xc0, 0x00); /* DPCD00400='h00, Parade OUI = > + * 'h001cf8 */ > + err |= ps8622_set(cl, 0x01, 0xc1, 0x1c); /* DPCD00401='h1c */ > + err |= ps8622_set(cl, 0x01, 0xc2, 0xf8); /* DPCD00402='hf8 */ > + err |= ps8622_set(cl, 0x01, 0xc3, 0x44); /* DPCD403~408 = ASCII code > + * D2SLV5='h4432534c5635 */ > + err |= ps8622_set(cl, 0x01, 0xc4, 0x32); /* DPCD404 */ > + err |= ps8622_set(cl, 0x01, 0xc5, 0x53); /* DPCD405 */ > + err |= ps8622_set(cl, 0x01, 0xc6, 0x4c); /* DPCD406 */ > + err |= ps8622_set(cl, 0x01, 0xc7, 0x56); /* DPCD407 */ > + err |= ps8622_set(cl, 0x01, 0xc8, 0x35); /* DPCD408 */ > + err |= ps8622_set(cl, 0x01, 0xca, 0x01); /* DPCD40A, Initial Code major > + * revision '01' */ > + err |= ps8622_set(cl, 0x01, 0xcb, 0x05); /* DPCD40B, Initial Code minor > + * revision '05' */ > + if (ps_bridge->bl) { > + err |= ps8622_set(cl, 0x01, 0xa5, 0xa0); > + /* DPCD720, internal PWM */ > + err |= ps8622_set(cl, 0x01, 0xa7, > + ps_bridge->bl->props.brightness); > + /* FFh for 100% brightness, > + * 0h for 0% brightness */ > + } else { > + err |= ps8622_set(cl, 0x01, 0xa5, 0x80); > + /* DPCD720, external PWM */ > + } > + err |= ps8622_set(cl, 0x01, 0xcc, 0x13); /* Set LVDS output as 6bit-VESA > + * mapping, single LVDS channel > + * */ > + err |= ps8622_set(cl, 0x02, 0xb1, 0x20); /* Enable SSC set by register > + * */ > + err |= ps8622_set(cl, 0x04, 0x10, 0x16); /* Set SSC enabled and +/-1% > + * central spreading */ > + /* Logic end */ > + err |= ps8622_set(cl, 0x04, 0x59, 0x60); /* MPU Clock source: LC => RCO > + * */ > + err |= ps8622_set(cl, 0x04, 0x54, 0x14); /* LC -> RCO */ > + err |= ps8622_set(cl, 0x02, 0xa1, 0x91); /* HPD high */ > + > + return err ? -EIO : 0; > +} > + [.....] > + > +static void ps8622_pre_enable(struct drm_bridge *bridge) > +{ > + struct ps8622_bridge *ps_bridge = bridge->driver_private; > + int ret; > + > + mutex_lock(&ps_bridge->enable_mutex); > + if (ps_bridge->enabled) > + goto out; > + > + if (gpio_is_valid(ps_bridge->gpio_rst_n)) > + gpio_set_value(ps_bridge->gpio_rst_n, 0); > + > + if (ps_bridge->v12) { > + ret = regulator_enable(ps_bridge->v12); > + if (ret) > + DRM_ERROR("fails to enable ps_bridge->v12"); > + } > + > + drm_panel_pre_enable(ps_bridge->panel); > + > + if (gpio_is_valid(ps_bridge->gpio_slp_n)) > + gpio_set_value(ps_bridge->gpio_slp_n, 1); > + > + /* > + * T1 is the range of time that it takes for the power to rise after we > + * enable the lcd fet. T2 is the range of time in which the data sheet > + * specifies we should deassert the reset pin. > + * > + * If it takes T1.max for the power to rise, we need to wait atleast > + * T2.min before deasserting the reset pin. If it takes T1.min for the > + * power to rise, we need to wait at most T2.max before deasserting the > + * reset pin. > + */ > + usleep_range(PS8622_RST_HIGH_T2_MIN_US + PS8622_POWER_RISE_T1_MAX_US, > + PS8622_RST_HIGH_T2_MAX_US + PS8622_POWER_RISE_T1_MIN_US); > + > + if (gpio_is_valid(ps_bridge->gpio_rst_n)) > + gpio_set_value(ps_bridge->gpio_rst_n, 1); > + > + ret = ps8622_send_config(ps_bridge); > + if (ret) > + DRM_ERROR("Failed to send config to bridge (%d)\n", ret); > + > + ps_bridge->enabled = true; > + > + mutex_unlock(&ps_bridge->enable_mutex); > + return; > + > +out: > + > + mutex_unlock(&ps_bridge->enable_mutex); > +} mutex_unlock() is duplicated. Also, 'return' is unnecessary. Please fix it as below. + ps_bridge->enabled = true; + +out: + mutex_unlock(&ps_bridge->enable_mutex); +} > + > +static void ps8622_enable(struct drm_bridge *bridge) > +{ > + struct ps8622_bridge *ps_bridge = bridge->driver_private; > + > + mutex_lock(&ps_bridge->enable_mutex); > + if (ps_bridge->enabled) > + drm_panel_enable(ps_bridge->panel); > + mutex_unlock(&ps_bridge->enable_mutex); > +} > + > +static void ps8622_disable(struct drm_bridge *bridge) > +{ > + struct ps8622_bridge *ps_bridge = bridge->driver_private; > + > + mutex_lock(&ps_bridge->enable_mutex); > + > + if (!ps_bridge->enabled) > + goto out; > + > + ps_bridge->enabled = false; > + > + drm_panel_disable(ps_bridge->panel); > + msleep(PS8622_PWMO_END_T12_MS); > + > + /* > + * This doesn't matter if the regulators are turned off, but something > + * else might keep them on. In that case, we want to assert the slp gpio > + * to lower power. > + */ > + if (gpio_is_valid(ps_bridge->gpio_slp_n)) > + gpio_set_value(ps_bridge->gpio_slp_n, 0); > + > + drm_panel_post_disable(ps_bridge->panel); > + if (ps_bridge->v12) > + regulator_disable(ps_bridge->v12); > + > + /* > + * Sleep for at least the amount of time that it takes the power rail to > + * fall to prevent asserting the rst gpio from doing anything. > + */ > + usleep_range(PS8622_POWER_FALL_T16_MAX_US, > + 2 * PS8622_POWER_FALL_T16_MAX_US); > + if (gpio_is_valid(ps_bridge->gpio_rst_n)) > + gpio_set_value(ps_bridge->gpio_rst_n, 0); > + > + msleep(PS8622_POWER_OFF_T17_MS); > + > +out: > + mutex_unlock(&ps_bridge->enable_mutex); > +} > + > +static void ps8622_post_disable(struct drm_bridge *bridge) > +{ > +} How about just removing this empty function? [.....] Best regards, Jingoo Han -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, Apr 21, 2014 at 6:39 PM, Ajay Kumar <ajaykumar.rs@samsung.com> wrote: > This patch adds a drm_bridge driver for the PS8622 DisplayPort to LVDS > bridge chip. > > Signed-off-by: Andrew Bresticker <abrestic@chromium.org> > Signed-off-by: Sean Paul <seanpaul@chromium.org> > Signed-off-by: Rahul Sharma <rahul.sharma@samsung.com> > Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> Hi Ajay, I don't think that you should be claiming authorship of this driver, I don't see any commits from you in the chromium tree ([1] & [2]). The driver was originally written by vpalatin@chromium.org (who's completely missing from SoB), it should probably be attributed to him. Sean [1]- https://git.chromium.org/gitweb/?p=chromiumos/third_party/kernel-next.git;a=history;f=drivers/auxdisplay/ps8622.c;hb=fac579ad884d064d2c3cda988117e7dd66fc30c4 [2]- https://git.chromium.org/gitweb/?p=chromiumos/third_party/kernel-next.git;a=history;f=drivers/gpu/drm/bridge/ps8622.c;hb=refs/heads/chromeos-3.8 > --- > Changes since V1: > Pushing V1 for this as V2 because this patch holds good in this series. > > drivers/gpu/drm/bridge/Kconfig | 7 + > drivers/gpu/drm/bridge/Makefile | 1 + > drivers/gpu/drm/bridge/ps8622.c | 566 +++++++++++++++++++++++++++++++++++++++ > include/drm/bridge/ps8622.h | 42 +++ > 4 files changed, 616 insertions(+) > create mode 100644 drivers/gpu/drm/bridge/ps8622.c > create mode 100644 include/drm/bridge/ps8622.h > > diff --git a/drivers/gpu/drm/bridge/Kconfig b/drivers/gpu/drm/bridge/Kconfig > index 3bc6845..aba21fc 100644 > --- a/drivers/gpu/drm/bridge/Kconfig > +++ b/drivers/gpu/drm/bridge/Kconfig > @@ -4,3 +4,10 @@ config DRM_PTN3460 > select DRM_KMS_HELPER > select DRM_PANEL > ---help--- > + > +config DRM_PS8622 > + tristate "Parade eDP/LVDS bridge" > + depends on DRM > + select DRM_KMS_HELPER > + select DRM_PANEL > + ---help--- > diff --git a/drivers/gpu/drm/bridge/Makefile b/drivers/gpu/drm/bridge/Makefile > index b4733e1..d1b5daa 100644 > --- a/drivers/gpu/drm/bridge/Makefile > +++ b/drivers/gpu/drm/bridge/Makefile > @@ -1,3 +1,4 @@ > ccflags-y := -Iinclude/drm > > obj-$(CONFIG_DRM_PTN3460) += ptn3460.o > +obj-$(CONFIG_DRM_PS8622) += ps8622.o > diff --git a/drivers/gpu/drm/bridge/ps8622.c b/drivers/gpu/drm/bridge/ps8622.c > new file mode 100644 > index 0000000..1e6b3ca > --- /dev/null > +++ b/drivers/gpu/drm/bridge/ps8622.c > @@ -0,0 +1,566 @@ > +/* > + * Parade PS8622 eDP/LVDS bridge driver > + * > + * Copyright (C) 2014 Google, Inc. > + * > + * This software is licensed under the terms of the GNU General Public > + * License version 2, as published by the Free Software Foundation, and > + * may be copied, distributed, and modified under those terms. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + */ > + > +#include <linux/module.h> > +#include <linux/backlight.h> > +#include <linux/delay.h> > +#include <linux/err.h> > +#include <linux/fb.h> > +#include <linux/gpio.h> > +#include <linux/i2c.h> > +#include <linux/of.h> > +#include <linux/of_gpio.h> > +#include <linux/pm.h> > +#include <linux/regulator/consumer.h> > +#include <drm/drm_panel.h> > + > +#include "drmP.h" > +#include "drm_crtc.h" > +#include "drm_crtc_helper.h" > + > +struct ps8622_bridge { > + struct drm_connector connector; > + struct drm_bridge *bridge; > + struct drm_encoder *encoder; > + struct drm_panel *panel; > + struct i2c_client *client; > + struct regulator *v12; > + struct backlight_device *bl; > + struct mutex enable_mutex; > + > + int gpio_slp_n; > + int gpio_rst_n; > + > + u8 max_lane_count; > + u8 lane_count; > + > + bool enabled; > + > + struct drm_display_mode mode; > +}; > + > +struct ps8622_device_data { > + u8 max_lane_count; > +}; > + > +static const struct ps8622_device_data ps8622_data = { > + .max_lane_count = 1, > +}; > + > +static const struct ps8622_device_data ps8625_data = { > + .max_lane_count = 2, > +}; > + > +/* Brightness scale on the Parade chip */ > +#define PS8622_MAX_BRIGHTNESS 0xff > + > +/* Timings taken from the version 1.7 datasheet for the PS8622/PS8625 */ > +#define PS8622_POWER_RISE_T1_MIN_US 10 > +#define PS8622_POWER_RISE_T1_MAX_US 10000 > +#define PS8622_RST_HIGH_T2_MIN_US 3000 > +#define PS8622_RST_HIGH_T2_MAX_US 30000 > +#define PS8622_PWMO_END_T12_MS 200 > +#define PS8622_POWER_FALL_T16_MAX_US 10000 > +#define PS8622_POWER_OFF_T17_MS 500 > + > +#if ((PS8622_RST_HIGH_T2_MIN_US + PS8622_POWER_RISE_T1_MAX_US) > \ > + (PS8622_RST_HIGH_T2_MAX_US + PS8622_POWER_RISE_T1_MIN_US)) > +#error "T2.min + T1.max must be less than T2.max + T1.min" > +#endif > + > +static int ps8622_set(struct i2c_client *client, u8 page, u8 reg, u8 val) > +{ > + int ret; > + struct i2c_adapter *adap = client->adapter; > + struct i2c_msg msg; > + u8 data[] = {reg, val}; > + > + msg.addr = client->addr + page; > + msg.flags = 0; > + msg.len = sizeof(data); > + msg.buf = data; > + > + ret = i2c_transfer(adap, &msg, 1); > + if (ret != 1) > + pr_warn("PS8622 I2C write (0x%02x,0x%02x,0x%02x) failed: %d\n", > + client->addr + page, reg, val, ret); > + return !(ret == 1); > +} > + > +static int ps8622_send_config(struct ps8622_bridge *ps_bridge) > +{ > + struct i2c_client *cl = ps_bridge->client; > + int err = 0; > + > + /* wait 20ms after power ON */ > + usleep_range(20000, 30000); > + > + err |= ps8622_set(cl, 0x02, 0xa1, 0x01); /* HPD low */ > + /* SW setting */ > + err |= ps8622_set(cl, 0x04, 0x14, 0x01); /* [1:0] SW output 1.2V voltage > + * is lower to 96% */ > + /* RCO SS setting */ > + err |= ps8622_set(cl, 0x04, 0xe3, 0x20); /* [5:4] = b01 0.5%, b10 1%, > + * b11 1.5% */ > + err |= ps8622_set(cl, 0x04, 0xe2, 0x80); /* [7] RCO SS enable */ > + /* RPHY Setting */ > + err |= ps8622_set(cl, 0x04, 0x8a, 0x0c); /* [3:2] CDR tune wait cycle > + * before measure for fine tune > + * b00: 1us b01: 0.5us b10:2us > + * b11: 4us */ > + err |= ps8622_set(cl, 0x04, 0x89, 0x08); /* [3] RFD always on */ > + err |= ps8622_set(cl, 0x04, 0x71, 0x2d); /* CTN lock in/out: > + * 20000ppm/80000ppm. > + * Lock out 2 times. */ > + /* 2.7G CDR settings */ > + err |= ps8622_set(cl, 0x04, 0x7d, 0x07); /* NOF=40LSB for HBR CDR > + * setting */ > + err |= ps8622_set(cl, 0x04, 0x7b, 0x00); /* [1:0] Fmin=+4bands */ > + err |= ps8622_set(cl, 0x04, 0x7a, 0xfd); /* [7:5] DCO_FTRNG=+-40% */ > + /* 1.62G CDR settings */ > + err |= ps8622_set(cl, 0x04, 0xc0, 0x12); /* [5:2]NOF=64LSB [1:0]DCO > + * scale is 2/5 */ > + err |= ps8622_set(cl, 0x04, 0xc1, 0x92); /* Gitune=-37% */ > + err |= ps8622_set(cl, 0x04, 0xc2, 0x1c); /* Fbstep=100% */ > + err |= ps8622_set(cl, 0x04, 0x32, 0x80); /* [7] LOS signal disable */ > + /* RPIO Setting */ > + err |= ps8622_set(cl, 0x04, 0x00, 0xb0); /* [7:4] LVDS driver bias > + * current : 75% (250mV swing) > + * */ > + err |= ps8622_set(cl, 0x04, 0x15, 0x40); /* [7:6] Right-bar GPIO output > + * strength is 8mA */ > + /* EQ Training State Machine Setting */ > + err |= ps8622_set(cl, 0x04, 0x54, 0x10); /* RCO calibration start */ > + /* Logic, needs more than 10 I2C command */ > + err |= ps8622_set(cl, 0x01, 0x02, 0x80 | ps_bridge->max_lane_count); > + /* [4:0] MAX_LANE_COUNT set to > + * max supported lanes */ > + err |= ps8622_set(cl, 0x01, 0x21, 0x80 | ps_bridge->lane_count); > + /* [4:0] LANE_COUNT_SET set to > + * chosen lane count */ > + err |= ps8622_set(cl, 0x00, 0x52, 0x20); > + err |= ps8622_set(cl, 0x00, 0xf1, 0x03); /* HPD CP toggle enable */ > + err |= ps8622_set(cl, 0x00, 0x62, 0x41); > + err |= ps8622_set(cl, 0x00, 0xf6, 0x01); /* Counter number, add 1ms > + * counter delay */ > + err |= ps8622_set(cl, 0x00, 0x77, 0x06); /* [6]PWM function control by > + * DPCD0040f[7], default is PWM > + * block always works. */ > + err |= ps8622_set(cl, 0x00, 0x4c, 0x04); /* 04h Adjust VTotal tolerance > + * to fix the 30Hz no display > + * issue */ > + err |= ps8622_set(cl, 0x01, 0xc0, 0x00); /* DPCD00400='h00, Parade OUI = > + * 'h001cf8 */ > + err |= ps8622_set(cl, 0x01, 0xc1, 0x1c); /* DPCD00401='h1c */ > + err |= ps8622_set(cl, 0x01, 0xc2, 0xf8); /* DPCD00402='hf8 */ > + err |= ps8622_set(cl, 0x01, 0xc3, 0x44); /* DPCD403~408 = ASCII code > + * D2SLV5='h4432534c5635 */ > + err |= ps8622_set(cl, 0x01, 0xc4, 0x32); /* DPCD404 */ > + err |= ps8622_set(cl, 0x01, 0xc5, 0x53); /* DPCD405 */ > + err |= ps8622_set(cl, 0x01, 0xc6, 0x4c); /* DPCD406 */ > + err |= ps8622_set(cl, 0x01, 0xc7, 0x56); /* DPCD407 */ > + err |= ps8622_set(cl, 0x01, 0xc8, 0x35); /* DPCD408 */ > + err |= ps8622_set(cl, 0x01, 0xca, 0x01); /* DPCD40A, Initial Code major > + * revision '01' */ > + err |= ps8622_set(cl, 0x01, 0xcb, 0x05); /* DPCD40B, Initial Code minor > + * revision '05' */ > + if (ps_bridge->bl) { > + err |= ps8622_set(cl, 0x01, 0xa5, 0xa0); > + /* DPCD720, internal PWM */ > + err |= ps8622_set(cl, 0x01, 0xa7, > + ps_bridge->bl->props.brightness); > + /* FFh for 100% brightness, > + * 0h for 0% brightness */ > + } else { > + err |= ps8622_set(cl, 0x01, 0xa5, 0x80); > + /* DPCD720, external PWM */ > + } > + err |= ps8622_set(cl, 0x01, 0xcc, 0x13); /* Set LVDS output as 6bit-VESA > + * mapping, single LVDS channel > + * */ > + err |= ps8622_set(cl, 0x02, 0xb1, 0x20); /* Enable SSC set by register > + * */ > + err |= ps8622_set(cl, 0x04, 0x10, 0x16); /* Set SSC enabled and +/-1% > + * central spreading */ > + /* Logic end */ > + err |= ps8622_set(cl, 0x04, 0x59, 0x60); /* MPU Clock source: LC => RCO > + * */ > + err |= ps8622_set(cl, 0x04, 0x54, 0x14); /* LC -> RCO */ > + err |= ps8622_set(cl, 0x02, 0xa1, 0x91); /* HPD high */ > + > + return err ? -EIO : 0; > +} > + > +static int ps8622_backlight_update(struct backlight_device *bl) > +{ > + struct ps8622_bridge *ps_bridge = dev_get_drvdata(&bl->dev); > + int ret, brightness = bl->props.brightness; > + > + if (bl->props.power != FB_BLANK_UNBLANK || > + bl->props.state & (BL_CORE_SUSPENDED | BL_CORE_FBBLANK)) > + brightness = 0; > + > + mutex_lock(&ps_bridge->enable_mutex); > + > + if (!ps_bridge->enabled) { > + ret = -EINVAL; > + goto out; > + } > + > + ret = ps8622_set(ps_bridge->client, 0x01, 0xa7, brightness); > + > +out: > + mutex_unlock(&ps_bridge->enable_mutex); > + return ret; > +} > + > +static int ps8622_backlight_get(struct backlight_device *bl) > +{ > + return bl->props.brightness; > +} > + > +static const struct backlight_ops ps8622_backlight_ops = { > + .update_status = ps8622_backlight_update, > + .get_brightness = ps8622_backlight_get, > +}; > + > +static void ps8622_pre_enable(struct drm_bridge *bridge) > +{ > + struct ps8622_bridge *ps_bridge = bridge->driver_private; > + int ret; > + > + mutex_lock(&ps_bridge->enable_mutex); > + if (ps_bridge->enabled) > + goto out; > + > + if (gpio_is_valid(ps_bridge->gpio_rst_n)) > + gpio_set_value(ps_bridge->gpio_rst_n, 0); > + > + if (ps_bridge->v12) { > + ret = regulator_enable(ps_bridge->v12); > + if (ret) > + DRM_ERROR("fails to enable ps_bridge->v12"); > + } > + > + drm_panel_pre_enable(ps_bridge->panel); > + > + if (gpio_is_valid(ps_bridge->gpio_slp_n)) > + gpio_set_value(ps_bridge->gpio_slp_n, 1); > + > + /* > + * T1 is the range of time that it takes for the power to rise after we > + * enable the lcd fet. T2 is the range of time in which the data sheet > + * specifies we should deassert the reset pin. > + * > + * If it takes T1.max for the power to rise, we need to wait atleast > + * T2.min before deasserting the reset pin. If it takes T1.min for the > + * power to rise, we need to wait at most T2.max before deasserting the > + * reset pin. > + */ > + usleep_range(PS8622_RST_HIGH_T2_MIN_US + PS8622_POWER_RISE_T1_MAX_US, > + PS8622_RST_HIGH_T2_MAX_US + PS8622_POWER_RISE_T1_MIN_US); > + > + if (gpio_is_valid(ps_bridge->gpio_rst_n)) > + gpio_set_value(ps_bridge->gpio_rst_n, 1); > + > + ret = ps8622_send_config(ps_bridge); > + if (ret) > + DRM_ERROR("Failed to send config to bridge (%d)\n", ret); > + > + ps_bridge->enabled = true; > + > + mutex_unlock(&ps_bridge->enable_mutex); > + return; > + > +out: > + > + mutex_unlock(&ps_bridge->enable_mutex); > +} > + > +static void ps8622_enable(struct drm_bridge *bridge) > +{ > + struct ps8622_bridge *ps_bridge = bridge->driver_private; > + > + mutex_lock(&ps_bridge->enable_mutex); > + if (ps_bridge->enabled) > + drm_panel_enable(ps_bridge->panel); > + mutex_unlock(&ps_bridge->enable_mutex); > +} > + > +static void ps8622_disable(struct drm_bridge *bridge) > +{ > + struct ps8622_bridge *ps_bridge = bridge->driver_private; > + > + mutex_lock(&ps_bridge->enable_mutex); > + > + if (!ps_bridge->enabled) > + goto out; > + > + ps_bridge->enabled = false; > + > + drm_panel_disable(ps_bridge->panel); > + msleep(PS8622_PWMO_END_T12_MS); > + > + /* > + * This doesn't matter if the regulators are turned off, but something > + * else might keep them on. In that case, we want to assert the slp gpio > + * to lower power. > + */ > + if (gpio_is_valid(ps_bridge->gpio_slp_n)) > + gpio_set_value(ps_bridge->gpio_slp_n, 0); > + > + drm_panel_post_disable(ps_bridge->panel); > + if (ps_bridge->v12) > + regulator_disable(ps_bridge->v12); > + > + /* > + * Sleep for at least the amount of time that it takes the power rail to > + * fall to prevent asserting the rst gpio from doing anything. > + */ > + usleep_range(PS8622_POWER_FALL_T16_MAX_US, > + 2 * PS8622_POWER_FALL_T16_MAX_US); > + if (gpio_is_valid(ps_bridge->gpio_rst_n)) > + gpio_set_value(ps_bridge->gpio_rst_n, 0); > + > + msleep(PS8622_POWER_OFF_T17_MS); > + > +out: > + mutex_unlock(&ps_bridge->enable_mutex); > +} > + > +static void ps8622_post_disable(struct drm_bridge *bridge) > +{ > +} > + > +static void ps8622_destroy(struct drm_bridge *bridge) > +{ > + struct ps8622_bridge *ps_bridge = bridge->driver_private; > + > + drm_bridge_cleanup(bridge); > + if (ps_bridge->bl) > + backlight_device_unregister(ps_bridge->bl); > + > + put_device(&ps_bridge->client->dev); > +} > + > +static const struct drm_bridge_funcs ps8622_bridge_funcs = { > + .pre_enable = ps8622_pre_enable, > + .enable = ps8622_enable, > + .disable = ps8622_disable, > + .post_disable = ps8622_post_disable, > + .destroy = ps8622_destroy, > +}; > + > +static int ps8622_get_modes(struct drm_connector *connector) > +{ > + struct ps8622_bridge *ps_bridge; > + struct drm_display_mode *new_mode; > + > + ps_bridge = container_of(connector, struct ps8622_bridge, connector); > + > + new_mode = drm_mode_duplicate(connector->dev, &ps_bridge->mode); > + if (!new_mode) { > + DRM_ERROR("Failed to duplicate ps_bridge mode!\n"); > + return 0; > + } > + drm_mode_probed_add(connector, new_mode); > + return 1; > +} > + > +static int ps8622_mode_valid(struct drm_connector *connector, > + struct drm_display_mode *mode) > +{ > + struct ps8622_bridge *ps_bridge; > + > + ps_bridge = container_of(connector, struct ps8622_bridge, connector); > + > + return drm_mode_equal(mode, &ps_bridge->mode) ? MODE_OK : MODE_BAD; > +} > + > +static struct drm_encoder *ps8622_best_encoder(struct drm_connector *connector) > +{ > + struct ps8622_bridge *ps_bridge; > + > + ps_bridge = container_of(connector, struct ps8622_bridge, connector); > + > + return ps_bridge->encoder; > +} > + > +static const struct drm_connector_helper_funcs ps8622_connector_helper_funcs = { > + .get_modes = ps8622_get_modes, > + .mode_valid = ps8622_mode_valid, > + .best_encoder = ps8622_best_encoder, > +}; > + > +static enum drm_connector_status ps8622_detect(struct drm_connector *connector, > + bool force) > +{ > + return connector_status_connected; > +} > + > +static void ps8622_connector_destroy(struct drm_connector *connector) > +{ > + drm_connector_cleanup(connector); > +} > + > +static const struct drm_connector_funcs ps8622_connector_funcs = { > + .dpms = drm_helper_connector_dpms, > + .fill_modes = drm_helper_probe_single_connector_modes, > + .detect = ps8622_detect, > + .destroy = ps8622_connector_destroy, > +}; > + > +static const struct of_device_id ps8622_devices[] = { > + { > + .compatible = "parade,ps8622", > + .data = &ps8622_data, > + }, { > + .compatible = "parade,ps8625", > + .data = &ps8625_data, > + }, { > + /* end node */ > + } > +}; > + > +int ps8622_init(struct drm_device *dev, struct drm_encoder *encoder, > + struct i2c_client *client, struct device_node *node, > + struct drm_panel *panel) > +{ > + int ret; > + const struct of_device_id *match; > + const struct ps8622_device_data *device_data; > + struct drm_bridge *bridge; > + struct ps8622_bridge *ps_bridge; > + > + node = of_find_matching_node_and_match(NULL, ps8622_devices, &match); > + if (!node) > + return -ENODEV; > + > + bridge = devm_kzalloc(dev->dev, sizeof(*bridge), GFP_KERNEL); > + if (!bridge) { > + DRM_ERROR("Failed to allocate drm bridge\n"); > + return -ENOMEM; > + } > + > + ps_bridge = devm_kzalloc(dev->dev, sizeof(*ps_bridge), GFP_KERNEL); > + if (!ps_bridge) { > + DRM_ERROR("could not allocate ps bridge\n"); > + return -ENOMEM; > + } > + > + mutex_init(&ps_bridge->enable_mutex); > + > + device_data = match->data; > + ps_bridge->client = client; > + ps_bridge->encoder = encoder; > + ps_bridge->bridge = bridge; > + ps_bridge->v12 = devm_regulator_get(&client->dev, "vdd_bridge"); > + if (IS_ERR(ps_bridge->v12)) { > + DRM_INFO("no 1.2v regulator found for PS8622\n"); > + ps_bridge->v12 = NULL; > + } > + > + ps_bridge->gpio_slp_n = of_get_named_gpio(node, "sleep-gpio", 0); > + if (gpio_is_valid(ps_bridge->gpio_slp_n)) { > + ret = devm_gpio_request_one(&client->dev, ps_bridge->gpio_slp_n, > + GPIOF_OUT_INIT_HIGH, > + "PS8622_SLP_N"); > + if (ret) > + goto err_client; > + } > + > + ps_bridge->gpio_rst_n = of_get_named_gpio(node, "reset-gpio", 0); > + if (gpio_is_valid(ps_bridge->gpio_rst_n)) { > + /* > + * Assert the reset pin high to avoid the bridge being > + * initialized prematurely > + */ > + ret = devm_gpio_request_one(&client->dev, ps_bridge->gpio_rst_n, > + GPIOF_OUT_INIT_HIGH, > + "PS8622_RST_N"); > + if (ret) > + goto err_client; > + } > + > + ps_bridge->max_lane_count = device_data->max_lane_count; > + > + if (of_property_read_u8(node, "lane-count", &ps_bridge->lane_count)) > + ps_bridge->lane_count = ps_bridge->max_lane_count; > + else if (ps_bridge->lane_count > ps_bridge->max_lane_count) { > + DRM_ERROR("lane-count property is too high for DP bridge\n"); > + ps_bridge->lane_count = ps_bridge->max_lane_count; > + } > + > + if (!of_find_property(node, "use-external-pwm", NULL)) { > + ps_bridge->bl = backlight_device_register("ps8622-backlight", > + dev->dev, ps_bridge, &ps8622_backlight_ops, > + NULL); > + if (IS_ERR(ps_bridge->bl)) { > + DRM_ERROR("failed to register backlight\n"); > + ret = PTR_ERR(ps_bridge->bl); > + ps_bridge->bl = NULL; > + goto err_client; > + } > + ps_bridge->bl->props.max_brightness = PS8622_MAX_BRIGHTNESS; > + ps_bridge->bl->props.brightness = PS8622_MAX_BRIGHTNESS; > + } > + > + ret = of_get_drm_display_mode(node, &ps_bridge->mode, 0); > + if (ret) { > + DRM_ERROR("Failed to get display mode, ret=%d\n", ret); > + goto err_backlight; > + } > + > + ret = drm_bridge_init(dev, ps_bridge->bridge, &ps8622_bridge_funcs); > + if (ret) { > + DRM_ERROR("Failed to initialize bridge with drm\n"); > + goto err_backlight; > + } > + > + if (panel) { > + ps_bridge->panel = panel; > + drm_panel_attach(ps_bridge->panel, &ps_bridge->connector); > + } > + > + ps_bridge->bridge->driver_private = ps_bridge; > + encoder->bridge = ps_bridge->bridge; > + ps_bridge->enabled = false; > + ps_bridge->connector.polled = DRM_CONNECTOR_POLL_HPD; > + > + ret = drm_connector_init(dev, &ps_bridge->connector, > + &ps8622_connector_funcs, DRM_MODE_CONNECTOR_LVDS); > + if (ret) { > + DRM_ERROR("Failed to initialize connector with drm\n"); > + goto err_bridge; > + } > + drm_connector_helper_add(&ps_bridge->connector, > + &ps8622_connector_helper_funcs); > + drm_sysfs_connector_add(&ps_bridge->connector); > + drm_mode_connector_attach_encoder(&ps_bridge->connector, encoder); > + > + return 0; > + > +err_bridge: > + drm_bridge_cleanup(ps_bridge->bridge); > +err_backlight: > + if (ps_bridge->bl) > + backlight_device_unregister(ps_bridge->bl); > + > +err_client: > + put_device(&client->dev); > + DRM_ERROR("device probe failed : %d\n", ret); > + return ret; > +} > +EXPORT_SYMBOL(ps8622_init); > diff --git a/include/drm/bridge/ps8622.h b/include/drm/bridge/ps8622.h > new file mode 100644 > index 0000000..4fc1959 > --- /dev/null > +++ b/include/drm/bridge/ps8622.h > @@ -0,0 +1,42 @@ > +/* > + * Copyright (C) 2014 Google, Inc. > + * > + * This software is licensed under the terms of the GNU General Public > + * License version 2, as published by the Free Software Foundation, and > + * may be copied, distributed, and modified under those terms. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + */ > + > +#ifndef _DRM_BRIDGE_PS8622_H_ > +#define _DRM_BRIDGE_PS8622_H_ > + > +struct drm_device; > +struct drm_encoder; > +struct i2c_client; > +struct device_node; > +struct drm_panel; > + > +#ifdef CONFIG_DRM_PS8622 > + > +extern int ps8622_init(struct drm_device *dev, struct drm_encoder *encoder, > + struct i2c_client *client, struct device_node *node, > + struct drm_panel *panel); > + > +#else > + > +static inline int ps8622_init(struct drm_device *dev, > + struct drm_encoder *encoder, > + struct i2c_client *client, > + struct device_node *node, > + struct drm_panel *panel) > +{ > + return -ENODEV; > +} > + > +#endif > + > +#endif > -- > 1.7.9.5 > -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Sean, On Tue, Apr 22, 2014 at 7:01 PM, Sean Paul <seanpaul@google.com> wrote: > > On Mon, Apr 21, 2014 at 6:39 PM, Ajay Kumar <ajaykumar.rs@samsung.com> wrote: > > This patch adds a drm_bridge driver for the PS8622 DisplayPort to LVDS > > bridge chip. > > > > Signed-off-by: Andrew Bresticker <abrestic@chromium.org> > > Signed-off-by: Sean Paul <seanpaul@chromium.org> > > Signed-off-by: Rahul Sharma <rahul.sharma@samsung.com> > > Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> > > Hi Ajay, > I don't think that you should be claiming authorship of this driver, I > don't see any commits from you in the chromium tree ([1] & [2]). The > driver was originally written by vpalatin@chromium.org (who's > completely missing from SoB), it should probably be attributed to him. > > Sean > Oops. I am really sorry and I didn't really want to claim the authorship for this patch. I missed it while doing "git format-patch". And, thanks for letting me know that Vincent was the actual author. I will add his authorship. Regards, Ajay Kumar > > > [1]- https://git.chromium.org/gitweb/?p=chromiumos/third_party/kernel-next.git;a=history;f=drivers/auxdisplay/ps8622.c;hb=fac579ad884d064d2c3cda988117e7dd66fc30c4 > [2]- https://git.chromium.org/gitweb/?p=chromiumos/third_party/kernel-next.git;a=history;f=drivers/gpu/drm/bridge/ps8622.c;hb=refs/heads/chromeos-3.8 > > > > > --- > > Changes since V1: > > Pushing V1 for this as V2 because this patch holds good in this series. > > > > drivers/gpu/drm/bridge/Kconfig | 7 + > > drivers/gpu/drm/bridge/Makefile | 1 + > > drivers/gpu/drm/bridge/ps8622.c | 566 +++++++++++++++++++++++++++++++++++++++ > > include/drm/bridge/ps8622.h | 42 +++ > > 4 files changed, 616 insertions(+) > > create mode 100644 drivers/gpu/drm/bridge/ps8622.c > > create mode 100644 include/drm/bridge/ps8622.h > > > > diff --git a/drivers/gpu/drm/bridge/Kconfig b/drivers/gpu/drm/bridge/Kconfig > > index 3bc6845..aba21fc 100644 > > --- a/drivers/gpu/drm/bridge/Kconfig > > +++ b/drivers/gpu/drm/bridge/Kconfig > > @@ -4,3 +4,10 @@ config DRM_PTN3460 > > select DRM_KMS_HELPER > > select DRM_PANEL > > ---help--- > > + > > +config DRM_PS8622 > > + tristate "Parade eDP/LVDS bridge" > > + depends on DRM > > + select DRM_KMS_HELPER > > + select DRM_PANEL > > + ---help--- > > diff --git a/drivers/gpu/drm/bridge/Makefile b/drivers/gpu/drm/bridge/Makefile > > index b4733e1..d1b5daa 100644 > > --- a/drivers/gpu/drm/bridge/Makefile > > +++ b/drivers/gpu/drm/bridge/Makefile > > @@ -1,3 +1,4 @@ > > ccflags-y := -Iinclude/drm > > > > obj-$(CONFIG_DRM_PTN3460) += ptn3460.o > > +obj-$(CONFIG_DRM_PS8622) += ps8622.o > > diff --git a/drivers/gpu/drm/bridge/ps8622.c b/drivers/gpu/drm/bridge/ps8622.c > > new file mode 100644 > > index 0000000..1e6b3ca > > --- /dev/null > > +++ b/drivers/gpu/drm/bridge/ps8622.c > > @@ -0,0 +1,566 @@ > > +/* > > + * Parade PS8622 eDP/LVDS bridge driver > > + * > > + * Copyright (C) 2014 Google, Inc. > > + * > > + * This software is licensed under the terms of the GNU General Public > > + * License version 2, as published by the Free Software Foundation, and > > + * may be copied, distributed, and modified under those terms. > > + * > > + * This program is distributed in the hope that it will be useful, > > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > > + * GNU General Public License for more details. > > + */ > > + > > +#include <linux/module.h> > > +#include <linux/backlight.h> > > +#include <linux/delay.h> > > +#include <linux/err.h> > > +#include <linux/fb.h> > > +#include <linux/gpio.h> > > +#include <linux/i2c.h> > > +#include <linux/of.h> > > +#include <linux/of_gpio.h> > > +#include <linux/pm.h> > > +#include <linux/regulator/consumer.h> > > +#include <drm/drm_panel.h> > > + > > +#include "drmP.h" > > +#include "drm_crtc.h" > > +#include "drm_crtc_helper.h" > > + > > +struct ps8622_bridge { > > + struct drm_connector connector; > > + struct drm_bridge *bridge; > > + struct drm_encoder *encoder; > > + struct drm_panel *panel; > > + struct i2c_client *client; > > + struct regulator *v12; > > + struct backlight_device *bl; > > + struct mutex enable_mutex; > > + > > + int gpio_slp_n; > > + int gpio_rst_n; > > + > > + u8 max_lane_count; > > + u8 lane_count; > > + > > + bool enabled; > > + > > + struct drm_display_mode mode; > > +}; > > + > > +struct ps8622_device_data { > > + u8 max_lane_count; > > +}; > > + > > +static const struct ps8622_device_data ps8622_data = { > > + .max_lane_count = 1, > > +}; > > + > > +static const struct ps8622_device_data ps8625_data = { > > + .max_lane_count = 2, > > +}; > > + > > +/* Brightness scale on the Parade chip */ > > +#define PS8622_MAX_BRIGHTNESS 0xff > > + > > +/* Timings taken from the version 1.7 datasheet for the PS8622/PS8625 */ > > +#define PS8622_POWER_RISE_T1_MIN_US 10 > > +#define PS8622_POWER_RISE_T1_MAX_US 10000 > > +#define PS8622_RST_HIGH_T2_MIN_US 3000 > > +#define PS8622_RST_HIGH_T2_MAX_US 30000 > > +#define PS8622_PWMO_END_T12_MS 200 > > +#define PS8622_POWER_FALL_T16_MAX_US 10000 > > +#define PS8622_POWER_OFF_T17_MS 500 > > + > > +#if ((PS8622_RST_HIGH_T2_MIN_US + PS8622_POWER_RISE_T1_MAX_US) > \ > > + (PS8622_RST_HIGH_T2_MAX_US + PS8622_POWER_RISE_T1_MIN_US)) > > +#error "T2.min + T1.max must be less than T2.max + T1.min" > > +#endif > > + > > +static int ps8622_set(struct i2c_client *client, u8 page, u8 reg, u8 val) > > +{ > > + int ret; > > + struct i2c_adapter *adap = client->adapter; > > + struct i2c_msg msg; > > + u8 data[] = {reg, val}; > > + > > + msg.addr = client->addr + page; > > + msg.flags = 0; > > + msg.len = sizeof(data); > > + msg.buf = data; > > + > > + ret = i2c_transfer(adap, &msg, 1); > > + if (ret != 1) > > + pr_warn("PS8622 I2C write (0x%02x,0x%02x,0x%02x) failed: %d\n", > > + client->addr + page, reg, val, ret); > > + return !(ret == 1); > > +} > > + > > +static int ps8622_send_config(struct ps8622_bridge *ps_bridge) > > +{ > > + struct i2c_client *cl = ps_bridge->client; > > + int err = 0; > > + > > + /* wait 20ms after power ON */ > > + usleep_range(20000, 30000); > > + > > + err |= ps8622_set(cl, 0x02, 0xa1, 0x01); /* HPD low */ > > + /* SW setting */ > > + err |= ps8622_set(cl, 0x04, 0x14, 0x01); /* [1:0] SW output 1.2V voltage > > + * is lower to 96% */ > > + /* RCO SS setting */ > > + err |= ps8622_set(cl, 0x04, 0xe3, 0x20); /* [5:4] = b01 0.5%, b10 1%, > > + * b11 1.5% */ > > + err |= ps8622_set(cl, 0x04, 0xe2, 0x80); /* [7] RCO SS enable */ > > + /* RPHY Setting */ > > + err |= ps8622_set(cl, 0x04, 0x8a, 0x0c); /* [3:2] CDR tune wait cycle > > + * before measure for fine tune > > + * b00: 1us b01: 0.5us b10:2us > > + * b11: 4us */ > > + err |= ps8622_set(cl, 0x04, 0x89, 0x08); /* [3] RFD always on */ > > + err |= ps8622_set(cl, 0x04, 0x71, 0x2d); /* CTN lock in/out: > > + * 20000ppm/80000ppm. > > + * Lock out 2 times. */ > > + /* 2.7G CDR settings */ > > + err |= ps8622_set(cl, 0x04, 0x7d, 0x07); /* NOF=40LSB for HBR CDR > > + * setting */ > > + err |= ps8622_set(cl, 0x04, 0x7b, 0x00); /* [1:0] Fmin=+4bands */ > > + err |= ps8622_set(cl, 0x04, 0x7a, 0xfd); /* [7:5] DCO_FTRNG=+-40% */ > > + /* 1.62G CDR settings */ > > + err |= ps8622_set(cl, 0x04, 0xc0, 0x12); /* [5:2]NOF=64LSB [1:0]DCO > > + * scale is 2/5 */ > > + err |= ps8622_set(cl, 0x04, 0xc1, 0x92); /* Gitune=-37% */ > > + err |= ps8622_set(cl, 0x04, 0xc2, 0x1c); /* Fbstep=100% */ > > + err |= ps8622_set(cl, 0x04, 0x32, 0x80); /* [7] LOS signal disable */ > > + /* RPIO Setting */ > > + err |= ps8622_set(cl, 0x04, 0x00, 0xb0); /* [7:4] LVDS driver bias > > + * current : 75% (250mV swing) > > + * */ > > + err |= ps8622_set(cl, 0x04, 0x15, 0x40); /* [7:6] Right-bar GPIO output > > + * strength is 8mA */ > > + /* EQ Training State Machine Setting */ > > + err |= ps8622_set(cl, 0x04, 0x54, 0x10); /* RCO calibration start */ > > + /* Logic, needs more than 10 I2C command */ > > + err |= ps8622_set(cl, 0x01, 0x02, 0x80 | ps_bridge->max_lane_count); > > + /* [4:0] MAX_LANE_COUNT set to > > + * max supported lanes */ > > + err |= ps8622_set(cl, 0x01, 0x21, 0x80 | ps_bridge->lane_count); > > + /* [4:0] LANE_COUNT_SET set to > > + * chosen lane count */ > > + err |= ps8622_set(cl, 0x00, 0x52, 0x20); > > + err |= ps8622_set(cl, 0x00, 0xf1, 0x03); /* HPD CP toggle enable */ > > + err |= ps8622_set(cl, 0x00, 0x62, 0x41); > > + err |= ps8622_set(cl, 0x00, 0xf6, 0x01); /* Counter number, add 1ms > > + * counter delay */ > > + err |= ps8622_set(cl, 0x00, 0x77, 0x06); /* [6]PWM function control by > > + * DPCD0040f[7], default is PWM > > + * block always works. */ > > + err |= ps8622_set(cl, 0x00, 0x4c, 0x04); /* 04h Adjust VTotal tolerance > > + * to fix the 30Hz no display > > + * issue */ > > + err |= ps8622_set(cl, 0x01, 0xc0, 0x00); /* DPCD00400='h00, Parade OUI = > > + * 'h001cf8 */ > > + err |= ps8622_set(cl, 0x01, 0xc1, 0x1c); /* DPCD00401='h1c */ > > + err |= ps8622_set(cl, 0x01, 0xc2, 0xf8); /* DPCD00402='hf8 */ > > + err |= ps8622_set(cl, 0x01, 0xc3, 0x44); /* DPCD403~408 = ASCII code > > + * D2SLV5='h4432534c5635 */ > > + err |= ps8622_set(cl, 0x01, 0xc4, 0x32); /* DPCD404 */ > > + err |= ps8622_set(cl, 0x01, 0xc5, 0x53); /* DPCD405 */ > > + err |= ps8622_set(cl, 0x01, 0xc6, 0x4c); /* DPCD406 */ > > + err |= ps8622_set(cl, 0x01, 0xc7, 0x56); /* DPCD407 */ > > + err |= ps8622_set(cl, 0x01, 0xc8, 0x35); /* DPCD408 */ > > + err |= ps8622_set(cl, 0x01, 0xca, 0x01); /* DPCD40A, Initial Code major > > + * revision '01' */ > > + err |= ps8622_set(cl, 0x01, 0xcb, 0x05); /* DPCD40B, Initial Code minor > > + * revision '05' */ > > + if (ps_bridge->bl) { > > + err |= ps8622_set(cl, 0x01, 0xa5, 0xa0); > > + /* DPCD720, internal PWM */ > > + err |= ps8622_set(cl, 0x01, 0xa7, > > + ps_bridge->bl->props.brightness); > > + /* FFh for 100% brightness, > > + * 0h for 0% brightness */ > > + } else { > > + err |= ps8622_set(cl, 0x01, 0xa5, 0x80); > > + /* DPCD720, external PWM */ > > + } > > + err |= ps8622_set(cl, 0x01, 0xcc, 0x13); /* Set LVDS output as 6bit-VESA > > + * mapping, single LVDS channel > > + * */ > > + err |= ps8622_set(cl, 0x02, 0xb1, 0x20); /* Enable SSC set by register > > + * */ > > + err |= ps8622_set(cl, 0x04, 0x10, 0x16); /* Set SSC enabled and +/-1% > > + * central spreading */ > > + /* Logic end */ > > + err |= ps8622_set(cl, 0x04, 0x59, 0x60); /* MPU Clock source: LC => RCO > > + * */ > > + err |= ps8622_set(cl, 0x04, 0x54, 0x14); /* LC -> RCO */ > > + err |= ps8622_set(cl, 0x02, 0xa1, 0x91); /* HPD high */ > > + > > + return err ? -EIO : 0; > > +} > > + > > +static int ps8622_backlight_update(struct backlight_device *bl) > > +{ > > + struct ps8622_bridge *ps_bridge = dev_get_drvdata(&bl->dev); > > + int ret, brightness = bl->props.brightness; > > + > > + if (bl->props.power != FB_BLANK_UNBLANK || > > + bl->props.state & (BL_CORE_SUSPENDED | BL_CORE_FBBLANK)) > > + brightness = 0; > > + > > + mutex_lock(&ps_bridge->enable_mutex); > > + > > + if (!ps_bridge->enabled) { > > + ret = -EINVAL; > > + goto out; > > + } > > + > > + ret = ps8622_set(ps_bridge->client, 0x01, 0xa7, brightness); > > + > > +out: > > + mutex_unlock(&ps_bridge->enable_mutex); > > + return ret; > > +} > > + > > +static int ps8622_backlight_get(struct backlight_device *bl) > > +{ > > + return bl->props.brightness; > > +} > > + > > +static const struct backlight_ops ps8622_backlight_ops = { > > + .update_status = ps8622_backlight_update, > > + .get_brightness = ps8622_backlight_get, > > +}; > > + > > +static void ps8622_pre_enable(struct drm_bridge *bridge) > > +{ > > + struct ps8622_bridge *ps_bridge = bridge->driver_private; > > + int ret; > > + > > + mutex_lock(&ps_bridge->enable_mutex); > > + if (ps_bridge->enabled) > > + goto out; > > + > > + if (gpio_is_valid(ps_bridge->gpio_rst_n)) > > + gpio_set_value(ps_bridge->gpio_rst_n, 0); > > + > > + if (ps_bridge->v12) { > > + ret = regulator_enable(ps_bridge->v12); > > + if (ret) > > + DRM_ERROR("fails to enable ps_bridge->v12"); > > + } > > + > > + drm_panel_pre_enable(ps_bridge->panel); > > + > > + if (gpio_is_valid(ps_bridge->gpio_slp_n)) > > + gpio_set_value(ps_bridge->gpio_slp_n, 1); > > + > > + /* > > + * T1 is the range of time that it takes for the power to rise after we > > + * enable the lcd fet. T2 is the range of time in which the data sheet > > + * specifies we should deassert the reset pin. > > + * > > + * If it takes T1.max for the power to rise, we need to wait atleast > > + * T2.min before deasserting the reset pin. If it takes T1.min for the > > + * power to rise, we need to wait at most T2.max before deasserting the > > + * reset pin. > > + */ > > + usleep_range(PS8622_RST_HIGH_T2_MIN_US + PS8622_POWER_RISE_T1_MAX_US, > > + PS8622_RST_HIGH_T2_MAX_US + PS8622_POWER_RISE_T1_MIN_US); > > + > > + if (gpio_is_valid(ps_bridge->gpio_rst_n)) > > + gpio_set_value(ps_bridge->gpio_rst_n, 1); > > + > > + ret = ps8622_send_config(ps_bridge); > > + if (ret) > > + DRM_ERROR("Failed to send config to bridge (%d)\n", ret); > > + > > + ps_bridge->enabled = true; > > + > > + mutex_unlock(&ps_bridge->enable_mutex); > > + return; > > + > > +out: > > + > > + mutex_unlock(&ps_bridge->enable_mutex); > > +} > > + > > +static void ps8622_enable(struct drm_bridge *bridge) > > +{ > > + struct ps8622_bridge *ps_bridge = bridge->driver_private; > > + > > + mutex_lock(&ps_bridge->enable_mutex); > > + if (ps_bridge->enabled) > > + drm_panel_enable(ps_bridge->panel); > > + mutex_unlock(&ps_bridge->enable_mutex); > > +} > > + > > +static void ps8622_disable(struct drm_bridge *bridge) > > +{ > > + struct ps8622_bridge *ps_bridge = bridge->driver_private; > > + > > + mutex_lock(&ps_bridge->enable_mutex); > > + > > + if (!ps_bridge->enabled) > > + goto out; > > + > > + ps_bridge->enabled = false; > > + > > + drm_panel_disable(ps_bridge->panel); > > + msleep(PS8622_PWMO_END_T12_MS); > > + > > + /* > > + * This doesn't matter if the regulators are turned off, but something > > + * else might keep them on. In that case, we want to assert the slp gpio > > + * to lower power. > > + */ > > + if (gpio_is_valid(ps_bridge->gpio_slp_n)) > > + gpio_set_value(ps_bridge->gpio_slp_n, 0); > > + > > + drm_panel_post_disable(ps_bridge->panel); > > + if (ps_bridge->v12) > > + regulator_disable(ps_bridge->v12); > > + > > + /* > > + * Sleep for at least the amount of time that it takes the power rail to > > + * fall to prevent asserting the rst gpio from doing anything. > > + */ > > + usleep_range(PS8622_POWER_FALL_T16_MAX_US, > > + 2 * PS8622_POWER_FALL_T16_MAX_US); > > + if (gpio_is_valid(ps_bridge->gpio_rst_n)) > > + gpio_set_value(ps_bridge->gpio_rst_n, 0); > > + > > + msleep(PS8622_POWER_OFF_T17_MS); > > + > > +out: > > + mutex_unlock(&ps_bridge->enable_mutex); > > +} > > + > > +static void ps8622_post_disable(struct drm_bridge *bridge) > > +{ > > +} > > + > > +static void ps8622_destroy(struct drm_bridge *bridge) > > +{ > > + struct ps8622_bridge *ps_bridge = bridge->driver_private; > > + > > + drm_bridge_cleanup(bridge); > > + if (ps_bridge->bl) > > + backlight_device_unregister(ps_bridge->bl); > > + > > + put_device(&ps_bridge->client->dev); > > +} > > + > > +static const struct drm_bridge_funcs ps8622_bridge_funcs = { > > + .pre_enable = ps8622_pre_enable, > > + .enable = ps8622_enable, > > + .disable = ps8622_disable, > > + .post_disable = ps8622_post_disable, > > + .destroy = ps8622_destroy, > > +}; > > + > > +static int ps8622_get_modes(struct drm_connector *connector) > > +{ > > + struct ps8622_bridge *ps_bridge; > > + struct drm_display_mode *new_mode; > > + > > + ps_bridge = container_of(connector, struct ps8622_bridge, connector); > > + > > + new_mode = drm_mode_duplicate(connector->dev, &ps_bridge->mode); > > + if (!new_mode) { > > + DRM_ERROR("Failed to duplicate ps_bridge mode!\n"); > > + return 0; > > + } > > + drm_mode_probed_add(connector, new_mode); > > + return 1; > > +} > > + > > +static int ps8622_mode_valid(struct drm_connector *connector, > > + struct drm_display_mode *mode) > > +{ > > + struct ps8622_bridge *ps_bridge; > > + > > + ps_bridge = container_of(connector, struct ps8622_bridge, connector); > > + > > + return drm_mode_equal(mode, &ps_bridge->mode) ? MODE_OK : MODE_BAD; > > +} > > + > > +static struct drm_encoder *ps8622_best_encoder(struct drm_connector *connector) > > +{ > > + struct ps8622_bridge *ps_bridge; > > + > > + ps_bridge = container_of(connector, struct ps8622_bridge, connector); > > + > > + return ps_bridge->encoder; > > +} > > + > > +static const struct drm_connector_helper_funcs ps8622_connector_helper_funcs = { > > + .get_modes = ps8622_get_modes, > > + .mode_valid = ps8622_mode_valid, > > + .best_encoder = ps8622_best_encoder, > > +}; > > + > > +static enum drm_connector_status ps8622_detect(struct drm_connector *connector, > > + bool force) > > +{ > > + return connector_status_connected; > > +} > > + > > +static void ps8622_connector_destroy(struct drm_connector *connector) > > +{ > > + drm_connector_cleanup(connector); > > +} > > + > > +static const struct drm_connector_funcs ps8622_connector_funcs = { > > + .dpms = drm_helper_connector_dpms, > > + .fill_modes = drm_helper_probe_single_connector_modes, > > + .detect = ps8622_detect, > > + .destroy = ps8622_connector_destroy, > > +}; > > + > > +static const struct of_device_id ps8622_devices[] = { > > + { > > + .compatible = "parade,ps8622", > > + .data = &ps8622_data, > > + }, { > > + .compatible = "parade,ps8625", > > + .data = &ps8625_data, > > + }, { > > + /* end node */ > > + } > > +}; > > + > > +int ps8622_init(struct drm_device *dev, struct drm_encoder *encoder, > > + struct i2c_client *client, struct device_node *node, > > + struct drm_panel *panel) > > +{ > > + int ret; > > + const struct of_device_id *match; > > + const struct ps8622_device_data *device_data; > > + struct drm_bridge *bridge; > > + struct ps8622_bridge *ps_bridge; > > + > > + node = of_find_matching_node_and_match(NULL, ps8622_devices, &match); > > + if (!node) > > + return -ENODEV; > > + > > + bridge = devm_kzalloc(dev->dev, sizeof(*bridge), GFP_KERNEL); > > + if (!bridge) { > > + DRM_ERROR("Failed to allocate drm bridge\n"); > > + return -ENOMEM; > > + } > > + > > + ps_bridge = devm_kzalloc(dev->dev, sizeof(*ps_bridge), GFP_KERNEL); > > + if (!ps_bridge) { > > + DRM_ERROR("could not allocate ps bridge\n"); > > + return -ENOMEM; > > + } > > + > > + mutex_init(&ps_bridge->enable_mutex); > > + > > + device_data = match->data; > > + ps_bridge->client = client; > > + ps_bridge->encoder = encoder; > > + ps_bridge->bridge = bridge; > > + ps_bridge->v12 = devm_regulator_get(&client->dev, "vdd_bridge"); > > + if (IS_ERR(ps_bridge->v12)) { > > + DRM_INFO("no 1.2v regulator found for PS8622\n"); > > + ps_bridge->v12 = NULL; > > + } > > + > > + ps_bridge->gpio_slp_n = of_get_named_gpio(node, "sleep-gpio", 0); > > + if (gpio_is_valid(ps_bridge->gpio_slp_n)) { > > + ret = devm_gpio_request_one(&client->dev, ps_bridge->gpio_slp_n, > > + GPIOF_OUT_INIT_HIGH, > > + "PS8622_SLP_N"); > > + if (ret) > > + goto err_client; > > + } > > + > > + ps_bridge->gpio_rst_n = of_get_named_gpio(node, "reset-gpio", 0); > > + if (gpio_is_valid(ps_bridge->gpio_rst_n)) { > > + /* > > + * Assert the reset pin high to avoid the bridge being > > + * initialized prematurely > > + */ > > + ret = devm_gpio_request_one(&client->dev, ps_bridge->gpio_rst_n, > > + GPIOF_OUT_INIT_HIGH, > > + "PS8622_RST_N"); > > + if (ret) > > + goto err_client; > > + } > > + > > + ps_bridge->max_lane_count = device_data->max_lane_count; > > + > > + if (of_property_read_u8(node, "lane-count", &ps_bridge->lane_count)) > > + ps_bridge->lane_count = ps_bridge->max_lane_count; > > + else if (ps_bridge->lane_count > ps_bridge->max_lane_count) { > > + DRM_ERROR("lane-count property is too high for DP bridge\n"); > > + ps_bridge->lane_count = ps_bridge->max_lane_count; > > + } > > + > > + if (!of_find_property(node, "use-external-pwm", NULL)) { > > + ps_bridge->bl = backlight_device_register("ps8622-backlight", > > + dev->dev, ps_bridge, &ps8622_backlight_ops, > > + NULL); > > + if (IS_ERR(ps_bridge->bl)) { > > + DRM_ERROR("failed to register backlight\n"); > > + ret = PTR_ERR(ps_bridge->bl); > > + ps_bridge->bl = NULL; > > + goto err_client; > > + } > > + ps_bridge->bl->props.max_brightness = PS8622_MAX_BRIGHTNESS; > > + ps_bridge->bl->props.brightness = PS8622_MAX_BRIGHTNESS; > > + } > > + > > + ret = of_get_drm_display_mode(node, &ps_bridge->mode, 0); > > + if (ret) { > > + DRM_ERROR("Failed to get display mode, ret=%d\n", ret); > > + goto err_backlight; > > + } > > + > > + ret = drm_bridge_init(dev, ps_bridge->bridge, &ps8622_bridge_funcs); > > + if (ret) { > > + DRM_ERROR("Failed to initialize bridge with drm\n"); > > + goto err_backlight; > > + } > > + > > + if (panel) { > > + ps_bridge->panel = panel; > > + drm_panel_attach(ps_bridge->panel, &ps_bridge->connector); > > + } > > + > > + ps_bridge->bridge->driver_private = ps_bridge; > > + encoder->bridge = ps_bridge->bridge; > > + ps_bridge->enabled = false; > > + ps_bridge->connector.polled = DRM_CONNECTOR_POLL_HPD; > > + > > + ret = drm_connector_init(dev, &ps_bridge->connector, > > + &ps8622_connector_funcs, DRM_MODE_CONNECTOR_LVDS); > > + if (ret) { > > + DRM_ERROR("Failed to initialize connector with drm\n"); > > + goto err_bridge; > > + } > > + drm_connector_helper_add(&ps_bridge->connector, > > + &ps8622_connector_helper_funcs); > > + drm_sysfs_connector_add(&ps_bridge->connector); > > + drm_mode_connector_attach_encoder(&ps_bridge->connector, encoder); > > + > > + return 0; > > + > > +err_bridge: > > + drm_bridge_cleanup(ps_bridge->bridge); > > +err_backlight: > > + if (ps_bridge->bl) > > + backlight_device_unregister(ps_bridge->bl); > > + > > +err_client: > > + put_device(&client->dev); > > + DRM_ERROR("device probe failed : %d\n", ret); > > + return ret; > > +} > > +EXPORT_SYMBOL(ps8622_init); > > diff --git a/include/drm/bridge/ps8622.h b/include/drm/bridge/ps8622.h > > new file mode 100644 > > index 0000000..4fc1959 > > --- /dev/null > > +++ b/include/drm/bridge/ps8622.h > > @@ -0,0 +1,42 @@ > > +/* > > + * Copyright (C) 2014 Google, Inc. > > + * > > + * This software is licensed under the terms of the GNU General Public > > + * License version 2, as published by the Free Software Foundation, and > > + * may be copied, distributed, and modified under those terms. > > + * > > + * This program is distributed in the hope that it will be useful, > > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > > + * GNU General Public License for more details. > > + */ > > + > > +#ifndef _DRM_BRIDGE_PS8622_H_ > > +#define _DRM_BRIDGE_PS8622_H_ > > + > > +struct drm_device; > > +struct drm_encoder; > > +struct i2c_client; > > +struct device_node; > > +struct drm_panel; > > + > > +#ifdef CONFIG_DRM_PS8622 > > + > > +extern int ps8622_init(struct drm_device *dev, struct drm_encoder *encoder, > > + struct i2c_client *client, struct device_node *node, > > + struct drm_panel *panel); > > + > > +#else > > + > > +static inline int ps8622_init(struct drm_device *dev, > > + struct drm_encoder *encoder, > > + struct i2c_client *client, > > + struct device_node *node, > > + struct drm_panel *panel) > > +{ > > + return -ENODEV; > > +} > > + > > +#endif > > + > > +#endif > > -- > > 1.7.9.5 > > -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/gpu/drm/bridge/Kconfig b/drivers/gpu/drm/bridge/Kconfig index 3bc6845..aba21fc 100644 --- a/drivers/gpu/drm/bridge/Kconfig +++ b/drivers/gpu/drm/bridge/Kconfig @@ -4,3 +4,10 @@ config DRM_PTN3460 select DRM_KMS_HELPER select DRM_PANEL ---help--- + +config DRM_PS8622 + tristate "Parade eDP/LVDS bridge" + depends on DRM + select DRM_KMS_HELPER + select DRM_PANEL + ---help--- diff --git a/drivers/gpu/drm/bridge/Makefile b/drivers/gpu/drm/bridge/Makefile index b4733e1..d1b5daa 100644 --- a/drivers/gpu/drm/bridge/Makefile +++ b/drivers/gpu/drm/bridge/Makefile @@ -1,3 +1,4 @@ ccflags-y := -Iinclude/drm obj-$(CONFIG_DRM_PTN3460) += ptn3460.o +obj-$(CONFIG_DRM_PS8622) += ps8622.o diff --git a/drivers/gpu/drm/bridge/ps8622.c b/drivers/gpu/drm/bridge/ps8622.c new file mode 100644 index 0000000..1e6b3ca --- /dev/null +++ b/drivers/gpu/drm/bridge/ps8622.c @@ -0,0 +1,566 @@ +/* + * Parade PS8622 eDP/LVDS bridge driver + * + * Copyright (C) 2014 Google, Inc. + * + * This software is licensed under the terms of the GNU General Public + * License version 2, as published by the Free Software Foundation, and + * may be copied, distributed, and modified under those terms. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +#include <linux/module.h> +#include <linux/backlight.h> +#include <linux/delay.h> +#include <linux/err.h> +#include <linux/fb.h> +#include <linux/gpio.h> +#include <linux/i2c.h> +#include <linux/of.h> +#include <linux/of_gpio.h> +#include <linux/pm.h> +#include <linux/regulator/consumer.h> +#include <drm/drm_panel.h> + +#include "drmP.h" +#include "drm_crtc.h" +#include "drm_crtc_helper.h" + +struct ps8622_bridge { + struct drm_connector connector; + struct drm_bridge *bridge; + struct drm_encoder *encoder; + struct drm_panel *panel; + struct i2c_client *client; + struct regulator *v12; + struct backlight_device *bl; + struct mutex enable_mutex; + + int gpio_slp_n; + int gpio_rst_n; + + u8 max_lane_count; + u8 lane_count; + + bool enabled; + + struct drm_display_mode mode; +}; + +struct ps8622_device_data { + u8 max_lane_count; +}; + +static const struct ps8622_device_data ps8622_data = { + .max_lane_count = 1, +}; + +static const struct ps8622_device_data ps8625_data = { + .max_lane_count = 2, +}; + +/* Brightness scale on the Parade chip */ +#define PS8622_MAX_BRIGHTNESS 0xff + +/* Timings taken from the version 1.7 datasheet for the PS8622/PS8625 */ +#define PS8622_POWER_RISE_T1_MIN_US 10 +#define PS8622_POWER_RISE_T1_MAX_US 10000 +#define PS8622_RST_HIGH_T2_MIN_US 3000 +#define PS8622_RST_HIGH_T2_MAX_US 30000 +#define PS8622_PWMO_END_T12_MS 200 +#define PS8622_POWER_FALL_T16_MAX_US 10000 +#define PS8622_POWER_OFF_T17_MS 500 + +#if ((PS8622_RST_HIGH_T2_MIN_US + PS8622_POWER_RISE_T1_MAX_US) > \ + (PS8622_RST_HIGH_T2_MAX_US + PS8622_POWER_RISE_T1_MIN_US)) +#error "T2.min + T1.max must be less than T2.max + T1.min" +#endif + +static int ps8622_set(struct i2c_client *client, u8 page, u8 reg, u8 val) +{ + int ret; + struct i2c_adapter *adap = client->adapter; + struct i2c_msg msg; + u8 data[] = {reg, val}; + + msg.addr = client->addr + page; + msg.flags = 0; + msg.len = sizeof(data); + msg.buf = data; + + ret = i2c_transfer(adap, &msg, 1); + if (ret != 1) + pr_warn("PS8622 I2C write (0x%02x,0x%02x,0x%02x) failed: %d\n", + client->addr + page, reg, val, ret); + return !(ret == 1); +} + +static int ps8622_send_config(struct ps8622_bridge *ps_bridge) +{ + struct i2c_client *cl = ps_bridge->client; + int err = 0; + + /* wait 20ms after power ON */ + usleep_range(20000, 30000); + + err |= ps8622_set(cl, 0x02, 0xa1, 0x01); /* HPD low */ + /* SW setting */ + err |= ps8622_set(cl, 0x04, 0x14, 0x01); /* [1:0] SW output 1.2V voltage + * is lower to 96% */ + /* RCO SS setting */ + err |= ps8622_set(cl, 0x04, 0xe3, 0x20); /* [5:4] = b01 0.5%, b10 1%, + * b11 1.5% */ + err |= ps8622_set(cl, 0x04, 0xe2, 0x80); /* [7] RCO SS enable */ + /* RPHY Setting */ + err |= ps8622_set(cl, 0x04, 0x8a, 0x0c); /* [3:2] CDR tune wait cycle + * before measure for fine tune + * b00: 1us b01: 0.5us b10:2us + * b11: 4us */ + err |= ps8622_set(cl, 0x04, 0x89, 0x08); /* [3] RFD always on */ + err |= ps8622_set(cl, 0x04, 0x71, 0x2d); /* CTN lock in/out: + * 20000ppm/80000ppm. + * Lock out 2 times. */ + /* 2.7G CDR settings */ + err |= ps8622_set(cl, 0x04, 0x7d, 0x07); /* NOF=40LSB for HBR CDR + * setting */ + err |= ps8622_set(cl, 0x04, 0x7b, 0x00); /* [1:0] Fmin=+4bands */ + err |= ps8622_set(cl, 0x04, 0x7a, 0xfd); /* [7:5] DCO_FTRNG=+-40% */ + /* 1.62G CDR settings */ + err |= ps8622_set(cl, 0x04, 0xc0, 0x12); /* [5:2]NOF=64LSB [1:0]DCO + * scale is 2/5 */ + err |= ps8622_set(cl, 0x04, 0xc1, 0x92); /* Gitune=-37% */ + err |= ps8622_set(cl, 0x04, 0xc2, 0x1c); /* Fbstep=100% */ + err |= ps8622_set(cl, 0x04, 0x32, 0x80); /* [7] LOS signal disable */ + /* RPIO Setting */ + err |= ps8622_set(cl, 0x04, 0x00, 0xb0); /* [7:4] LVDS driver bias + * current : 75% (250mV swing) + * */ + err |= ps8622_set(cl, 0x04, 0x15, 0x40); /* [7:6] Right-bar GPIO output + * strength is 8mA */ + /* EQ Training State Machine Setting */ + err |= ps8622_set(cl, 0x04, 0x54, 0x10); /* RCO calibration start */ + /* Logic, needs more than 10 I2C command */ + err |= ps8622_set(cl, 0x01, 0x02, 0x80 | ps_bridge->max_lane_count); + /* [4:0] MAX_LANE_COUNT set to + * max supported lanes */ + err |= ps8622_set(cl, 0x01, 0x21, 0x80 | ps_bridge->lane_count); + /* [4:0] LANE_COUNT_SET set to + * chosen lane count */ + err |= ps8622_set(cl, 0x00, 0x52, 0x20); + err |= ps8622_set(cl, 0x00, 0xf1, 0x03); /* HPD CP toggle enable */ + err |= ps8622_set(cl, 0x00, 0x62, 0x41); + err |= ps8622_set(cl, 0x00, 0xf6, 0x01); /* Counter number, add 1ms + * counter delay */ + err |= ps8622_set(cl, 0x00, 0x77, 0x06); /* [6]PWM function control by + * DPCD0040f[7], default is PWM + * block always works. */ + err |= ps8622_set(cl, 0x00, 0x4c, 0x04); /* 04h Adjust VTotal tolerance + * to fix the 30Hz no display + * issue */ + err |= ps8622_set(cl, 0x01, 0xc0, 0x00); /* DPCD00400='h00, Parade OUI = + * 'h001cf8 */ + err |= ps8622_set(cl, 0x01, 0xc1, 0x1c); /* DPCD00401='h1c */ + err |= ps8622_set(cl, 0x01, 0xc2, 0xf8); /* DPCD00402='hf8 */ + err |= ps8622_set(cl, 0x01, 0xc3, 0x44); /* DPCD403~408 = ASCII code + * D2SLV5='h4432534c5635 */ + err |= ps8622_set(cl, 0x01, 0xc4, 0x32); /* DPCD404 */ + err |= ps8622_set(cl, 0x01, 0xc5, 0x53); /* DPCD405 */ + err |= ps8622_set(cl, 0x01, 0xc6, 0x4c); /* DPCD406 */ + err |= ps8622_set(cl, 0x01, 0xc7, 0x56); /* DPCD407 */ + err |= ps8622_set(cl, 0x01, 0xc8, 0x35); /* DPCD408 */ + err |= ps8622_set(cl, 0x01, 0xca, 0x01); /* DPCD40A, Initial Code major + * revision '01' */ + err |= ps8622_set(cl, 0x01, 0xcb, 0x05); /* DPCD40B, Initial Code minor + * revision '05' */ + if (ps_bridge->bl) { + err |= ps8622_set(cl, 0x01, 0xa5, 0xa0); + /* DPCD720, internal PWM */ + err |= ps8622_set(cl, 0x01, 0xa7, + ps_bridge->bl->props.brightness); + /* FFh for 100% brightness, + * 0h for 0% brightness */ + } else { + err |= ps8622_set(cl, 0x01, 0xa5, 0x80); + /* DPCD720, external PWM */ + } + err |= ps8622_set(cl, 0x01, 0xcc, 0x13); /* Set LVDS output as 6bit-VESA + * mapping, single LVDS channel + * */ + err |= ps8622_set(cl, 0x02, 0xb1, 0x20); /* Enable SSC set by register + * */ + err |= ps8622_set(cl, 0x04, 0x10, 0x16); /* Set SSC enabled and +/-1% + * central spreading */ + /* Logic end */ + err |= ps8622_set(cl, 0x04, 0x59, 0x60); /* MPU Clock source: LC => RCO + * */ + err |= ps8622_set(cl, 0x04, 0x54, 0x14); /* LC -> RCO */ + err |= ps8622_set(cl, 0x02, 0xa1, 0x91); /* HPD high */ + + return err ? -EIO : 0; +} + +static int ps8622_backlight_update(struct backlight_device *bl) +{ + struct ps8622_bridge *ps_bridge = dev_get_drvdata(&bl->dev); + int ret, brightness = bl->props.brightness; + + if (bl->props.power != FB_BLANK_UNBLANK || + bl->props.state & (BL_CORE_SUSPENDED | BL_CORE_FBBLANK)) + brightness = 0; + + mutex_lock(&ps_bridge->enable_mutex); + + if (!ps_bridge->enabled) { + ret = -EINVAL; + goto out; + } + + ret = ps8622_set(ps_bridge->client, 0x01, 0xa7, brightness); + +out: + mutex_unlock(&ps_bridge->enable_mutex); + return ret; +} + +static int ps8622_backlight_get(struct backlight_device *bl) +{ + return bl->props.brightness; +} + +static const struct backlight_ops ps8622_backlight_ops = { + .update_status = ps8622_backlight_update, + .get_brightness = ps8622_backlight_get, +}; + +static void ps8622_pre_enable(struct drm_bridge *bridge) +{ + struct ps8622_bridge *ps_bridge = bridge->driver_private; + int ret; + + mutex_lock(&ps_bridge->enable_mutex); + if (ps_bridge->enabled) + goto out; + + if (gpio_is_valid(ps_bridge->gpio_rst_n)) + gpio_set_value(ps_bridge->gpio_rst_n, 0); + + if (ps_bridge->v12) { + ret = regulator_enable(ps_bridge->v12); + if (ret) + DRM_ERROR("fails to enable ps_bridge->v12"); + } + + drm_panel_pre_enable(ps_bridge->panel); + + if (gpio_is_valid(ps_bridge->gpio_slp_n)) + gpio_set_value(ps_bridge->gpio_slp_n, 1); + + /* + * T1 is the range of time that it takes for the power to rise after we + * enable the lcd fet. T2 is the range of time in which the data sheet + * specifies we should deassert the reset pin. + * + * If it takes T1.max for the power to rise, we need to wait atleast + * T2.min before deasserting the reset pin. If it takes T1.min for the + * power to rise, we need to wait at most T2.max before deasserting the + * reset pin. + */ + usleep_range(PS8622_RST_HIGH_T2_MIN_US + PS8622_POWER_RISE_T1_MAX_US, + PS8622_RST_HIGH_T2_MAX_US + PS8622_POWER_RISE_T1_MIN_US); + + if (gpio_is_valid(ps_bridge->gpio_rst_n)) + gpio_set_value(ps_bridge->gpio_rst_n, 1); + + ret = ps8622_send_config(ps_bridge); + if (ret) + DRM_ERROR("Failed to send config to bridge (%d)\n", ret); + + ps_bridge->enabled = true; + + mutex_unlock(&ps_bridge->enable_mutex); + return; + +out: + + mutex_unlock(&ps_bridge->enable_mutex); +} + +static void ps8622_enable(struct drm_bridge *bridge) +{ + struct ps8622_bridge *ps_bridge = bridge->driver_private; + + mutex_lock(&ps_bridge->enable_mutex); + if (ps_bridge->enabled) + drm_panel_enable(ps_bridge->panel); + mutex_unlock(&ps_bridge->enable_mutex); +} + +static void ps8622_disable(struct drm_bridge *bridge) +{ + struct ps8622_bridge *ps_bridge = bridge->driver_private; + + mutex_lock(&ps_bridge->enable_mutex); + + if (!ps_bridge->enabled) + goto out; + + ps_bridge->enabled = false; + + drm_panel_disable(ps_bridge->panel); + msleep(PS8622_PWMO_END_T12_MS); + + /* + * This doesn't matter if the regulators are turned off, but something + * else might keep them on. In that case, we want to assert the slp gpio + * to lower power. + */ + if (gpio_is_valid(ps_bridge->gpio_slp_n)) + gpio_set_value(ps_bridge->gpio_slp_n, 0); + + drm_panel_post_disable(ps_bridge->panel); + if (ps_bridge->v12) + regulator_disable(ps_bridge->v12); + + /* + * Sleep for at least the amount of time that it takes the power rail to + * fall to prevent asserting the rst gpio from doing anything. + */ + usleep_range(PS8622_POWER_FALL_T16_MAX_US, + 2 * PS8622_POWER_FALL_T16_MAX_US); + if (gpio_is_valid(ps_bridge->gpio_rst_n)) + gpio_set_value(ps_bridge->gpio_rst_n, 0); + + msleep(PS8622_POWER_OFF_T17_MS); + +out: + mutex_unlock(&ps_bridge->enable_mutex); +} + +static void ps8622_post_disable(struct drm_bridge *bridge) +{ +} + +static void ps8622_destroy(struct drm_bridge *bridge) +{ + struct ps8622_bridge *ps_bridge = bridge->driver_private; + + drm_bridge_cleanup(bridge); + if (ps_bridge->bl) + backlight_device_unregister(ps_bridge->bl); + + put_device(&ps_bridge->client->dev); +} + +static const struct drm_bridge_funcs ps8622_bridge_funcs = { + .pre_enable = ps8622_pre_enable, + .enable = ps8622_enable, + .disable = ps8622_disable, + .post_disable = ps8622_post_disable, + .destroy = ps8622_destroy, +}; + +static int ps8622_get_modes(struct drm_connector *connector) +{ + struct ps8622_bridge *ps_bridge; + struct drm_display_mode *new_mode; + + ps_bridge = container_of(connector, struct ps8622_bridge, connector); + + new_mode = drm_mode_duplicate(connector->dev, &ps_bridge->mode); + if (!new_mode) { + DRM_ERROR("Failed to duplicate ps_bridge mode!\n"); + return 0; + } + drm_mode_probed_add(connector, new_mode); + return 1; +} + +static int ps8622_mode_valid(struct drm_connector *connector, + struct drm_display_mode *mode) +{ + struct ps8622_bridge *ps_bridge; + + ps_bridge = container_of(connector, struct ps8622_bridge, connector); + + return drm_mode_equal(mode, &ps_bridge->mode) ? MODE_OK : MODE_BAD; +} + +static struct drm_encoder *ps8622_best_encoder(struct drm_connector *connector) +{ + struct ps8622_bridge *ps_bridge; + + ps_bridge = container_of(connector, struct ps8622_bridge, connector); + + return ps_bridge->encoder; +} + +static const struct drm_connector_helper_funcs ps8622_connector_helper_funcs = { + .get_modes = ps8622_get_modes, + .mode_valid = ps8622_mode_valid, + .best_encoder = ps8622_best_encoder, +}; + +static enum drm_connector_status ps8622_detect(struct drm_connector *connector, + bool force) +{ + return connector_status_connected; +} + +static void ps8622_connector_destroy(struct drm_connector *connector) +{ + drm_connector_cleanup(connector); +} + +static const struct drm_connector_funcs ps8622_connector_funcs = { + .dpms = drm_helper_connector_dpms, + .fill_modes = drm_helper_probe_single_connector_modes, + .detect = ps8622_detect, + .destroy = ps8622_connector_destroy, +}; + +static const struct of_device_id ps8622_devices[] = { + { + .compatible = "parade,ps8622", + .data = &ps8622_data, + }, { + .compatible = "parade,ps8625", + .data = &ps8625_data, + }, { + /* end node */ + } +}; + +int ps8622_init(struct drm_device *dev, struct drm_encoder *encoder, + struct i2c_client *client, struct device_node *node, + struct drm_panel *panel) +{ + int ret; + const struct of_device_id *match; + const struct ps8622_device_data *device_data; + struct drm_bridge *bridge; + struct ps8622_bridge *ps_bridge; + + node = of_find_matching_node_and_match(NULL, ps8622_devices, &match); + if (!node) + return -ENODEV; + + bridge = devm_kzalloc(dev->dev, sizeof(*bridge), GFP_KERNEL); + if (!bridge) { + DRM_ERROR("Failed to allocate drm bridge\n"); + return -ENOMEM; + } + + ps_bridge = devm_kzalloc(dev->dev, sizeof(*ps_bridge), GFP_KERNEL); + if (!ps_bridge) { + DRM_ERROR("could not allocate ps bridge\n"); + return -ENOMEM; + } + + mutex_init(&ps_bridge->enable_mutex); + + device_data = match->data; + ps_bridge->client = client; + ps_bridge->encoder = encoder; + ps_bridge->bridge = bridge; + ps_bridge->v12 = devm_regulator_get(&client->dev, "vdd_bridge"); + if (IS_ERR(ps_bridge->v12)) { + DRM_INFO("no 1.2v regulator found for PS8622\n"); + ps_bridge->v12 = NULL; + } + + ps_bridge->gpio_slp_n = of_get_named_gpio(node, "sleep-gpio", 0); + if (gpio_is_valid(ps_bridge->gpio_slp_n)) { + ret = devm_gpio_request_one(&client->dev, ps_bridge->gpio_slp_n, + GPIOF_OUT_INIT_HIGH, + "PS8622_SLP_N"); + if (ret) + goto err_client; + } + + ps_bridge->gpio_rst_n = of_get_named_gpio(node, "reset-gpio", 0); + if (gpio_is_valid(ps_bridge->gpio_rst_n)) { + /* + * Assert the reset pin high to avoid the bridge being + * initialized prematurely + */ + ret = devm_gpio_request_one(&client->dev, ps_bridge->gpio_rst_n, + GPIOF_OUT_INIT_HIGH, + "PS8622_RST_N"); + if (ret) + goto err_client; + } + + ps_bridge->max_lane_count = device_data->max_lane_count; + + if (of_property_read_u8(node, "lane-count", &ps_bridge->lane_count)) + ps_bridge->lane_count = ps_bridge->max_lane_count; + else if (ps_bridge->lane_count > ps_bridge->max_lane_count) { + DRM_ERROR("lane-count property is too high for DP bridge\n"); + ps_bridge->lane_count = ps_bridge->max_lane_count; + } + + if (!of_find_property(node, "use-external-pwm", NULL)) { + ps_bridge->bl = backlight_device_register("ps8622-backlight", + dev->dev, ps_bridge, &ps8622_backlight_ops, + NULL); + if (IS_ERR(ps_bridge->bl)) { + DRM_ERROR("failed to register backlight\n"); + ret = PTR_ERR(ps_bridge->bl); + ps_bridge->bl = NULL; + goto err_client; + } + ps_bridge->bl->props.max_brightness = PS8622_MAX_BRIGHTNESS; + ps_bridge->bl->props.brightness = PS8622_MAX_BRIGHTNESS; + } + + ret = of_get_drm_display_mode(node, &ps_bridge->mode, 0); + if (ret) { + DRM_ERROR("Failed to get display mode, ret=%d\n", ret); + goto err_backlight; + } + + ret = drm_bridge_init(dev, ps_bridge->bridge, &ps8622_bridge_funcs); + if (ret) { + DRM_ERROR("Failed to initialize bridge with drm\n"); + goto err_backlight; + } + + if (panel) { + ps_bridge->panel = panel; + drm_panel_attach(ps_bridge->panel, &ps_bridge->connector); + } + + ps_bridge->bridge->driver_private = ps_bridge; + encoder->bridge = ps_bridge->bridge; + ps_bridge->enabled = false; + ps_bridge->connector.polled = DRM_CONNECTOR_POLL_HPD; + + ret = drm_connector_init(dev, &ps_bridge->connector, + &ps8622_connector_funcs, DRM_MODE_CONNECTOR_LVDS); + if (ret) { + DRM_ERROR("Failed to initialize connector with drm\n"); + goto err_bridge; + } + drm_connector_helper_add(&ps_bridge->connector, + &ps8622_connector_helper_funcs); + drm_sysfs_connector_add(&ps_bridge->connector); + drm_mode_connector_attach_encoder(&ps_bridge->connector, encoder); + + return 0; + +err_bridge: + drm_bridge_cleanup(ps_bridge->bridge); +err_backlight: + if (ps_bridge->bl) + backlight_device_unregister(ps_bridge->bl); + +err_client: + put_device(&client->dev); + DRM_ERROR("device probe failed : %d\n", ret); + return ret; +} +EXPORT_SYMBOL(ps8622_init); diff --git a/include/drm/bridge/ps8622.h b/include/drm/bridge/ps8622.h new file mode 100644 index 0000000..4fc1959 --- /dev/null +++ b/include/drm/bridge/ps8622.h @@ -0,0 +1,42 @@ +/* + * Copyright (C) 2014 Google, Inc. + * + * This software is licensed under the terms of the GNU General Public + * License version 2, as published by the Free Software Foundation, and + * may be copied, distributed, and modified under those terms. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +#ifndef _DRM_BRIDGE_PS8622_H_ +#define _DRM_BRIDGE_PS8622_H_ + +struct drm_device; +struct drm_encoder; +struct i2c_client; +struct device_node; +struct drm_panel; + +#ifdef CONFIG_DRM_PS8622 + +extern int ps8622_init(struct drm_device *dev, struct drm_encoder *encoder, + struct i2c_client *client, struct device_node *node, + struct drm_panel *panel); + +#else + +static inline int ps8622_init(struct drm_device *dev, + struct drm_encoder *encoder, + struct i2c_client *client, + struct device_node *node, + struct drm_panel *panel) +{ + return -ENODEV; +} + +#endif + +#endif