mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Peter Ujfalusi <peter.ujfalusi@ti.com>
To: <tomi.valkeinen@ti.com>, <plagnioj@jcrosoft.com>,
	<robdclark@gmail.com>, <airlied@linux.ie>, <tony@atomide.com>
Cc: <linux-fbdev@vger.kernel.org>, <linux-omap@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>,
	<laurent.pinchart@ideasonboard.com>, <jsarha@ti.com>,
	<dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH v3 00/27] fb/drm: omapdss: Clean up the headers and separate the two stack
Date: Fri, 3 Jun 2016 16:08:18 +0300	[thread overview]
Message-ID: <bca39779-6fda-a718-3fa3-5089f4d5bfbc@ti.com> (raw)
In-Reply-To: <20160603110329.9953-1-peter.ujfalusi@ti.com>

Tony,

On 06/03/16 14:03, Peter Ujfalusi wrote:
> Hi,
> 
> Changes since v2:
> - Collected the patches (4 of them) at the beginning which touches mach-omap2
> - Smaller changes in the moved patches to make sure they compile.
> 
> Changes since v1:
> - patches (2) added to remove the inclusion of video/omap-panel-data.h when it
>   is not needed
> - Transitional patch to create the video/omapfb_dss.h has been changed to copy
>   the content of the omapdss.h in one step.
> 
> I have prepared two branches on top of v4.7-rc1:
> [1] https://github.com/omap-audio/linux-audio.git peter/for-4.8_omapdss_part1
> 
> containing:
>   ARM: OMAP: rx51-video: Do not set TV connector_type
>   ARM/video: omap2: Move omap_display_init declaration to
>     mach-omap2/display.h
>   video/platform_data: omapdss: Create new header file for platform data
>   ARM: OMAP2: Use the platform_data header for omapdss

I have added your Acked-by to the ARM: OMAP patches to speed things up, please
let me know if that is OK.

> 
> [2] https://github.com/omap-audio/linux-audio.git peter/for-4.8_omapdss_part2
> 
> containing the whole set, including part1.
> 
> Only patches in part1 branch are touching arch/arm/mach-omap2/ files.
> 
> omapfb is deprecated. It will not receive new features, only bug fixes. On the.
> other hand omapdrm is in active development and the fact that both
> implementation (omapfb and omapdrm) is using the same header file
> (video/omapdss.h) makes implementing new features or doing bigger clean ups in
> omapdrm harder and hared as the change should not break omapfb.
> 
> To overcome this issue we need to separate the two implementation. This is the
> aim of this series:
> Create platform_data header for omapdss,
> clean up the header usage and dependencies,
> new header file for omapfb stack (video/omapfb_dss.h)
> local omapdss.h header file for omapdrm.
> 
> Regards,
> Peter
> ---
> Peter Ujfalusi (27):
>   ARM: OMAP: rx51-video: Do not set TV connector_type
>   ARM/video: omap2: Move omap_display_init declaration to
>     mach-omap2/display.h
>   video/platform_data: omapdss: Create new header file for platform data
>   ARM: OMAP2: Use the platform_data header for omapdss
>   omapfb: panel-tpo-td028ttec1: Remove legacy boot support
>   omapfb: panel-nec-nl8048hl11: Remove legacy boot support
>   omapfb: panel-tpo-td043mtea1: Remove legacy boot support
>   omapfb: panel-sharp-ls037v7dw01: Remove legacy boot support
>   omapfb: panel-lgphilips-lb035q02: Remove legacy boot support
>   omapfb: panel-dsi-cm: Remove legacy boot support
>   omapfb: connector-hdmi: Remove legacy boot support
>   omapfb: connector-dvi: Remove legacy boot support
>   omapfb: encoder-tfp410: Remove legacy boot support
>   omapfb: encoder-tpd12s015: No need to include video/omap-panle-data.h
>   drm/omap: displays: Do not include video/omap-panel-data.h if not
>     needed
>   omapdss: omap-panel-data.h: Remove struct omap_dss_device declaration
>   drm/omap: connector-analog-tv: Support only Composite type in legacy
>     boot
>   omapfb: connector-analog-tv: Support only Composite type in legacy
>     boot
>   omapdss: omap-panel-data.h: Remove connector_type from atv pdata
>   drm/omap: Remove reference to pdata->default_device
>   omapfb: Remove reference to pdata->default_device
>   platform_data: omapdss: Remove unused members from omap_dss_board_info
>   omapdss: hdmi audio: Make header file independent of video/omapdss.h
>   drm/omap: Do not include video/omapdss.h directly in drivers
>   omapfb: Create new header file for omapfb DSS implementation
>   [media] omap_vout: Switch to use the video/omapfb_dss.h header file
>   drm/omap: Remove the video/omapdss.h and move it's content to local
>     header file
> 
>  arch/arm/mach-omap2/board-ldp.c                    |   3 +-
>  arch/arm/mach-omap2/board-rx51-video.c             |   4 +-
>  arch/arm/mach-omap2/display.c                      |   2 +-
>  arch/arm/mach-omap2/display.h                      |   5 +
>  arch/arm/mach-omap2/dss-common.c                   |   2 +-
>  .../gpu/drm/omapdrm/displays/connector-analog-tv.c |  11 +-
>  drivers/gpu/drm/omapdrm/displays/connector-dvi.c   |   4 +-
>  drivers/gpu/drm/omapdrm/displays/connector-hdmi.c  |   4 +-
>  drivers/gpu/drm/omapdrm/displays/encoder-opa362.c  |   2 +-
>  drivers/gpu/drm/omapdrm/displays/encoder-tfp410.c  |   3 +-
>  .../gpu/drm/omapdrm/displays/encoder-tpd12s015.c   |   3 +-
>  drivers/gpu/drm/omapdrm/displays/panel-dpi.c       |   3 +-
>  drivers/gpu/drm/omapdrm/displays/panel-dsi-cm.c    |   4 +-
>  .../omapdrm/displays/panel-lgphilips-lb035q02.c    |   3 +-
>  .../drm/omapdrm/displays/panel-nec-nl8048hl11.c    |   2 +-
>  .../drm/omapdrm/displays/panel-sharp-ls037v7dw01.c |   3 +-
>  .../drm/omapdrm/displays/panel-sony-acx565akm.c    |   3 +-
>  .../drm/omapdrm/displays/panel-tpo-td028ttec1.c    |   3 +-
>  .../drm/omapdrm/displays/panel-tpo-td043mtea1.c    |   2 +-
>  drivers/gpu/drm/omapdrm/dss/core.c                 |   5 +-
>  drivers/gpu/drm/omapdrm/dss/dispc.c                |   3 +-
>  drivers/gpu/drm/omapdrm/dss/dispc_coefs.c          |   2 +-
>  drivers/gpu/drm/omapdrm/dss/display.c              |   2 +-
>  drivers/gpu/drm/omapdrm/dss/dpi.c                  |   3 +-
>  drivers/gpu/drm/omapdrm/dss/dsi.c                  |   2 +-
>  drivers/gpu/drm/omapdrm/dss/dss-of.c               |   3 +-
>  drivers/gpu/drm/omapdrm/dss/dss.c                  |   3 +-
>  drivers/gpu/drm/omapdrm/dss/dss_features.c         |   3 +-
>  drivers/gpu/drm/omapdrm/dss/hdmi.h                 |   3 +-
>  drivers/gpu/drm/omapdrm/dss/hdmi4.c                |   2 +-
>  drivers/gpu/drm/omapdrm/dss/hdmi5.c                |   2 +-
>  drivers/gpu/drm/omapdrm/dss/hdmi_common.c          |   2 +-
>  drivers/gpu/drm/omapdrm/dss/hdmi_phy.c             |   2 +-
>  drivers/gpu/drm/omapdrm/dss/hdmi_pll.c             |   3 +-
>  drivers/gpu/drm/omapdrm/dss/hdmi_wp.c              |   2 +-
>  drivers/gpu/drm/omapdrm/dss/omapdss.h              | 866 ++++++++++++++++++++-
>  drivers/gpu/drm/omapdrm/dss/output.c               |   3 +-
>  drivers/gpu/drm/omapdrm/dss/pll.c                  |   3 +-
>  drivers/gpu/drm/omapdrm/dss/rfbi.c                 |   2 +-
>  drivers/gpu/drm/omapdrm/dss/sdi.c                  |   2 +-
>  drivers/gpu/drm/omapdrm/dss/venc.c                 |   3 +-
>  drivers/gpu/drm/omapdrm/dss/video-pll.c            |   3 +-
>  drivers/gpu/drm/omapdrm/omap_drv.h                 |   1 -
>  drivers/media/platform/omap/omap_vout.c            |   2 +-
>  drivers/media/platform/omap/omap_voutdef.h         |   2 +-
>  drivers/media/platform/omap/omap_voutlib.c         |   2 +-
>  .../omap2/omapfb/displays/connector-analog-tv.c    |  10 +-
>  .../fbdev/omap2/omapfb/displays/connector-dvi.c    |  60 +-
>  .../fbdev/omap2/omapfb/displays/connector-hdmi.c   |  44 +-
>  .../fbdev/omap2/omapfb/displays/encoder-opa362.c   |   2 +-
>  .../fbdev/omap2/omapfb/displays/encoder-tfp410.c   |  46 +-
>  .../omap2/omapfb/displays/encoder-tpd12s015.c      |   3 +-
>  .../video/fbdev/omap2/omapfb/displays/panel-dpi.c  |   2 +-
>  .../fbdev/omap2/omapfb/displays/panel-dsi-cm.c     |  54 +-
>  .../omapfb/displays/panel-lgphilips-lb035q02.c     |  58 +-
>  .../omap2/omapfb/displays/panel-nec-nl8048hl11.c   |  47 +-
>  .../omapfb/displays/panel-sharp-ls037v7dw01.c      |  83 +-
>  .../omap2/omapfb/displays/panel-sony-acx565akm.c   |   2 +-
>  .../omap2/omapfb/displays/panel-tpo-td028ttec1.c   |  45 +-
>  .../omap2/omapfb/displays/panel-tpo-td043mtea1.c   |  46 +-
>  drivers/video/fbdev/omap2/omapfb/dss/apply.c       |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/core.c        |   4 +-
>  .../video/fbdev/omap2/omapfb/dss/dispc-compat.c    |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/dispc.c       |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/dispc_coefs.c |   2 +-
>  .../video/fbdev/omap2/omapfb/dss/display-sysfs.c   |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/display.c     |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/dpi.c         |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/dsi.c         |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/dss-of.c      |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/dss.c         |   2 +-
>  .../video/fbdev/omap2/omapfb/dss/dss_features.c    |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/hdmi.h        |   3 +-
>  drivers/video/fbdev/omap2/omapfb/dss/hdmi4.c       |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/hdmi5.c       |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/hdmi_common.c |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/hdmi_phy.c    |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/hdmi_pll.c    |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/hdmi_wp.c     |   2 +-
>  .../video/fbdev/omap2/omapfb/dss/manager-sysfs.c   |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/manager.c     |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/output.c      |   2 +-
>  .../video/fbdev/omap2/omapfb/dss/overlay-sysfs.c   |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/overlay.c     |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/pll.c         |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/rfbi.c        |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/sdi.c         |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/venc.c        |   2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/video-pll.c   |   2 +-
>  drivers/video/fbdev/omap2/omapfb/omapfb-ioctl.c    |   2 +-
>  drivers/video/fbdev/omap2/omapfb/omapfb-main.c     |   2 +-
>  drivers/video/fbdev/omap2/omapfb/omapfb-sysfs.c    |   2 +-
>  drivers/video/fbdev/omap2/omapfb/omapfb.h          |   2 +-
>  include/linux/platform_data/omapdss.h              |  37 +
>  include/sound/omap-hdmi-audio.h                    |   9 +-
>  include/video/omap-panel-data.h                    | 157 ----
>  include/video/{omapdss.h => omapfb_dss.h}          |  60 +-
>  sound/soc/omap/omap-hdmi-audio.c                   |   1 -
>  98 files changed, 1081 insertions(+), 751 deletions(-)
>  create mode 100644 include/linux/platform_data/omapdss.h
>  rename include/video/{omapdss.h => omapfb_dss.h} (93%)
> 


-- 
Péter

  parent reply	other threads:[~2016-06-03 13:08 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-03 11:03 Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 01/27] ARM: OMAP: rx51-video: Do not set TV connector_type Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 02/27] ARM/video: omap2: Move omap_display_init declaration to mach-omap2/display.h Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 03/27] video/platform_data: omapdss: Create new header file for platform data Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 04/27] ARM: OMAP2: Use the platform_data header for omapdss Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 05/27] omapfb: panel-tpo-td028ttec1: Remove legacy boot support Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 06/27] omapfb: panel-nec-nl8048hl11: " Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 07/27] omapfb: panel-tpo-td043mtea1: " Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 08/27] omapfb: panel-sharp-ls037v7dw01: " Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 09/27] omapfb: panel-lgphilips-lb035q02: " Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 10/27] omapfb: panel-dsi-cm: " Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 11/27] omapfb: connector-hdmi: " Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 12/27] omapfb: connector-dvi: " Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 13/27] omapfb: encoder-tfp410: " Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 14/27] omapfb: encoder-tpd12s015: No need to include video/omap-panle-data.h Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 15/27] drm/omap: displays: Do not include video/omap-panel-data.h if not needed Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 16/27] omapdss: omap-panel-data.h: Remove struct omap_dss_device declaration Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 17/27] drm/omap: connector-analog-tv: Support only Composite type in legacy boot Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 18/27] omapfb: " Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 19/27] omapdss: omap-panel-data.h: Remove connector_type from atv pdata Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 20/27] drm/omap: Remove reference to pdata->default_device Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 21/27] omapfb: " Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 22/27] platform_data: omapdss: Remove unused members from omap_dss_board_info Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 23/27] omapdss: hdmi audio: Make header file independent of video/omapdss.h Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 24/27] drm/omap: Do not include video/omapdss.h directly in drivers Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 25/27] omapfb: Create new header file for omapfb DSS implementation Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 26/27] [media] omap_vout: Switch to use the video/omapfb_dss.h header file Peter Ujfalusi
2016-06-03 11:03 ` [PATCH v3 27/27] drm/omap: Remove the video/omapdss.h and move it's content to local " Peter Ujfalusi
2016-06-03 13:08 ` Peter Ujfalusi [this message]
2016-06-07  5:58   ` [PATCH v3 00/27] fb/drm: omapdss: Clean up the headers and separate the two stack Tony Lindgren
  -- strict thread matches above, loose matches on Subject: below --
2016-06-03 10:52 Peter Ujfalusi
2016-06-03 11:02 ` Peter Ujfalusi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bca39779-6fda-a718-3fa3-5089f4d5bfbc@ti.com \
    --to=peter.ujfalusi@ti.com \
    --cc=airlied@linux.ie \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=jsarha@ti.com \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-fbdev@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=plagnioj@jcrosoft.com \
    --cc=robdclark@gmail.com \
    --cc=tomi.valkeinen@ti.com \
    --cc=tony@atomide.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®