Re: [PATCH] ocxl: fix driver function comment typo

2023-12-06 Thread jiangyunshui
> Date: Tue, 5 Dec 2023 11:00:16 +  [thread overview]
> Message-ID: <2f2aca95-f5a6-45fa-9e3b-37aecf657...@csgroup.eu> (raw)
> In-Reply-To: <20231205094319.32114-1-jiangyuns...@kylinos.cn>

> Please add a description explaining why you want to do that change.

> When I grep I see cxlflash driver, I don't know what ocxlflash driver is:

> $ git grep ocxl_config_read_afu
> ...
> drivers/scsi/cxlflash/ocxl_hw.c:rc = ocxl_config_read_afu(pdev, 
> fcfg, acfg, 0);
> drivers/scsi/cxlflash/ocxl_hw.c:dev_err(dev, "%s: 
> ocxl_config_read_afu failed rc=%d\n",
> include/misc/ocxl.h:int ocxl_config_read_afu(struct pci_dev *dev,

> Christophe

I checked my commit again and found I'd mismatched the driver's name with other 
things.
As you said it doesn't make sense to change cxlflash to ocxlflash.
Sorry for the inconvenience, you could just drop my commit please.
Thanks,

Yunshui

> ---
> Reported-by: k2ci 
> Signed-off-by: yunshui 
> ---
>   include/misc/ocxl.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/misc/ocxl.h b/include/misc/ocxl.h
> index 3ed736da02c8..ef7d26009a36 100644
> --- a/include/misc/ocxl.h
> +++ b/include/misc/ocxl.h
> @@ -324,7 +324,7 @@ int ocxl_global_mmio_clear32(struct ocxl_afu *afu, size_t 
> offset,
>   int ocxl_global_mmio_clear64(struct ocxl_afu *afu, size_t offset,
>   enum ocxl_endian endian, u64 mask);
>
> -// Functions left here are for compatibility with the cxlflash driver
> +// Functions left here are for compatibility with the ocxlflash driver
>
>   /*
>* Read the configuration space of a function for the AFU specified by
> --
> 2.25.1
>


[PATCH] ocxl: fix driver function comment typo

2023-12-05 Thread jiangyunshui
Reported-by: k2ci 
Signed-off-by: yunshui 
---
 include/misc/ocxl.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/misc/ocxl.h b/include/misc/ocxl.h
index 3ed736da02c8..ef7d26009a36 100644
--- a/include/misc/ocxl.h
+++ b/include/misc/ocxl.h
@@ -324,7 +324,7 @@ int ocxl_global_mmio_clear32(struct ocxl_afu *afu, size_t 
offset,
 int ocxl_global_mmio_clear64(struct ocxl_afu *afu, size_t offset,
 enum ocxl_endian endian, u64 mask);
 
-// Functions left here are for compatibility with the cxlflash driver
+// Functions left here are for compatibility with the ocxlflash driver
 
 /*
  * Read the configuration space of a function for the AFU specified by
-- 
2.25.1