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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <a6f6731b-9478-b91d-f4ce-aa018fb44d7b@arm.com>
Date:   Wed, 12 Jul 2023 12:14:30 +0530
From:   Mohammad Faiz Abbas Rizvi <faiz.abbas@....com>
To:     Liviu Dudau <liviu.dudau@....com>
Cc:     linux-kernel@...r.kernel.org, dri-devel@...ts.freedesktop.org,
        airlied@...il.com, daniel@...ll.ch
Subject: Re: [PATCH v2] drm/arm/komeda: Remove component framework and add a
 simple encoder

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