Re: [PATCH v2] drm/arm/komeda: Remove component framework and add a simple encoder

[Mohammad Faiz Abbas Rizvi <faiz.abbas@arm.com>]

Hi Liviu,

On 7/5/2023 10:44 PM, Liviu Dudau wrote:
> Hi Faiz,
> 
> On Tue, Jul 04, 2023 at 10:04:54PM +0530, Faiz Abbas wrote:
>> The Komeda driver always expects the remote connector node to initialize
>> an encoder. It uses the component aggregator framework which consists
>> of component->bind() calls used to initialize the remote encoder and attach
>> it to the crtc. This makes it incompatible with connector drivers which
>> implement drm_bridge APIs.
>>
>> Remove all component framework calls from the komeda driver and declare and
>> attach an encoder inside komeda_crtc_add().
>>
>> The remote connector driver has to implement the DRM bridge APIs which
>> can be used to glue the encoder to the remote connector. Since we
>> usually pair this with a component encoder that also implements a
>> drm_bridge, dropping support is not expected to affect users of this
>> driver.
> 
> Thanks for updating the commit description, I think it shows the intent better.
> 
> When I'm trying to apply your patch to drm-misc next (or any branch that matters)
> it fails because ...
> 
>>
>> Signed-off-by: Faiz Abbas <faiz.abbas@arm.com>
>> ---
>>  .../gpu/drm/arm/display/komeda/komeda_crtc.c  | 23 +++++++-
>>  .../gpu/drm/arm/display/komeda/komeda_drv.c   | 57 ++-----------------
>>  .../gpu/drm/arm/display/komeda/komeda_kms.c   | 10 +---
>>  .../gpu/drm/arm/display/komeda/komeda_kms.h   |  3 +
>>  4 files changed, 32 insertions(+), 61 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c b/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c
>> index 4cc07d6bb9d82..e5a8a80b173f4 100644
>> --- a/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c
>> +++ b/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c
>> @@ -13,6 +13,8 @@
>>  #include <drm/drm_crtc_helper.h>
> 
> ... this line is different in my tree. It looks like your tree is missing
> commit e3b63718827880 ("drm/arm/komeda: Remove unnecessary include
> statements for drm_crtc_helper.h"), which has been applied in early January.
> 


Right. Let me rebase to drm-misc-next and send another version.

Thanks,
Faiz