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@xxxxxxx>
>> ---
>> .../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
