[PATCH] drivers: ufs: remove unused ufs_post_bind() declaration

Neil Armstrong neil.armstrong at linaro.org
Tue Feb 3 10:34:24 CET 2026


On 2/3/26 10:21, Julien Stephan wrote:
> Commit 067c1b033282 ("ufs: Call ufs_scsi_bind() from uclass .post_bind")
> inlined ufs_scsi_bind() into ufs_post_bind() as trivial
> device_bind_driver() call.
> 
> ufs_scsi_bind() is no longer referenced anywhere in the codebase, so
> drop its declaration from include/ufs.h.
> 
> Drivers used to include <ufs.h> to include prototype of ufs_scsi_bind()
> function, so we can now safely remove such includes.
> 
> Fixes: 067c1b033282 ("ufs: Call ufs_scsi_bind() from uclass .post_bind")
> Signed-off-by: Julien Stephan <jstephan at baylibre.com>
> ---
>   drivers/ufs/cdns-platform.c         | 1 -
>   drivers/ufs/ufs-amd-versal2.c       | 1 -
>   drivers/ufs/ufs-pci.c               | 1 -
>   drivers/ufs/ufs-qcom.c              | 1 -
>   drivers/ufs/ufs-renesas-rcar-gen5.c | 1 -
>   drivers/ufs/ufs-renesas.c           | 1 -
>   drivers/ufs/ufs-rockchip.c          | 1 -
>   drivers/ufs/ufshcd-dwc.c            | 1 -
>   include/ufs.h                       | 9 ---------
>   9 files changed, 17 deletions(-)
> 
> diff --git a/drivers/ufs/cdns-platform.c b/drivers/ufs/cdns-platform.c
> index 87d9c5bad79..40883a187b0 100644
> --- a/drivers/ufs/cdns-platform.c
> +++ b/drivers/ufs/cdns-platform.c
> @@ -7,7 +7,6 @@
>   
>   #include <clk.h>
>   #include <dm.h>
> -#include <ufs.h>
>   #include <asm/io.h>
>   #include <dm/device_compat.h>
>   #include <linux/bitops.h>
> diff --git a/drivers/ufs/ufs-amd-versal2.c b/drivers/ufs/ufs-amd-versal2.c
> index dd62c9819ba..6c949b2ca76 100644
> --- a/drivers/ufs/ufs-amd-versal2.c
> +++ b/drivers/ufs/ufs-amd-versal2.c
> @@ -5,7 +5,6 @@
>   
>   #include <clk.h>
>   #include <dm.h>
> -#include <ufs.h>
>   #include <asm/io.h>
>   #include <dm/device_compat.h>
>   #include <zynqmp_firmware.h>
> diff --git a/drivers/ufs/ufs-pci.c b/drivers/ufs/ufs-pci.c
> index 5b9c72a695d..e1e010d027c 100644
> --- a/drivers/ufs/ufs-pci.c
> +++ b/drivers/ufs/ufs-pci.c
> @@ -7,7 +7,6 @@
>   #include <dm.h>
>   #include <errno.h>
>   #include <pci.h>
> -#include <ufs.h>
>   #include <dm/device_compat.h>
>   #include "ufs.h"
>   
> diff --git a/drivers/ufs/ufs-qcom.c b/drivers/ufs/ufs-qcom.c
> index ee43958d5d8..dc40ee62daf 100644
> --- a/drivers/ufs/ufs-qcom.c
> +++ b/drivers/ufs/ufs-qcom.c
> @@ -14,7 +14,6 @@
>   #include <dm.h>
>   #include <dm/device_compat.h>
>   #include <generic-phy.h>
> -#include <ufs.h>
>   #include <asm/gpio.h>
>   #include <interconnect.h>
>   
> diff --git a/drivers/ufs/ufs-renesas-rcar-gen5.c b/drivers/ufs/ufs-renesas-rcar-gen5.c
> index cc53e91449c..a21ae3f390e 100644
> --- a/drivers/ufs/ufs-renesas-rcar-gen5.c
> +++ b/drivers/ufs/ufs-renesas-rcar-gen5.c
> @@ -7,7 +7,6 @@
>   
>   #include <clk.h>
>   #include <dm.h>
> -#include <ufs.h>
>   #include <asm/io.h>
>   #include <dm/device_compat.h>
>   #include <linux/iopoll.h>
> diff --git a/drivers/ufs/ufs-renesas.c b/drivers/ufs/ufs-renesas.c
> index 5652309911e..a206e3c6d58 100644
> --- a/drivers/ufs/ufs-renesas.c
> +++ b/drivers/ufs/ufs-renesas.c
> @@ -7,7 +7,6 @@
>   
>   #include <clk.h>
>   #include <dm.h>
> -#include <ufs.h>
>   #include <asm/io.h>
>   #include <dm/device_compat.h>
>   #include <linux/bitops.h>
> diff --git a/drivers/ufs/ufs-rockchip.c b/drivers/ufs/ufs-rockchip.c
> index 0384244387d..643a6ffb9bc 100644
> --- a/drivers/ufs/ufs-rockchip.c
> +++ b/drivers/ufs/ufs-rockchip.c
> @@ -13,7 +13,6 @@
>   #include <linux/err.h>
>   #include <linux/ioport.h>
>   #include <reset.h>
> -#include <ufs.h>
>   
>   #include "ufs.h"
>   #include "unipro.h"
> diff --git a/drivers/ufs/ufshcd-dwc.c b/drivers/ufs/ufshcd-dwc.c
> index 3f62e59a060..98422d742a0 100644
> --- a/drivers/ufs/ufshcd-dwc.c
> +++ b/drivers/ufs/ufshcd-dwc.c
> @@ -7,7 +7,6 @@
>    */
>   #include <clk.h>
>   #include <dm.h>
> -#include <ufs.h>
>   #include <asm/io.h>
>   #include <dm/device_compat.h>
>   #include <linux/bitops.h>
> diff --git a/include/ufs.h b/include/ufs.h
> index 702b8359dbd..f6e27d90e43 100644
> --- a/include/ufs.h
> +++ b/include/ufs.h
> @@ -20,13 +20,4 @@ int ufs_probe(void);
>    */
>   int ufs_probe_dev(int index);
>   
> -/*
> - * ufs_scsi_bind() - Create a new scsi device as a child of the UFS device and
> - *		     bind it to the ufs_scsi driver
> - * @ufs_dev: UFS device
> - * @scsi_devp: Pointer to scsi device
> - *
> - * Return: 0 if Ok, -ve on error
> - */
> -int ufs_scsi_bind(struct udevice *ufs_dev, struct udevice **scsi_devp);
>   #endif
> 
> ---
> base-commit: eed514b11d04a2f8a949521ad3bffba3ec98bd2f
> change-id: 20260203-cleanup-ufs-header-e385e21fac1c
> 
> Best regards,

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

Thanks,
Neil


More information about the U-Boot mailing list