[PATCH V2 8/9] drm/bridge: Add PS8622 bridge driver
Sean Paul
seanpaul at google.com
Tue Apr 22 06:31:21 PDT 2014
On Mon, Apr 21, 2014 at 6:39 PM, Ajay Kumar <ajaykumar.rs at samsung.com> wrote:
> This patch adds a drm_bridge driver for the PS8622 DisplayPort to LVDS
> bridge chip.
>
> Signed-off-by: Andrew Bresticker <abrestic at chromium.org>
> Signed-off-by: Sean Paul <seanpaul at chromium.org>
> Signed-off-by: Rahul Sharma <rahul.sharma at samsung.com>
> Signed-off-by: Ajay Kumar <ajaykumar.rs at 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 at 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
>
More information about the dri-devel
mailing list