Re: [PATCH 4/4] rt2800: comment and simplify AGC init for RT6352

2018-10-10 Thread Daniel Golle
On Wed, Oct 10, 2018 at 10:09:14PM +0200, Tom Psyborg wrote: > it's 6352 dude true. thanks for the review! > > On 10/10/2018, Stanislaw Gruszka wrote: > > We do not need separate lines for calculating register values. > > Also add comment that value is different than in vendor driver. > > > > S

Re: [PATCH 4/4] rt2800: comment and simplify AGC init for RT6352

2018-10-10 Thread Tom Psyborg
it's 6352 dude On 10/10/2018, Stanislaw Gruszka wrote: > We do not need separate lines for calculating register values. > Also add comment that value is different than in vendor driver. > > Suggested-by: Daniel Golle > Signed-off-by: Stanislaw Gruszka > --- > drivers/net/wireless/ralink/rt2x00

[PATCH 4/4] rt2800: comment and simplify AGC init for RT6352

2018-10-10 Thread Stanislaw Gruszka
We do not need separate lines for calculating register values. Also add comment that value is different than in vendor driver. Suggested-by: Daniel Golle Signed-off-by: Stanislaw Gruszka --- drivers/net/wireless/ralink/rt2x00/rt2800lib.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletio