Re: [PATCH] hpsa: add missing pci_set_master in kdump path

[Tomas Henzl <thenzl@redhat.com>]

On 09/25/2014 02:20 PM, Christoph Hellwig wrote:
> Tomas, can you also give me a signed-off-by for this one?

Oh, I forgot to add it before.

Patch is 
Signed-off-by: Tomas Henzl <thenzl@redhat.com>

>
> On Fri, Sep 12, 2014 at 02:44:15PM +0200, Tomas Henzl wrote:
>> Add a call to pci_set_master(...)  missing in the previous
>> patch "hpsa: refine the pci enable/disable handling".
>> Found thanks to Rob Elliot.
>> ---
>>  drivers/scsi/hpsa.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
>> index 7828834..cef5d49 100644
>> --- a/drivers/scsi/hpsa.c
>> +++ b/drivers/scsi/hpsa.c
>> @@ -6544,7 +6544,7 @@ static int hpsa_init_reset_devices(struct pci_dev *pdev)
>>  		dev_warn(&pdev->dev, "failed to enable device.\n");
>>  		return -ENODEV;
>>  	}
>> -
>> +	pci_set_master(pdev);
>>  	/* Reset the controller with a PCI power-cycle or via doorbell */
>>  	rc = hpsa_kdump_hard_reset_controller(pdev);
>>  
>> -- 
>> 1.8.3.1
>>
>>
>> --
>> 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
> ---end quoted text---
> --
> 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