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


Groups > linux.kernel > #1570540 > unrolled thread

Re: stmmac: GMAC_RGSMIIIS reports bogus values

Started byGiuseppe CAVALLARO <peppe.cavallaro@st.com>
First post2017-01-31 11:00 +0100
Last post2017-01-31 14:50 +0100
Articles 3 — 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: stmmac: GMAC_RGSMIIIS reports bogus values Giuseppe CAVALLARO <peppe.cavallaro@st.com> - 2017-01-31 11:00 +0100
    Re: stmmac: GMAC_RGSMIIIS reports bogus values Alexey Brodkin <Alexey.Brodkin@synopsys.com> - 2017-01-31 14:30 +0100
      Re: stmmac: GMAC_RGSMIIIS reports bogus values Giuseppe CAVALLARO <peppe.cavallaro@st.com> - 2017-01-31 14:50 +0100

#1570540 — Re: stmmac: GMAC_RGSMIIIS reports bogus values

FromGiuseppe CAVALLARO <peppe.cavallaro@st.com>
Date2017-01-31 11:00 +0100
SubjectRe: stmmac: GMAC_RGSMIIIS reports bogus values
Message-ID<t5Doe-7zo-23@gated-at.bofh.it>
On 1/27/2017 11:23 AM, Alexey Brodkin wrote:
> That's why my initial proposal was to ignore whatever we read from this register
> if we have MDIO bus instantiated already.

sorry for my late reply, I agree with this approach, according to the
HW and platform configuration the driver has to understand and then
work to use either SMA or PCS module.

Regards
Peppe

[toc] | [next] | [standalone]


#1570744

FromAlexey Brodkin <Alexey.Brodkin@synopsys.com>
Date2017-01-31 14:30 +0100
Message-ID<t5GFs-1cI-33@gated-at.bofh.it>
In reply to#1570540
HiĀ Giuseppe,

On Tue, 2017-01-31 at 10:55 +0100, Giuseppe CAVALLARO wrote:
> On 1/27/2017 11:23 AM, Alexey Brodkin wrote:
> > 
> > That's why my initial proposal was to ignore whatever we read from this register
> > if we have MDIO bus instantiated already.
> 
> sorry for my late reply, I agree with this approach, according to the
> HW and platform configuration the driver has to understand and then
> work to use either SMA or PCS module.

I already submitted another solution which IMHO is much cleaner and appropriate,
see David has it already in master tree here:
http://git.kernel.org/cgit/linux/kernel/git/davem/net.git/commit/?id=0a764db103376cf69d04449b10688f3516cc0b88

-Alexey

[toc] | [prev] | [next] | [standalone]


#1570761

FromGiuseppe CAVALLARO <peppe.cavallaro@st.com>
Date2017-01-31 14:50 +0100
Message-ID<t5GYO-1ja-9@gated-at.bofh.it>
In reply to#1570744
Hello Alexey

On 1/31/2017 2:24 PM, Alexey Brodkin wrote:
> Hi Giuseppe,
>
> On Tue, 2017-01-31 at 10:55 +0100, Giuseppe CAVALLARO wrote:
>> On 1/27/2017 11:23 AM, Alexey Brodkin wrote:
>>>
>>> That's why my initial proposal was to ignore whatever we read from this register
>>> if we have MDIO bus instantiated already.
>>
>> sorry for my late reply, I agree with this approach, according to the
>> HW and platform configuration the driver has to understand and then
>> work to use either SMA or PCS module.
>
> I already submitted another solution which IMHO is much cleaner and appropriate,
> see David has it already in master tree here:
> http://git.kernel.org/cgit/linux/kernel/git/davem/net.git/commit/?id=0a764db103376cf69d04449b10688f3516cc0b88
>

it's ok, thx for that

Peppe

> -Alexey
>

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web