Thread (23 messages) flat view 23 messages, 4 authors, 2017-05-08

Re: [PATCH v3 net-next 01/11] net: stmmac: prepare dma op mode config for multiple queues

From: Andy Shevchenko <hidden>
Date: 2017-05-08 11:56:15
Also in: lkml

On Mon, May 8, 2017 at 2:40 PM, Joao Pinto [off-list ref] wrote:
Às 12:34 PM de 5/8/2017, Andy Shevchenko escreveu:
quoted
On Mon, May 8, 2017 at 1:42 PM, Joao Pinto [off-list ref] wrote:
quoted
Às 11:12 AM de 5/8/2017, Andy Shevchenko escreveu:
quoted
On Mon, May 8, 2017 at 12:54 PM, Joao Pinto [off-list ref] wrote:
quoted
Às 10:36 AM de 5/8/2017, Andy Shevchenko escreveu:
quoted
[   44.374161] stmmac_dvr_probe <<< 0 0
Ok, so this is the cause of the problem. The driver is geting 0 for real RX and
TX queues.

Your setup uses standard DT parsing from stmmac_platform or a custom one?

If you are using stmmac_probe_config_dt():
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c#n363

You will find a function named stmmac_mtl_setup() being called:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/net/ethernet/stmicro/stmmac/stmmac_platform.c#n492

In this function, the number of RX and TX queues is being set to 1 by default.
Ah-ha, now I know how it's happened.
You forget to update all setup() hooks in PCI bus driver :-)

I will prepare a fix.
Just tell me should I put Fixes tag or not? And if yes, what commit
should I refer to?

-- 
With Best Regards,
Andy Shevchenko
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help