public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Ben Warren <biggerbadderben@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [PATCH] sh: Add Ethernet support for MPR2 board
Date: Fri, 07 Mar 2008 09:43:13 -0500	[thread overview]
Message-ID: <47D15481.1060902@gmail.com> (raw)
In-Reply-To: <9b3c8e40803070125g244e3f3fv86bdbee2388d0023@mail.gmail.com>

Hi Mark,

Mark Jonas wrote:
> Hello,
>
> the attached patch adds a driver for the SMSC LAN911x and LAN921x
> family of Ethernet controllers. The patch also
> enables the use of this driver on the SH7720 based MPR2 board.
>
> Please note that the network driver does not obey the U-Boot coding
> style at all. The reason is that the driver is
> supplied like that by SMSC on their web page. I only modified it such
> that it not only supports the LAN9x18 but
> also the LAN9x15. And I removed CRLFs.
>
> My reasoning for leaving the driver as is that if SMSC updates the
> driver on their web page it will be easier to
> get the changes into U-Boot as well.
>
>   
Your intentions are good, but you have to know that there's no way this 
driver is going into U-boot unless it meets the coding standards.  There 
are many reasons for this policy, but suffice to say it just ain't gonna 
happen.

If you clean it up, we'll put it in.  Also, sending patches zipped up is 
uncool.  Please send as inline text and break up logically if you need to.

regards,
Ben

  reply	other threads:[~2008-03-07 14:43 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-07  9:25 [U-Boot-Users] [PATCH] sh: Add Ethernet support for MPR2 board Mark Jonas
2008-03-07 14:43 ` Ben Warren [this message]
2008-03-07 21:50   ` Mark Jonas

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=47D15481.1060902@gmail.com \
    --to=biggerbadderben@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox