Re: [PATCH v4 0/3] Use *switch*es instead of *if*s in the Artop PATA driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2/6/22 05:34, Sergey Shtylyov wrote:
> Here are 3 patches against the 'for-next' branch of Damien Le Moal's
> 'libata.git' repo. The driver abuses the strings of the *if* statements
> where a *switch* statements would fit better...
> 
> Sergey Shtylyov (3):
>   pata_artop: eliminate local variable in artop_init_one()
>   pata_artop: use *switch* in artop_init_one()
>   pata_artop: use *switch* in atp8xx_fixup()

Please change the patch title prefix to "ata: <drv>: xxx".
So "ata: pata_artop: xxx" in this case.

> 
>  drivers/ata/pata_artop.c | 28 +++++++++++++++++-----------
>  1 file changed, 17 insertions(+), 11 deletions(-)
> 


-- 
Damien Le Moal
Western Digital Research



[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux