| Date | Sat, 18 May 2024 21:20:52 +0800 | Subject | Re: [RFT,v2,26/48] drm/panel: simple: Stop tracking prepared/enabled | From | Sui Jingfeng <> |
| |
Hi,
On 2024/5/4 05:33, Douglas Anderson wrote: > As talked about in commit d2aacaf07395 ("drm/panel: Check for already > prepared/enabled in drm_panel"), we want to remove needless code from > panel drivers that was storing and double-checking the > prepared/enabled state. Even if someone was relying on the > double-check before, that double-check is now in the core and not > needed in individual drivers.
I think you are right, as I see drm_panel already has embed the 'prepared' and 'enabled' as members, remove them from the derived structure could probably save memory footprint. Reducing boilerplate is also a side benefit.
> Signed-off-by: Douglas Anderson <dianders@chromium.org>
Acked-by: Sui Jingfeng <sui.jingfeng@linux.dev>
-- Best regards, Sui
|