Messages in this thread | | | Subject | Re: [PATCH net-next v3 0/7] Add support for QCA8334 switch | From | Michal Vokáč <> | Date | Thu, 24 May 2018 11:34:57 +0200 |
| |
On 23.5.2018 17:39, Florian Fainelli wrote: > > > On 05/22/2018 11:20 PM, Michal Vokáč wrote: >> This series basically adds support for a QCA8334 ethernet switch to the >> qca8k driver. It is a four-port variant of the already supported seven >> port QCA8337. Register map is the same for the whole familly and all chips >> have the same device ID. >> >> Major part of this series enhances the CPU port setting. Currently the CPU >> port is not set to any sensible defaults compatible with the xGMII >> interface. This series forces the CPU port to its maximum bandwidth and >> also allows to adjust the new defaults using fixed-link device tree >> sub-node. >> >> Alongside these changes I fixed two checkpatch warnings regarding SPDX and >> redundant parentheses. > > Looks great, thanks Michal! Do you have any features or things you are > working on that would be added later to the driver?
Thank you too Florian. And also big thank to you Andrew. You helped me a lot to debug the RGMII issue. I have been stuck at that for more than a month and would not resolve it without your help.
As I have done this in a process of upgrading our BSP to a more recent kernel, and hopefully mainline, I now need to move on to other parts of the board. So unfortunately no, I do not have any other enhancements planned to this driver for now. But as we are probably one of the few with access to the NDA covered Qualcomm documentation I see a great opportunity to work on that later. I am afraid "later" means something like next year in this case as I am basically the only kernel developer in our company and not yet very experienced.
Thank you all for your time, Michal
| |