[PATCH v4 2/4] drm: bridge: Add LVDS encoder driver

[Archit Taneja]

Hi,

On 3/2/2017 4:17 PM, Laurent Pinchart wrote:
> The LVDS encoder driver is a DRM bridge driver that supports the
> parallel to LVDS encoders that don't require any configuration. The
> driver thus doesn't interact with the device, but creates an LVDS
> connector for the panel and exposes its size and timing based on
> information retrieved from DT.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas at ideasonboard.com>
> Acked-by: Daniel Vetter <daniel at ffwll.ch>
> ---
> Changes since v3:
>
> - Dropped encoder_type
> - Added a Kconfig dependency on DRM_PANEL
> ---
>  drivers/gpu/drm/bridge/Kconfig        |   9 ++
>  drivers/gpu/drm/bridge/Makefile       |   1 +
>  drivers/gpu/drm/bridge/lvds-encoder.c | 209 ++++++++++++++++++++++++++++++++++
>  3 files changed, 219 insertions(+)
>  create mode 100644 drivers/gpu/drm/bridge/lvds-encoder.c
>

<snip>

> +
> +static int lvds_encoder_remove(struct platform_device *pdev)
> +{
> +	struct lvds_encoder *encoder = platform_get_drvdata(pdev);
> +
> +	drm_bridge_remove(&encoder->bridge);
> +
> +	return 0;
> +}
> +
> +static const struct of_device_id lvds_encoder_match[] = {
> +	{ .compatible = "lvds-encoder" },
> +	{},
> +};
> +MODULE_DEVICE_TABLE(of, lvds_encoder_match);
> +
> +struct platform_driver lvds_encoder_driver = {

I changed this to static to fix a sparse warning. Pushed
this and others in this set to drm-misc-next.

Thanks,
Archit

> +	.probe	= lvds_encoder_probe,
> +	.remove	= lvds_encoder_remove,
> +	.driver		= {
> +		.name		= "lvds-encoder",
> +		.of_match_table	= lvds_encoder_match,
> +	},
> +};
> +module_platform_driver(lvds_encoder_driver);
> +
> +MODULE_AUTHOR("Laurent Pinchart <laurent.pinchart at ideasonboard.com>");
> +MODULE_DESCRIPTION("Transparent parallel to LVDS encoder");
> +MODULE_LICENSE("GPL");
>

-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
hosted by The Linux Foundation