From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ajay kumar Subject: Re: [PATCH V4 06/10] drm/bridge: Add a driver which binds drm_bridge with drm_panel Date: Fri, 20 Jun 2014 13:38:26 +0530 Message-ID: References: <1402511228-18945-1-git-send-email-ajaykumar.rs@samsung.com> <1402511228-18945-7-git-send-email-ajaykumar.rs@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <1402511228-18945-7-git-send-email-ajaykumar.rs-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Ajay Kumar Cc: "dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org" , "linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Dae , Sean Paul , Rob Clark , Daniel Vetter , Thierry Reding , sunil joshi , Prashanth G , =?UTF-8?Q?St=C3=A9phane_Marchesin?= List-Id: devicetree@vger.kernel.org ping. On Wed, Jun 11, 2014 at 11:57 PM, Ajay Kumar 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 > --- > 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 > +#include > + > +#include > + > +#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 > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html