Re: [RFC][PATCH 0/4] p54: various updates

[Chr <chunkeey@web.de>]

On Friday 04 April 2008 00:34:35 Luis R. Rodriguez wrote:
> On Thu, Apr 3, 2008 at 11:40 AM, Chr <chunkeey@web.de> wrote:
> > On Thursday 03 April 2008 01:42:59 Luis R. Rodriguez wrote:
> >  > Thanks, can you resend as inline patches so it can help get better
> >  > review. Also while you're at it can you stuff all p54 files into its
> >  > own folder?
> >
> >  hmm, I can.... but:
> >  - in which folder? for now.. I put everything into a new one folder
> > named p54, but as you know the fullmac driver has already the prism54
> > subdirectory... should I put it there? or is this confusing?
>
> p54 directory works nice. Now that you are introducing master mode the
> only thing left I think is IBSS compared to prism54, and if that is
> done I think we should *consider* removing prism54 driver after one
> release cycle of the kernel. I once was told that for some cards the
> softmac driver wouldn't work on some fullmac chipsets. I have yet to
> hear of someone who found these type of cards though. I know we also
> had some WDS code for prism54 lying around, it just never got merged
> in because Jeff didn't find it widely used and IIRC it needed a little
> more cleanup.
>
> Think you can get IBSS and maybe WDS to work too?
>
Hmm, AFAIK they were thinking of the 3877 (802.11a only)... I guess this one
won't work with p54. But have you ever see anyone (except the 
Intersil/Conexant people) with a 5Ghz prism card at all? It'll be interesting 
to get a eeprom dump from them... 

About WDS: is there anything special about WDS, that I should know? 
Since, I thought it's entirely done by mac80211 by now?

About IBSS: yes, it is kind of working... I'll put some code for this in the 
next round.

> >  - the resulting patch is too big to post it here, ~ 271kb as
> > plaintext... is a bz2 (32kb)  attachment ok too?
>
> Oh OK, I see. Can you perhaps split the big patch up into a few series?
that'll be about 3 or 4 parts... tomorrow! I have to figure out how
I can split up the _big_ one.

Regards,
	Chr