[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CY4PR07MB27579495272871B120875A83C1BE9@CY4PR07MB2757.namprd07.prod.outlook.com>
Date: Wed, 20 Oct 2021 03:38:47 +0000
From: Parshuram Raju Thombare <pthombar@...ence.com>
To: Bjorn Helgaas <helgaas@...nel.org>
CC: "kishon@...com" <kishon@...com>, Tom Joseph <tjoseph@...ence.com>,
"lorenzo.pieralisi@....com" <lorenzo.pieralisi@....com>,
"robh@...nel.org" <robh@...nel.org>, "kw@...ux.com" <kw@...ux.com>,
"bhelgaas@...gle.com" <bhelgaas@...gle.com>,
"linux-omap@...r.kernel.org" <linux-omap@...r.kernel.org>,
"linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Milind Parab <mparab@...ence.com>
Subject: RE: [PATCH] PCI: cadence: Disable Function Level Reset support
>Add blank lines between paragraphs.
>Write the text in imperative mood, e.g.,
Ok
> Disable FLR (Function Level Reset) support on all functions.
>It looks like this patch clears PCI_EXP_DEVCAP_FLR in the Device
>Capabilities register. From the point of view of Linux, that means
>the device doesn't *advertise* FLR support.
>
>That's different from actualy *disabling* FLR support, but maybe
>there's internal logic in the device that ignores
>PCI_EXP_DEVCTL_BCR_FLR when PCI_EXP_DEVCAP_FLR is cleared?
Yes, this patch is just to prevent device from advertising FLR support.
Regards,
Parshuram Thombare
Powered by blists - more mailing lists