Re: [PATCH 1/2] drm/panel: Remove redundant checks in multiple panels

2024-03-25 Thread Emilio Mendoza Reyes
Hi, I'd like to first thank you for your time and for looking at my patch. On Monday, March 25, 2024 8:21:20 PM EDT Doug Anderson wrote: > Aside from the formatting issues (several lines start with an extra > "-" and there is the PGP stuff), there are a few high-level issues > here: Yeah so sorry

Re: [PATCH 2/2] drm/panel: Promote warn_dev to WARN for invalid calls in drm_panel.c

2024-03-25 Thread Emilio Mendoza Reyes
ed-in-panels Signed-off-by: Emilio Mendoza Reyes --- drivers/gpu/drm/drm_panel.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/drm_panel.c b/drivers/gpu/drm/drm_panel.c index cfbe020de54e..4459f4f5fe66 100644 --- a/drivers/gpu/drm/drm_panel.c +++ b/dri

[PATCH 2/2] drm/panel: Promote warn_dev to WARN for invalid calls in drm_panel.c

2024-03-24 Thread Emilio Mendoza Reyes
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 From: Emilio Mendoza Reyes This patch is the second of a set of two. In the kernel docs there is a todo: Link: https://www.kernel.org/doc/html/v6.8/gpu/todo.html#clean-up-checks-for-already-prepared-enabled-in-panels The first patch addresses

[PATCH 1/2] drm/panel: Remove redundant checks in multiple panels

2024-03-24 Thread Emilio Mendoza Reyes
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 From: Emilio Mendoza Reyes The patch ("drm/panel: Check for already prepared/enabled in drm_panel") moved checking for (en/dis)abled and [un]prepared panels before specific function calls to drm_panel.c.Those checks that still ex