[PATCH v2 02/10] drm/simpledrm: Inline device-init helpers
Javier Martinez Canillas
javierm at redhat.com
Wed Jul 27 19:30:31 AEST 2022
Hello Thomas,
On 7/27/22 09:50, Thomas Zimmermann wrote:
> Hi
>
> Am 25.07.22 um 17:01 schrieb Javier Martinez Canillas:
>> Hello Thomas,
>>
>> On 7/20/22 16:27, Thomas Zimmermann wrote:
>>> Inline the helpers for initializing the hardware FB, the memory
>>> management and the modesetting into the device-creation function.
>>> No functional changes.
>>>
>>
>> Could you please elaborate in the commit message why this change is
>> desirable? Without this additional context, this feels like going
>> backwards, since you are dropping few helpers that have quite self
>> contained code and making simpledrm_device_create() much larger.
>
> To clarify: I want to make the init code more easy to follow. These old
> init functions still had to be called in the right order as each > possibly depends on settings from the others. It also feels like it's
> easier to extract common code for ofdrm. And the pipeline is static, so
> it doesn't require complex chains of helper calls. Having everything in
> one helper seems beneficial. (It's a trade-off, I know.)
>
I see. That makes sense to me. Could you please add the explanation to
the commit message ? And feel free to add my Acked-by for this one too.
--
Best regards,
Javier Martinez Canillas
Linux Engineering
Red Hat
More information about the Linuxppc-dev
mailing list