All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michal Simek <michal.simek@xilinx.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 11/12] net: emaclite: Move driver to DM
Date: Wed, 16 Dec 2015 10:52:33 +0100	[thread overview]
Message-ID: <56713461.10306@xilinx.com> (raw)
In-Reply-To: <CANr=Z=Zb66xUJSRcPLqXOkA9xnNR3Vot8T0ruo5t8Ph5s+PK_A@mail.gmail.com>

On 15.12.2015 23:34, Joe Hershberger wrote:
> On Fri, Dec 11, 2015 at 6:03 AM, Michal Simek <michal.simek@xilinx.com> wrote:
>> Move driver to DM.
>>
>> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> 
> A few nits below, but otherwise,
> 
> Acked-by: Joe Hershberger <joe.hershberger@ni.com>

Thanks.

>> @@ -523,78 +520,105 @@ try_again:
>>
>>         debug("Packet receive from 0x%p, length %dB\n", addr, length);
>>         net_process_received_packet((uchar *)etherrxbuff, length);
>> -       return length;
>> +       return 0;
> 
> Since there is no clean-up here, it would be better to return the
> length and remove the net_process_received_packet() call above.

I will send it in separate patch. I wanted to do it in this way to be
smaller patch.

> 
>>
> 
> Remove this line.

done.

>> +
>> +static const struct eth_ops emaclite_ops = {
>> +       .start = emaclite_init,
> 
> Name this emaclite_start.
> 
>> +       .send = emaclite_send,
>> +       .recv = emaclite_recv,
>> +       .stop = emaclite_halt,
> 
> Name this emaclite_stop.

there will be separate patch for this.

Thanks,
Michal

  reply	other threads:[~2015-12-16  9:52 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-11 12:03 [U-Boot] [PATCH 00/12] Moving Emaclite to DM + MDIO support Michal Simek
2015-12-11 12:03 ` [U-Boot] [PATCH 01/12] net: emaclite: Remove ancient OF probe function Michal Simek
2015-12-15 21:08   ` Joe Hershberger
2015-12-11 12:03 ` [U-Boot] [PATCH 02/12] net: emaclite: Add MDIO support to driver Michal Simek
2015-12-15 21:22   ` Joe Hershberger
2015-12-16  8:30     ` Michal Simek
2015-12-11 12:03 ` [U-Boot] [PATCH 03/12] net: emaclite: Convert MDIO to use register offset Michal Simek
2015-12-15 21:32   ` Joe Hershberger
2015-12-11 12:03 ` [U-Boot] [PATCH 04/12] net: emaclite: Use indirect register access for tx_ping/pong Michal Simek
2015-12-15 21:36   ` Joe Hershberger
2015-12-11 12:03 ` [U-Boot] [PATCH 05/12] " Michal Simek
2015-12-15 21:36   ` Joe Hershberger
2015-12-16 10:08     ` Michal Simek
2015-12-11 12:03 ` [U-Boot] [PATCH 06/12] net: emaclite: Use indirect register access for TX reset Michal Simek
2015-12-15 21:37   ` Joe Hershberger
2015-12-11 12:03 ` [U-Boot] [PATCH 07/12] net: emaclite: Fix logic around available TX buffers Michal Simek
2015-12-15 21:38   ` Joe Hershberger
2015-12-11 12:03 ` [U-Boot] [PATCH 08/12] net: emaclite: Remove XEL_TSR_XMIT_ACTIVE_MASK flag Michal Simek
2015-12-11 12:03 ` [U-Boot] [PATCH 09/12] net: emaclite: Use indirect reg access in send Michal Simek
2015-12-15 21:44   ` Joe Hershberger
2015-12-11 12:03 ` [U-Boot] [PATCH 10/12] net: emaclite: Use indirect access in emaclite_recv Michal Simek
2015-12-15 22:20   ` Joe Hershberger
2015-12-17 11:24     ` Michal Simek
2015-12-11 12:03 ` [U-Boot] [PATCH 11/12] net: emaclite: Move driver to DM Michal Simek
2015-12-15 22:34   ` Joe Hershberger
2015-12-16  9:52     ` Michal Simek [this message]
2015-12-11 12:03 ` [U-Boot] [PATCH 12/12] net: emaclite: Move emaclite to Kconfig Michal Simek
2015-12-15 22:35   ` Joe Hershberger

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=56713461.10306@xilinx.com \
    --to=michal.simek@xilinx.com \
    --cc=u-boot@lists.denx.de \
    /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.