On 06/01/2014 05:06 PM, Fabian Frederick wrote:
> __constant_cpu_to_be16 converted to cpu_to_be16
> 
> This patch fixes checkpatch warnings:
> 
> "WARNING: __constant_cpu_to_be16 should be cpu_to_be16"
> 
> Cc: Boaz Harrosh <bharr...@panasas.com>
> Cc: Benny Halevy <bhal...@primarydata.com>
> Signed-off-by: Fabian Frederick <f...@skynet.be>

Ack-by: Boaz Harrosh <bharr...@panasas.com>

James please apply
Thanks
Boaz

> ---
>  include/scsi/osd_protocol.h | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/include/scsi/osd_protocol.h b/include/scsi/osd_protocol.h
> index 25ac628..a2594af 100644
> --- a/include/scsi/osd_protocol.h
> +++ b/include/scsi/osd_protocol.h
> @@ -263,16 +263,16 @@ static inline struct osd_cdb_head *osd_cdb_head(struct 
> osd_cdb *ocdb)
>   * Ex name = FORMAT_OSD we have OSD_ACT_FORMAT_OSD && OSDv1_ACT_FORMAT_OSD
>   */
>  #define OSD_ACT___(Name, Num) \
> -     OSD_ACT_##Name = __constant_cpu_to_be16(0x8880 + Num), \
> -     OSDv1_ACT_##Name = __constant_cpu_to_be16(0x8800 + Num),
> +     OSD_ACT_##Name = cpu_to_be16(0x8880 + Num), \
> +     OSDv1_ACT_##Name = cpu_to_be16(0x8800 + Num),
>  
>  /* V2 only actions */
>  #define OSD_ACT_V2(Name, Num) \
> -     OSD_ACT_##Name = __constant_cpu_to_be16(0x8880 + Num),
> +     OSD_ACT_##Name = cpu_to_be16(0x8880 + Num),
>  
>  #define OSD_ACT_V1_V2(Name, Num1, Num2) \
> -     OSD_ACT_##Name = __constant_cpu_to_be16(Num2), \
> -     OSDv1_ACT_##Name = __constant_cpu_to_be16(Num1),
> +     OSD_ACT_##Name = cpu_to_be16(Num2), \
> +     OSDv1_ACT_##Name = cpu_to_be16(Num1),
>  
>  enum osd_service_actions {
>       OSD_ACT_V2(OBJECT_STRUCTURE_CHECK,      0x00)
> 

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

Reply via email to