ping. On Wed, Jun 11, 2014 at 11:57 PM, Ajay Kumar <ajaykumar.rs@xxxxxxxxxxx> wrote: > Add a dummy bridge which binds all of the drm_bridge callbacks > to corresponding drm_panel callbacks. > > In theory, this is just a glue layer for the last bridge and > the panel attached to it. > > This driver also implements the required drm_connector ops > for the encoder(on which the entire bridge chain is hanging off). > > Signed-off-by: Ajay Kumar <ajaykumar.rs@xxxxxxxxxxx> > --- > drivers/gpu/drm/bridge/Kconfig | 7 ++ > drivers/gpu/drm/bridge/Makefile | 1 + > drivers/gpu/drm/bridge/panel_binder.c | 193 +++++++++++++++++++++++++++++++++ > include/drm/bridge/panel_binder.h | 44 ++++++++ > 4 files changed, 245 insertions(+) > create mode 100644 drivers/gpu/drm/bridge/panel_binder.c > create mode 100644 include/drm/bridge/panel_binder.h > > diff --git a/drivers/gpu/drm/bridge/Kconfig b/drivers/gpu/drm/bridge/Kconfig > index 884923f..e3fb487 100644 > --- a/drivers/gpu/drm/bridge/Kconfig > +++ b/drivers/gpu/drm/bridge/Kconfig > @@ -3,3 +3,10 @@ config DRM_PTN3460 > depends on DRM > select DRM_KMS_HELPER > ---help--- > + > +config DRM_PANEL_BINDER > + tristate "bridge panel binder" > + 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..ba8b5b8 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_PANEL_BINDER) += panel_binder.o > diff --git a/drivers/gpu/drm/bridge/panel_binder.c b/drivers/gpu/drm/bridge/panel_binder.c > new file mode 100644 > index 0000000..93d976b > --- /dev/null > +++ b/drivers/gpu/drm/bridge/panel_binder.c > @@ -0,0 +1,193 @@ > +/* > + * Copyright (C) 2014 Samsung Electronics Co., Ltd. > + * > + * 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/of.h> > + > +#include <drm/drm_panel.h> > + > +#include "drmP.h" > +#include "drm_crtc.h" > +#include "drm_crtc_helper.h" > + > +#include "bridge/panel_binder.h" > + > +struct panel_binder { > + struct drm_connector connector; > + struct i2c_client *client; > + struct drm_encoder *encoder; > + struct drm_bridge *bridge; > + struct drm_panel *panel; > +}; > + > +static void panel_binder_pre_enable(struct drm_bridge *bridge) > +{ > + struct panel_binder *panel_binder = bridge->driver_private; > + > + drm_panel_prepare(panel_binder->panel); > +} > + > +static void panel_binder_enable(struct drm_bridge *bridge) > +{ > + struct panel_binder *panel_binder = bridge->driver_private; > + > + drm_panel_enable(panel_binder->panel); > +} > + > +static void panel_binder_disable(struct drm_bridge *bridge) > +{ > + struct panel_binder *panel_binder = bridge->driver_private; > + > + drm_panel_disable(panel_binder->panel); > +} > + > +static void panel_binder_post_disable(struct drm_bridge *bridge) > +{ > + struct panel_binder *panel_binder = bridge->driver_private; > + > + drm_panel_unprepare(panel_binder->panel); > +} > + > +void panel_binder_destroy(struct drm_bridge *bridge) > +{ > + struct panel_binder *panel_binder = bridge->driver_private; > + > + drm_panel_detach(panel_binder->panel); > + drm_bridge_cleanup(bridge); > +} > + > +struct drm_bridge_funcs panel_binder_funcs = { > + .pre_enable = panel_binder_pre_enable, > + .enable = panel_binder_enable, > + .disable = panel_binder_disable, > + .post_disable = panel_binder_post_disable, > + .destroy = panel_binder_destroy, > +}; > + > +static int panel_binder_mode_valid(struct drm_connector *connector, > + struct drm_display_mode *mode) > +{ > + return MODE_OK; > +} > + > +static int panel_binder_get_modes(struct drm_connector *connector) > +{ > + struct panel_binder *panel_binder; > + > + panel_binder = container_of(connector, struct panel_binder, connector); > + > + return panel_binder->panel->funcs->get_modes(panel_binder->panel); > +} > + > +static struct drm_encoder *panel_binder_best_encoder(struct drm_connector > + *connector) > +{ > + struct panel_binder *panel_binder; > + > + panel_binder = container_of(connector, struct panel_binder, connector); > + > + return panel_binder->encoder; > +} > + > +static const struct drm_connector_helper_funcs > + panel_binder_connector_helper_funcs = { > + .get_modes = panel_binder_get_modes, > + .mode_valid = panel_binder_mode_valid, > + .best_encoder = panel_binder_best_encoder, > +}; > + > +static enum drm_connector_status panel_binder_detect(struct drm_connector > + *connector, bool force) > +{ > + return connector_status_connected; > +} > + > +static void panel_binder_connector_destroy(struct drm_connector *connector) > +{ > + drm_connector_cleanup(connector); > +} > + > +static const struct drm_connector_funcs panel_binder_connector_funcs = { > + .dpms = drm_helper_connector_dpms, > + .fill_modes = drm_helper_probe_single_connector_modes, > + .detect = panel_binder_detect, > + .destroy = panel_binder_connector_destroy, > +}; > + > +struct drm_bridge *panel_binder_init(struct drm_device *dev, > + struct drm_encoder *encoder, > + struct i2c_client *client, > + struct device_node *node, > + struct drm_panel *panel, > + int connector_type, > + uint8_t polled) > +{ > + int ret; > + struct drm_bridge *bridge; > + struct panel_binder *panel_binder; > + > + if (IS_ERR_OR_NULL(panel)) { > + DRM_ERROR("invalid drm_panel pointer\n"); > + return NULL; > + } > + > + bridge = devm_kzalloc(dev->dev, sizeof(*bridge), GFP_KERNEL); > + if (!bridge) { > + DRM_ERROR("failed to allocate drm bridge\n"); > + return NULL; > + } > + > + panel_binder = devm_kzalloc(dev->dev, sizeof(*panel_binder), > + GFP_KERNEL); > + if (!panel_binder) { > + DRM_ERROR("failed to allocate bridge panel_binder\n"); > + return NULL; > + } > + > + panel_binder->client = client; > + panel_binder->encoder = encoder; > + panel_binder->bridge = bridge; > + panel_binder->panel = panel; > + > + ret = drm_bridge_init(dev, bridge, &panel_binder_funcs); > + if (ret) { > + DRM_ERROR("failed to initialize bridge with drm\n"); > + goto err; > + } > + > + bridge->driver_private = panel_binder; > + > + drm_panel_attach(panel_binder->panel, &panel_binder->connector); > + > + if (!encoder->bridge) > + /* First entry in the bridge chain */ > + encoder->bridge = bridge; > + > + panel_binder->connector.polled = polled; > + ret = drm_connector_init(dev, &panel_binder->connector, > + &panel_binder_connector_funcs, connector_type); > + if (ret) { > + DRM_ERROR("failed to initialize connector with drm\n"); > + goto err; > + } > + drm_connector_helper_add(&panel_binder->connector, > + &panel_binder_connector_helper_funcs); > + drm_sysfs_connector_add(&panel_binder->connector); > + drm_mode_connector_attach_encoder(&panel_binder->connector, encoder); > + > + return bridge; > + > +err: > + return NULL; > +} > +EXPORT_SYMBOL(panel_binder_init); > diff --git a/include/drm/bridge/panel_binder.h b/include/drm/bridge/panel_binder.h > new file mode 100644 > index 0000000..b5ebbc2 > --- /dev/null > +++ b/include/drm/bridge/panel_binder.h > @@ -0,0 +1,44 @@ > +/* > + * Copyright (C) 2014 Samsung Electronics Co., Ltd. > + * > + * 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_PANEL_H_ > +#define _DRM_BRIDGE_PANEL_H_ > + > +struct drm_device; > +struct drm_encoder; > +struct i2c_client; > +struct device_node; > +struct drm_panel; > + > +#if defined(CONFIG_DRM_PANEL_BINDER) > +struct drm_bridge *panel_binder_init(struct drm_device *dev, > + struct drm_encoder *encoder, > + struct i2c_client *client, > + struct device_node *node, > + struct drm_panel *panel, > + int connector_type, > + uint8_t polled); > +#else > +static inline struct drm_bridge *panel_binder_init(struct drm_device *dev, > + struct drm_encoder *encoder, > + struct i2c_client *client, > + struct device_node *node, > + struct drm_panel *panel, > + int connector_type, > + uint8_t polled) > +{ > + return 0; > +} > +#endif > + > +#endif > -- > 1.7.9.5 > _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel