Re: [PATCH v2 04/10] qla2xxx: Reset reserved field in firmware options to 0.

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

To: Bart Van Assche <Bart.VanAssche@xxxxxxxxxxx>, "hch@xxxxxxxxxxxxx" <hch@xxxxxxxxxxxxx>, "target-devel@xxxxxxxxxxxxxxx" <target-devel@xxxxxxxxxxxxxxx>, "nab@xxxxxxxxxxxxxxx" <nab@xxxxxxxxxxxxxxx>
Subject: Re: [PATCH v2 04/10] qla2xxx: Reset reserved field in firmware options to 0.
From: "Madhani, Himanshu" <Himanshu.Madhani@xxxxxxxxxx>
Date: Sat, 24 Dec 2016 00:07:19 +0000
Accept-language: en-US
Cc: "linux-scsi@xxxxxxxxxxxxxxx" <linux-scsi@xxxxxxxxxxxxxxx>, "Malavali, Giridhar" <Giridhar.Malavali@xxxxxxxxxx>
In-reply-to: <1482482240.3510.5.camel@sandisk.com>
References: <1482357459-31079-1-git-send-email-himanshu.madhani@cavium.com> <1482357459-31079-5-git-send-email-himanshu.madhani@cavium.com> <1482482240.3510.5.camel@sandisk.com>

On 12/23/16, 12:37 AM, "Bart Van Assche" <Bart.VanAssche@xxxxxxxxxxx> wrote:

>Please use cpu_to_le32() in new code instead of __constant_cpu_to_le32().
>gcc generates the same code for both conversion functions but the former
>function makes source code easier to read.

Ack. Will update the patch.

>
��.n��������+%������w��{.n����j�����{ay�ʇڙ���f���h������_�(�階�ݢj"��������G����?���&��

References:
- [PATCH v2 00/10] qla2xxx: Bug fixes for driver.
  - From: Himanshu Madhani
- [PATCH v2 04/10] qla2xxx: Reset reserved field in firmware options to 0.
  - From: Himanshu Madhani
- Re: [PATCH v2 04/10] qla2xxx: Reset reserved field in firmware options to 0.
  - From: Bart Van Assche

Prev by Date: Re: [PATCH v2 06/10] qla2xxx: Fix crash due to null pointer access.
Next by Date: Re: [PATCH v2 02/10] qla2xxx: Include ATIO queue in firmware dump when in target mode
Previous by thread: Re: [PATCH v2 04/10] qla2xxx: Reset reserved field in firmware options to 0.
Next by thread: [PATCH v2 05/10] qla2xxx: Collect additional information to debug fw dump.
Index(es):
- Date
- Thread

[Index of Archives] [Linux SCSI] [Kernel Newbies] [Linux SCSI Target Infrastructure] [Share Photos] [IDE] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux ATA RAID] [Linux IIO] [Device Mapper]

Powered by Linux