Re: [PPC] Boot problems after the pci-v6.18-changes
- To: Herve Codina <herve.codina@bootlin.com>
- Cc: Manivannan Sadhasivam <mani@kernel.org>, Bjorn Helgaas <helgaas@kernel.org>, Lukas Wunner <lukas@wunner.de>, Manivannan Sadhasivam <manivannan.sadhasivam@oss.qualcomm.com>, Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>, linux-pci@vger.kernel.org, mad skateman <madskateman@gmail.com>, "R.T.Dickinson" <rtd2@xtra.co.nz>, Christian Zigotzky <info@xenosoft.de>, linuxppc-dev <linuxppc-dev@lists.ozlabs.org>, hypexed@yahoo.com.au, Darren Stevens <darren@stevens-zone.net>, debian-powerpc@lists.debian.org, Thomas Petazzoni <thomas.petazzoni@bootlin.com>
- Subject: Re: [PPC] Boot problems after the pci-v6.18-changes
- From: Christian Zigotzky <chzigotzky@xenosoft.de>
- Date: Wed, 15 Oct 2025 15:07:09 +0200
- Message-id: <[🔎] CA1ADCB8-17B8-4903-8E1E-6451B8944657@xenosoft.de>
- In-reply-to: <[🔎] EF4D5B4B-9A61-4CF8-A5CC-5F6A49E824C1@xenosoft.de>
- References: <[🔎] EF4D5B4B-9A61-4CF8-A5CC-5F6A49E824C1@xenosoft.de>
> Am 15 October 2025 um 02:27 pm, Christian Zigotzky <chzigotzky@xenosoft.de> wrote:
>
>
>> On 15 October 2025 at 01:58 pm, Herve Codina <herve.codina@bootlin.com> wrote:
>>
>> Hi Christian,
>>
>>> On Wed, 15 Oct 2025 13:30:44 +0200
>>> Christian Zigotzky <chzigotzky@xenosoft.de> wrote:
>>>
>>> Hello Herve,
>>>
>>>>>> On 15 October 2025 at 10:39 am, Herve Codina <herve.codina@bootlin.com> wrote:
>>>>>
>>>>> Hi All,
>>>>>
>>>>> I also observed issues with the commit f3ac2ff14834 ("PCI/ASPM: Enable all
>>>>> ClockPM and ASPM states for devicetree platforms")
>>>
>>> Thanks for reporting.
>>>
>>>>
>>>> Also tried the quirk proposed in this discussion (quirk_disable_aspm_all)
>>>> an the quirk also fixes the timing issue.
>>>
>>> Where have you added quirk_disable_aspm_all?
>>
>> --- 8< ---
>> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
>> index 214ed060ca1b..a3808ab6e92e 100644
>> --- a/drivers/pci/quirks.c
>> +++ b/drivers/pci/quirks.c
>> @@ -2525,6 +2525,17 @@ static void quirk_disable_aspm_l0s_l1(struct pci_dev *dev)
>> */
>> DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_ASMEDIA, 0x1080, quirk_disable_aspm_l0s_l1);
>>
>> +static void quirk_disable_aspm_all(struct pci_dev *dev)
>> +{
>> + pci_info(dev, "Disabling ASPM\n");
>> + pci_disable_link_state(dev, PCIE_LINK_STATE_ALL);
>> +}
>> +
>> +/* LAN966x PCI board */
>> +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_EFAR, 0x9660, quirk_disable_aspm_all);
>> +
>> /*
>> * Some Pericom PCIe-to-PCI bridges in reverse mode need the PCIe Retrain
>> * Link bit cleared after starting the link retrain process to allow this
>> --- 8< ---
>>
>> Best regards,
>> Hervé
>
> It is the same patch, I use for my AMD Radeon cards.
I use
+DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_ATI, PCI_ANY_ID, quirk_disable_aspm_all);
in this patch.
>
> In my point of view we have to add a lot of other devices.
>
> But if the computer does not boot, will the average user know that there is a problem with the power management and their graphics card?
> I am unsure whether I can deliver the kernel to average users later on.
>
> Thanks,
> Christian
Reply to: