Re: [PATCH] misc: ocxl: context: Remove unnecessary (void*) conversions

2023-12-07 Thread Michael Ellerman 
On Mon, 13 Nov 2023 09:15:43 +0800, Li zeming wrote:
> The ctx pointer does not need to cast the type.
> 
> 

Applied to powerpc/next.

[1/1] misc: ocxl: context: Remove unnecessary (void*) conversions
  https://git.kernel.org/powerpc/c/82d30723d58fccbd2d7d707fab7649b541fafa1b

cheers

Re: [PATCH] misc: ocxl: context: Remove unnecessary (void*) conversions

2023-11-19 Thread Andrew Donnellan 
On Mon, 2023-11-13 at 09:15 +0800, Li zeming wrote:
> The ctx pointer does not need to cast the type.
> 
> Signed-off-by: Li zeming 

Acked-by: Andrew Donnellan 

> ---
>  drivers/misc/ocxl/context.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/misc/ocxl/context.c
> b/drivers/misc/ocxl/context.c
> index 7f83116ae11a6..cded7d1caf328 100644
> --- a/drivers/misc/ocxl/context.c
> +++ b/drivers/misc/ocxl/context.c
> @@ -55,7 +55,7 @@ EXPORT_SYMBOL_GPL(ocxl_context_alloc);
>   */
>  static void xsl_fault_error(void *data, u64 addr, u64 dsisr)
>  {
> - struct ocxl_context *ctx = (struct ocxl_context *) data;
> + struct ocxl_context *ctx = data;
>  
>   mutex_lock(>xsl_error_lock);
>   ctx->xsl_error.addr = addr;

-- 
Andrew DonnellanOzLabs, ADL Canberra
a...@linux.ibm.com   IBM Australia Limited

Re: [PATCH] misc: ocxl: context: Remove unnecessary (void*) conversions

2023-11-14 Thread Frederic Barrat 




On 13/11/2023 02:15, Li zeming wrote:

The ctx pointer does not need to cast the type.

Signed-off-by: Li zeming 
---



Indeed, it's useless.
Acked-by: Frederic Barrat 



  drivers/misc/ocxl/context.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/misc/ocxl/context.c b/drivers/misc/ocxl/context.c
index 7f83116ae11a6..cded7d1caf328 100644
--- a/drivers/misc/ocxl/context.c
+++ b/drivers/misc/ocxl/context.c
@@ -55,7 +55,7 @@ EXPORT_SYMBOL_GPL(ocxl_context_alloc);
   */
  static void xsl_fault_error(void *data, u64 addr, u64 dsisr)
  {
-   struct ocxl_context *ctx = (struct ocxl_context *) data;
+   struct ocxl_context *ctx = data;
  
  	mutex_lock(>xsl_error_lock);

ctx->xsl_error.addr = addr;

[PATCH] misc: ocxl: context: Remove unnecessary (void*) conversions

2023-11-12 Thread Li zeming 
The ctx pointer does not need to cast the type.

Signed-off-by: Li zeming 
---
 drivers/misc/ocxl/context.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/misc/ocxl/context.c b/drivers/misc/ocxl/context.c
index 7f83116ae11a6..cded7d1caf328 100644
--- a/drivers/misc/ocxl/context.c
+++ b/drivers/misc/ocxl/context.c
@@ -55,7 +55,7 @@ EXPORT_SYMBOL_GPL(ocxl_context_alloc);
  */
 static void xsl_fault_error(void *data, u64 addr, u64 dsisr)
 {
-   struct ocxl_context *ctx = (struct ocxl_context *) data;
+   struct ocxl_context *ctx = data;
 
mutex_lock(>xsl_error_lock);
ctx->xsl_error.addr = addr;
-- 
2.18.2