[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <02bb01db8460$03c4d640$0b4e82c0$@samsung.com>
Date: Fri, 21 Feb 2025 18:26:15 +0530
From: "Swathi K S" <swathi.ks@...sung.com>
To: "'Russell King \(Oracle\)'" <linux@...linux.org.uk>
Cc: <krzk+dt@...nel.org>, <andrew+netdev@...n.ch>, <davem@...emloft.net>,
<edumazet@...gle.com>, <kuba@...nel.org>, <pabeni@...hat.com>,
<robh@...nel.org>, <conor+dt@...nel.org>, <richardcochran@...il.com>,
<mcoquelin.stm32@...il.com>, <alexandre.torgue@...s.st.com>,
<netdev@...r.kernel.org>, <devicetree@...r.kernel.org>,
<linux-stm32@...md-mailman.stormreply.com>,
<linux-arm-kernel@...ts.infradead.org>, <linux-kernel@...r.kernel.org>,
<pankaj.dubey@...sung.com>, <ravi.patel@...sung.com>, <gost.dev@...sung.com>
Subject: RE: [PATCH v7 2/2] net: stmmac: dwc-qos: Add FSD EQoS support
> -----Original Message-----
> From: Russell King (Oracle) <linux@...linux.org.uk>
> Sent: 21 February 2025 17:20
> To: Swathi K S <swathi.ks@...sung.com>
> Cc: krzk+dt@...nel.org; andrew+netdev@...n.ch; davem@...emloft.net;
> edumazet@...gle.com; kuba@...nel.org; pabeni@...hat.com;
> robh@...nel.org; conor+dt@...nel.org; richardcochran@...il.com;
> mcoquelin.stm32@...il.com; alexandre.torgue@...s.st.com;
> netdev@...r.kernel.org; devicetree@...r.kernel.org; linux-stm32@...md-
> mailman.stormreply.com; linux-arm-kernel@...ts.infradead.org; linux-
> kernel@...r.kernel.org; pankaj.dubey@...sung.com;
> ravi.patel@...sung.com; gost.dev@...sung.com
> Subject: Re: [PATCH v7 2/2] net: stmmac: dwc-qos: Add FSD EQoS support
>
> On Fri, Feb 21, 2025 at 04:04:25PM +0530, Swathi K S wrote:
> >
> >
> > > -----Original Message-----
> > > From: Russell King (Oracle) <linux@...linux.org.uk>
> > > Sent: 20 February 2025 18:10
> > > To: Swathi K S <swathi.ks@...sung.com>
> > > Cc: krzk+dt@...nel.org; andrew+netdev@...n.ch;
> davem@...emloft.net;
> > > edumazet@...gle.com; kuba@...nel.org; pabeni@...hat.com;
> > > robh@...nel.org; conor+dt@...nel.org; richardcochran@...il.com;
> > > mcoquelin.stm32@...il.com; alexandre.torgue@...s.st.com;
> > > netdev@...r.kernel.org; devicetree@...r.kernel.org;
> > > linux-stm32@...md- mailman.stormreply.com;
> > > linux-arm-kernel@...ts.infradead.org; linux- kernel@...r.kernel.org;
> > > pankaj.dubey@...sung.com; ravi.patel@...sung.com;
> > > gost.dev@...sung.com
> > > Subject: Re: [PATCH v7 2/2] net: stmmac: dwc-qos: Add FSD EQoS
> > > support
> > >
> > > On Thu, Feb 20, 2025 at 10:07:12AM +0530, Swathi K S wrote:
> > > > +static int fsd_eqos_probe(struct platform_device *pdev,
> > > > + struct plat_stmmacenet_data *data,
> > > > + struct stmmac_resources *res) {
> > > > + struct clk *clk_rx1 = NULL;
> > > > + struct clk *clk_rx2 = NULL;
> > > > +
> > > > + for (int i = 0; i < data->num_clks; i++) {
> > > > + if (strcmp(data->clks[i].id, "slave_bus") == 0)
> > > > + data->stmmac_clk = data->clks[i].clk;
> > > > + else if (strcmp(data->clks[i].id, "eqos_rxclk_mux")
== 0)
> > > > + clk_rx1 = data->clks[i].clk;
> > > > + else if (strcmp(data->clks[i].id, "eqos_phyrxclk")
== 0)
> > > > + clk_rx2 = data->clks[i].clk;
> > > > + }
> > > > +
> > > > + /* Eth0 RX clock doesn't support MUX */
> > > > + if (clk_rx1)
> > > > + clk_set_parent(clk_rx1, clk_rx2);
> > >
> > > Isn't there support in DT for automatically setting the clock tree?
> > > See
> > > https://protect2.fireeye.com/v1/url?k=f0089f78-90ea0225-f0091437-
> > > 000babd9f1ba-cf835b8b94ccd94a&q=1&e=4ae794ec-f443-4d77-aee4-
> > > 449f53a3a1a4&u=https%3A%2F%2Fgithub.com%2Fdevicetree-org%2Fdt-
> > >
> schema%2Fblob%2Fmain%2Fdtschema%2Fschemas%2Fclock%2Fclock.yaml
> > > %23L24
> > >
> > > Also, I think a cleanup like the below (sorry, it's on top of other
> > patches I'm
> > > working on at the moment but could be rebased) would make sense.
> > >
> > > With both of these, this should mean that your changes amount to:
> > >
> > > 1. making data->probe optional
> > > 2. providing a dwc_eth_dwmac_data structure that has
> .stmmac_clk_name
> > > filled in
> > > 3. adding your compatible to the match data with a pointer to the
> > > above structure.
> >
> > Hi Russell,
> > Thanks for your input.
> > Will implement this in v8.
> > But I could not find your patch 'net: stmmac: clean up clock
initialisation'
> > in mailing list
> > Could you point me to that?
> > Or do you want me to integrate the below changes into my patch series
> > and post?
> >
> > Please let me know
>
> Please have patience - I'm a volunteer here, and I included the patch in
the
> email for you. You're not the only one whom I'm addressing issues in the
> stmmac driver for. Since Sunday, I have a total of 16 new stmmac patches
> plus been debugging a regression someone has reported.
I understand your concern and thanks for improving the code.
>
> Coincidentally, I just sent out the patch as a stand-alone patch, it
should be
> quicker to get it into net-next rather than trying to get all the other
patches
> I'd *already* had queued up in first.
>
> https://lore.kernel.org/r/E1tlRMP-004Vt5-W1@rmk-PC.armlinux.org.uk
Will test your patch on FSD platform.
- Swathi
>
> However, I'm expecting someone to say that dwc_eth_find_clk() should be
> moved into the stmmac platform code, and the other platforms need to be
> converted to use it... so it may be some time before we're at a stage
where
> you can proceed.
>
> Please be patient.
>
> --
> RMK's Patch system: https://protect2.fireeye.com/v1/url?k=9539722c-
> f442d8a5-9538f963-74fe48600034-5bc44de329219ee1&q=1&e=0bc950e5-
> 2fde-481e-bce0-
> 534d0f94352b&u=https%3A%2F%2Fwww.armlinux.org.uk%2Fdeveloper%2F
> patches%2F
> FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!
Powered by blists - more mailing lists