Hello Bjorn!
On Thursday 23 April 2020 12:27:13 Bjorn Helgaas wrote:
> In the next round, please capitalize the first word of the subjects of
> the whole series to match:
Thank you for the review, I will fix subjects of all patches it in V3.
> $ git log --oneline drivers/pci/controller/pci-aardvark.c
> 4e5be6f81be7 ("PCI: aardvark: Use pci_parse_request_of_pci_ranges()")
> e078723f9ccc ("PCI: aardvark: Fix big endian support")
> 7fbcb5da811b ("PCI: aardvark: Don't rely on jiffies while holding spinlock")
> c0f05a6ab525 ("PCI: aardvark: Fix PCI_EXP_RTCTL register configuration")
> f4c7d053d7f7 ("PCI: aardvark: Wait for endpoint to be ready before training link")
> 364b3f1ff8f0 ("PCI: aardvark: Use LTSSM state to build link training flag")
>
> The important thing for the subject of this patch is not the "don't
> write to read-only register" part; it's true that there's no point in
> writing to read-only registers, but removing that write would not fix
> any bugs.
>
> The important thing is that we shouldn't blindly enable ASPM L0s, so
> that's what the subject should mention.
Ok understood, I will fix it in V3.
