staging: most: dim2: Replace function dim_norm_ctrl_async_buffer_size()
authorNishka Dasgupta <nishkadg.linux@gmail.com>
Mon, 8 Jul 2019 06:41:43 +0000 (12:11 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 22 Jul 2019 05:34:10 +0000 (07:34 +0200)
Remove function dim_norm_ctrl_async_buffer_size as it does nothing
except call norm_ctrl_async_buffer_size.
Rename norm_ctrl_async_buffer_size to dim_norm_ctrl_async_buffer_size to
maintain compatibility with call sites of the latter.
Change type of new dim_norm_ctrl_async_buffer_size from static inline to
non-static to match the old version.
Modify only remaining call site of norm_ctrl_async_buffer_size to call
dim_norm_ctrl_async_buffer_size instead.
Issue found with Coccinelle.

Signed-off-by: Nishka Dasgupta <nishkadg.linux@gmail.com>
Link: https://lore.kernel.org/r/20190708064145.3250-1-nishkadg.linux@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/most/dim2/hal.c

index 699e02f83bd41bad4419e59e021eb7e597e3275b..d4d532e761474ccb51ee8b1940464a6c91b1efdc 100644 (file)
@@ -471,7 +471,7 @@ static inline bool check_bytes_per_frame(u32 bytes_per_frame)
        return true;
 }
 
-static inline u16 norm_ctrl_async_buffer_size(u16 buf_size)
+u16 dim_norm_ctrl_async_buffer_size(u16 buf_size)
 {
        u16 const max_size = (u16)ADT1_CTRL_ASYNC_BD_MASK + 1u;
 
@@ -652,7 +652,7 @@ static bool channel_start(struct dim_channel *ch, u32 buf_addr, u16 buf_size)
                return dim_on_error(DIM_ERR_BAD_BUFFER_SIZE, "Bad buffer size");
 
        if (ch->packet_length == 0 && ch->bytes_per_frame == 0 &&
-           buf_size != norm_ctrl_async_buffer_size(buf_size))
+           buf_size != dim_norm_ctrl_async_buffer_size(buf_size))
                return dim_on_error(DIM_ERR_BAD_BUFFER_SIZE,
                                    "Bad control/async buffer size");
 
@@ -780,11 +780,6 @@ void dim_service_mlb_int_irq(void)
        dimcb_io_write(&g.dim2->MS1, 0);
 }
 
-u16 dim_norm_ctrl_async_buffer_size(u16 buf_size)
-{
-       return norm_ctrl_async_buffer_size(buf_size);
-}
-
 /**
  * Retrieves maximal possible correct buffer size for isochronous data type
  * conform to given packet length and not bigger than given buffer size.