[PATCH] drm/bridge: ti-sn65dsi86: Remove extra semicolon in ti_sn_bridge_probe()

Douglas Anderson posted 1 patch 2 months, 3 weeks ago
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] drm/bridge: ti-sn65dsi86: Remove extra semicolon in ti_sn_bridge_probe()
Posted by Douglas Anderson 2 months, 3 weeks ago
As report by the kernel test robot, a recent patch introduced an
unnecessary semicolon. Remove it.

Fixes: 55e8ff842051 ("drm/bridge: ti-sn65dsi86: Add HPD for DisplayPort connector type")
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202506301704.0SBj6ply-lkp@intel.com/
Signed-off-by: Douglas Anderson <dianders@chromium.org>
---

 drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.c b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
index de9c23537465..834b42a4d31f 100644
--- a/drivers/gpu/drm/bridge/ti-sn65dsi86.c
+++ b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
@@ -1373,7 +1373,7 @@ static int ti_sn_bridge_probe(struct auxiliary_device *adev,
 			regmap_update_bits(pdata->regmap, SN_HPD_DISABLE_REG,
 					   HPD_DISABLE, 0);
 		mutex_unlock(&pdata->comms_mutex);
-	};
+	}
 
 	drm_bridge_add(&pdata->bridge);
 
-- 
2.50.0.727.gbf7dc18ff4-goog
Re: [PATCH] drm/bridge: ti-sn65dsi86: Remove extra semicolon in ti_sn_bridge_probe()
Posted by Devarsh Thakkar 2 months, 2 weeks ago
On 15/07/25 01:36, Douglas Anderson wrote:
> As report by the kernel test robot, a recent patch introduced an
> unnecessary semicolon. Remove it.
> 
> Fixes: 55e8ff842051 ("drm/bridge: ti-sn65dsi86: Add HPD for DisplayPort connector type")
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202506301704.0SBj6ply-lkp@intel.com/
> Signed-off-by: Douglas Anderson <dianders@chromium.org>

Reviewed-by: Devarsh Thakkar <devarsht@ti.com>

Regards
Devarsh
> ---
> 
>   drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.c b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> index de9c23537465..834b42a4d31f 100644
> --- a/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> +++ b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> @@ -1373,7 +1373,7 @@ static int ti_sn_bridge_probe(struct auxiliary_device *adev,
>   			regmap_update_bits(pdata->regmap, SN_HPD_DISABLE_REG,
>   					   HPD_DISABLE, 0);
>   		mutex_unlock(&pdata->comms_mutex);
> -	};
> +	}
>   
>   	drm_bridge_add(&pdata->bridge);
>
Re: [PATCH] drm/bridge: ti-sn65dsi86: Remove extra semicolon in ti_sn_bridge_probe()
Posted by Doug Anderson 2 months, 2 weeks ago
Hi,

On Tue, Jul 22, 2025 at 6:53 AM Devarsh Thakkar <devarsht@ti.com> wrote:
>
> On 15/07/25 01:36, Douglas Anderson wrote:
> > As report by the kernel test robot, a recent patch introduced an
> > unnecessary semicolon. Remove it.
> >
> > Fixes: 55e8ff842051 ("drm/bridge: ti-sn65dsi86: Add HPD for DisplayPort connector type")
> > Reported-by: kernel test robot <lkp@intel.com>
> > Closes: https://lore.kernel.org/oe-kbuild-all/202506301704.0SBj6ply-lkp@intel.com/
> > Signed-off-by: Douglas Anderson <dianders@chromium.org>
>
> Reviewed-by: Devarsh Thakkar <devarsht@ti.com>
>
> Regards
> Devarsh

Thanks! Pushed to drm-misc-fixes:

[1/1] drm/bridge: ti-sn65dsi86: Remove extra semicolon in ti_sn_bridge_probe()
      commit: 15a7ca747d9538c2ad8b0c81dd4c1261e0736c82
Re: [PATCH] drm/bridge: ti-sn65dsi86: Remove extra semicolon in ti_sn_bridge_probe()
Posted by Doug Anderson 2 months, 2 weeks ago
Hi,

On Mon, Jul 14, 2025 at 1:07 PM Douglas Anderson <dianders@chromium.org> wrote:
>
> As report by the kernel test robot, a recent patch introduced an
> unnecessary semicolon. Remove it.
>
> Fixes: 55e8ff842051 ("drm/bridge: ti-sn65dsi86: Add HPD for DisplayPort connector type")
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202506301704.0SBj6ply-lkp@intel.com/
> Signed-off-by: Douglas Anderson <dianders@chromium.org>
> ---
>
>  drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

I can't land this without some sort of Reviewed-by tag. Anyone care to
do the honors? Jayesh, maybe?

-Doug