On Mon, 2007-09-24 at 14:07 +0100, Christoph Hellwig wrote:
> On Mon, Sep 24, 2007 at 08:59:35AM -0400, Pavel Roskin wrote:
> > On Sat, 2007-09-22 at 12:22 +0200, Johannes Berg wrote:
> >
> > > struct ieee80211_channel {
> > > /* first part set by driver */
> > > u16 freq; /* in MHz */
> > > int hardware_value; /* arbitrary */
> >
> > Please consider using "long" for hardware_value. Drivers should be able
> > to use this value as a pointer to some driver-specific structure.
>
> If you want to use it as pointer better make it a pointer..
Then we'd have to have two values. I don't see a need for having a
pointer at all though. Pavel, can you show a driver that really benefits
from that? And explain why it can't use an array index here?
johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
