[PATCH] drm/panel: lvds: Remove unused members from main structure

Liu Ying posted 1 patch 1 month ago
drivers/gpu/drm/panel/panel-lvds.c | 2 --
1 file changed, 2 deletions(-)
[PATCH] drm/panel: lvds: Remove unused members from main structure
Posted by Liu Ying 1 month ago
Since commit 03fa454bb666 ("drm/panel: lvds: Simplify mode parsing"),
the width and height members of struct panel_lvds are no longer used.
Remove them.  No functional change.

Signed-off-by: Liu Ying <victor.liu@nxp.com>
---
 drivers/gpu/drm/panel/panel-lvds.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/gpu/drm/panel/panel-lvds.c b/drivers/gpu/drm/panel/panel-lvds.c
index 23fd535d8f479045a87b0f51bec17cd7c66b0f75..46b07f38559f35aa3b0a41662cc29480e53e885f 100644
--- a/drivers/gpu/drm/panel/panel-lvds.c
+++ b/drivers/gpu/drm/panel/panel-lvds.c
@@ -28,8 +28,6 @@ struct panel_lvds {
 	struct device *dev;
 
 	const char *label;
-	unsigned int width;
-	unsigned int height;
 	struct drm_display_mode dmode;
 	u32 bus_flags;
 	unsigned int bus_format;

---
base-commit: 3cace99d63192a7250461b058279a42d91075d0c
change-id: 20250829-panel-lvds-remove-width-height-1d063f0bb178

Best regards,
-- 
Liu Ying <victor.liu@nxp.com>
Re: [PATCH] drm/panel: lvds: Remove unused members from main structure
Posted by Liu Ying 4 weeks ago
On 08/29/2025, Liu Ying wrote:
> Since commit 03fa454bb666 ("drm/panel: lvds: Simplify mode parsing"),
> the width and height members of struct panel_lvds are no longer used.
> Remove them.  No functional change.
> 
> Signed-off-by: Liu Ying <victor.liu@nxp.com>
> ---
>  drivers/gpu/drm/panel/panel-lvds.c | 2 --
>  1 file changed, 2 deletions(-)

Applied to misc/kernel.git (drm-misc-next).

-- 
Regards,
Liu Ying
Re: [PATCH] drm/panel: lvds: Remove unused members from main structure
Posted by Raphael Gallais-Pou 1 month ago

On 8/29/25 09:53, Liu Ying wrote:
> Since commit 03fa454bb666 ("drm/panel: lvds: Simplify mode parsing"),
> the width and height members of struct panel_lvds are no longer used.
> Remove them.  No functional change.
>
> Signed-off-by: Liu Ying <victor.liu@nxp.com>
> ---

Hi,

Reviewed-by: Raphael Gallais-Pou <raphael.gallais-pou@foss.st.com>

Best regards,
Raphaël
>  drivers/gpu/drm/panel/panel-lvds.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/panel/panel-lvds.c b/drivers/gpu/drm/panel/panel-lvds.c
> index 23fd535d8f479045a87b0f51bec17cd7c66b0f75..46b07f38559f35aa3b0a41662cc29480e53e885f 100644
> --- a/drivers/gpu/drm/panel/panel-lvds.c
> +++ b/drivers/gpu/drm/panel/panel-lvds.c
> @@ -28,8 +28,6 @@ struct panel_lvds {
>  	struct device *dev;
>  
>  	const char *label;
> -	unsigned int width;
> -	unsigned int height;
>  	struct drm_display_mode dmode;
>  	u32 bus_flags;
>  	unsigned int bus_format;
>
> ---
> base-commit: 3cace99d63192a7250461b058279a42d91075d0c
> change-id: 20250829-panel-lvds-remove-width-height-1d063f0bb178
>
> Best regards,

Re: [PATCH] drm/panel: lvds: Remove unused members from main structure
Posted by Neil Armstrong 1 month ago
On 29/08/2025 09:53, Liu Ying wrote:
> Since commit 03fa454bb666 ("drm/panel: lvds: Simplify mode parsing"),
> the width and height members of struct panel_lvds are no longer used.
> Remove them.  No functional change.
> 
> Signed-off-by: Liu Ying <victor.liu@nxp.com>
> ---
>   drivers/gpu/drm/panel/panel-lvds.c | 2 --
>   1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/panel/panel-lvds.c b/drivers/gpu/drm/panel/panel-lvds.c
> index 23fd535d8f479045a87b0f51bec17cd7c66b0f75..46b07f38559f35aa3b0a41662cc29480e53e885f 100644
> --- a/drivers/gpu/drm/panel/panel-lvds.c
> +++ b/drivers/gpu/drm/panel/panel-lvds.c
> @@ -28,8 +28,6 @@ struct panel_lvds {
>   	struct device *dev;
>   
>   	const char *label;
> -	unsigned int width;
> -	unsigned int height;
>   	struct drm_display_mode dmode;
>   	u32 bus_flags;
>   	unsigned int bus_format;
> 
> ---
> base-commit: 3cace99d63192a7250461b058279a42d91075d0c
> change-id: 20250829-panel-lvds-remove-width-height-1d063f0bb178
> 
> Best regards,

Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>