[U-Boot-Users] [PATCH] sh: Add Ethernet support for MPR2 board

[U-Boot-Users] [PATCH] sh: Add Ethernet support for MPR2 board

[Mark Jonas]

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.

Regards,
Mark Jonas

CHANGELOG:
----------
sh: Added support for SMSC LAN9115 networking on MPR2 board.

Signed-off-by: Mark Jonas <mark.jonas@de.bosch.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: u-boot_mpr2_lan9115_20080307.patch.bz2
Type: application/x-bzip2
Size: 10375 bytes
Desc: not available
Url : http://lists.denx.de/pipermail/u-boot/attachments/20080307/fe1917b3/attachment.bin 

[U-Boot-Users] [PATCH] sh: Add Ethernet support for MPR2 board

[Ben Warren]

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

[U-Boot-Users] [PATCH] sh: Add Ethernet support for MPR2 board

[Mark Jonas]

Hi Ben,

>  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.

OK, I somehow thought so but isn't there this clause in the U-Boot Readme:

"Source files originating from a different project (for example the
MTD subsystem) are generally exempt from these guidelines and are not
reformated to ease subsequent migration to newer versions of those
sources."

>  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.

Maybe I am uncool but I just obeyed the Readme:

"* Remember that there is a size limit of 40 kB per message on the
 u-boot-users mailing list. Compression may help."

IMHO I broke it up logically into two patches. One containing the mpr2
board code and one the Ethernet driver extension. The driver itself
does not make sense to add to U-Boot if nobody is using it. My changes
to the mpr2 board to add Ethernet support do not make sense without
the driver.

Is it worth the effort splitting this patch into two pieces without
adjusting the driver's formatting or should I really forget it?

Regards,
Mark