[PATCH] scsi: ufs: rpmb: register and unregister the ufs_rpmb bus

[PATCH] scsi: ufs: rpmb: register and unregister the ufs_rpmb bus

[Dmitry Baryshkov]

The ufs_rpmb bus isn't registered in the device framework, so any
attempt to register RPMB devices on UFS will fail with:

 bus_add_device: cannot add device 'ufs_rpmb0' to unregistered bus 'ufs_rpmb'
 ufshcd-qcom 1d84000.ufshc: Failed to register UFS RPMB device 0

Add module_init() / exit() functions to handle bus registration and
unregistration.

Fixes: b06b8c421485 ("scsi: ufs: core: Add OP-TEE based RPMB driver for UFS devices")
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
---
 drivers/ufs/core/ufs-rpmb.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/drivers/ufs/core/ufs-rpmb.c b/drivers/ufs/core/ufs-rpmb.c
index 50ef63afd528..8ae917adbc98 100644
--- a/drivers/ufs/core/ufs-rpmb.c
+++ b/drivers/ufs/core/ufs-rpmb.c
@@ -275,5 +275,18 @@ void ufs_rpmb_remove(struct ufs_hba *hba)
 	dev_info(hba->dev, "All UFS RPMB devices unregistered\n");
 }
 
+static int __init ufs_rpmb_init(void)
+{
+	return bus_register(&ufs_rpmb_bus_type);
+}
+
+static void __exit ufs_rpmb_exit(void)
+{
+	bus_unregister(&ufs_rpmb_bus_type);
+}
+
+module_init(ufs_rpmb_init);
+module_exit(ufs_rpmb_exit);
+
 MODULE_LICENSE("GPL v2");
 MODULE_DESCRIPTION("OP-TEE UFS RPMB driver");

---
base-commit: 3f2425f5b5bbbdd991ca9cdfd5502e68d8895998
change-id: 20260511-fix-ufs-rpmb-487d73d091b7

Best regards,
--  
With best wishes
Dmitry

Re: [PATCH] scsi: ufs: rpmb: register and unregister the ufs_rpmb bus

[Stanley Jhu]

On Sun, 20 Sep 2026 17:50:18 +0300, Dmitry Baryshkov wrote:
> The ufs_rpmb bus isn't registered in the device framework, so any
> attempt to register RPMB devices on UFS will fail with:
>
>  bus_add_device: cannot add device 'ufs_rpmb0' to unregistered bus 'ufs_rpmb'
>  ufshcd-qcom 1d84000.ufshc: Failed to register UFS RPMB device 0
>
> Add module_init() / exit() functions to handle bus registration and
> unregistration.

We have an existing patch on the list addressing this from the opposite
direction (`[PATCH v5 3/3]`, Reviewed-by Bean Huo) [1]:

Since ufs_rpmb_bus_type declares no .match and no .probe and consumers
access RPMB devices via /sys/class/rpmb/, patch 3/3 drops
ufs_rpmb_bus_type (leaving ufs_rpmb->dev.bus NULL) rather than
registering an empty bus.

Could you take a look at [1] and let us know which approach you prefer?

[1] https://lore.kernel.org/linux-scsi/20260914144943.2571104-1-stanleyjhu@google.com/

Re: [PATCH] scsi: ufs: rpmb: register and unregister the ufs_rpmb bus

[kernel test robot]

Hi Dmitry,

kernel test robot noticed the following build errors:

[auto build test ERROR on 3f2425f5b5bbbdd991ca9cdfd5502e68d8895998]

url:    https://github.com/intel-lab-lkp/linux/commits/Dmitry-Baryshkov/scsi-ufs-rpmb-register-and-unregister-the-ufs_rpmb-bus/20260920-175018
base:   3f2425f5b5bbbdd991ca9cdfd5502e68d8895998
patch link:    https://lore.kernel.org/r/20260920-fix-ufs-rpmb-v1-1-560b842c589d%40oss.qualcomm.com
patch subject: [PATCH] scsi: ufs: rpmb: register and unregister the ufs_rpmb bus
config: sparc64-randconfig-1001-20260926 (https://download.01.org/0day-ci/archive/20260926/202609260636.G8CEOXSF-lkp@intel.com/config)
compiler: sparc64-linux-gcc (GCC) 9.5.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260926/202609260636.G8CEOXSF-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202609260636.G8CEOXSF-lkp@intel.com/

All errors (new ones prefixed by >>):

   sparc64-linux-ld: drivers/ufs/core/ufs-rpmb.o: in function `ufs_rpmb_init':
>> ufs-rpmb.c:(.init.text+0x0): multiple definition of `init_module'; drivers/ufs/core/ufshcd.o:ufshcd.c:(.init.text+0x0): first defined here
   sparc64-linux-ld: drivers/ufs/core/ufs-rpmb.o: in function `ufs_rpmb_exit':
>> ufs-rpmb.c:(.exit.text+0x0): multiple definition of `cleanup_module'; drivers/ufs/core/ufshcd.o:ufshcd.c:(.exit.text+0x0): first defined here

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

Re: [PATCH] scsi: ufs: rpmb: register and unregister the ufs_rpmb bus

[kernel test robot]

Hi Dmitry,

kernel test robot noticed the following build errors:

[auto build test ERROR on 3f2425f5b5bbbdd991ca9cdfd5502e68d8895998]

url:    https://github.com/intel-lab-lkp/linux/commits/Dmitry-Baryshkov/scsi-ufs-rpmb-register-and-unregister-the-ufs_rpmb-bus/20260920-175018
base:   3f2425f5b5bbbdd991ca9cdfd5502e68d8895998
patch link:    https://lore.kernel.org/r/20260920-fix-ufs-rpmb-v1-1-560b842c589d%40oss.qualcomm.com
patch subject: [PATCH] scsi: ufs: rpmb: register and unregister the ufs_rpmb bus
config: arc-randconfig-1001-20260926 (https://download.01.org/0day-ci/archive/20260926/202609260913.FY1vuM99-lkp@intel.com/config)
compiler: arc-linux-gcc (GCC) 12.5.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260926/202609260913.FY1vuM99-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202609260913.FY1vuM99-lkp@intel.com/

All errors (new ones prefixed by >>):

   arc-linux-ld: drivers/ufs/core/ufs-rpmb.o: in function `ufs_rpmb_init':
>> drivers/ufs/core/ufs-rpmb.c:280: multiple definition of `init_module'; drivers/ufs/core/ufshcd.o:drivers/ufs/core/ufshcd.c:11609: first defined here
   arc-linux-ld: drivers/ufs/core/ufs-rpmb.o: in function `ufs_rpmb_exit':
>> drivers/ufs/core/ufs-rpmb.c:285: multiple definition of `cleanup_module'; drivers/ufs/core/ufshcd.o:drivers/ufs/core/ufshcd.c:11623: first defined here


vim +280 drivers/ufs/core/ufs-rpmb.c

   277	
   278	static int __init ufs_rpmb_init(void)
   279	{
 > 280		return bus_register(&ufs_rpmb_bus_type);
   281	}
   282	
   283	static void __exit ufs_rpmb_exit(void)
   284	{
 > 285		bus_unregister(&ufs_rpmb_bus_type);
   286	}
   287	

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki