Re: [PATCH] drm/ast: Inline drm_simple_encoder_init()

2024-06-28 Thread Daniel Vetter
On Thu, Jun 27, 2024 at 09:18:15AM +0200, Thomas Zimmermann wrote: > Hi > > Am 26.06.24 um 19:59 schrieb Daniel Vetter: > > On Wed, Jun 26, 2024 at 11:01:11AM +0200, Thomas Zimmermann wrote: > > > Hi > > > > > > Am 26.06.24 um 06:34 schrieb Dmitry Baryshkov: > > > > On Tue, Jun 25, 2024 at 03:18:

Re: [PATCH] drm/ast: Inline drm_simple_encoder_init()

2024-06-28 Thread Thomas Zimmermann
Hi Am 27.06.24 um 09:47 schrieb Daniel Vetter: On Wed, Jun 26, 2024 at 07:59:52PM +0200, Daniel Vetter wrote: On Wed, Jun 26, 2024 at 11:01:11AM +0200, Thomas Zimmermann wrote: Hi Am 26.06.24 um 06:34 schrieb Dmitry Baryshkov: On Tue, Jun 25, 2024 at 03:18:09PM GMT, Thomas Zimmermann wrote:

Re: [PATCH] drm/ast: Inline drm_simple_encoder_init()

2024-06-27 Thread Thomas Zimmermann
Hi Am 25.06.24 um 16:18 schrieb Jocelyn Falempe: On 25/06/2024 15:18, Thomas Zimmermann wrote: The function drm_simple_encoder_init() is a trivial helper and deprecated. Replace it with the regular call to drm_encoder_init(). Resolves the dependency on drm_simple_kms_helper.h. No functional ch

Re: [PATCH] drm/ast: Inline drm_simple_encoder_init()

2024-06-27 Thread Daniel Vetter
On Wed, Jun 26, 2024 at 07:59:52PM +0200, Daniel Vetter wrote: > On Wed, Jun 26, 2024 at 11:01:11AM +0200, Thomas Zimmermann wrote: > > Hi > > > > Am 26.06.24 um 06:34 schrieb Dmitry Baryshkov: > > > On Tue, Jun 25, 2024 at 03:18:09PM GMT, Thomas Zimmermann wrote: > > > > The function drm_simple_e

Re: [PATCH] drm/ast: Inline drm_simple_encoder_init()

2024-06-27 Thread Thomas Zimmermann
Hi Am 26.06.24 um 19:59 schrieb Daniel Vetter: On Wed, Jun 26, 2024 at 11:01:11AM +0200, Thomas Zimmermann wrote: Hi Am 26.06.24 um 06:34 schrieb Dmitry Baryshkov: On Tue, Jun 25, 2024 at 03:18:09PM GMT, Thomas Zimmermann wrote: The function drm_simple_encoder_init() is a trivial helper and

Re: [PATCH] drm/ast: Inline drm_simple_encoder_init()

2024-06-26 Thread Daniel Vetter
On Wed, Jun 26, 2024 at 11:01:11AM +0200, Thomas Zimmermann wrote: > Hi > > Am 26.06.24 um 06:34 schrieb Dmitry Baryshkov: > > On Tue, Jun 25, 2024 at 03:18:09PM GMT, Thomas Zimmermann wrote: > > > The function drm_simple_encoder_init() is a trivial helper and > > > deprecated. Replace it with the

Re: [PATCH] drm/ast: Inline drm_simple_encoder_init()

2024-06-26 Thread Thomas Zimmermann
Hi Am 26.06.24 um 11:19 schrieb Dmitry Baryshkov: On Wed, 26 Jun 2024 at 12:19, Thomas Zimmermann wrote: Hi Am 26.06.24 um 11:10 schrieb Dmitry Baryshkov: On Wed, Jun 26, 2024 at 11:01:11AM GMT, Thomas Zimmermann wrote: Hi Am 26.06.24 um 06:34 schrieb Dmitry Baryshkov: On Tue, Jun 25, 202

Re: [PATCH] drm/ast: Inline drm_simple_encoder_init()

2024-06-26 Thread Dmitry Baryshkov
On Wed, 26 Jun 2024 at 12:19, Thomas Zimmermann wrote: > > Hi > > Am 26.06.24 um 11:10 schrieb Dmitry Baryshkov: > > On Wed, Jun 26, 2024 at 11:01:11AM GMT, Thomas Zimmermann wrote: > >> Hi > >> > >> Am 26.06.24 um 06:34 schrieb Dmitry Baryshkov: > >>> On Tue, Jun 25, 2024 at 03:18:09PM GMT, Thoma

Re: [PATCH] drm/ast: Inline drm_simple_encoder_init()

2024-06-26 Thread Thomas Zimmermann
Hi Am 26.06.24 um 11:10 schrieb Dmitry Baryshkov: On Wed, Jun 26, 2024 at 11:01:11AM GMT, Thomas Zimmermann wrote: Hi Am 26.06.24 um 06:34 schrieb Dmitry Baryshkov: On Tue, Jun 25, 2024 at 03:18:09PM GMT, Thomas Zimmermann wrote: The function drm_simple_encoder_init() is a trivial helper and

Re: [PATCH] drm/ast: Inline drm_simple_encoder_init()

2024-06-26 Thread Dmitry Baryshkov
On Wed, Jun 26, 2024 at 11:01:11AM GMT, Thomas Zimmermann wrote: > Hi > > Am 26.06.24 um 06:34 schrieb Dmitry Baryshkov: > > On Tue, Jun 25, 2024 at 03:18:09PM GMT, Thomas Zimmermann wrote: > > > The function drm_simple_encoder_init() is a trivial helper and > > > deprecated. Replace it with the r

Re: [PATCH] drm/ast: Inline drm_simple_encoder_init()

2024-06-26 Thread Thomas Zimmermann
Hi Am 26.06.24 um 06:34 schrieb Dmitry Baryshkov: On Tue, Jun 25, 2024 at 03:18:09PM GMT, Thomas Zimmermann wrote: The function drm_simple_encoder_init() is a trivial helper and deprecated. Replace it with the regular call to drm_encoder_init(). Resolves the dependency on drm_simple_kms_helper.

Re: [PATCH] drm/ast: Inline drm_simple_encoder_init()

2024-06-26 Thread Daniel Vetter
On Tue, Jun 25, 2024 at 04:03:21PM +0200, Thomas Zimmermann wrote: > Hi > > Am 25.06.24 um 15:55 schrieb Jocelyn Falempe: > > > > > > On 25/06/2024 15:18, Thomas Zimmermann wrote: > > > The function drm_simple_encoder_init() is a trivial helper and > > > deprecated. Replace it with the regular c

Re: [PATCH] drm/ast: Inline drm_simple_encoder_init()

2024-06-25 Thread Dmitry Baryshkov
On Tue, Jun 25, 2024 at 03:18:09PM GMT, Thomas Zimmermann wrote: > The function drm_simple_encoder_init() is a trivial helper and > deprecated. Replace it with the regular call to drm_encoder_init(). > Resolves the dependency on drm_simple_kms_helper.h. No functional > changes. > > Signed-off-by:

Re: [PATCH] drm/ast: Inline drm_simple_encoder_init()

2024-06-25 Thread Jocelyn Falempe
On 25/06/2024 15:18, Thomas Zimmermann wrote: The function drm_simple_encoder_init() is a trivial helper and deprecated. Replace it with the regular call to drm_encoder_init(). Resolves the dependency on drm_simple_kms_helper.h. No functional changes. Thanks for your patch, it looks good to m

Re: [PATCH] drm/ast: Inline drm_simple_encoder_init()

2024-06-25 Thread Thomas Zimmermann
Hi Am 25.06.24 um 15:55 schrieb Jocelyn Falempe: On 25/06/2024 15:18, Thomas Zimmermann wrote: The function drm_simple_encoder_init() is a trivial helper and deprecated. Replace it with the regular call to drm_encoder_init(). Resolves the dependency on drm_simple_kms_helper.h. No functional c

Re: [PATCH] drm/ast: Inline drm_simple_encoder_init()

2024-06-25 Thread Jocelyn Falempe
On 25/06/2024 15:18, Thomas Zimmermann wrote: The function drm_simple_encoder_init() is a trivial helper and deprecated. Replace it with the regular call to drm_encoder_init(). Resolves the dependency on drm_simple_kms_helper.h. No functional changes. Do you think it's possible to add a defa