linux-kernel - Re: [PATCH 2/5] PCI: mediatek-gen3: Add MT8196 support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <36c5011e0df6a706d7389aebeda8f8912f403fe8.camel@mediatek.com>
Date: Tue, 7 Jan 2025 01:51:52 +0000
From: Jianjun Wang (王建军) <Jianjun.Wang@...iatek.com>
To: "helgaas@...nel.org" <helgaas@...nel.org>
CC: "linux-mediatek@...ts.infradead.org" <linux-mediatek@...ts.infradead.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	Xavier Chang (張獻文) <Xavier.Chang@...iatek.com>,
	"manivannan.sadhasivam@...aro.org" <manivannan.sadhasivam@...aro.org>,
	"conor+dt@...nel.org" <conor+dt@...nel.org>, "robh@...nel.org"
	<robh@...nel.org>, "kw@...ux.com" <kw@...ux.com>,
	"linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "matthias.bgg@...il.com"
	<matthias.bgg@...il.com>, "bhelgaas@...gle.com" <bhelgaas@...gle.com>,
	"lpieralisi@...nel.org" <lpieralisi@...nel.org>, "krzk+dt@...nel.org"
	<krzk+dt@...nel.org>, AngeloGioacchino Del Regno
	<angelogioacchino.delregno@...labora.com>, Ryder Lee <Ryder.Lee@...iatek.com>
Subject: Re: [PATCH 2/5] PCI: mediatek-gen3: Add MT8196 support

On Fri, 2025-01-03 at 13:02 -0600, Bjorn Helgaas wrote:
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
> 
> 
> On Fri, Jan 03, 2025 at 02:00:12PM +0800, Jianjun Wang wrote:
> > The MT8196 is an ARM platform SoC that has the same PCIe IP as the
> > MT8195.
> > However, it requires additional settings in the pextpcfg registers.
> > Introduce pextpcfg in PCIe driver for these settings.
> 
> Add blank lines between paragraphs.
> 
> > +      * The values of some registers are different in RC and EP
> > mode. Therefore,
> > +      * call soc->pre_init after the mode change in case it
> > depends on these registers.
> 
> Wrap this to fit in 80 columns like the rest of the file.
> 
> > +     /* Adjust SYS_CLK_RDY_TIME ot 10us to avoid glitch */
> 
> s/ot/to/
> 
> Is this an erratum?  Is there any spec or erratum citation you can
> include in the comment?

Yes, the default time for sys_clk to be ready is 3us, which is the
value we use for all other platforms.

However, for MT8196, we need to set it to 10us to avoid glitch, I'll
add more description in the comment in the next version.

Thanks.

> 
> > +     val = readl_relaxed(pcie->base + PCIE_RESOURCE_CTRL_REG);
> > +     val &= ~PCIE_SYS_CLK_RDY_TIME_MASK;
> > +     val |= PCIE_SYS_CLK_RDY_TIME_TO_10US;
> > +     writel_relaxed(val, pcie->base + PCIE_RESOURCE_CTRL_REG);
> 
>