Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Philippe CORNU - foss <philippe.cornu@foss.st.com>
To:     Yannick FERTRE - foss <yannick.fertre@foss.st.com>,
        Thomas Zimmermann <tzimmermann@suse.de>,
        Jagan Teki <jagan@amarulasolutions.com>,
        "Yannick FERTRE" <yannick.fertre@st.com>,
        Philippe CORNU <philippe.cornu@st.com>,
        Benjamin Gaignard <benjamin.gaignard@linaro.org>,
        Vincent ABRIOU <vincent.abriou@st.com>
CC:     "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>,
        "linux-amarula@amarulasolutions.com" 
        <linux-amarula@amarulasolutions.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-stm32@st-md-mailman.stormreply.com" 
        <linux-stm32@st-md-mailman.stormreply.com>
Subject: RE: [Linux-stm32] [PATCH] drm/stm: ltdc: Use simple encoder
Thread-Topic: [Linux-stm32] [PATCH] drm/stm: ltdc: Use simple encoder
Thread-Index: AQHXD42Bu6QwdpjwfE2bZcobc4w1VKpzbsWAgAAIDoCABntyNg==
Date:   Mon, 8 Mar 2021 10:50:06 +0000
Message-ID: <1615200602091.7557@foss.st.com>
References: <20210302175700.28640-1-jagan@amarulasolutions.com>
 <791a6f75-3603-9b84-c267-76c24fb77ee2@suse.de>,<1d6ff4e2-9213-6e13-214d-b1705199610e@foss.st.com>
In-Reply-To: <1d6ff4e2-9213-6e13-214d-b1705199610e@foss.st.com>
Accept-Language: fr-FR, en-US
Content-Language: fr-FR
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Precedence: bulk

Applied on drm-misc-next.=0A=
Many thanks Jagan for your patch and many thanks Thomas & Yannick for your =
reviews & tests.=0A=
Philippe :-)=0A=
=0A=
________________________________________=0A=
De : Linux-stm32 <linux-stm32-bounces@st-md-mailman.stormreply.com> de la p=
art de Yannick FERTRE - foss=0A=
Envoy=E9 : jeudi 4 mars 2021 09:49=0A=
=C0 : Thomas Zimmermann; Jagan Teki; Yannick FERTRE; Philippe CORNU; Benjam=
in Gaignard; Vincent ABRIOU=0A=
Cc : dri-devel@lists.freedesktop.org; linux-amarula@amarulasolutions.com; l=
inux-kernel@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-st=
m32@st-md-mailman.stormreply.com=0A=
Objet : Re: [Linux-stm32] [PATCH] drm/stm: ltdc: Use simple encoder=0A=
=0A=
Hi Thomas,=0A=
I wait a few days before merging it.=0A=
Thank you for your help.=0A=
=0A=
Best regards=0A=
=0A=
Yannick=0A=
=0A=
=0A=
On 3/4/21 9:21 AM, Thomas Zimmermann wrote:=0A=
> Hi,=0A=
>=0A=
> shall I merge this patch?=0A=
>=0A=
> Am 02.03.21 um 18:57 schrieb Jagan Teki:=0A=
>> STM ltdc driver uses an empty implementation for its encoder.=0A=
>> Replace the code with the generic simple encoder.=0A=
>>=0A=
>> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>=0A=
>> ---=0A=
>>   drivers/gpu/drm/stm/ltdc.c | 12 ++----------=0A=
>>   1 file changed, 2 insertions(+), 10 deletions(-)=0A=
>>=0A=
>> diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c=0A=
>> index 7812094f93d6..aeeb43524ca0 100644=0A=
>> --- a/drivers/gpu/drm/stm/ltdc.c=0A=
>> +++ b/drivers/gpu/drm/stm/ltdc.c=0A=
>> @@ -31,6 +31,7 @@=0A=
>>   #include <drm/drm_of.h>=0A=
>>   #include <drm/drm_plane_helper.h>=0A=
>>   #include <drm/drm_probe_helper.h>=0A=
>> +#include <drm/drm_simple_kms_helper.h>=0A=
>>   #include <drm/drm_vblank.h>=0A=
>>   #include <video/videomode.h>=0A=
>> @@ -1020,14 +1021,6 @@ static int ltdc_crtc_init(struct drm_device=0A=
>> *ddev, struct drm_crtc *crtc)=0A=
>>       return ret;=0A=
>>   }=0A=
>> -/*=0A=
>> - * DRM_ENCODER=0A=
>> - */=0A=
>> -=0A=
>> -static const struct drm_encoder_funcs ltdc_encoder_funcs =3D {=0A=
>> -    .destroy =3D drm_encoder_cleanup,=0A=
>> -};=0A=
>> -=0A=
>>   static void ltdc_encoder_disable(struct drm_encoder *encoder)=0A=
>>   {=0A=
>>       struct drm_device *ddev =3D encoder->dev;=0A=
>> @@ -1088,8 +1081,7 @@ static int ltdc_encoder_init(struct drm_device=0A=
>> *ddev, struct drm_bridge *bridge)=0A=
>>       encoder->possible_crtcs =3D CRTC_MASK;=0A=
>>       encoder->possible_clones =3D 0;    /* No cloning support */=0A=
>> -    drm_encoder_init(ddev, encoder, &ltdc_encoder_funcs,=0A=
>> -             DRM_MODE_ENCODER_DPI, NULL);=0A=
>> +    drm_simple_encoder_init(ddev, encoder, DRM_MODE_ENCODER_DPI);=0A=
>>       drm_encoder_helper_add(encoder, &ltdc_encoder_helper_funcs);=0A=
>>=0A=
>=0A=
>=0A=
> _______________________________________________=0A=
> dri-devel mailing list=0A=
> dri-devel@lists.freedesktop.org=0A=
> https://lists.freedesktop.org/mailman/listinfo/dri-devel=0A=
>=0A=
_______________________________________________=0A=
Linux-stm32 mailing list=0A=
Linux-stm32@st-md-mailman.stormreply.com=0A=
https://st-md-mailman.stormreply.com/mailman/listinfo/linux-stm32=0A=