Re: [PATCH v2 0/4] PCI: qcom: Cleanups

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

 



Hi Bjorn,

On 10/12/2016 07:06 PM, Bjorn Helgaas wrote:
> On Wed, Oct 12, 2016 at 08:59:54AM -0500, Bjorn Helgaas wrote:
>>   - Remove platform drvdata because it appears unused (we called
>>     platform_set_drvdata() but not platform_get_drvdata()).
>>
>>   - Remove redundant struct members.
>>
>> Nothing here should change the behavior of the driver.
>>
>> Changes from v1:
>>   I dropped the following patch because it was a lot of churn for
>>   questionable benefit:
>>     PCI: qcom: Name private struct pointer "qcom" consistently
>>
>> ---
>>
>> Bjorn Helgaas (4):
>>       PCI: qcom: Remove unused platform data
>>       PCI: qcom: Remove redundant struct qcom_pcie.dbi
>>       PCI: qcom: Remove redundant struct qcom_pcie.dev
>>       PCI: qcom: Reorder struct qcom_pcie
>>
>>
>>  drivers/pci/host/pcie-qcom.c |   30 ++++++++++++------------------
>>  1 file changed, 12 insertions(+), 18 deletions(-)
> 
> I applied these to pci/host-qcom for v4.9.  I hope to ask Linus to
> pull them tomorrow, so if you see any issues, let me know soon.

Thanks for the clean up!

Reviewed-by: Stanimir Varbanov <svarbanov@xxxxxxxxxx>

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



[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux