All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sebastian Reichel <sre@kernel.org>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>,
	Sam Ravnborg <sam@ravnborg.org>,
	dri-devel@lists.freedesktop.org,
	Boris Brezillon <bbrezillon@kernel.org>
Subject: Re: [PATCH v6 49/51] drm/omap: dss: Remove unused omap_dss_device operations
Date: Sat, 22 Feb 2020 04:13:48 +0100	[thread overview]
Message-ID: <20200222031348.ozzn2zyhr57n4aka@earth.universe> (raw)
In-Reply-To: <20200220214530.GS4998@pendragon.ideasonboard.com>


[-- Attachment #1.1: Type: text/plain, Size: 1255 bytes --]

Hi,

On Thu, Feb 20, 2020 at 11:45:30PM +0200, Laurent Pinchart wrote:
> On Thu, Feb 20, 2020 at 10:39:38PM +0100, Sebastian Reichel wrote:
> > On Sun, Feb 16, 2020 at 11:03:06PM +0200, Laurent Pinchart wrote:
> > > The omap_dss_device .pre_enable(), .post_disable() and .set_timings()
> > > are not used anymore. Remove them.
> > > 
> > > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > > Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
> > > ---
> > 
> > Actually it would be good to postpone this patch a bit, since 
> > I need those functions as intermediate step for converting DSI
> > code (I'm currently rebasing my series on yours).
> 
> I'm fine with that. I'll move the patch to the end of the series
> (nothing depends on it) and postpone it for now.

Great, Thanks. I got my DSI cleanup patches working on top of your
series:

https://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-n900.git/log/?h=omapdrm/bridge/devel-with-dsi

I added a similar patch towards the end, which removes all
of the omap_dss_device ops:

https://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-n900.git/commit/?h=omapdrm/bridge/devel-with-dsi&id=4269becc2bf39631a4ae5aba8e21f4aab83fc837

-- Sebastian

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2020-02-22  3:13 UTC|newest]

Thread overview: 68+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-16 21:02 [PATCH v6 00/51] drm/omap: Replace custom display drivers with drm_bridge and drm_panel Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 01/51] video: hdmi: Change return type of hdmi_avi_infoframe_init() to void Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 02/51] drm/connector: Add helper to get a connector type name Laurent Pinchart
2020-02-17  9:43   ` Daniel Vetter
2020-02-21 14:15     ` Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 03/51] drm/edid: Add flag to drm_display_info to identify HDMI sinks Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 04/51] drm/bridge: Add connector-related bridge operations and data Laurent Pinchart
2020-02-17  9:53   ` Daniel Vetter
2020-02-18  1:21     ` Laurent Pinchart
2020-02-18  6:44       ` Daniel Vetter
2020-02-21 14:23         ` Laurent Pinchart
2020-02-21 17:05           ` Daniel Vetter
2020-02-21 17:18           ` Sam Ravnborg
2020-02-22  9:02             ` Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 05/51] drm/bridge: Add interlace_allowed flag to drm_bridge Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 06/51] drm/bridge: Extend bridge API to disable connector creation Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 07/51] drm/bridge: dumb-vga-dac: Rename internal symbols to simple-bridge Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 08/51] drm/bridge: dumb-vga-dac: Rename driver " Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 09/51] drm/bridge: simple-bridge: Add support for non-VGA bridges Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 10/51] drm/bridge: simple-bridge: Add support for enable GPIO Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 11/51] drm/bridge: simple-bridge: Add support for the TI OPA362 Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 12/51] drm/bridge: Add bridge driver for display connectors Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 13/51] drm/bridge: Add driver for the TI TPD12S015 HDMI level shifter Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 14/51] drm/bridge: panel: Implement bridge connector operations Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 15/51] drm/bridge: tfp410: Replace manual connector handling with bridge Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 16/51] drm/bridge: tfp410: Allow operation without drm_connector Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 17/51] drm: Add helper to create a connector for a chain of bridges Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 18/51] drm/omap: dss: Cleanup DSS ports on initialisation failure Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 19/51] drm/omap: Simplify HDMI mode and infoframe configuration Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 20/51] drm/omap: Factor out display type to connector type conversion Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 21/51] drm/omap: Use the drm_panel_bridge API Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 22/51] drm/omap: dss: Fix output next device lookup in DT Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 23/51] drm/omap: Add infrastructure to support drm_bridge local to DSS outputs Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 24/51] drm/omap: dss: Make omap_dss_device_ops optional Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 25/51] drm/omap: hdmi: Allocate EDID in the .read_edid() operation Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 26/51] drm/omap: hdmi4: Rework EDID read to isolate data read Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 27/51] drm/omap: hdmi5: " Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 28/51] drm/omap: hdmi4: Register a drm_bridge for EDID read Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 29/51] drm/omap: hdmi5: " Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 30/51] drm/omap: hdmi4: Move mode set, enable and disable operations to bridge Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 31/51] drm/omap: hdmi5: " Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 32/51] drm/omap: hdmi4: Implement drm_bridge .hpd_notify() operation Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 33/51] drm/omap: dss: Remove .set_hdmi_mode() and .set_infoframe() operations Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 34/51] drm/omap: venc: Register a drm_bridge Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 35/51] drm/omap: Create connector for bridges Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 36/51] drm/omap: Switch the HDMI and VENC outputs to drm_bridge Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 37/51] drm/omap: Remove HPD, detect and EDID omapdss operations Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 38/51] drm/omap: hdmi: Remove omap_dss_device operations Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 39/51] drm/omap: venc: " Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 40/51] drm/omap: hdmi4: Simplify EDID read Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 41/51] drm/omap: hdmi5: " Laurent Pinchart
2020-02-16 21:02 ` [PATCH v6 42/51] drm/omap: dpi: Sort includes alphabetically Laurent Pinchart
2020-02-16 21:03 ` [PATCH v6 43/51] drm/omap: dpi: Reorder functions in sections Laurent Pinchart
2020-02-16 21:03 ` [PATCH v6 44/51] drm/omap: dpi: Simplify clock setting API Laurent Pinchart
2020-02-16 21:03 ` [PATCH v6 45/51] drm/omap: dpi: Register a drm_bridge Laurent Pinchart
2020-02-21 12:25   ` Tomi Valkeinen
2020-02-16 21:03 ` [PATCH v6 46/51] drm/omap: sdi: Sort includes alphabetically Laurent Pinchart
2020-02-16 21:03 ` [PATCH v6 47/51] drm/omap: sdi: Register a drm_bridge Laurent Pinchart
2020-02-16 21:03 ` [PATCH v6 48/51] drm/omap: Hardcode omap_connector type to DSI Laurent Pinchart
2020-02-21 12:20   ` Tomi Valkeinen
2020-02-16 21:03 ` [PATCH v6 49/51] drm/omap: dss: Remove unused omap_dss_device operations Laurent Pinchart
2020-02-20 21:39   ` Sebastian Reichel
2020-02-20 21:45     ` Laurent Pinchart
2020-02-22  3:13       ` Sebastian Reichel [this message]
2020-02-16 21:03 ` [PATCH v6 50/51] drm/omap: dss: Inline the omapdss_display_get() function Laurent Pinchart
2020-02-16 21:03 ` [PATCH v6 51/51] drm/omap: dss: Remove unused omapdss_of_find_connected_device() function Laurent Pinchart
2020-02-17 23:00 ` [PATCH v6 00/51] drm/omap: Replace custom display drivers with drm_bridge and drm_panel Sebastian Reichel
2020-02-17 23:16   ` Laurent Pinchart

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=20200222031348.ozzn2zyhr57n4aka@earth.universe \
    --to=sre@kernel.org \
    --cc=bbrezillon@kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=sam@ravnborg.org \
    --cc=tomi.valkeinen@ti.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.