From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?B?TWljaGFsIFZva8OhxI0=?= Subject: Re: i.MX6S/DL and QCA8334 switch using DSA driver - CPU port not working Date: Wed, 16 May 2018 14:50:46 +0200 Message-ID: References: <037faf3c-8e8f-a696-8312-d1380c3b8656@gmail.com> <649c06a4-ca63-cb38-f105-ffd9dc17f5d2@gmail.com> <20180426140629.GB15370@lunn.ch> <48c029f1-1632-573f-c628-86b4972d668c@gmail.com> <20180430132025.GF10066@lunn.ch> <573258a9-b115-ca4a-121c-5174798d5621@gmail.com> <20180504133014.GB9666@lunn.ch> <5e8ecfed-c772-5668-4bac-1549143f599a@gmail.com> <20180510142904.GB7698@lunn.ch> <46a92995-b976-130c-7658-4d9576d5bee9@gmail.com> <20180515160824.GJ27867@lunn.ch> <21fc20c7-77d2-cba0-8f58-6bacb41f0970@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, Vivien Didelot To: Florian Fainelli , Andrew Lunn Return-path: Received: from mail-wm0-f47.google.com ([74.125.82.47]:53283 "EHLO mail-wm0-f47.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752654AbeEPMut (ORCPT ); Wed, 16 May 2018 08:50:49 -0400 Received: by mail-wm0-f47.google.com with SMTP id a67-v6so1213860wmf.3 for ; Wed, 16 May 2018 05:50:49 -0700 (PDT) In-Reply-To: <21fc20c7-77d2-cba0-8f58-6bacb41f0970@gmail.com> Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: On 15.5.2018 18:17, Florian Fainelli wrote: >>> I would like to have confirmed that I understand it correctly and that >>> the problem is really in the driver not handling fixed-link. >> >> I would actually skip fixed-link, if you don't need it. Just hardwire >> the CPU port, like the Marvell driver does: >> >> https://elixir.bootlin.com/linux/latest/source/drivers/net/dsa/mv88e6xxx/chip.c#L1780 >> >> I would do it here: >> >> https://elixir.bootlin.com/linux/latest/source/drivers/net/dsa/qca8k.c#L518 Thanks for the references, I will look there. > > Agreed with Andrew here, though if you can implement fixed-link, this > should be more future proof. I do not actually need fixed-link but I agree with Florian that it is much better to have that option. I see it a as way to override defaults that do not work for the user. That is what I was hoping for but it did not work. So as the fixed-link subnode is optional we still should force some sensible defaults if it is not used. Same as Marvell drives does. Can I say that we agreed that the current CPU port setting is not correct and the fastest link speed and duplex supported by the chip should be set as a default? > As far as people using this driver, John submitted it, but we have not > see many fixes/enhancements, so I am not clear who is actually using it > these days... glad you are though! Thanks. To the compatibility of the driver with the QCA8334 that I use. I am not sure what should be the correct way to deal with that. For example Marvell binding uses only two compatible strings for a large range of chips in one family. While Broadcom has one compatible string for each chip. As I mentioned earlier in this thread the QCA8334 switch has four ports while the already supported QCA8337 has seven ports. That is the only difference. Register address space is the same. Should I: - add a new compatible string "qualcomm,qca8334" and update the docs, - or just update the documentation and describe the compatibility? As I need to update the docs I understand it should go as a separate patch. Is it enough to CC devicetree list with the whole series? Any other list/maintainer? Thank you for your time, Michal