mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Paul Cercueil <paul@crapouillou.net>
To: David Airlie <airlied@linux.ie>, Daniel Vetter <daniel@ffwll.ch>,
	Laurent Pinchart <Laurent.pinchart@ideasonboard.com>
Cc: Sam Ravnborg <sam@ravnborg.org>,
	od@zcrc.me, dri-devel@lists.freedesktop.org,
	linux-kernel@vger.kernel.org, linux-mips@vger.kernel.org
Subject: Re: [PATCH v3 0/4] Fixes to bridge/panel and ingenic-drm
Date: Wed, 24 Feb 2021 13:44:37 +0000	[thread overview]
Message-ID: <DUC1PQ.KO33KJE3BP5L@crapouillou.net> (raw)
In-Reply-To: <20210124085552.29146-1-paul@crapouillou.net>

Hi,

Some feedback for patches 1-3? Laurent?

Cheers,
-Paul


Le dim. 24 janv. 2021 à 8:55, Paul Cercueil <paul@crapouillou.net> a 
écrit :
> Hi,
> 
> Here are three independent fixes. The first one addresses a
> use-after-free in bridge/panel.c; the second one addresses a
> use-after-free in the ingenic-drm driver; finally, the third one makes
> the ingenic-drm driver work again on older Ingenic SoCs.
> 
> Changes from v2:
> - patch [1/4] added a FIXME.
> - patch [2/4] is new. It introduces a 
> drmm_plain_simple_encoder_alloc()
>   macro that will be used in patch [3/4].
> - patch [3/4] uses the macro introduced in patch [2/4].
> - patch [4/4] is unmodified.
> 
> Note to linux-stable guys: patch [v2 2/3] will only apply on the 
> current
> drm-misc-next branch, to fix it for v5.11 and older kernels, use the 
> V1
> of that patch.
> 
> Cheers,
> -Paul
> 
> Paul Cercueil (4):
>   drm: bridge/panel: Cleanup connector on bridge detach
>   drm/simple_kms_helper: Add macro drmm_plain_simple_encoder_alloc()
>   drm/ingenic: Register devm action to cleanup encoders
>   drm/ingenic: Fix non-OSD mode
> 
>  drivers/gpu/drm/bridge/panel.c            | 12 +++++++++++
>  drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 26 
> +++++++++++------------
>  include/drm/drm_simple_kms_helper.h       | 17 +++++++++++++++
>  3 files changed, 42 insertions(+), 13 deletions(-)
> 
> --
> 2.29.2
> 



  parent reply	other threads:[~2021-02-24 14:56 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-24  8:55 Paul Cercueil
2021-01-24  8:55 ` [PATCH v3 1/4] drm: bridge/panel: Cleanup connector on bridge detach Paul Cercueil
2021-03-24  2:16   ` Laurent Pinchart
2021-01-24  8:55 ` [PATCH v3 2/4] drm/simple_kms_helper: Add macro drmm_plain_simple_encoder_alloc() Paul Cercueil
2021-03-24  2:24   ` Laurent Pinchart
2021-01-24  8:55 ` [PATCH v3 3/4] drm/ingenic: Register devm action to cleanup encoders Paul Cercueil
2021-03-24  2:27   ` Laurent Pinchart
2021-01-24  8:55 ` [PATCH v3 4/4] drm/ingenic: Fix non-OSD mode Paul Cercueil
2021-01-24  9:30   ` H. Nikolaus Schaller
2021-01-24  9:43     ` Paul Cercueil
2021-01-24  9:47       ` H. Nikolaus Schaller
2021-01-25 18:56         ` H. Nikolaus Schaller
2021-03-24  2:31   ` Laurent Pinchart
2021-02-24 13:44 ` Paul Cercueil [this message]
2021-03-23 16:03   ` [PATCH v3 0/4] Fixes to bridge/panel and ingenic-drm Paul Cercueil
2021-03-24  2:32     ` 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=DUC1PQ.KO33KJE3BP5L@crapouillou.net \
    --to=paul@crapouillou.net \
    --cc=Laurent.pinchart@ideasonboard.com \
    --cc=airlied@linux.ie \
    --cc=daniel@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mips@vger.kernel.org \
    --cc=od@zcrc.me \
    --cc=sam@ravnborg.org \
    /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®