Re: [PATCH V5 12/12] pm80xx: thermal, sas controller config and error handling update

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

 



On Mon, 2013-05-06 at 09:36 -0700, Anand wrote:
> diff --git a/drivers/scsi/pm8001/pm80xx_hwi.h b/drivers/scsi/pm8001/pm80xx_hwi.h
> index 0c15627..ea127b6 100644
> --- a/drivers/scsi/pm8001/pm80xx_hwi.h
> +++ b/drivers/scsi/pm8001/pm80xx_hwi.h
[...]
> @@ -996,6 +1020,23 @@ struct ssp_coalesced_comp_resp {
>  
>  /* new outbound structure for spcv - ends */
>  
> +/* brief data structure for SAS protocol timer configuration page.
> + *
> + */
> +struct SASProtocolTimerConfig{
> +    __le32 pageCode;                        /* 0 */
> +    __le32 MST_MSI;                         /* 1 */
> +    __le32 STP_SSP_MCT_TMO;                 /* 2 */
> +    __le32 STP_FRM_TMO;                     /* 3 */
> +    __le32 STP_IDLE_TMO;                    /* 4 */
> +    __le32 OPNRJT_RTRY_INTVL;               /* 5 */
> +    __le32 Data_Cmd_OPNRJT_RTRY_TMO;        /* 6 */
> +    __le32 Data_Cmd_OPNRJT_RTRY_THR;        /* 7 */
> +    __le32 MAX_AIP;                         /* 8 */
> +} __attribute__((packed, aligned(4)));

This is wrongly aligned using spaces instead of tabs ... since the rest
of the file is OK, you cut and pasted this from somewhere.
checkpatch.pl warns about this stuff, please use it.

James


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




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux