On Fri, Nov 29, 2024 at 01:55:37PM -0600, Bjorn Helgaas wrote:
> On Fri, Nov 29, 2024 at 02:54:12PM +0530, Manivannan Sadhasivam wrote:
> > On all Qcom endpoint SoCs, BAR0/BAR2 are 64bit BARs by default and software
> > cannot change the type. So mark the those BARs as 64bit BARs and also mark
> > the successive BAR1/BAR3 as RESERVED BARs so that the EPF drivers cannot
> > use them.
>
> "Default" implies an initial setting that can be changed, but you say
> "by default" and also "software cannot change the type." Can they be
> anything *other* than 64-bit BARs?
>
> If they're hardwired to be 64-bit BARs, I would just say that.
>
> > Cc: stable+noauto...@kernel.org # depends on patch introducing only_64bit
> > flag
>
> If stable maintainers need to act on this, do they need to search for
> the patch introducing only_64bit flag? That seems onerous; is there a
> SHA1 that would make it easier?
>
> > Fixes: f55fee56a631 ("PCI: qcom-ep: Add Qualcomm PCIe Endpoint controller
> > driver")
Yes, having a sha1 that has that "feature addition" would be great,
surely that isn't 6.13-rc1, is it?
thanks,
greg k-h
