All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
To: Ben Dooks <ben.dooks@codethink.co.uk>, cm-hiep@jinso.co.jp
Cc: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>,
	netdev@vger.kernel.org, linux-sh@vger.kernel.org,
	Magnus Damm <magnus.damm@gmail.com>,
	Simon Horman <horms@verge.net.au>
Subject: Re: Ethernet bug at Upstream-v3.15-rcx on Lager.
Date: Tue, 03 Jun 2014 02:43:26 +0000	[thread overview]
Message-ID: <538D364E.6040605@renesas.com> (raw)
In-Reply-To: <53888FF2.8040505@codethink.co.uk>

Hi Ben,

(2014/05/30 23:04), Ben Dooks wrote:
> On 16/05/14 06:56, Yoshihiro Shimoda wrote:
>> Hi Hiep-san,
>>
>> (2014/05/16 10:08), カオ ミン ヒェップ wrote:
>>>>
>>>>> Please review it when you have time.
>>>>
>>>>    Please try this patch:
>>>>
>>>> http://marc.info/?l=linux-netdev&m\x139602519809181
>>>>
>>> I have tested this patch on v3.15.0-rc5-00077-g14186fe, and results are the message of
>>> "sh-eth r8a7790-ether eth0: Receive FIFO Overflow" has disappeared and sending datagrams is good.
>>> (Bandwidth 10M to 100M: Test with 5/5 times by script and by hand are good)
>>>
>>
>> It's nice!
>> So, I hope the patch is merged in upstream.
>> However, the patch's change log mentioned r8a7790 only.
>> The r8a7791's ethernet controller also has this bit.
> 
> The bit update covers both of them as the r8a7790 and r8a7791 both share
> the same configuration structure.
> 
> I re-sent the patch but didn't find your email post until after I had
> sent it, otherwise I would have mentioned your testing specifically. In
> future it would be helpful if you could do the following
> 
> - Mention driver by name in the subject line
> - Search the list for previous mentions and reply to that/those threads.
> 

Thank you very much for the comment and your v2 patch!
I understood it.

Best regards,
Yoshihiro Shimoda

WARNING: multiple messages have this Message-ID (diff)
From: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
To: Ben Dooks <ben.dooks@codethink.co.uk>, <cm-hiep@jinso.co.jp>
Cc: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>,
	<netdev@vger.kernel.org>, <linux-sh@vger.kernel.org>,
	Magnus Damm <magnus.damm@gmail.com>,
	Simon Horman <horms@verge.net.au>
Subject: Re: Ethernet bug at Upstream-v3.15-rcx on Lager.
Date: Tue, 3 Jun 2014 11:43:26 +0900	[thread overview]
Message-ID: <538D364E.6040605@renesas.com> (raw)
In-Reply-To: <53888FF2.8040505@codethink.co.uk>

Hi Ben,

(2014/05/30 23:04), Ben Dooks wrote:
> On 16/05/14 06:56, Yoshihiro Shimoda wrote:
>> Hi Hiep-san,
>>
>> (2014/05/16 10:08), カオ ミン ヒェップ wrote:
>>>>
>>>>> Please review it when you have time.
>>>>
>>>>    Please try this patch:
>>>>
>>>> http://marc.info/?l=linux-netdev&m=139602519809181
>>>>
>>> I have tested this patch on v3.15.0-rc5-00077-g14186fe, and results are the message of
>>> "sh-eth r8a7790-ether eth0: Receive FIFO Overflow" has disappeared and sending datagrams is good.
>>> (Bandwidth 10M to 100M: Test with 5/5 times by script and by hand are good)
>>>
>>
>> It's nice!
>> So, I hope the patch is merged in upstream.
>> However, the patch's change log mentioned r8a7790 only.
>> The r8a7791's ethernet controller also has this bit.
> 
> The bit update covers both of them as the r8a7790 and r8a7791 both share
> the same configuration structure.
> 
> I re-sent the patch but didn't find your email post until after I had
> sent it, otherwise I would have mentioned your testing specifically. In
> future it would be helpful if you could do the following
> 
> - Mention driver by name in the subject line
> - Search the list for previous mentions and reply to that/those threads.
> 

Thank you very much for the comment and your v2 patch!
I understood it.

Best regards,
Yoshihiro Shimoda

  reply	other threads:[~2014-06-03  2:43 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-15  6:28 Ethernet bug at Upstream-v3.15-rcx on Lager カオ ミン ヒェップ
2014-05-15  6:28 ` カオ ミン ヒェップ
2014-05-15 17:20 ` Sergei Shtylyov
2014-05-15 17:20   ` Sergei Shtylyov
2014-05-16  1:08   ` カオ ミン ヒェップ
2014-05-16  1:08     ` カオ ミン ヒェップ
2014-05-16  5:56     ` Yoshihiro Shimoda
2014-05-16  5:56       ` Yoshihiro Shimoda
2014-05-30 14:04       ` Ben Dooks
2014-05-30 14:04         ` Ben Dooks
2014-06-03  2:43         ` Yoshihiro Shimoda [this message]
2014-06-03  2:43           ` Yoshihiro Shimoda

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=538D364E.6040605@renesas.com \
    --to=yoshihiro.shimoda.uh@renesas.com \
    --cc=ben.dooks@codethink.co.uk \
    --cc=cm-hiep@jinso.co.jp \
    --cc=horms@verge.net.au \
    --cc=linux-sh@vger.kernel.org \
    --cc=magnus.damm@gmail.com \
    --cc=netdev@vger.kernel.org \
    --cc=sergei.shtylyov@cogentembedded.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.