Re: [PATCH 4/4] libata: clean up the SFF code for coding style

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

 



Alan Cox <alan@xxxxxxxxxxxxxxxxxxx> wrote:
> From: Alan Cox <alan@xxxxxxxxxx>
>
> Signed-off-by: Alan Cox <alan@xxxxxxxxxx>
> ---
>
>  drivers/ata/libata-sff.c |  136 +++++++++++++++++++++++-----------------------
>  1 files changed, 69 insertions(+), 67 deletions(-)
>
>
> diff --git a/drivers/ata/libata-sff.c b/drivers/ata/libata-sff.c
> index 1afaa96..03c2d41 100644
> --- a/drivers/ata/libata-sff.c
> +++ b/drivers/ata/libata-sff.c
[...]
> @@ -1016,18 +1036,19 @@ static void atapi_pio_bytes(struct ata_queued_cmd *qc)
>   *	RETURNS:
>   *	1 if ok in workqueue, 0 otherwise.
>   */
> -static inline int ata_hsm_ok_in_wq(struct ata_port *ap, struct ata_queued_cmd *qc)
> +static inline int ata_hsm_ok_in_wq(struct ata_port *ap,
> +						struct ata_queued_cmd *qc)
>  {
>  	if (qc->tf.flags & ATA_TFLAG_POLLING)
>  		return 1;
>  
>  	if (ap->hsm_task_state == HSM_ST_FIRST) {
>  		if (qc->tf.protocol == ATA_PROT_PIO &&
> -		    (qc->tf.flags & ATA_TFLAG_WRITE))
> +		   (qc->tf.flags & ATA_TFLAG_WRITE))
>  		    return 1;

Sorry, I don't understand that at all. In my opinion, the nesting of
parentheses is more obvious and easier to understand without that
change. Moreover, the indentation of the return statement would have
needed fixing much more desperately ;-).

>  
>  		if (ata_is_atapi(qc->tf.protocol) &&
> -		    !(qc->dev->flags & ATA_DFLAG_CDB_INTR))
> +		   !(qc->dev->flags & ATA_DFLAG_CDB_INTR))

Ditto.

[...]
> @@ -2711,17 +2760,18 @@ int ata_pci_sff_prepare_host(struct pci_dev *pdev,
>  	*r_host = host;
>  	return 0;
>  
> - err_bmdma:
> +err_bmdma:
>  	/* This is necessary because PCI and iomap resources are
>  	 * merged and releasing the top group won't release the
>  	 * acquired resources if some of those have been acquired
>  	 * before entering this function.
>  	 */
>  	pcim_iounmap_regions(pdev, 0xf);
> - err_out:
> +err_out:

Yes, I've wondered about these myself occasionally. Personally, I don't
insert a blank before those labels either and would be in favour of such
a change. But do we actually have a convention regarding this matter?

Regards,

Elias
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[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