On Thu, Jun 11, 2026 at 10:49:36AM +0300, Raz Ben Yehuda wrote: > > Remove the static qualifier from selected functions to allow > future reuse. Add flags according to UFS 3.1. > > Signed-off-by: Raz Ben Yehuda <[email protected]> > --- > drivers/ufs/ufs-uclass.c | 18 +- > drivers/ufs/ufs.h | 52 ---- > include/ufs.h | 603 > ++++++++++++++++++++++++++++++++++++++++++++++- > 3 files changed, 610 insertions(+), 63 deletions(-) > > diff --git a/drivers/ufs/ufs-uclass.c b/drivers/ufs/ufs-uclass.c > index 6a51f337e47..78f6064ddea 100644 > --- a/drivers/ufs/ufs-uclass.c > +++ b/drivers/ufs/ufs-uclass.c > @@ -757,7 +757,7 @@ static inline u8 ufshcd_get_upmcrs(struct ufs_hba *hba) > * > * Flush cache in aligned address..address+size range. > */ > -static void ufshcd_cache_flush(void *addr, unsigned long size) > +void ufshcd_cache_flush(void *addr, unsigned long size) > { > uintptr_t start_addr = (uintptr_t)addr & ~(ARCH_DMA_MINALIGN - 1); > uintptr_t end_addr = ALIGN((uintptr_t)addr + size, ARCH_DMA_MINALIGN);
We should be removing the comments here as well, when making these functions public, so that we don't end up with conflicting descriptions in the future. [snip] > diff --git a/include/ufs.h b/include/ufs.h > index f6e27d90e43..b8af475661a 100644 > --- a/include/ufs.h > +++ b/include/ufs.h > @@ -1,6 +1,64 @@ > /* SPDX-License-Identifier: GPL-2.0+ */ > -#ifndef _UFS_H > -#define _UFS_H > +#ifndef _UFS_CMD_H > +#define _UFS_CMD_H Why did you do that? -- Tom
signature.asc
Description: PGP signature
