diff mbox series

[v2] drm/bridge: analogix: remove unused struct 'bridge_init'

Message ID 20240520125551.11534-1-linux@treblig.org (mailing list archive)
State New, archived
Headers show
Series [v2] drm/bridge: analogix: remove unused struct 'bridge_init' | expand

Commit Message

Dr. David Alan Gilbert May 20, 2024, 12:55 p.m. UTC
From: "Dr. David Alan Gilbert" <linux@treblig.org>

commit 6a1688ae8794 ("drm/bridge: ptn3460: Convert to I2C driver model")
has dropped all the users of the struct bridge_init from the
exynos_dp_core, while retaining unused structure definition.
Later on the driver was reworked and the definition migrated
to the analogix_dp driver. Remove unused struct bridge_init definition.

Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
---
 drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 5 -----
 1 file changed, 5 deletions(-)

Comments

Dmitry Baryshkov May 20, 2024, 9:20 p.m. UTC | #1
On Mon, May 20, 2024 at 01:55:51PM +0100, linux@treblig.org wrote:
> From: "Dr. David Alan Gilbert" <linux@treblig.org>
> 
> commit 6a1688ae8794 ("drm/bridge: ptn3460: Convert to I2C driver model")
> has dropped all the users of the struct bridge_init from the
> exynos_dp_core, while retaining unused structure definition.
> Later on the driver was reworked and the definition migrated
> to the analogix_dp driver. Remove unused struct bridge_init definition.
> 
> Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
> ---
>  drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 5 -----
>  1 file changed, 5 deletions(-)
> 

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Dr. David Alan Gilbert May 20, 2024, 9:46 p.m. UTC | #2
* Dmitry Baryshkov (dmitry.baryshkov@linaro.org) wrote:
> On Mon, May 20, 2024 at 01:55:51PM +0100, linux@treblig.org wrote:
> > From: "Dr. David Alan Gilbert" <linux@treblig.org>
> > 
> > commit 6a1688ae8794 ("drm/bridge: ptn3460: Convert to I2C driver model")
> > has dropped all the users of the struct bridge_init from the
> > exynos_dp_core, while retaining unused structure definition.
> > Later on the driver was reworked and the definition migrated
> > to the analogix_dp driver. Remove unused struct bridge_init definition.
> > 
> > Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
> > ---
> >  drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 5 -----
> >  1 file changed, 5 deletions(-)
> > 
> 
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

Thanks.

Dave

> 
> -- 
> With best wishes
> Dmitry
>
Dr. David Alan Gilbert July 18, 2024, 5:49 p.m. UTC | #3
* Dmitry Baryshkov (dmitry.baryshkov@linaro.org) wrote:
> On Mon, May 20, 2024 at 01:55:51PM +0100, linux@treblig.org wrote:
> > From: "Dr. David Alan Gilbert" <linux@treblig.org>
> > 
> > commit 6a1688ae8794 ("drm/bridge: ptn3460: Convert to I2C driver model")
> > has dropped all the users of the struct bridge_init from the
> > exynos_dp_core, while retaining unused structure definition.
> > Later on the driver was reworked and the definition migrated
> > to the analogix_dp driver. Remove unused struct bridge_init definition.
> > 
> > Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
> > ---
> >  drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 5 -----
> >  1 file changed, 5 deletions(-)
> > 
> 
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

Hi Dmitry,
  Do you know who is likely to pick this one up?  I think all
my other drm patches have found their way into -next.

Thanks,

Dave

> 
> -- 
> With best wishes
> Dmitry
>
Dmitry Baryshkov July 18, 2024, 9:47 p.m. UTC | #4
On Thu, Jul 18, 2024 at 05:49:22PM GMT, Dr. David Alan Gilbert wrote:
> * Dmitry Baryshkov (dmitry.baryshkov@linaro.org) wrote:
> > On Mon, May 20, 2024 at 01:55:51PM +0100, linux@treblig.org wrote:
> > > From: "Dr. David Alan Gilbert" <linux@treblig.org>
> > > 
> > > commit 6a1688ae8794 ("drm/bridge: ptn3460: Convert to I2C driver model")
> > > has dropped all the users of the struct bridge_init from the
> > > exynos_dp_core, while retaining unused structure definition.
> > > Later on the driver was reworked and the definition migrated
> > > to the analogix_dp driver. Remove unused struct bridge_init definition.
> > > 
> > > Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
> > > ---
> > >  drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 5 -----
> > >  1 file changed, 5 deletions(-)
> > > 
> > 
> > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> 
> Hi Dmitry,
>   Do you know who is likely to pick this one up?  I think all
> my other drm patches have found their way into -next.

Applied and pushed.
Dr. David Alan Gilbert July 19, 2024, 12:28 a.m. UTC | #5
* Dmitry Baryshkov (dmitry.baryshkov@linaro.org) wrote:
> On Thu, Jul 18, 2024 at 05:49:22PM GMT, Dr. David Alan Gilbert wrote:
> > * Dmitry Baryshkov (dmitry.baryshkov@linaro.org) wrote:
> > > On Mon, May 20, 2024 at 01:55:51PM +0100, linux@treblig.org wrote:
> > > > From: "Dr. David Alan Gilbert" <linux@treblig.org>
> > > > 
> > > > commit 6a1688ae8794 ("drm/bridge: ptn3460: Convert to I2C driver model")
> > > > has dropped all the users of the struct bridge_init from the
> > > > exynos_dp_core, while retaining unused structure definition.
> > > > Later on the driver was reworked and the definition migrated
> > > > to the analogix_dp driver. Remove unused struct bridge_init definition.
> > > > 
> > > > Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
> > > > ---
> > > >  drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 5 -----
> > > >  1 file changed, 5 deletions(-)
> > > > 
> > > 
> > > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> > 
> > Hi Dmitry,
> >   Do you know who is likely to pick this one up?  I think all
> > my other drm patches have found their way into -next.
> 
> Applied and pushed.

Thanks!

Dave

> -- 
> With best wishes
> Dmitry
diff mbox series

Patch

diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c
index df9370e0ff23..1e03f3525a92 100644
--- a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c
+++ b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c
@@ -36,11 +36,6 @@ 
 
 static const bool verify_fast_training;
 
-struct bridge_init {
-	struct i2c_client *client;
-	struct device_node *node;
-};
-
 static int analogix_dp_init_dp(struct analogix_dp_device *dp)
 {
 	int ret;