On 28-08-20, 15:04, Dmitry Baryshkov wrote:
> +#define EDID_BLOCK_SIZE 128
> +#define EDID_NUM_BLOCKS 2
tab or space either one, not both ;)
> +static struct mipi_dsi_device *lt9611uxc_attach_dsi(struct lt9611uxc
> *lt9611uxc,
> + struct device_node *dsi_node)
Please align this with open parenthesis of preceding line (checkpatch
with --strict option will check this)
> +static int lt9611uxc_bridge_attach(struct drm_bridge *bridge,
> + enum drm_bridge_attach_flags flags)
> +{
> + struct lt9611uxc *lt9611uxc = bridge_to_lt9611uxc(bridge);
> + int ret;
> +
> + if (!(flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR)) {
> + dev_err(lt9611uxc->dev, "Fix bridge driver to make connector
> optional!");
Can we support both modes as I have done in lt9611, that way once the
conversion is done we can drop the init part and support conversion.
I have patch for msm driver to set DRM_BRIDGE_ATTACH_NO_CONNECTOR, you
can use that to test
> +static int lt9611uxc_hdmi_hw_params(struct device *dev, void *data,
> + struct hdmi_codec_daifmt *fmt,
> + struct hdmi_codec_params *hparms)
> +{
> + /*
> + * LT9611UXC will automatically detect rate and sample size, so no need
> + * to setup anything here.
> + */
> + return 0;
> +}
Do we need dummy function?
--
~Vinod
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
