Re: [PATCH net] net: dsa: point out the tail taggers

2020-10-16 Thread Jakub Kicinski
On Sat, 17 Oct 2020 00:33:02 +0300 Vladimir Oltean wrote: > On Sat, Oct 17, 2020 at 12:16:28AM +0300, Vladimir Oltean wrote: > > On Fri, Oct 16, 2020 at 11:03:18PM +0200, Andrew Lunn wrote: > > > 2ecbc1f684482b4ed52447a39903bd9b0f222898 does not have net-next, as > > > far as i see, > > > > No

Re: [PATCH net] net: dsa: point out the tail taggers

2020-10-16 Thread Vladimir Oltean
On Sat, Oct 17, 2020 at 12:16:28AM +0300, Vladimir Oltean wrote: > On Fri, Oct 16, 2020 at 11:03:18PM +0200, Andrew Lunn wrote: > > 2ecbc1f684482b4ed52447a39903bd9b0f222898 does not have net-next, as > > far as i see, > > Not sure what you mean by that. Ah, I do understand what you mean now. In g

Re: [PATCH net] net: dsa: point out the tail taggers

2020-10-16 Thread Vladimir Oltean
On Fri, Oct 16, 2020 at 11:03:18PM +0200, Andrew Lunn wrote: > 2ecbc1f684482b4ed52447a39903bd9b0f222898 does not have net-next, as > far as i see, Not sure what you mean by that. > and tail_tag only hit net when net-next was merged into > net. net-next is only merged into net via Linus Torvalds,

Re: [PATCH net] net: dsa: point out the tail taggers

2020-10-16 Thread Andrew Lunn
On Fri, Oct 16, 2020 at 11:19:30PM +0300, Vladimir Oltean wrote: > On Fri, Oct 16, 2020 at 10:14:28PM +0200, Andrew Lunn wrote: > > On Fri, Oct 16, 2020 at 08:33:17PM +0300, Vladimir Oltean wrote: > > > On Sat, Oct 17, 2020 at 01:25:08AM +0800, kernel test robot wrote: > > > > Hi Christian, > > > >

Re: [PATCH net] net: dsa: point out the tail taggers

2020-10-16 Thread Vladimir Oltean
On Fri, Oct 16, 2020 at 10:14:28PM +0200, Andrew Lunn wrote: > On Fri, Oct 16, 2020 at 08:33:17PM +0300, Vladimir Oltean wrote: > > On Sat, Oct 17, 2020 at 01:25:08AM +0800, kernel test robot wrote: > > > Hi Christian, > > > > > > Thank you for the patch! Yet something to improve: > > > > > > [auto

Re: [PATCH net] net: dsa: point out the tail taggers

2020-10-16 Thread Andrew Lunn
On Fri, Oct 16, 2020 at 08:33:17PM +0300, Vladimir Oltean wrote: > On Sat, Oct 17, 2020 at 01:25:08AM +0800, kernel test robot wrote: > > Hi Christian, > > > > Thank you for the patch! Yet something to improve: > > > > [auto build test ERROR on net/master] > > > > url: > > https://github.com/0d

Re: [PATCH net] net: dsa: point out the tail taggers

2020-10-16 Thread Vladimir Oltean
On Sat, Oct 17, 2020 at 01:25:08AM +0800, kernel test robot wrote: > Hi Christian, > > Thank you for the patch! Yet something to improve: > > [auto build test ERROR on net/master] > > url: > https://github.com/0day-ci/linux/commits/Christian-Eggers/net-dsa-point-out-the-tail-taggers/20201017-00

Re: [PATCH net] net: dsa: point out the tail taggers

2020-10-16 Thread kernel test robot
Hi Christian, Thank you for the patch! Yet something to improve: [auto build test ERROR on net/master] url: https://github.com/0day-ci/linux/commits/Christian-Eggers/net-dsa-point-out-the-tail-taggers/20201017-003007 base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net.git 2ecb

Re: [PATCH net] net: dsa: point out the tail taggers

2020-10-16 Thread Vladimir Oltean
Hi Christian, On Fri, Oct 16, 2020 at 06:28:00PM +0200, Christian Eggers wrote: > From a recent commit with the same summary: > > "The Marvell 88E6060 uses tag_trailer.c and the KSZ8795, KSZ9477 and > KSZ9893 switches also use tail tags." > > Set "tail_tag" to true for KSZ8795 and KSZ9477 which

Re: [PATCH net] net: dsa: point out the tail taggers

2020-10-16 Thread Florian Fainelli
On 10/16/20 9:28 AM, Christian Eggers wrote: > From a recent commit with the same summary: > > "The Marvell 88E6060 uses tag_trailer.c and the KSZ8795, KSZ9477 and > KSZ9893 switches also use tail tags." > > Set "tail_tag" to true for KSZ8795 and KSZ9477 which were missing in the > original comm

[PATCH net] net: dsa: point out the tail taggers

2020-10-16 Thread Christian Eggers
>From a recent commit with the same summary: "The Marvell 88E6060 uses tag_trailer.c and the KSZ8795, KSZ9477 and KSZ9893 switches also use tail tags." Set "tail_tag" to true for KSZ8795 and KSZ9477 which were missing in the original commit. Fixes: 7a6ffe764be3 [net] ("net: dsa: point out the t