mbox series

[v2,0/6] drm/tidss: Use new connector model for tidss

Message ID 20201109170601.21557-1-nikhil.nd@ti.com (mailing list archive)
Headers show
Series drm/tidss: Use new connector model for tidss | expand

Message

Nikhil Devshatwar Nov. 9, 2020, 5:05 p.m. UTC
This series moves the tidss to using new connectoe model, where the 
SoC driver (tidss) creates the connector and all the bridges are 
attached with the flag DRM_BRIDGE_ATTACH_NO_CONNECTOR

Since the bridges do not create the connector, the bus format and
bus_flag is set after the format negotiation.
Support format negotiations in the tfp410 and mhdp bridge drivers
as a first step before moving the connector model.

Nikhil Devshatwar (6):
  drm: bridge: Propagate the bus flags from bridge->timings
  drm/bridge: tfp410: Support format negotiation hooks
  drm/bridge: mhdp8546: Add minimal format negotiation
  drm/tidss: Set bus_format correctly from bridge/connector
  drm/tidss: Move to newer connector model
  drm/bridge: cdns-mhdp8546: Fix the interrupt enable/disable

 .../drm/bridge/cadence/cdns-mhdp8546-core.c   | 32 ++++++++++++++++--
 drivers/gpu/drm/bridge/ti-tfp410.c            | 33 +++++++++++++++++++
 drivers/gpu/drm/drm_bridge.c                  |  8 +++++
 drivers/gpu/drm/tidss/tidss_drv.h             |  3 ++
 drivers/gpu/drm/tidss/tidss_encoder.c         | 22 ++++++++-----
 drivers/gpu/drm/tidss/tidss_kms.c             | 19 ++++++++++-
 6 files changed, 106 insertions(+), 11 deletions(-)

Comments

Tomi Valkeinen Nov. 10, 2020, 9:02 a.m. UTC | #1
Hi Nikhil,

On 09/11/2020 19:05, Nikhil Devshatwar wrote:
> This series moves the tidss to using new connectoe model, where the 
> SoC driver (tidss) creates the connector and all the bridges are 
> attached with the flag DRM_BRIDGE_ATTACH_NO_CONNECTOR
> 
> Since the bridges do not create the connector, the bus format and
> bus_flag is set after the format negotiation.
> Support format negotiations in the tfp410 and mhdp bridge drivers
> as a first step before moving the connector model.
> 
> Nikhil Devshatwar (6):
>   drm: bridge: Propagate the bus flags from bridge->timings
>   drm/bridge: tfp410: Support format negotiation hooks
>   drm/bridge: mhdp8546: Add minimal format negotiation
>   drm/tidss: Set bus_format correctly from bridge/connector
>   drm/tidss: Move to newer connector model
>   drm/bridge: cdns-mhdp8546: Fix the interrupt enable/disable
> 
>  .../drm/bridge/cadence/cdns-mhdp8546-core.c   | 32 ++++++++++++++++--
>  drivers/gpu/drm/bridge/ti-tfp410.c            | 33 +++++++++++++++++++
>  drivers/gpu/drm/drm_bridge.c                  |  8 +++++
>  drivers/gpu/drm/tidss/tidss_drv.h             |  3 ++
>  drivers/gpu/drm/tidss/tidss_encoder.c         | 22 ++++++++-----
>  drivers/gpu/drm/tidss/tidss_kms.c             | 19 ++++++++++-
>  6 files changed, 106 insertions(+), 11 deletions(-)
> 

Please add a change log when sending new versions of a series.

 Tomi
Nikhil Devshatwar Nov. 10, 2020, 10:30 a.m. UTC | #2
On 11:02-20201110, Tomi Valkeinen wrote:
> Hi Nikhil,
> 
> On 09/11/2020 19:05, Nikhil Devshatwar wrote:
> > This series moves the tidss to using new connectoe model, where the 
> > SoC driver (tidss) creates the connector and all the bridges are 
> > attached with the flag DRM_BRIDGE_ATTACH_NO_CONNECTOR
> > 
> > Since the bridges do not create the connector, the bus format and
> > bus_flag is set after the format negotiation.
> > Support format negotiations in the tfp410 and mhdp bridge drivers
> > as a first step before moving the connector model.
> > 
> > Nikhil Devshatwar (6):
> >   drm: bridge: Propagate the bus flags from bridge->timings
    changes from v1:
    * Check for timings
    * Prioritize timings flags over next bridge's flags

> >   drm/bridge: tfp410: Support format negotiation hooks
    Changes from v1:
    * Use only MEDIA_BUS_FMT_FIXED for output

> >   drm/bridge: mhdp8546: Add minimal format negotiation
    Changes from v1:
    * cosmetic fixes, commit message update

> >   drm/tidss: Set bus_format correctly from bridge/connector
> >   drm/tidss: Move to newer connector model
    Changes from v1:
    * Add error handling

> >   drm/bridge: cdns-mhdp8546: Fix the interrupt enable/disable

> > 
> >  .../drm/bridge/cadence/cdns-mhdp8546-core.c   | 32 ++++++++++++++++--
> >  drivers/gpu/drm/bridge/ti-tfp410.c            | 33 +++++++++++++++++++
> >  drivers/gpu/drm/drm_bridge.c                  |  8 +++++
> >  drivers/gpu/drm/tidss/tidss_drv.h             |  3 ++
> >  drivers/gpu/drm/tidss/tidss_encoder.c         | 22 ++++++++-----
> >  drivers/gpu/drm/tidss/tidss_kms.c             | 19 ++++++++++-
> >  6 files changed, 106 insertions(+), 11 deletions(-)
> > 
> 
> Please add a change log when sending new versions of a series.
> 

Actually, I had added the git notes, but missed to add --notes while
sending the patches using git send-email.

I am updating the change log here manually.

Nikhil D

>  Tomi
> 
> -- 
> Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
> Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki