2022-10-05 04:41:18

by Chaitanya Kulkarni

Subject: [PATCH 6/6] null_blk: remove extra space in switch condition

The extra space in after switch condition does not follow kernel coding
standards, remove extra space in switch condition end_cmd().

Signed-off-by: Chaitanya Kulkarni <[email protected]>
---
drivers/block/null_blk/main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/block/null_blk/main.c b/drivers/block/null_blk/main.c
index eda5050d6dee..e030f87d0208 100644
--- a/drivers/block/null_blk/main.c
+++ b/drivers/block/null_blk/main.c
@@ -789,7 +789,7 @@ static void end_cmd(struct nullb_cmd *cmd)
{
int queue_mode = cmd->nq->dev->queue_mode;

- switch (queue_mode) {
+ switch (queue_mode) {
case NULL_Q_MQ:
blk_mq_end_request(cmd->rq, cmd->error);
return;
--
2.29.0

2022-10-05 05:58:56

by Damien Le Moal

Subject: Re: [PATCH 6/6] null_blk: remove extra space in switch condition

On 10/5/22 12:17, Chaitanya Kulkarni wrote:
> The extra space in after switch condition does not follow kernel coding
> standards, remove extra space in switch condition end_cmd().
>
> Signed-off-by: Chaitanya Kulkarni <[email protected]>

Reviewed-by: Damien Le Moal <[email protected]>

> ---
> drivers/block/null_blk/main.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/block/null_blk/main.c b/drivers/block/null_blk/main.c
> index eda5050d6dee..e030f87d0208 100644
> --- a/drivers/block/null_blk/main.c
> +++ b/drivers/block/null_blk/main.c
> @@ -789,7 +789,7 @@ static void end_cmd(struct nullb_cmd *cmd)
> {
> int queue_mode = cmd->nq->dev->queue_mode;
>
> - switch (queue_mode) {
> + switch (queue_mode) {
> case NULL_Q_MQ:
> blk_mq_end_request(cmd->rq, cmd->error);
> return;

--
Damien Le Moal
Western Digital Research