Linux PCI subsystem development
 help / color / mirror / Atom feed
From: Bjorn Helgaas <helgaas@kernel.org>
To: Huacai Chen <chenhuacai@loongson.cn>
Cc: "Bjorn Helgaas" <bhelgaas@google.com>,
	"Lorenzo Pieralisi" <lorenzo.pieralisi@arm.com>,
	"Rob Herring" <robh@kernel.org>,
	"Krzysztof Wilczyński" <kw@linux.com>,
	linux-pci@vger.kernel.org, "Jianmin Lv" <lvjianmin@loongson.cn>,
	"Xuefeng Li" <lixuefeng@loongson.cn>,
	"Huacai Chen" <chenhuacai@gmail.com>,
	"Jiaxun Yang" <jiaxun.yang@flygoat.com>,
	"Tiezhu Yang" <yangtiezhu@loongson.cn>
Subject: Re: [PATCH V4 0/2] PCI: Resolve Loongson's LS7A PCI problems
Date: Wed, 1 Feb 2023 12:57:56 -0600	[thread overview]
Message-ID: <20230201185756.GA1882554@bhelgaas> (raw)
In-Reply-To: <20230201043018.778499-1-chenhuacai@loongson.cn>

On Wed, Feb 01, 2023 at 12:30:16PM +0800, Huacai Chen wrote:
> This patchset attempt to resolves Loongson's LS7A PCI problems: the
> first patch remove pci_disable_device() in pcie_portdrv_remove() to
> avoid poweroff/reboot failure; the second patch improves the mrrs quirk
> for LS7A chipset; 
> 
> V1 -> V2:
> 1, Update commit messages and comments.
> 
> V2 -> V3:
> 1, Simply remove pci_disable_device() in pcie_port_device_remove() to
>    solve poweroff/reboot failure.
> 2, Update commit messages and comments.
> 
> V3 -> V4:
> 1, Just remove pci_disable_device() in pcie_portdrv_shutdown() and keep
>    pcie_portdrv_remove() be the same logic as before.
> 
> Huacai Chen, Tiezhu Yang and Jianmin Lv(2):
>  PCI: Omit pci_disable_device() in .shutdown().
>  PCI: loongson: Improve the MRRS quirk for LS7A.
> 
> Signed-off-by: Huacai Chen <chenhuacai@loongson.cn>
> Signed-off-by: Jianmin Lv <lvjianmin@loongson.cn> 
> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>

I applied both to pci/enumeration for v6.3, thanks!

I added a diagnostic message in pcie_set_readq() and reworked the
commit logs, so take a look and make sure I didn't mess it up:

https://git.kernel.org/cgit/linux/kernel/git/pci/pci.git/log/?h=pci/enumeration&id=8b3517f88ff2

  parent reply	other threads:[~2023-02-01 18:58 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-01  4:30 [PATCH V4 0/2] PCI: Resolve Loongson's LS7A PCI problems Huacai Chen
2023-02-01  4:30 ` [PATCH V4 1/2] PCI: Omit pci_disable_device() in .shutdown() Huacai Chen
2023-02-01 18:17   ` Bjorn Helgaas
2023-02-02 13:27     ` Huacai Chen
2023-02-02 20:30       ` Bjorn Helgaas
2023-02-03  4:00         ` Huacai Chen
2023-02-03 18:03           ` Bjorn Helgaas
2023-02-01  4:30 ` [PATCH V4 2/2] PCI: loongson: Improve the MRRS quirk for LS7A Huacai Chen
2023-02-01 18:57 ` Bjorn Helgaas [this message]
2023-02-02 13:28   ` [PATCH V4 0/2] PCI: Resolve Loongson's LS7A PCI problems Huacai Chen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230201185756.GA1882554@bhelgaas \
    --to=helgaas@kernel.org \
    --cc=bhelgaas@google.com \
    --cc=chenhuacai@gmail.com \
    --cc=chenhuacai@loongson.cn \
    --cc=jiaxun.yang@flygoat.com \
    --cc=kw@linux.com \
    --cc=linux-pci@vger.kernel.org \
    --cc=lixuefeng@loongson.cn \
    --cc=lorenzo.pieralisi@arm.com \
    --cc=lvjianmin@loongson.cn \
    --cc=robh@kernel.org \
    --cc=yangtiezhu@loongson.cn \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox