* [PATCH v4 1/2] drm: bridge: Add sii902x driver
@ 2016-05-16 14:04 Boris Brezillon
2016-05-16 14:04 ` [PATCH v4 2/2] drm: bridge: add sii902x DT bindings doc Boris Brezillon
2016-05-17 6:47 ` [PATCH v4 1/2] drm: bridge: Add sii902x driver Daniel Vetter
0 siblings, 2 replies; 8+ messages in thread
From: Boris Brezillon @ 2016-05-16 14:04 UTC (permalink / raw)
To: David Airlie, Daniel Vetter, dri-devel
Cc: Mark Rutland, Meng Yi, Xiubo Li, Pawel Moll, Ian Campbell,
Alison Wang, Nicolas Ferre, devicetree, Rob Herring,
Alexandre Belloni, Kumar Gala, Jean-Christophe Plagniol-Villard,
Jianwei Wang
Add basic support for the sii902x RGB -> HDMI bridge.
This driver does not support audio output yet.
Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
Tested-by: Nicolas Ferre <nicolas.ferre@atmel.com>
---
Hello,
This patch is only adding basic support for the sii9022 chip.
As stated in the commit log, there's no audio support, but the
driver also hardcodes a lot of things (like the RGB input format to
use).
Best Regards,
Boris
Changes in v4:
- make reset GPIO optional
- only support attaching to DRM devices supporting atomic updates
Changes in v3:
- fix get_modes() implementation to avoid turning the screen in power
save mode
- rename the driver (sil902x -> sii902x)
Changes in v2:
- fix errors reported by the kbuild robot
---
drivers/gpu/drm/bridge/Kconfig | 8 +
drivers/gpu/drm/bridge/Makefile | 1 +
drivers/gpu/drm/bridge/sii902x.c | 477 +++++++++++++++++++++++++++++++++++++++
3 files changed, 486 insertions(+)
create mode 100644 drivers/gpu/drm/bridge/sii902x.c
diff --git a/drivers/gpu/drm/bridge/Kconfig b/drivers/gpu/drm/bridge/Kconfig
index efd94e0..a692c5b 100644
--- a/drivers/gpu/drm/bridge/Kconfig
+++ b/drivers/gpu/drm/bridge/Kconfig
@@ -40,6 +40,14 @@ config DRM_PARADE_PS8622
---help---
Parade eDP-LVDS bridge chip driver.
+config DRM_SII902X
+ tristate "Silicon Image sii902x RGB/HDMI bridge"
+ depends on OF
+ select DRM_KMS_HELPER
+ select REGMAP_I2C
+ ---help---
+ Silicon Image sii902x bridge chip driver.
+
source "drivers/gpu/drm/bridge/analogix/Kconfig"
endmenu
diff --git a/drivers/gpu/drm/bridge/Makefile b/drivers/gpu/drm/bridge/Makefile
index ff821f4..b2a37e8 100644
--- a/drivers/gpu/drm/bridge/Makefile
+++ b/drivers/gpu/drm/bridge/Makefile
@@ -4,4 +4,5 @@ obj-$(CONFIG_DRM_DW_HDMI) += dw-hdmi.o
obj-$(CONFIG_DRM_DW_HDMI_AHB_AUDIO) += dw-hdmi-ahb-audio.o
obj-$(CONFIG_DRM_NXP_PTN3460) += nxp-ptn3460.o
obj-$(CONFIG_DRM_PARADE_PS8622) += parade-ps8622.o
+obj-$(CONFIG_DRM_SII902X) += sii902x.o
obj-$(CONFIG_DRM_ANALOGIX_DP) += analogix/
diff --git a/drivers/gpu/drm/bridge/sii902x.c b/drivers/gpu/drm/bridge/sii902x.c
new file mode 100644
index 0000000..76ad73a
--- /dev/null
+++ b/drivers/gpu/drm/bridge/sii902x.c
@@ -0,0 +1,477 @@
+/*
+ * Copyright (C) 2014 Atmel
+ * Bo Shen <voice.shen@atmel.com>
+ *
+ * Authors: Bo Shen <voice.shen@atmel.com>
+ * Boris Brezillon <boris.brezillon@free-electrons.com>
+ * Wu, Songjun <Songjun.Wu@atmel.com>
+ *
+ *
+ * Copyright (C) 2010-2011 Freescale Semiconductor, Inc. All Rights Reserved.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * 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/component.h>
+#include <linux/gpio/consumer.h>
+#include <linux/i2c.h>
+#include <linux/module.h>
+#include <linux/of.h>
+#include <linux/of_gpio.h>
+#include <linux/regmap.h>
+
+#include <drm/drmP.h>
+#include <drm/drm_atomic.h>
+#include <drm/drm_atomic_helper.h>
+#include <drm/drm_crtc_helper.h>
+#include <drm/drm_edid.h>
+#include <drm/drm_encoder_slave.h>
+
+#define SIL902X_TPI_VIDEO_DATA 0x0
+
+#define SIL902X_TPI_PIXEL_REPETITION 0x8
+#define SIL902X_TPI_AVI_PIXEL_REP_BUS_24BIT BIT(5)
+#define SIL902X_TPI_AVI_PIXEL_REP_RISING_EDGE BIT(4)
+#define SIL902X_TPI_AVI_PIXEL_REP_4X 3
+#define SIL902X_TPI_AVI_PIXEL_REP_2X 1
+#define SIL902X_TPI_AVI_PIXEL_REP_NONE 0
+#define SIL902X_TPI_CLK_RATIO_HALF (0 << 6)
+#define SIL902X_TPI_CLK_RATIO_1X (1 << 6)
+#define SIL902X_TPI_CLK_RATIO_2X (2 << 6)
+#define SIL902X_TPI_CLK_RATIO_4X (3 << 6)
+
+#define SIL902X_TPI_AVI_IN_FORMAT 0x9
+#define SIL902X_TPI_AVI_INPUT_BITMODE_12BIT BIT(7)
+#define SIL902X_TPI_AVI_INPUT_DITHER BIT(6)
+#define SIL902X_TPI_AVI_INPUT_RANGE_LIMITED (2 << 2)
+#define SIL902X_TPI_AVI_INPUT_RANGE_FULL (1 << 2)
+#define SIL902X_TPI_AVI_INPUT_RANGE_AUTO (0 << 2)
+#define SIL902X_TPI_AVI_INPUT_COLORSPACE_BLACK (3 << 0)
+#define SIL902X_TPI_AVI_INPUT_COLORSPACE_YUV422 (2 << 0)
+#define SIL902X_TPI_AVI_INPUT_COLORSPACE_YUV444 (1 << 0)
+#define SIL902X_TPI_AVI_INPUT_COLORSPACE_RGB (0 << 0)
+
+#define SIL902X_TPI_AVI_INFOFRAME 0x0c
+
+#define SIL902X_SYS_CTRL_DATA 0x1a
+#define SIL902X_SYS_CTRL_PWR_DWN BIT(4)
+#define SIL902X_SYS_CTRL_AV_MUTE BIT(3)
+#define SIL902X_SYS_CTRL_DDC_BUS_REQ BIT(2)
+#define SIL902X_SYS_CTRL_DDC_BUS_GRTD BIT(1)
+#define SIL902X_SYS_CTRL_OUTPUT_MODE BIT(0)
+#define SIL902X_SYS_CTRL_OUTPUT_HDMI 1
+#define SIL902X_SYS_CTRL_OUTPUT_DVI 0
+
+#define SIL902X_REG_CHIPID(n) (0x1b + (n))
+
+#define SIL902X_PWR_STATE_CTRL 0x1e
+#define SIL902X_AVI_POWER_STATE_MSK GENMASK(1, 0)
+#define SIL902X_AVI_POWER_STATE_D(l) ((l) & SIL902X_AVI_POWER_STATE_MSK)
+
+#define SI902X_INT_ENABLE 0x3c
+#define SI902X_INT_STATUS 0x3d
+#define SI902X_HOTPLUG_EVENT BIT(0)
+#define SI902X_PLUGGED_STATUS BIT(2)
+
+#define SIL902X_REG_TPI_RQB 0xc7
+
+struct sii902x {
+ struct i2c_client *i2c;
+ struct regmap *regmap;
+ struct drm_bridge bridge;
+ struct drm_connector connector;
+ struct gpio_desc *reset_gpio;
+ struct work_struct hotplug_work;
+};
+
+static inline struct sii902x *bridge_to_sii902x(struct drm_bridge *bridge)
+{
+ return container_of(bridge, struct sii902x, bridge);
+}
+
+static inline struct sii902x *connector_to_sii902x(struct drm_connector *con)
+{
+ return container_of(con, struct sii902x, connector);
+}
+
+static void sii902x_reset(struct sii902x *sii902x)
+{
+ if (!sii902x->reset_gpio)
+ return;
+
+ gpiod_set_value(sii902x->reset_gpio, 1);
+
+ msleep(100);
+
+ gpiod_set_value(sii902x->reset_gpio, 0);
+}
+
+static void sii902x_connector_destroy(struct drm_connector *connector)
+{
+ drm_connector_unregister(connector);
+ drm_connector_cleanup(connector);
+}
+
+static enum drm_connector_status
+sii902x_connector_detect(struct drm_connector *connector, bool force)
+{
+ struct sii902x *sii902x = connector_to_sii902x(connector);
+ unsigned int status;
+
+ regmap_read(sii902x->regmap, SI902X_INT_STATUS, &status);
+
+ return (status & SI902X_PLUGGED_STATUS) ?
+ connector_status_connected : connector_status_disconnected;
+}
+
+static const struct drm_connector_funcs sii902x_connector_funcs = {
+ .dpms = drm_atomic_helper_connector_dpms,
+ .detect = sii902x_connector_detect,
+ .fill_modes = drm_helper_probe_single_connector_modes,
+ .destroy = sii902x_connector_destroy,
+ .reset = drm_atomic_helper_connector_reset,
+ .atomic_duplicate_state = drm_atomic_helper_connector_duplicate_state,
+ .atomic_destroy_state = drm_atomic_helper_connector_destroy_state,
+};
+
+static int sii902x_get_modes(struct drm_connector *connector)
+{
+ struct sii902x *sii902x = connector_to_sii902x(connector);
+ struct regmap *regmap = sii902x->regmap;
+ u32 bus_format = MEDIA_BUS_FMT_RGB888_1X24;
+ unsigned int status;
+ struct edid *edid;
+ int num = 0;
+ int ret;
+ int i;
+
+ ret = regmap_update_bits(regmap, SIL902X_SYS_CTRL_DATA,
+ SIL902X_SYS_CTRL_DDC_BUS_REQ,
+ SIL902X_SYS_CTRL_DDC_BUS_REQ);
+ if (ret)
+ return ret;
+
+ i = 0;
+ do {
+ ret = regmap_read(regmap, SIL902X_SYS_CTRL_DATA, &status);
+ if (ret)
+ return ret;
+ i++;
+ } while (!(status & SIL902X_SYS_CTRL_DDC_BUS_GRTD));
+
+ ret = regmap_write(regmap, SIL902X_SYS_CTRL_DATA, status);
+ if (ret)
+ return ret;
+
+ edid = drm_get_edid(connector, sii902x->i2c->adapter);
+ drm_mode_connector_update_edid_property(connector, edid);
+ if (edid) {
+ num += drm_add_edid_modes(connector, edid);
+ kfree(edid);
+ }
+
+ ret = drm_display_info_set_bus_formats(&connector->display_info,
+ &bus_format, 1);
+ if (ret)
+ return ret;
+
+ regmap_read(regmap, SIL902X_SYS_CTRL_DATA, &status);
+ if (ret)
+ return ret;
+
+ ret = regmap_update_bits(regmap, SIL902X_SYS_CTRL_DATA,
+ SIL902X_SYS_CTRL_DDC_BUS_REQ |
+ SIL902X_SYS_CTRL_DDC_BUS_GRTD, 0);
+ if (ret)
+ return ret;
+
+ i = 0;
+ do {
+ ret = regmap_read(regmap, SIL902X_SYS_CTRL_DATA, &status);
+ if (ret)
+ return ret;
+ i++;
+ } while (status & (SIL902X_SYS_CTRL_DDC_BUS_REQ |
+ SIL902X_SYS_CTRL_DDC_BUS_GRTD));
+
+ return num;
+}
+
+static enum drm_mode_status sii902x_mode_valid(struct drm_connector *connector,
+ struct drm_display_mode *mode)
+{
+ /* TODO: check mode */
+
+ return MODE_OK;
+}
+
+static struct drm_encoder *sii902x_best_encoder(struct drm_connector *connector)
+{
+ struct sii902x *sii902x = connector_to_sii902x(connector);
+
+ return sii902x->bridge.encoder;
+}
+
+static const struct drm_connector_helper_funcs sii902x_connector_helper_funcs = {
+ .get_modes = sii902x_get_modes,
+ .mode_valid = sii902x_mode_valid,
+ .best_encoder = sii902x_best_encoder,
+};
+
+static void sii902x_bridge_disable(struct drm_bridge *bridge)
+{
+ struct sii902x *sii902x = bridge_to_sii902x(bridge);
+
+ regmap_update_bits(sii902x->regmap, SIL902X_SYS_CTRL_DATA,
+ SIL902X_SYS_CTRL_PWR_DWN,
+ SIL902X_SYS_CTRL_PWR_DWN);
+}
+
+static void sii902x_bridge_enable(struct drm_bridge *bridge)
+{
+ struct sii902x *sii902x = bridge_to_sii902x(bridge);
+
+ regmap_update_bits(sii902x->regmap, SIL902X_PWR_STATE_CTRL,
+ SIL902X_AVI_POWER_STATE_MSK,
+ SIL902X_AVI_POWER_STATE_D(0));
+ regmap_update_bits(sii902x->regmap, SIL902X_SYS_CTRL_DATA,
+ SIL902X_SYS_CTRL_PWR_DWN, 0);
+}
+
+static void sii902x_bridge_mode_set(struct drm_bridge *bridge,
+ struct drm_display_mode *mode,
+ struct drm_display_mode *adj)
+{
+ u8 buf[HDMI_INFOFRAME_HEADER_SIZE + HDMI_AVI_INFOFRAME_SIZE];
+ struct sii902x *sii902x = bridge_to_sii902x(bridge);
+ struct regmap *regmap = sii902x->regmap;
+ struct hdmi_avi_infoframe frame;
+ int ret;
+
+ buf[0] = adj->clock;
+ buf[1] = adj->clock >> 8;
+ buf[2] = adj->vrefresh;
+ buf[3] = 0x00;
+ buf[4] = adj->hdisplay;
+ buf[5] = adj->hdisplay >> 8;
+ buf[6] = adj->vdisplay;
+ buf[7] = adj->vdisplay >> 8;
+ buf[8] = SIL902X_TPI_CLK_RATIO_1X | SIL902X_TPI_AVI_PIXEL_REP_NONE |
+ SIL902X_TPI_AVI_PIXEL_REP_BUS_24BIT;
+ buf[9] = SIL902X_TPI_AVI_INPUT_RANGE_AUTO |
+ SIL902X_TPI_AVI_INPUT_COLORSPACE_RGB;
+
+ ret = regmap_bulk_write(regmap, SIL902X_TPI_VIDEO_DATA, buf, 10);
+ if (ret)
+ return;
+
+ ret = drm_hdmi_avi_infoframe_from_display_mode(&frame, adj);
+ if (ret < 0) {
+ DRM_ERROR("couldn't fill AVI infoframe\n");
+ return;
+ }
+
+ ret = hdmi_avi_infoframe_pack(&frame, buf, sizeof(buf));
+ if (ret < 0) {
+ DRM_ERROR("failed to pack AVI infoframe: %d\n", ret);
+ return;
+ }
+
+ /* Do not send the infoframe header, but keep the CRC field. */
+ regmap_bulk_write(regmap, SIL902X_TPI_AVI_INFOFRAME,
+ buf + HDMI_INFOFRAME_HEADER_SIZE - 1,
+ HDMI_AVI_INFOFRAME_SIZE + 1);
+}
+
+static int sii902x_bridge_attach(struct drm_bridge *bridge)
+{
+ struct sii902x *sii902x = bridge_to_sii902x(bridge);
+ struct drm_device *drm = bridge->dev;
+ int ret;
+
+ drm_connector_helper_add(&sii902x->connector,
+ &sii902x_connector_helper_funcs);
+
+ if (!drm_core_check_feature(drm, DRIVER_ATOMIC)) {
+ dev_err(&sii902x->i2c->dev,
+ "sii902x driver is only compatible with DRM device supporting atomic updates");
+ return -ENOTSUPP;
+ }
+
+ ret = drm_connector_init(drm, &sii902x->connector,
+ &sii902x_connector_funcs,
+ DRM_MODE_CONNECTOR_HDMIA);
+ if (ret)
+ return ret;
+
+ if (sii902x->i2c->irq > 0)
+ sii902x->connector.polled = DRM_CONNECTOR_POLL_HPD;
+ else
+ sii902x->connector.polled = DRM_CONNECTOR_POLL_CONNECT;
+
+ drm_mode_connector_attach_encoder(&sii902x->connector, bridge->encoder);
+
+ return 0;
+}
+
+static void sii902x_bridge_nop(struct drm_bridge *bridge)
+{
+}
+
+static const struct drm_bridge_funcs sii902x_bridge_funcs = {
+ .attach = sii902x_bridge_attach,
+ .mode_set = sii902x_bridge_mode_set,
+ .disable = sii902x_bridge_disable,
+ .post_disable = sii902x_bridge_nop,
+ .pre_enable = sii902x_bridge_nop,
+ .enable = sii902x_bridge_enable,
+};
+
+static const struct regmap_range sii902x_volatile_ranges[] = {
+ { .range_min = 0, .range_max = 0xff },
+};
+
+static const struct regmap_access_table sii902x_volatile_table = {
+ .yes_ranges = sii902x_volatile_ranges,
+ .n_yes_ranges = ARRAY_SIZE(sii902x_volatile_ranges),
+};
+
+static const struct regmap_config sii902x_regmap_config = {
+ .reg_bits = 8,
+ .val_bits = 8,
+ .volatile_table = &sii902x_volatile_table,
+ .cache_type = REGCACHE_NONE,
+};
+
+static irqreturn_t sii902x_interrupt(int irq, void *data)
+{
+ struct sii902x *sii902x = data;
+ unsigned int status = 0;
+
+ regmap_read(sii902x->regmap, SI902X_INT_STATUS, &status);
+ regmap_write(sii902x->regmap, SI902X_INT_STATUS, status);
+
+ if ((status & SI902X_HOTPLUG_EVENT) && sii902x->bridge.dev)
+ drm_helper_hpd_irq_event(sii902x->bridge.dev);
+
+ return IRQ_HANDLED;
+}
+
+static int sii902x_probe(struct i2c_client *client,
+ const struct i2c_device_id *id)
+{
+ struct device *dev = &client->dev;
+ unsigned int status = 0;
+ struct sii902x *sii902x;
+ u8 chipid[4];
+ int ret;
+
+ sii902x = devm_kzalloc(dev, sizeof(*sii902x), GFP_KERNEL);
+ if (!sii902x)
+ return -ENOMEM;
+
+ sii902x->i2c = client;
+ sii902x->regmap = devm_regmap_init_i2c(client, &sii902x_regmap_config);
+ if (IS_ERR(sii902x->regmap))
+ return PTR_ERR(sii902x->regmap);
+
+ sii902x->reset_gpio = devm_gpiod_get_optional(dev, "reset",
+ GPIOD_OUT_LOW);
+ if (IS_ERR(sii902x->reset_gpio))
+ dev_warn(dev, "Failed to retrieve/request reset gpio: %ld\n",
+ PTR_ERR(sii902x->reset_gpio));
+
+ sii902x_reset(sii902x);
+
+ ret = regmap_write(sii902x->regmap, SIL902X_REG_TPI_RQB, 0x0);
+ if (ret)
+ return ret;
+
+ ret = regmap_bulk_read(sii902x->regmap, SIL902X_REG_CHIPID(0),
+ &chipid, 4);
+ if (ret) {
+ dev_err(dev, "regmap_read failed %d\n", ret);
+ return ret;
+ }
+
+ if (chipid[0] != 0xb0) {
+ dev_err(dev, "Invalid chipid: %02x (expecting 0xb0)\n",
+ chipid[0]);
+ return -EINVAL;
+ }
+
+ /* Clear all pending interrupts */
+ regmap_read(sii902x->regmap, SI902X_INT_STATUS, &status);
+ regmap_write(sii902x->regmap, SI902X_INT_STATUS, status);
+
+ if (client->irq > 0) {
+ regmap_write(sii902x->regmap, SI902X_INT_ENABLE,
+ SI902X_HOTPLUG_EVENT);
+
+ ret = devm_request_threaded_irq(dev, client->irq, NULL,
+ sii902x_interrupt,
+ IRQF_ONESHOT, dev_name(dev),
+ sii902x);
+ if (ret)
+ return ret;
+ }
+
+ sii902x->bridge.funcs = &sii902x_bridge_funcs;
+ sii902x->bridge.of_node = dev->of_node;
+ ret = drm_bridge_add(&sii902x->bridge);
+ if (ret) {
+ dev_err(dev, "Failed to add drm_bridge\n");
+ return ret;
+ }
+
+ i2c_set_clientdata(client, sii902x);
+
+ return 0;
+}
+
+static int sii902x_remove(struct i2c_client *client)
+
+{
+ struct sii902x *sii902x = i2c_get_clientdata(client);
+
+ drm_bridge_remove(&sii902x->bridge);
+
+ return 0;
+}
+
+#ifdef CONFIG_OF
+static const struct of_device_id sii902x_dt_ids[] = {
+ { .compatible = "sil,sii9022", },
+ { }
+};
+MODULE_DEVICE_TABLE(of, sii902x_dt_ids);
+#endif
+
+static const struct i2c_device_id sii902x_i2c_ids[] = {
+ { "sii9022", 0 },
+ { },
+};
+MODULE_DEVICE_TABLE(i2c, sii902x_i2c_ids);
+
+static struct i2c_driver sii902x_driver = {
+ .probe = sii902x_probe,
+ .remove = sii902x_remove,
+ .driver = {
+ .name = "sii902x",
+ .of_match_table = of_match_ptr(sii902x_dt_ids),
+ },
+ .id_table = sii902x_i2c_ids,
+};
+module_i2c_driver(sii902x_driver);
+
+MODULE_AUTHOR("Boris Brezillon <boris.brezillon@free-electrons.com>");
+MODULE_DESCRIPTION("SIL902x RGB -> HDMI bridges");
+MODULE_LICENSE("GPL");
--
2.7.4
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v4 2/2] drm: bridge: add sii902x DT bindings doc
2016-05-16 14:04 [PATCH v4 1/2] drm: bridge: Add sii902x driver Boris Brezillon
@ 2016-05-16 14:04 ` Boris Brezillon
2016-05-16 16:46 ` Rob Herring
2016-05-17 6:47 ` [PATCH v4 1/2] drm: bridge: Add sii902x driver Daniel Vetter
1 sibling, 1 reply; 8+ messages in thread
From: Boris Brezillon @ 2016-05-16 14:04 UTC (permalink / raw)
To: David Airlie, Daniel Vetter, dri-devel
Cc: Mark Rutland, Meng Yi, Xiubo Li, Pawel Moll, Ian Campbell,
Alison Wang, Nicolas Ferre, devicetree, Rob Herring,
Alexandre Belloni, Kumar Gala, Jean-Christophe Plagniol-Villard,
Jianwei Wang
Add Sii9022 DT bindings description.
Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
---
Changes since v1:
- rename doc file
- s/sil902/sii902/
---
.../devicetree/bindings/display/bridge/sii902x.txt | 35 ++++++++++++++++++++++
1 file changed, 35 insertions(+)
create mode 100644 Documentation/devicetree/bindings/display/bridge/sii902x.txt
diff --git a/Documentation/devicetree/bindings/display/bridge/sii902x.txt b/Documentation/devicetree/bindings/display/bridge/sii902x.txt
new file mode 100644
index 0000000..e8d0ffa
--- /dev/null
+++ b/Documentation/devicetree/bindings/display/bridge/sii902x.txt
@@ -0,0 +1,35 @@
+sii902x HDMI bridge bindings
+
+Required properties:
+ - compatible: "sil,sii9022"
+ - reg: i2c address of the bridge
+ - reset-gpios: OF device-tree gpio specification for RST_N pin.
+
+Optional properties:
+ - interrupts-extended or interrupt-parent + interrupts: describe
+ the interrupt line used to inform the host about hotplug events.
+
+Optional subnodes:
+ - video input: this subnode can contain a video input port node
+ to connect the bridge to a display controller output (See this
+ documentation [1]).
+
+[1]: Documentation/devicetree/bindings/media/video-interfaces.txt
+
+Example:
+ hdmi-bridge@39 {
+ compatible = "sil,sii9022";
+ reg = <0x39>;
+ reset-gpios = <&pioA 1 0>;
+ ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ port@0 {
+ reg = <0>;
+ bridge_in: endpoint {
+ remote-endpoint = <&dc_out>;
+ };
+ };
+ };
+ };
--
2.7.4
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v4 2/2] drm: bridge: add sii902x DT bindings doc
2016-05-16 14:04 ` [PATCH v4 2/2] drm: bridge: add sii902x DT bindings doc Boris Brezillon
@ 2016-05-16 16:46 ` Rob Herring
0 siblings, 0 replies; 8+ messages in thread
From: Rob Herring @ 2016-05-16 16:46 UTC (permalink / raw)
To: Boris Brezillon
Cc: Mark Rutland, Meng Yi, Xiubo Li, Pawel Moll, Ian Campbell,
Alison Wang, Nicolas Ferre, dri-devel, devicetree,
Alexandre Belloni, Kumar Gala, Jean-Christophe Plagniol-Villard,
Jianwei Wang
On Mon, May 16, 2016 at 04:04:29PM +0200, Boris Brezillon wrote:
> Add Sii9022 DT bindings description.
>
> Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
> ---
> Changes since v1:
> - rename doc file
> - s/sil902/sii902/
> ---
> .../devicetree/bindings/display/bridge/sii902x.txt | 35 ++++++++++++++++++++++
> 1 file changed, 35 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/display/bridge/sii902x.txt
Acked-by: Rob Herring <robh@kernel.org>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 1/2] drm: bridge: Add sii902x driver
2016-05-16 14:04 [PATCH v4 1/2] drm: bridge: Add sii902x driver Boris Brezillon
2016-05-16 14:04 ` [PATCH v4 2/2] drm: bridge: add sii902x DT bindings doc Boris Brezillon
@ 2016-05-17 6:47 ` Daniel Vetter
2016-06-01 16:03 ` Boris Brezillon
1 sibling, 1 reply; 8+ messages in thread
From: Daniel Vetter @ 2016-05-17 6:47 UTC (permalink / raw)
To: Boris Brezillon
Cc: Mark Rutland, Meng Yi, Xiubo Li, Pawel Moll, Ian Campbell,
Alison Wang, Nicolas Ferre, dri-devel, devicetree, Rob Herring,
Alexandre Belloni, Kumar Gala, Jean-Christophe Plagniol-Villard,
Jianwei Wang
On Mon, May 16, 2016 at 04:04:28PM +0200, Boris Brezillon wrote:
> Add basic support for the sii902x RGB -> HDMI bridge.
> This driver does not support audio output yet.
>
> Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
> Tested-by: Nicolas Ferre <nicolas.ferre@atmel.com>
> ---
> Hello,
>
> This patch is only adding basic support for the sii9022 chip.
> As stated in the commit log, there's no audio support, but the
> driver also hardcodes a lot of things (like the RGB input format to
> use).
>
> Best Regards,
>
> Boris
>
> Changes in v4:
> - make reset GPIO optional
> - only support attaching to DRM devices supporting atomic updates
>
> Changes in v3:
> - fix get_modes() implementation to avoid turning the screen in power
> save mode
> - rename the driver (sil902x -> sii902x)
>
> Changes in v2:
> - fix errors reported by the kbuild robot
> ---
> drivers/gpu/drm/bridge/Kconfig | 8 +
> drivers/gpu/drm/bridge/Makefile | 1 +
> drivers/gpu/drm/bridge/sii902x.c | 477 +++++++++++++++++++++++++++++++++++++++
> 3 files changed, 486 insertions(+)
> create mode 100644 drivers/gpu/drm/bridge/sii902x.c
>
> diff --git a/drivers/gpu/drm/bridge/Kconfig b/drivers/gpu/drm/bridge/Kconfig
> index efd94e0..a692c5b 100644
> --- a/drivers/gpu/drm/bridge/Kconfig
> +++ b/drivers/gpu/drm/bridge/Kconfig
> @@ -40,6 +40,14 @@ config DRM_PARADE_PS8622
> ---help---
> Parade eDP-LVDS bridge chip driver.
>
> +config DRM_SII902X
> + tristate "Silicon Image sii902x RGB/HDMI bridge"
> + depends on OF
> + select DRM_KMS_HELPER
> + select REGMAP_I2C
> + ---help---
> + Silicon Image sii902x bridge chip driver.
> +
> source "drivers/gpu/drm/bridge/analogix/Kconfig"
>
> endmenu
> diff --git a/drivers/gpu/drm/bridge/Makefile b/drivers/gpu/drm/bridge/Makefile
> index ff821f4..b2a37e8 100644
> --- a/drivers/gpu/drm/bridge/Makefile
> +++ b/drivers/gpu/drm/bridge/Makefile
> @@ -4,4 +4,5 @@ obj-$(CONFIG_DRM_DW_HDMI) += dw-hdmi.o
> obj-$(CONFIG_DRM_DW_HDMI_AHB_AUDIO) += dw-hdmi-ahb-audio.o
> obj-$(CONFIG_DRM_NXP_PTN3460) += nxp-ptn3460.o
> obj-$(CONFIG_DRM_PARADE_PS8622) += parade-ps8622.o
> +obj-$(CONFIG_DRM_SII902X) += sii902x.o
> obj-$(CONFIG_DRM_ANALOGIX_DP) += analogix/
> diff --git a/drivers/gpu/drm/bridge/sii902x.c b/drivers/gpu/drm/bridge/sii902x.c
> new file mode 100644
> index 0000000..76ad73a
> --- /dev/null
> +++ b/drivers/gpu/drm/bridge/sii902x.c
> @@ -0,0 +1,477 @@
> +/*
> + * Copyright (C) 2014 Atmel
> + * Bo Shen <voice.shen@atmel.com>
> + *
> + * Authors: Bo Shen <voice.shen@atmel.com>
> + * Boris Brezillon <boris.brezillon@free-electrons.com>
> + * Wu, Songjun <Songjun.Wu@atmel.com>
> + *
> + *
> + * Copyright (C) 2010-2011 Freescale Semiconductor, Inc. All Rights Reserved.
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + *
> + * 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/component.h>
> +#include <linux/gpio/consumer.h>
> +#include <linux/i2c.h>
> +#include <linux/module.h>
> +#include <linux/of.h>
> +#include <linux/of_gpio.h>
> +#include <linux/regmap.h>
> +
> +#include <drm/drmP.h>
> +#include <drm/drm_atomic.h>
> +#include <drm/drm_atomic_helper.h>
> +#include <drm/drm_crtc_helper.h>
> +#include <drm/drm_edid.h>
> +#include <drm/drm_encoder_slave.h>
> +
> +#define SIL902X_TPI_VIDEO_DATA 0x0
> +
> +#define SIL902X_TPI_PIXEL_REPETITION 0x8
> +#define SIL902X_TPI_AVI_PIXEL_REP_BUS_24BIT BIT(5)
> +#define SIL902X_TPI_AVI_PIXEL_REP_RISING_EDGE BIT(4)
> +#define SIL902X_TPI_AVI_PIXEL_REP_4X 3
> +#define SIL902X_TPI_AVI_PIXEL_REP_2X 1
> +#define SIL902X_TPI_AVI_PIXEL_REP_NONE 0
> +#define SIL902X_TPI_CLK_RATIO_HALF (0 << 6)
> +#define SIL902X_TPI_CLK_RATIO_1X (1 << 6)
> +#define SIL902X_TPI_CLK_RATIO_2X (2 << 6)
> +#define SIL902X_TPI_CLK_RATIO_4X (3 << 6)
> +
> +#define SIL902X_TPI_AVI_IN_FORMAT 0x9
> +#define SIL902X_TPI_AVI_INPUT_BITMODE_12BIT BIT(7)
> +#define SIL902X_TPI_AVI_INPUT_DITHER BIT(6)
> +#define SIL902X_TPI_AVI_INPUT_RANGE_LIMITED (2 << 2)
> +#define SIL902X_TPI_AVI_INPUT_RANGE_FULL (1 << 2)
> +#define SIL902X_TPI_AVI_INPUT_RANGE_AUTO (0 << 2)
> +#define SIL902X_TPI_AVI_INPUT_COLORSPACE_BLACK (3 << 0)
> +#define SIL902X_TPI_AVI_INPUT_COLORSPACE_YUV422 (2 << 0)
> +#define SIL902X_TPI_AVI_INPUT_COLORSPACE_YUV444 (1 << 0)
> +#define SIL902X_TPI_AVI_INPUT_COLORSPACE_RGB (0 << 0)
> +
> +#define SIL902X_TPI_AVI_INFOFRAME 0x0c
> +
> +#define SIL902X_SYS_CTRL_DATA 0x1a
> +#define SIL902X_SYS_CTRL_PWR_DWN BIT(4)
> +#define SIL902X_SYS_CTRL_AV_MUTE BIT(3)
> +#define SIL902X_SYS_CTRL_DDC_BUS_REQ BIT(2)
> +#define SIL902X_SYS_CTRL_DDC_BUS_GRTD BIT(1)
> +#define SIL902X_SYS_CTRL_OUTPUT_MODE BIT(0)
> +#define SIL902X_SYS_CTRL_OUTPUT_HDMI 1
> +#define SIL902X_SYS_CTRL_OUTPUT_DVI 0
> +
> +#define SIL902X_REG_CHIPID(n) (0x1b + (n))
> +
> +#define SIL902X_PWR_STATE_CTRL 0x1e
> +#define SIL902X_AVI_POWER_STATE_MSK GENMASK(1, 0)
> +#define SIL902X_AVI_POWER_STATE_D(l) ((l) & SIL902X_AVI_POWER_STATE_MSK)
> +
> +#define SI902X_INT_ENABLE 0x3c
> +#define SI902X_INT_STATUS 0x3d
> +#define SI902X_HOTPLUG_EVENT BIT(0)
> +#define SI902X_PLUGGED_STATUS BIT(2)
> +
> +#define SIL902X_REG_TPI_RQB 0xc7
> +
> +struct sii902x {
> + struct i2c_client *i2c;
> + struct regmap *regmap;
> + struct drm_bridge bridge;
> + struct drm_connector connector;
> + struct gpio_desc *reset_gpio;
> + struct work_struct hotplug_work;
> +};
> +
> +static inline struct sii902x *bridge_to_sii902x(struct drm_bridge *bridge)
> +{
> + return container_of(bridge, struct sii902x, bridge);
> +}
> +
> +static inline struct sii902x *connector_to_sii902x(struct drm_connector *con)
> +{
> + return container_of(con, struct sii902x, connector);
> +}
> +
> +static void sii902x_reset(struct sii902x *sii902x)
> +{
> + if (!sii902x->reset_gpio)
> + return;
> +
> + gpiod_set_value(sii902x->reset_gpio, 1);
> +
> + msleep(100);
> +
> + gpiod_set_value(sii902x->reset_gpio, 0);
> +}
> +
> +static void sii902x_connector_destroy(struct drm_connector *connector)
> +{
> + drm_connector_unregister(connector);
> + drm_connector_cleanup(connector);
> +}
> +
> +static enum drm_connector_status
> +sii902x_connector_detect(struct drm_connector *connector, bool force)
> +{
> + struct sii902x *sii902x = connector_to_sii902x(connector);
> + unsigned int status;
> +
> + regmap_read(sii902x->regmap, SI902X_INT_STATUS, &status);
> +
> + return (status & SI902X_PLUGGED_STATUS) ?
> + connector_status_connected : connector_status_disconnected;
> +}
> +
> +static const struct drm_connector_funcs sii902x_connector_funcs = {
> + .dpms = drm_atomic_helper_connector_dpms,
> + .detect = sii902x_connector_detect,
> + .fill_modes = drm_helper_probe_single_connector_modes,
> + .destroy = sii902x_connector_destroy,
> + .reset = drm_atomic_helper_connector_reset,
> + .atomic_duplicate_state = drm_atomic_helper_connector_duplicate_state,
> + .atomic_destroy_state = drm_atomic_helper_connector_destroy_state,
> +};
> +
> +static int sii902x_get_modes(struct drm_connector *connector)
> +{
> + struct sii902x *sii902x = connector_to_sii902x(connector);
> + struct regmap *regmap = sii902x->regmap;
> + u32 bus_format = MEDIA_BUS_FMT_RGB888_1X24;
> + unsigned int status;
> + struct edid *edid;
> + int num = 0;
> + int ret;
> + int i;
> +
> + ret = regmap_update_bits(regmap, SIL902X_SYS_CTRL_DATA,
> + SIL902X_SYS_CTRL_DDC_BUS_REQ,
> + SIL902X_SYS_CTRL_DDC_BUS_REQ);
> + if (ret)
> + return ret;
> +
> + i = 0;
> + do {
> + ret = regmap_read(regmap, SIL902X_SYS_CTRL_DATA, &status);
> + if (ret)
> + return ret;
> + i++;
> + } while (!(status & SIL902X_SYS_CTRL_DDC_BUS_GRTD));
> +
> + ret = regmap_write(regmap, SIL902X_SYS_CTRL_DATA, status);
> + if (ret)
> + return ret;
> +
> + edid = drm_get_edid(connector, sii902x->i2c->adapter);
> + drm_mode_connector_update_edid_property(connector, edid);
> + if (edid) {
> + num += drm_add_edid_modes(connector, edid);
> + kfree(edid);
> + }
> +
> + ret = drm_display_info_set_bus_formats(&connector->display_info,
> + &bus_format, 1);
> + if (ret)
> + return ret;
> +
> + regmap_read(regmap, SIL902X_SYS_CTRL_DATA, &status);
> + if (ret)
> + return ret;
> +
> + ret = regmap_update_bits(regmap, SIL902X_SYS_CTRL_DATA,
> + SIL902X_SYS_CTRL_DDC_BUS_REQ |
> + SIL902X_SYS_CTRL_DDC_BUS_GRTD, 0);
> + if (ret)
> + return ret;
> +
> + i = 0;
> + do {
> + ret = regmap_read(regmap, SIL902X_SYS_CTRL_DATA, &status);
> + if (ret)
> + return ret;
> + i++;
> + } while (status & (SIL902X_SYS_CTRL_DDC_BUS_REQ |
> + SIL902X_SYS_CTRL_DDC_BUS_GRTD));
> +
> + return num;
> +}
> +
> +static enum drm_mode_status sii902x_mode_valid(struct drm_connector *connector,
> + struct drm_display_mode *mode)
> +{
> + /* TODO: check mode */
> +
> + return MODE_OK;
> +}
> +
> +static struct drm_encoder *sii902x_best_encoder(struct drm_connector *connector)
> +{
> + struct sii902x *sii902x = connector_to_sii902x(connector);
> +
> + return sii902x->bridge.encoder;
> +}
drm_atomic_helper_best_encoder should do exactly this for you. If you feel
board pimp the atomic helpers to call that one by default to even remove
the vfunc assingment line ;-)
-Daniel
> +
> +static const struct drm_connector_helper_funcs sii902x_connector_helper_funcs = {
> + .get_modes = sii902x_get_modes,
> + .mode_valid = sii902x_mode_valid,
> + .best_encoder = sii902x_best_encoder,
> +};
> +
> +static void sii902x_bridge_disable(struct drm_bridge *bridge)
> +{
> + struct sii902x *sii902x = bridge_to_sii902x(bridge);
> +
> + regmap_update_bits(sii902x->regmap, SIL902X_SYS_CTRL_DATA,
> + SIL902X_SYS_CTRL_PWR_DWN,
> + SIL902X_SYS_CTRL_PWR_DWN);
> +}
> +
> +static void sii902x_bridge_enable(struct drm_bridge *bridge)
> +{
> + struct sii902x *sii902x = bridge_to_sii902x(bridge);
> +
> + regmap_update_bits(sii902x->regmap, SIL902X_PWR_STATE_CTRL,
> + SIL902X_AVI_POWER_STATE_MSK,
> + SIL902X_AVI_POWER_STATE_D(0));
> + regmap_update_bits(sii902x->regmap, SIL902X_SYS_CTRL_DATA,
> + SIL902X_SYS_CTRL_PWR_DWN, 0);
> +}
> +
> +static void sii902x_bridge_mode_set(struct drm_bridge *bridge,
> + struct drm_display_mode *mode,
> + struct drm_display_mode *adj)
> +{
> + u8 buf[HDMI_INFOFRAME_HEADER_SIZE + HDMI_AVI_INFOFRAME_SIZE];
> + struct sii902x *sii902x = bridge_to_sii902x(bridge);
> + struct regmap *regmap = sii902x->regmap;
> + struct hdmi_avi_infoframe frame;
> + int ret;
> +
> + buf[0] = adj->clock;
> + buf[1] = adj->clock >> 8;
> + buf[2] = adj->vrefresh;
> + buf[3] = 0x00;
> + buf[4] = adj->hdisplay;
> + buf[5] = adj->hdisplay >> 8;
> + buf[6] = adj->vdisplay;
> + buf[7] = adj->vdisplay >> 8;
> + buf[8] = SIL902X_TPI_CLK_RATIO_1X | SIL902X_TPI_AVI_PIXEL_REP_NONE |
> + SIL902X_TPI_AVI_PIXEL_REP_BUS_24BIT;
> + buf[9] = SIL902X_TPI_AVI_INPUT_RANGE_AUTO |
> + SIL902X_TPI_AVI_INPUT_COLORSPACE_RGB;
> +
> + ret = regmap_bulk_write(regmap, SIL902X_TPI_VIDEO_DATA, buf, 10);
> + if (ret)
> + return;
> +
> + ret = drm_hdmi_avi_infoframe_from_display_mode(&frame, adj);
> + if (ret < 0) {
> + DRM_ERROR("couldn't fill AVI infoframe\n");
> + return;
> + }
> +
> + ret = hdmi_avi_infoframe_pack(&frame, buf, sizeof(buf));
> + if (ret < 0) {
> + DRM_ERROR("failed to pack AVI infoframe: %d\n", ret);
> + return;
> + }
> +
> + /* Do not send the infoframe header, but keep the CRC field. */
> + regmap_bulk_write(regmap, SIL902X_TPI_AVI_INFOFRAME,
> + buf + HDMI_INFOFRAME_HEADER_SIZE - 1,
> + HDMI_AVI_INFOFRAME_SIZE + 1);
> +}
> +
> +static int sii902x_bridge_attach(struct drm_bridge *bridge)
> +{
> + struct sii902x *sii902x = bridge_to_sii902x(bridge);
> + struct drm_device *drm = bridge->dev;
> + int ret;
> +
> + drm_connector_helper_add(&sii902x->connector,
> + &sii902x_connector_helper_funcs);
> +
> + if (!drm_core_check_feature(drm, DRIVER_ATOMIC)) {
> + dev_err(&sii902x->i2c->dev,
> + "sii902x driver is only compatible with DRM device supporting atomic updates");
> + return -ENOTSUPP;
> + }
> +
> + ret = drm_connector_init(drm, &sii902x->connector,
> + &sii902x_connector_funcs,
> + DRM_MODE_CONNECTOR_HDMIA);
> + if (ret)
> + return ret;
> +
> + if (sii902x->i2c->irq > 0)
> + sii902x->connector.polled = DRM_CONNECTOR_POLL_HPD;
> + else
> + sii902x->connector.polled = DRM_CONNECTOR_POLL_CONNECT;
> +
> + drm_mode_connector_attach_encoder(&sii902x->connector, bridge->encoder);
> +
> + return 0;
> +}
> +
> +static void sii902x_bridge_nop(struct drm_bridge *bridge)
> +{
> +}
> +
> +static const struct drm_bridge_funcs sii902x_bridge_funcs = {
> + .attach = sii902x_bridge_attach,
> + .mode_set = sii902x_bridge_mode_set,
> + .disable = sii902x_bridge_disable,
> + .post_disable = sii902x_bridge_nop,
> + .pre_enable = sii902x_bridge_nop,
> + .enable = sii902x_bridge_enable,
> +};
> +
> +static const struct regmap_range sii902x_volatile_ranges[] = {
> + { .range_min = 0, .range_max = 0xff },
> +};
> +
> +static const struct regmap_access_table sii902x_volatile_table = {
> + .yes_ranges = sii902x_volatile_ranges,
> + .n_yes_ranges = ARRAY_SIZE(sii902x_volatile_ranges),
> +};
> +
> +static const struct regmap_config sii902x_regmap_config = {
> + .reg_bits = 8,
> + .val_bits = 8,
> + .volatile_table = &sii902x_volatile_table,
> + .cache_type = REGCACHE_NONE,
> +};
> +
> +static irqreturn_t sii902x_interrupt(int irq, void *data)
> +{
> + struct sii902x *sii902x = data;
> + unsigned int status = 0;
> +
> + regmap_read(sii902x->regmap, SI902X_INT_STATUS, &status);
> + regmap_write(sii902x->regmap, SI902X_INT_STATUS, status);
> +
> + if ((status & SI902X_HOTPLUG_EVENT) && sii902x->bridge.dev)
> + drm_helper_hpd_irq_event(sii902x->bridge.dev);
> +
> + return IRQ_HANDLED;
> +}
> +
> +static int sii902x_probe(struct i2c_client *client,
> + const struct i2c_device_id *id)
> +{
> + struct device *dev = &client->dev;
> + unsigned int status = 0;
> + struct sii902x *sii902x;
> + u8 chipid[4];
> + int ret;
> +
> + sii902x = devm_kzalloc(dev, sizeof(*sii902x), GFP_KERNEL);
> + if (!sii902x)
> + return -ENOMEM;
> +
> + sii902x->i2c = client;
> + sii902x->regmap = devm_regmap_init_i2c(client, &sii902x_regmap_config);
> + if (IS_ERR(sii902x->regmap))
> + return PTR_ERR(sii902x->regmap);
> +
> + sii902x->reset_gpio = devm_gpiod_get_optional(dev, "reset",
> + GPIOD_OUT_LOW);
> + if (IS_ERR(sii902x->reset_gpio))
> + dev_warn(dev, "Failed to retrieve/request reset gpio: %ld\n",
> + PTR_ERR(sii902x->reset_gpio));
> +
> + sii902x_reset(sii902x);
> +
> + ret = regmap_write(sii902x->regmap, SIL902X_REG_TPI_RQB, 0x0);
> + if (ret)
> + return ret;
> +
> + ret = regmap_bulk_read(sii902x->regmap, SIL902X_REG_CHIPID(0),
> + &chipid, 4);
> + if (ret) {
> + dev_err(dev, "regmap_read failed %d\n", ret);
> + return ret;
> + }
> +
> + if (chipid[0] != 0xb0) {
> + dev_err(dev, "Invalid chipid: %02x (expecting 0xb0)\n",
> + chipid[0]);
> + return -EINVAL;
> + }
> +
> + /* Clear all pending interrupts */
> + regmap_read(sii902x->regmap, SI902X_INT_STATUS, &status);
> + regmap_write(sii902x->regmap, SI902X_INT_STATUS, status);
> +
> + if (client->irq > 0) {
> + regmap_write(sii902x->regmap, SI902X_INT_ENABLE,
> + SI902X_HOTPLUG_EVENT);
> +
> + ret = devm_request_threaded_irq(dev, client->irq, NULL,
> + sii902x_interrupt,
> + IRQF_ONESHOT, dev_name(dev),
> + sii902x);
> + if (ret)
> + return ret;
> + }
> +
> + sii902x->bridge.funcs = &sii902x_bridge_funcs;
> + sii902x->bridge.of_node = dev->of_node;
> + ret = drm_bridge_add(&sii902x->bridge);
> + if (ret) {
> + dev_err(dev, "Failed to add drm_bridge\n");
> + return ret;
> + }
> +
> + i2c_set_clientdata(client, sii902x);
> +
> + return 0;
> +}
> +
> +static int sii902x_remove(struct i2c_client *client)
> +
> +{
> + struct sii902x *sii902x = i2c_get_clientdata(client);
> +
> + drm_bridge_remove(&sii902x->bridge);
> +
> + return 0;
> +}
> +
> +#ifdef CONFIG_OF
> +static const struct of_device_id sii902x_dt_ids[] = {
> + { .compatible = "sil,sii9022", },
> + { }
> +};
> +MODULE_DEVICE_TABLE(of, sii902x_dt_ids);
> +#endif
> +
> +static const struct i2c_device_id sii902x_i2c_ids[] = {
> + { "sii9022", 0 },
> + { },
> +};
> +MODULE_DEVICE_TABLE(i2c, sii902x_i2c_ids);
> +
> +static struct i2c_driver sii902x_driver = {
> + .probe = sii902x_probe,
> + .remove = sii902x_remove,
> + .driver = {
> + .name = "sii902x",
> + .of_match_table = of_match_ptr(sii902x_dt_ids),
> + },
> + .id_table = sii902x_i2c_ids,
> +};
> +module_i2c_driver(sii902x_driver);
> +
> +MODULE_AUTHOR("Boris Brezillon <boris.brezillon@free-electrons.com>");
> +MODULE_DESCRIPTION("SIL902x RGB -> HDMI bridges");
> +MODULE_LICENSE("GPL");
> --
> 2.7.4
>
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 1/2] drm: bridge: Add sii902x driver
2016-05-17 6:47 ` [PATCH v4 1/2] drm: bridge: Add sii902x driver Daniel Vetter
@ 2016-06-01 16:03 ` Boris Brezillon
2016-06-01 21:56 ` Daniel Vetter
0 siblings, 1 reply; 8+ messages in thread
From: Boris Brezillon @ 2016-06-01 16:03 UTC (permalink / raw)
To: Daniel Vetter
Cc: Mark Rutland, Meng Yi, Xiubo Li, Pawel Moll, Ian Campbell,
Alison Wang, Nicolas Ferre, dri-devel, devicetree, Rob Herring,
Alexandre Belloni, Kumar Gala, Jean-Christophe Plagniol-Villard,
Jianwei Wang
On Tue, 17 May 2016 08:47:11 +0200
Daniel Vetter <daniel@ffwll.ch> wrote:
> > +static struct drm_encoder *sii902x_best_encoder(struct drm_connector *connector)
> > +{
> > + struct sii902x *sii902x = connector_to_sii902x(connector);
> > +
> > + return sii902x->bridge.encoder;
> > +}
>
> drm_atomic_helper_best_encoder should do exactly this for you. If you feel
> board pimp the atomic helpers to call that one by default to even remove
> the vfunc assingment line ;-)
Just to be sure, is that what you had in mind?
--->8---
From d218b7ea16ca35452b3174c83a207ecd406e5c88 Mon Sep 17 00:00:00 2001
From: Boris Brezillon <boris.brezillon@free-electrons.com>
Date: Wed, 1 Jun 2016 17:57:18 +0200
Subject: [PATCH] drm: atomic: Handle funcs->best_encoder == NULL case
Fallback drm_atomic_helper_best_encoder() is funcs->best_encoder() is NULL
so that DRM drivers can leave this hook unassigned if they know they want
to use drm_atomic_helper_best_encoder().
Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
---
drivers/gpu/drm/drm_atomic_helper.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
index ddfa0d1..f6a3350 100644
--- a/drivers/gpu/drm/drm_atomic_helper.c
+++ b/drivers/gpu/drm/drm_atomic_helper.c
@@ -110,8 +110,10 @@ static int handle_conflicting_encoders(struct drm_atomic_state *state,
if (funcs->atomic_best_encoder)
new_encoder = funcs->atomic_best_encoder(connector, conn_state);
- else
+ else if (funcs->best_encoder)
new_encoder = funcs->best_encoder(connector);
+ else
+ new_encoder = drm_atomic_helper_best_encoder(connector);
if (new_encoder) {
if (encoder_mask & (1 << drm_encoder_index(new_encoder))) {
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v4 1/2] drm: bridge: Add sii902x driver
2016-06-01 16:03 ` Boris Brezillon
@ 2016-06-01 21:56 ` Daniel Vetter
2016-06-02 7:40 ` Boris Brezillon
2016-06-02 8:04 ` Boris Brezillon
0 siblings, 2 replies; 8+ messages in thread
From: Daniel Vetter @ 2016-06-01 21:56 UTC (permalink / raw)
To: Boris Brezillon
Cc: Mark Rutland, Meng Yi, Xiubo Li, Pawel Moll, Ian Campbell,
Alison Wang, Nicolas Ferre, dri-devel, devicetree, Rob Herring,
Alexandre Belloni, Kumar Gala, Jean-Christophe Plagniol-Villard,
Jianwei Wang
On Wed, Jun 01, 2016 at 06:03:37PM +0200, Boris Brezillon wrote:
> On Tue, 17 May 2016 08:47:11 +0200
> Daniel Vetter <daniel@ffwll.ch> wrote:
>
> > > +static struct drm_encoder *sii902x_best_encoder(struct drm_connector *connector)
> > > +{
> > > + struct sii902x *sii902x = connector_to_sii902x(connector);
> > > +
> > > + return sii902x->bridge.encoder;
> > > +}
> >
> > drm_atomic_helper_best_encoder should do exactly this for you. If you feel
> > board pimp the atomic helpers to call that one by default to even remove
> > the vfunc assingment line ;-)
>
> Just to be sure, is that what you had in mind?
Exactly, I'll pick this one up for drm-misc. btw if you're even more
motivated, you can go through all drivers with a static mapping from
connector to encoder, and nuke those functions. Since if it's just a
static mapping, the only way to do it is like the helper does.
Thanks, Daniel
>
> --->8---
> From d218b7ea16ca35452b3174c83a207ecd406e5c88 Mon Sep 17 00:00:00 2001
> From: Boris Brezillon <boris.brezillon@free-electrons.com>
> Date: Wed, 1 Jun 2016 17:57:18 +0200
> Subject: [PATCH] drm: atomic: Handle funcs->best_encoder == NULL case
>
> Fallback drm_atomic_helper_best_encoder() is funcs->best_encoder() is NULL
> so that DRM drivers can leave this hook unassigned if they know they want
> to use drm_atomic_helper_best_encoder().
>
> Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
> ---
> drivers/gpu/drm/drm_atomic_helper.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
> index ddfa0d1..f6a3350 100644
> --- a/drivers/gpu/drm/drm_atomic_helper.c
> +++ b/drivers/gpu/drm/drm_atomic_helper.c
> @@ -110,8 +110,10 @@ static int handle_conflicting_encoders(struct drm_atomic_state *state,
>
> if (funcs->atomic_best_encoder)
> new_encoder = funcs->atomic_best_encoder(connector, conn_state);
> - else
> + else if (funcs->best_encoder)
> new_encoder = funcs->best_encoder(connector);
> + else
> + new_encoder = drm_atomic_helper_best_encoder(connector);
>
> if (new_encoder) {
> if (encoder_mask & (1 << drm_encoder_index(new_encoder))) {
>
>
>
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 1/2] drm: bridge: Add sii902x driver
2016-06-01 21:56 ` Daniel Vetter
@ 2016-06-02 7:40 ` Boris Brezillon
2016-06-02 8:04 ` Boris Brezillon
1 sibling, 0 replies; 8+ messages in thread
From: Boris Brezillon @ 2016-06-02 7:40 UTC (permalink / raw)
To: Daniel Vetter
Cc: Mark Rutland, Meng Yi, Xiubo Li, Pawel Moll, Ian Campbell,
Alison Wang, Nicolas Ferre, dri-devel, devicetree, Rob Herring,
Alexandre Belloni, Kumar Gala, Jean-Christophe Plagniol-Villard,
Jianwei Wang
Hi Daniel,
On Wed, 1 Jun 2016 23:56:02 +0200
Daniel Vetter <daniel@ffwll.ch> wrote:
> On Wed, Jun 01, 2016 at 06:03:37PM +0200, Boris Brezillon wrote:
> > On Tue, 17 May 2016 08:47:11 +0200
> > Daniel Vetter <daniel@ffwll.ch> wrote:
> >
> > > > +static struct drm_encoder *sii902x_best_encoder(struct drm_connector *connector)
> > > > +{
> > > > + struct sii902x *sii902x = connector_to_sii902x(connector);
> > > > +
> > > > + return sii902x->bridge.encoder;
> > > > +}
> > >
> > > drm_atomic_helper_best_encoder should do exactly this for you. If you feel
> > > board pimp the atomic helpers to call that one by default to even remove
> > > the vfunc assingment line ;-)
> >
> > Just to be sure, is that what you had in mind?
>
> Exactly, I'll pick this one up for drm-misc. btw if you're even more
> motivated, you can go through all drivers with a static mapping from
> connector to encoder, and nuke those functions. Since if it's just a
> static mapping, the only way to do it is like the helper does.
>
> Thanks, Daniel
> >
> > --->8---
> > From d218b7ea16ca35452b3174c83a207ecd406e5c88 Mon Sep 17 00:00:00 2001
> > From: Boris Brezillon <boris.brezillon@free-electrons.com>
> > Date: Wed, 1 Jun 2016 17:57:18 +0200
> > Subject: [PATCH] drm: atomic: Handle funcs->best_encoder == NULL case
> >
> > Fallback drm_atomic_helper_best_encoder() is funcs->best_encoder() is NULL
Oops, can you fix the commit message before applying?
"
Fallback *to* drm_atomic_helper_best_encoder() *if*
funcs->best_encoder() is NULL
"
Maybe I should resend the patch along with the other modifications you
asked for. This way it will in a dedicated thread, instead of being
hidden in an unrelated patch series.
Regards,
Boris
> > so that DRM drivers can leave this hook unassigned if they know they want
> > to use drm_atomic_helper_best_encoder().
> >
> > Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
> > ---
> > drivers/gpu/drm/drm_atomic_helper.c | 4 +++-
> > 1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
> > index ddfa0d1..f6a3350 100644
> > --- a/drivers/gpu/drm/drm_atomic_helper.c
> > +++ b/drivers/gpu/drm/drm_atomic_helper.c
> > @@ -110,8 +110,10 @@ static int handle_conflicting_encoders(struct drm_atomic_state *state,
> >
> > if (funcs->atomic_best_encoder)
> > new_encoder = funcs->atomic_best_encoder(connector, conn_state);
> > - else
> > + else if (funcs->best_encoder)
> > new_encoder = funcs->best_encoder(connector);
> > + else
> > + new_encoder = drm_atomic_helper_best_encoder(connector);
> >
> > if (new_encoder) {
> > if (encoder_mask & (1 << drm_encoder_index(new_encoder))) {
> >
> >
> >
>
--
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v4 1/2] drm: bridge: Add sii902x driver
2016-06-01 21:56 ` Daniel Vetter
2016-06-02 7:40 ` Boris Brezillon
@ 2016-06-02 8:04 ` Boris Brezillon
1 sibling, 0 replies; 8+ messages in thread
From: Boris Brezillon @ 2016-06-02 8:04 UTC (permalink / raw)
To: Daniel Vetter
Cc: Mark Rutland, Meng Yi, Xiubo Li, Pawel Moll, Ian Campbell,
Alison Wang, Nicolas Ferre, dri-devel, devicetree, Rob Herring,
Alexandre Belloni, Kumar Gala, Jean-Christophe Plagniol-Villard,
Jianwei Wang
On Wed, 1 Jun 2016 23:56:02 +0200
Daniel Vetter <daniel@ffwll.ch> wrote:
> On Wed, Jun 01, 2016 at 06:03:37PM +0200, Boris Brezillon wrote:
> > On Tue, 17 May 2016 08:47:11 +0200
> > Daniel Vetter <daniel@ffwll.ch> wrote:
> >
> > > > +static struct drm_encoder *sii902x_best_encoder(struct drm_connector *connector)
> > > > +{
> > > > + struct sii902x *sii902x = connector_to_sii902x(connector);
> > > > +
> > > > + return sii902x->bridge.encoder;
> > > > +}
> > >
> > > drm_atomic_helper_best_encoder should do exactly this for you. If you feel
> > > board pimp the atomic helpers to call that one by default to even remove
> > > the vfunc assingment line ;-)
> >
> > Just to be sure, is that what you had in mind?
>
> Exactly, I'll pick this one up for drm-misc. btw if you're even more
> motivated, you can go through all drivers with a static mapping from
> connector to encoder, and nuke those functions. Since if it's just a
> static mapping, the only way to do it is like the helper does.
>
> Thanks, Daniel
> >
> > --->8---
> > From d218b7ea16ca35452b3174c83a207ecd406e5c88 Mon Sep 17 00:00:00 2001
> > From: Boris Brezillon <boris.brezillon@free-electrons.com>
> > Date: Wed, 1 Jun 2016 17:57:18 +0200
> > Subject: [PATCH] drm: atomic: Handle funcs->best_encoder == NULL case
> >
> > Fallback drm_atomic_helper_best_encoder() is funcs->best_encoder() is NULL
> > so that DRM drivers can leave this hook unassigned if they know they want
> > to use drm_atomic_helper_best_encoder().
Please don't apply the patch, it's buggy.
> >
> > Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
> > ---
> > drivers/gpu/drm/drm_atomic_helper.c | 4 +++-
> > 1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c
> > index ddfa0d1..f6a3350 100644
> > --- a/drivers/gpu/drm/drm_atomic_helper.c
> > +++ b/drivers/gpu/drm/drm_atomic_helper.c
> > @@ -110,8 +110,10 @@ static int handle_conflicting_encoders(struct drm_atomic_state *state,
> >
> > if (funcs->atomic_best_encoder)
> > new_encoder = funcs->atomic_best_encoder(connector, conn_state);
> > - else
> > + else if (funcs->best_encoder)
> > new_encoder = funcs->best_encoder(connector);
> > + else
> > + new_encoder = drm_atomic_helper_best_encoder(connector);
> >
> > if (new_encoder) {
> > if (encoder_mask & (1 << drm_encoder_index(new_encoder))) {
> >
> >
> >
>
--
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2016-06-02 8:04 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-16 14:04 [PATCH v4 1/2] drm: bridge: Add sii902x driver Boris Brezillon
2016-05-16 14:04 ` [PATCH v4 2/2] drm: bridge: add sii902x DT bindings doc Boris Brezillon
2016-05-16 16:46 ` Rob Herring
2016-05-17 6:47 ` [PATCH v4 1/2] drm: bridge: Add sii902x driver Daniel Vetter
2016-06-01 16:03 ` Boris Brezillon
2016-06-01 21:56 ` Daniel Vetter
2016-06-02 7:40 ` Boris Brezillon
2016-06-02 8:04 ` Boris Brezillon
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.