Re: [PATCH 4/4] scsi: pm8001: fix pm8001_store_update_fw

[Tomas Henzl <thenzl@redhat.com>]

On 07/10/2014 03:30 PM, Tomas Henzl wrote:
> On 07/10/2014 08:43 AM, Christoph Hellwig wrote:
>> On Mon, Jul 07, 2014 at 05:20:01PM +0200, Tomas Henzl wrote:
>>> The current implementation may mix the negative value returned
>>> from pm8001_set_nvmd with with count. -(-ENOMEM) could be interpreted
>>> as bytes programmed, this patch fixes it.
>> This still doesn;t look correct to me as err mixes up the driver
>> internal FAIL_* codes with Linux error codes.  It seems like for the
>> FAIL_* codes should only go into ->fw_status and the return value
>> should be a proper Linux error code.
> And the fw_status might be later used to show error strings in pm8001_show_update_fw,
> if it is so it depends on the flash utility but it seems likely. 
>
>> Funny fact: the FAIL_* / FLASH_IN_PROGRESS codes seems to be the same
>> between aic94xx and pm8001.
> And similar story there too - asd_store_update_bios -...- asd_poll_flash(might return -ENOENT)
>
> Maybe the flash utility ignores the return value or it has never happened.
> ---------------------
>
> I'll try to find what seems to be the most probable way and post it in few days.

Christoph,
from few days it is three weeks and the patch has been just replaced
by "[PATCH 1/3] pm8001: fix pm8001_store_update_fw" sent by Suresh,
so you can drop this patch.
(I've asked Suresh off-list to test it, I didn't want to brick my card by testing
fw flashes.)
Thanks,
Tomas

> This patch is not related to the patches 1-3/4, so just wait with this one.
>
> Thanks, Tomas
>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html