Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1611262 > unrolled thread

Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time

Started byVivien Didelot <vivien.didelot@savoirfairelinux.com>
First post2017-03-28 20:20 +0200
Last post2017-03-29 07:00 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2017-03-28 20:20 +0200
    Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time David Miller <davem@davemloft.net> - 2017-03-29 07:00 +0200

#1611262 — Re: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time

FromVivien Didelot <vivien.didelot@savoirfairelinux.com>
Date2017-03-28 20:20 +0200
SubjectRe: [PATCH net-next] net: dsa: mv88e6xxx: debug ATU Age Time
Message-ID<tq3SN-37G-3@gated-at.bofh.it>
Hi David,

Vivien Didelot <vivien.didelot@savoirfairelinux.com> writes:

> The ATU ageing time value programmed in the switch is rounded up to the
> nearest multiple of its coefficient (variable depending on the model.)
>
> Add a debug message to inform the user about the exact programmed value.
>
> On 6352, "brctl setageing br0 18" gives "AgeTime set to 0x01 (15000 ms)"
> while on 6390 we get "AgeTime set to 0x05 (18750 ms)".
>
> Signed-off-by: Vivien Didelot <vivien.didelot@savoirfairelinux.com>

Can you pick this patch?

Thanks,

        Vivien

[toc] | [next] | [standalone]


#1611565

FromDavid Miller <davem@davemloft.net>
Date2017-03-29 07:00 +0200
Message-ID<tqdS9-1Qy-15@gated-at.bofh.it>
In reply to#1611262
From: Vivien Didelot <vivien.didelot@savoirfairelinux.com>
Date: Tue, 28 Mar 2017 14:13:53 -0400

> Hi David,
> 
> Vivien Didelot <vivien.didelot@savoirfairelinux.com> writes:
> 
>> The ATU ageing time value programmed in the switch is rounded up to the
>> nearest multiple of its coefficient (variable depending on the model.)
>>
>> Add a debug message to inform the user about the exact programmed value.
>>
>> On 6352, "brctl setageing br0 18" gives "AgeTime set to 0x01 (15000 ms)"
>> while on 6390 we get "AgeTime set to 0x05 (18750 ms)".
>>
>> Signed-off-by: Vivien Didelot <vivien.didelot@savoirfairelinux.com>
> 
> Can you pick this patch?

If it's not in a pending state in patchwork, there must be a reason.  And
that reason will tell you why I didn't apply it, and what needs to be
resolved in order to change that.

In any event, you have to at a minimum resubmit the patch.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web