lkml.org 
[lkml]   [2018]   [Jun]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 3/3] message: fusion: Fix line over 80 characters
From
Date
On Sun, 2018-06-03 at 18:52 +0530, Varsha Rao wrote:
> Line over 80 characters is split to fix the following checkpatch issue:
>
> WARNING: line over 80 characters
[]
> diff --git a/drivers/message/fusion/mptbase.c b/drivers/message/fusion/mptbase.c
[]
> @@ -5690,7 +5690,8 @@ mpt_inactive_raid_volumes(MPT_ADAPTER *ioc, u8 channel, u8 id)
> buffer->PhysDisk[i].PhysDiskNum, &phys_disk) != 0)
> continue;
>
> - if (!(component_info = kmalloc(sizeof(*component_info), GFP_KERNEL)))
> + if (!(component_info = kmalloc(sizeof(*component_info),
> + GFP_KERNEL)))
> continue;

It's much more common to split this into multiple statements like

component_info = kmalloc(sizeof(*component_info), GFP_KERNEL);
if (!component_info)
continue;

but the continue in a loop with a known alloc failure
seems silly as the next alloc is also extremely likely
to fail as well.

btw: Are you actually using this code?

If not, perhaps it's better to send cleanup only patches
for files in drivers/staging as practice instead.

\
 
 \ /
  Last update: 2018-06-03 22:03    [W:0.033 / U:0.880 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site