mbox series

[v1,0/7] drm_connector: drop legacy drm_bus_flags

Message ID 20200630180545.1132217-1-sam@ravnborg.org (mailing list archive)
Headers show
Series drm_connector: drop legacy drm_bus_flags | expand

Message

Sam Ravnborg June 30, 2020, 6:05 p.m. UTC
Drop all uses of legacy drm_bus_flags, and then drop the flags.
Follow-up with a patch to inline the documentation of the flags.

The conversion was triggered by Laurent's clean-up of
the bus_flags use in panel-simple.
https://lore.kernel.org/dri-devel/20200630135802.GA581802@ravnborg.org/T/#t

Build tested only.
Patches on top of latest drm-misc-next.

	Sam

(Procrastination, was supposed to review patches, not producing them).

Sam Ravnborg (7):
      drm/tidss: drop use of legacy drm_bus_flags
      drm/ingenic-drm: drop use of legacy drm_bus_flags
      drm/panel: raydium-rm67191: drop use of legacy drm_bus_flags
      drm/panel: novatek-nt39016: drop use of legacy drm_bus_flags
      drm/panel: panel-simple: drop use of legacy drm_bus_flags
      drm/drm_connector: drop legacy drm_bus_flags values
      drm/drm_connector: use inline comments for drm_bus_flags

 drivers/gpu/drm/ingenic/ingenic-drm.c         |   2 +-
 drivers/gpu/drm/panel/panel-novatek-nt39016.c |   2 +-
 drivers/gpu/drm/panel/panel-raydium-rm67191.c |   2 +-
 drivers/gpu/drm/panel/panel-simple.c          |  28 +++---
 drivers/gpu/drm/tidss/tidss_dispc.c           |   4 +-
 include/drm/drm_connector.h                   | 124 ++++++++++++++++++--------
 6 files changed, 104 insertions(+), 58 deletions(-)

Comments

Laurent Pinchart July 1, 2020, 5:47 a.m. UTC | #1
Hi Sam,

Thank you for the patches.

On Tue, Jun 30, 2020 at 08:05:38PM +0200, Sam Ravnborg wrote:
> Drop all uses of legacy drm_bus_flags, and then drop the flags.
> Follow-up with a patch to inline the documentation of the flags.
> 
> The conversion was triggered by Laurent's clean-up of
> the bus_flags use in panel-simple.
> https://lore.kernel.org/dri-devel/20200630135802.GA581802@ravnborg.org/T/#t
> 
> Build tested only.
> Patches on top of latest drm-misc-next.

For the whole series,

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> (Procrastination, was supposed to review patches, not producing them).
> 
> Sam Ravnborg (7):
>       drm/tidss: drop use of legacy drm_bus_flags
>       drm/ingenic-drm: drop use of legacy drm_bus_flags
>       drm/panel: raydium-rm67191: drop use of legacy drm_bus_flags
>       drm/panel: novatek-nt39016: drop use of legacy drm_bus_flags
>       drm/panel: panel-simple: drop use of legacy drm_bus_flags
>       drm/drm_connector: drop legacy drm_bus_flags values
>       drm/drm_connector: use inline comments for drm_bus_flags
> 
>  drivers/gpu/drm/ingenic/ingenic-drm.c         |   2 +-
>  drivers/gpu/drm/panel/panel-novatek-nt39016.c |   2 +-
>  drivers/gpu/drm/panel/panel-raydium-rm67191.c |   2 +-
>  drivers/gpu/drm/panel/panel-simple.c          |  28 +++---
>  drivers/gpu/drm/tidss/tidss_dispc.c           |   4 +-
>  include/drm/drm_connector.h                   | 124 ++++++++++++++++++--------
>  6 files changed, 104 insertions(+), 58 deletions(-)
Sam Ravnborg July 1, 2020, 9:40 a.m. UTC | #2
On Tue, Jun 30, 2020 at 08:05:38PM +0200, Sam Ravnborg wrote:
> Drop all uses of legacy drm_bus_flags, and then drop the flags.
> Follow-up with a patch to inline the documentation of the flags.
> 
> The conversion was triggered by Laurent's clean-up of
> the bus_flags use in panel-simple.
> https://lore.kernel.org/dri-devel/20200630135802.GA581802@ravnborg.org/T/#t
> 
> Build tested only.
> Patches on top of latest drm-misc-next.
> 
> 	Sam
> 
> (Procrastination, was supposed to review patches, not producing them).
> 
> Sam Ravnborg (7):
>       drm/tidss: drop use of legacy drm_bus_flags
>       drm/ingenic-drm: drop use of legacy drm_bus_flags
>       drm/panel: raydium-rm67191: drop use of legacy drm_bus_flags
>       drm/panel: novatek-nt39016: drop use of legacy drm_bus_flags
>       drm/panel: panel-simple: drop use of legacy drm_bus_flags
>       drm/drm_connector: drop legacy drm_bus_flags values
>       drm/drm_connector: use inline comments for drm_bus_flags

With the quick feedback from Jyri ack and Laurent I have applied this set to
drm-misc-next.

	Sam

> 
>  drivers/gpu/drm/ingenic/ingenic-drm.c         |   2 +-
>  drivers/gpu/drm/panel/panel-novatek-nt39016.c |   2 +-
>  drivers/gpu/drm/panel/panel-raydium-rm67191.c |   2 +-
>  drivers/gpu/drm/panel/panel-simple.c          |  28 +++---
>  drivers/gpu/drm/tidss/tidss_dispc.c           |   4 +-
>  include/drm/drm_connector.h                   | 124 ++++++++++++++++++--------
>  6 files changed, 104 insertions(+), 58 deletions(-)
>