netdev - Re: [PATCH 00/39] merge request for WiMAX kernel stack and i2400m driver v2

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [day] [month] [year] [list]

Message-Id: <AC362BF0-D9FE-42F5-97DA-754A31446B16@holtmann.org>
Date:	Thu, 27 Nov 2008 11:47:17 +0100
From:	Marcel Holtmann <marcel@...tmann.org>
To:	Inaky Perez-Gonzalez <inaky@...ux.intel.com>
Cc:	netdev@...r.kernel.org, wimax@...uxwimax.org
Subject: Re: [PATCH 00/39] merge request for WiMAX kernel stack and i2400m driver v2

Hi Inaky,

> This code is broken up in a WiMAX subsystem and a driver for the Intel
> 2400m Wireless WiMAX Link. To be able to fully utilize WiMAX services,
> you need extra pieces of device-specific user space software; for now,
> there is a C level API (the Common API) to use/control it, but a high
> level API over DBUS that should work for any device is in the
> works. See details further below.

this is not a big deal, but a more general question on how the layout  
of the kernel source code tree. So the whole WiMAX subsystem should be  
in net/wimax/, but for the drivers I think using drivers/net/wimax/ is  
wrong. None of these drivers should ever register an Ethernet or  
network device directly. They register themselves with the WiMAX  
subystem. So I would propose to use drivers/wimax/ here. We do the  
same with Bluetooth. All the drivers are under drivers/bluetooth/,  
because they don't register any kind of network device.

Jeff, what is your take on this. Shouldn't be drivers/net/ only  
contain drivers that actually use struct net_device somehow?

Regards

Marcel

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html