linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* RE: [PATCH] /driver/net/wan/sbs520
@ 2005-01-10 19:49 Adam Anthony
  2005-01-10 20:11 ` Stephen Hemminger
  2005-01-10 20:29 ` Francois Romieu
  0 siblings, 2 replies; 8+ messages in thread
From: Adam Anthony @ 2005-01-10 19:49 UTC (permalink / raw)
  To: Matthias-Christian Ott, Alexey Dobriyan
  Cc: Adam Anthony, netdev, linux-kernel

Thank you for the heads up A&M.  I have destroyed the evil [^M]'s, and
updated the package.
http://prdownloads.sourceforge.net/sbs520lnxdrv/sbs520patch.bz2?download
-AA


-----Original Message-----
From: Matthias-Christian Ott [mailto:matthias.christian@tiscali.de] 
Sent: Monday, January 10, 2005 12:26 PM
To: Alexey Dobriyan
Cc: Adam Anthony; netdev@oss.sgi.com; linux-kernel@vger.kernel.org
Subject: Re: [PATCH] /driver/net/wan/sbs520

Alexey Dobriyan wrote:

>On Mon, 10 Jan 2005 07:46:52 -0700, Adam Anthony wrote:
>
>  
>
>>With the permission of my employer, SBS Technologies, Inc., I have
>>released a patch for 2.4 kernels that supports the 520 Series of WAN
>>adapters.
>>    
>>
>
>My editor shows ^M at the end of every line of new
Documentation/Configure.help,
>MAINTAINERS (add ~63400 bogus lines!). Please, look at the patch _after_
>generating it.
>
>  
>
>>+obj-$(CONFIG_LANMEDIA)		+=		syncppp.o^M
>>    
>>
>
>  
>
>>+subdir-$(CONFIG_LANMEDIA) += lmc^M
>>    
>>
>
>Also random ^M's.
> 
>--- linux-2.4.28-virgin/drivers/net/wan/sbs520/lnxosl.c
>+++ /usr/src/linux-2.4.28/drivers/net/wan/sbs520/lnxosl.c
>
>  
>
>>+// Programming Language:	C^M
>>+// Target Processor:		Any^M
>>+// Target Operating System: Linux^M
>>    
>>
>
>Well, this is pretty obvious to everyone here. :-)
>
>  
>
>>+// This software may be used and distributed according to the terms^M
>>+// of the GNU General Public License, incorporated herein by reference.^M
>>    
>>
>
>Stupid question: do you mean GPL version 2 or something else?
>
>	Alexey
>-
>To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>the body of a message to majordomo@vger.kernel.org
>More majordomo info at  http://vger.kernel.org/majordomo-info.html
>Please read the FAQ at  http://www.tux.org/lkml/
>
>  
>
That's ugly, that are Microsoft line endings.

Matthias-Christian Ott

***This message has been scanned for virus, spam, and undesirable
content.***
***For further information, contact your mail administrator.***

For limitations on the use and distribution of this message, please visit www.sbs.com/emaildisclaimer.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] /driver/net/wan/sbs520
  2005-01-10 19:49 [PATCH] /driver/net/wan/sbs520 Adam Anthony
@ 2005-01-10 20:11 ` Stephen Hemminger
  2005-01-10 20:29 ` Francois Romieu
  1 sibling, 0 replies; 8+ messages in thread
From: Stephen Hemminger @ 2005-01-10 20:11 UTC (permalink / raw)
  To: linux-kernel

Like most vendor drivers, this looks ugly full of obfuscating typedef's
and MixedCase. Please make it conform to linux style.

Also, all global's should be static where possible to avoid name space pollution.
Replace volatile with wmb() and rmb() which is probably what you need anyway.
Don't bury volatile inside typedef.s



^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] /driver/net/wan/sbs520
  2005-01-10 19:49 [PATCH] /driver/net/wan/sbs520 Adam Anthony
  2005-01-10 20:11 ` Stephen Hemminger
@ 2005-01-10 20:29 ` Francois Romieu
  1 sibling, 0 replies; 8+ messages in thread
From: Francois Romieu @ 2005-01-10 20:29 UTC (permalink / raw)
  To: Adam Anthony
  Cc: Matthias-Christian Ott, Alexey Dobriyan, netdev, linux-kernel

Adam Anthony <AAnthony@sbs.com> :
> Thank you for the heads up A&M.  I have destroyed the evil [^M]'s, and
> updated the package.
> http://prdownloads.sourceforge.net/sbs520lnxdrv/sbs520patch.bz2?download

o OsMapPhysToVirt
  -> should be ioremap/pci_iomap()

o OsUnMapVirt
  -> iounmap, etc.

o OsAllocateNonPagedMemory/OsMemcpy/OsStall/OsSleep/OsZeroMem
  -> useless wrappers.

o OsAllocateDeviceMemory
  Yuck, virt_to_bus !
  Please read:
  - linux-2.6.x/Documentation/DMA-mapping.txt
  - linux-2.6.x/Documentation/DMA-API.txt
  drivers/net/*.c provides a lot of good examples for recent PCI
  devices.

o OsReadPciConfiguration
  Please see pci_resource_{start/len} and friends.

At this point, lnxosl.c will be removable and nobody will regret it.

--
Ueimor

^ permalink raw reply	[flat|nested] 8+ messages in thread

* RE: [PATCH] /driver/net/wan/sbs520
@ 2005-01-10 20:49 Adam Anthony
  0 siblings, 0 replies; 8+ messages in thread
From: Adam Anthony @ 2005-01-10 20:49 UTC (permalink / raw)
  To: Francois Romieu
  Cc: Matthias-Christian Ott, Alexey Dobriyan, netdev, linux-kernel

	The SBS Communications group structured their software this way
(w/useless wrappers) to make files portable from one operating system to
another.  Now that this package will be taking a life of its own, the change
you propose makes perfect sense.
Thanks,
Adam

-----Original Message-----
From: Francois Romieu [mailto:romieu@fr.zoreil.com] 
Sent: Monday, January 10, 2005 3:30 PM
To: Adam Anthony
Cc: Matthias-Christian Ott; Alexey Dobriyan; netdev@oss.sgi.com;
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] /driver/net/wan/sbs520

o OsMapPhysToVirt
  -> should be ioremap/pci_iomap()

o OsUnMapVirt
  -> iounmap, etc.

o OsAllocateNonPagedMemory/OsMemcpy/OsStall/OsSleep/OsZeroMem
  -> useless wrappers.

o OsAllocateDeviceMemory
  Yuck, virt_to_bus !
  Please read:
  - linux-2.6.x/Documentation/DMA-mapping.txt
  - linux-2.6.x/Documentation/DMA-API.txt
  drivers/net/*.c provides a lot of good examples for recent PCI
  devices.

o OsReadPciConfiguration
  Please see pci_resource_{start/len} and friends.

At this point, lnxosl.c will be removable and nobody will regret it.

--
Ueimor

For limitations on the use and distribution of this message, please visit www.sbs.com/emaildisclaimer.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] /driver/net/wan/sbs520
  2005-01-10 14:46 Adam Anthony
@ 2005-01-10 18:00 ` Francois Romieu
  0 siblings, 0 replies; 8+ messages in thread
From: Francois Romieu @ 2005-01-10 18:00 UTC (permalink / raw)
  To: Adam Anthony; +Cc: netdev, linux-kernel

Adam Anthony <AAnthony@sbs.com> :
[...]
>        It would be great to receive some feedback on our work, and we hope
> that this driver will eventually be added to the kernel.

It will probably require a few extra steps:
- read Documentation/CodingStyle (mixed case, typedef from hell, ugly #ifdef);
- grep ^static
  -> no static functions ? Uh ?
- use non-obsolete API (pci_find_device in 2005 ?);
- convert the os independant wrappers.

Btw it would probably make sense 1) to figure out what can be merged with
the in-tree DSCC4 driver and 2) to integrate the driver with the existing
hdlc stack. Imho there is some duplicated work/code.

--
Ueimor

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] /driver/net/wan/sbs520
@ 2005-01-10 17:47 Alexey Dobriyan
  2005-01-10 17:26 ` Matthias-Christian Ott
  0 siblings, 1 reply; 8+ messages in thread
From: Alexey Dobriyan @ 2005-01-10 17:47 UTC (permalink / raw)
  To: Adam Anthony; +Cc: netdev, linux-kernel

On Mon, 10 Jan 2005 07:46:52 -0700, Adam Anthony wrote:

> With the permission of my employer, SBS Technologies, Inc., I have
> released a patch for 2.4 kernels that supports the 520 Series of WAN
> adapters.

My editor shows ^M at the end of every line of new Documentation/Configure.help,
MAINTAINERS (add ~63400 bogus lines!). Please, look at the patch _after_
generating it.

> +obj-$(CONFIG_LANMEDIA)		+=		syncppp.o^M

> +subdir-$(CONFIG_LANMEDIA) += lmc^M

Also random ^M's.
 
--- linux-2.4.28-virgin/drivers/net/wan/sbs520/lnxosl.c
+++ /usr/src/linux-2.4.28/drivers/net/wan/sbs520/lnxosl.c

> +// Programming Language:	C^M
> +// Target Processor:		Any^M
> +// Target Operating System: Linux^M

Well, this is pretty obvious to everyone here. :-)

> +// This software may be used and distributed according to the terms^M
> +// of the GNU General Public License, incorporated herein by reference.^M

Stupid question: do you mean GPL version 2 or something else?

	Alexey

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] /driver/net/wan/sbs520
  2005-01-10 17:47 Alexey Dobriyan
@ 2005-01-10 17:26 ` Matthias-Christian Ott
  0 siblings, 0 replies; 8+ messages in thread
From: Matthias-Christian Ott @ 2005-01-10 17:26 UTC (permalink / raw)
  To: Alexey Dobriyan; +Cc: Adam Anthony, netdev, linux-kernel

Alexey Dobriyan wrote:

>On Mon, 10 Jan 2005 07:46:52 -0700, Adam Anthony wrote:
>
>  
>
>>With the permission of my employer, SBS Technologies, Inc., I have
>>released a patch for 2.4 kernels that supports the 520 Series of WAN
>>adapters.
>>    
>>
>
>My editor shows ^M at the end of every line of new Documentation/Configure.help,
>MAINTAINERS (add ~63400 bogus lines!). Please, look at the patch _after_
>generating it.
>
>  
>
>>+obj-$(CONFIG_LANMEDIA)		+=		syncppp.o^M
>>    
>>
>
>  
>
>>+subdir-$(CONFIG_LANMEDIA) += lmc^M
>>    
>>
>
>Also random ^M's.
> 
>--- linux-2.4.28-virgin/drivers/net/wan/sbs520/lnxosl.c
>+++ /usr/src/linux-2.4.28/drivers/net/wan/sbs520/lnxosl.c
>
>  
>
>>+// Programming Language:	C^M
>>+// Target Processor:		Any^M
>>+// Target Operating System: Linux^M
>>    
>>
>
>Well, this is pretty obvious to everyone here. :-)
>
>  
>
>>+// This software may be used and distributed according to the terms^M
>>+// of the GNU General Public License, incorporated herein by reference.^M
>>    
>>
>
>Stupid question: do you mean GPL version 2 or something else?
>
>	Alexey
>-
>To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>the body of a message to majordomo@vger.kernel.org
>More majordomo info at  http://vger.kernel.org/majordomo-info.html
>Please read the FAQ at  http://www.tux.org/lkml/
>
>  
>
That's ugly, that are Microsoft line endings.

Matthias-Christian Ott

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH] /driver/net/wan/sbs520
@ 2005-01-10 14:46 Adam Anthony
  2005-01-10 18:00 ` Francois Romieu
  0 siblings, 1 reply; 8+ messages in thread
From: Adam Anthony @ 2005-01-10 14:46 UTC (permalink / raw)
  To: netdev, linux-kernel; +Cc: Adam Anthony

SGI and Kernel Maintainers,
	With the permission of my employer, SBS Technologies, Inc., I have
released a patch for 2.4 kernels that supports the 520 Series of WAN
adapters.  (A similar patch for 2.6 kernels will soon follow)  The driver
itself has been tested by many customers, but as of this morning, the patch
has only been tested by me.
       I have created a Source Forge project for the patch/GPL driver, which
can be viewed at the following location:
"https://sourceforge.net/projects/sbs520lnxdrv/".  The patch can be
downloaded directly from the following link:
"http://prdownloads.sourceforge.net/sbs520lnxdrv/sbs520patch.bz2?download"
       It would be great to receive some feedback on our work, and we hope
that this driver will eventually be added to the kernel.
Best regards,
Adam

Signed-off-by: Adam T. Anthony <aanthony@sbs.com>



For limitations on the use and distribution of this message, please visit www.sbs.com/emaildisclaimer.

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2005-01-10 20:52 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-01-10 19:49 [PATCH] /driver/net/wan/sbs520 Adam Anthony
2005-01-10 20:11 ` Stephen Hemminger
2005-01-10 20:29 ` Francois Romieu
  -- strict thread matches above, loose matches on Subject: below --
2005-01-10 20:49 Adam Anthony
2005-01-10 17:47 Alexey Dobriyan
2005-01-10 17:26 ` Matthias-Christian Ott
2005-01-10 14:46 Adam Anthony
2005-01-10 18:00 ` Francois Romieu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).