[PATCH 6/6] [WIP] libertas: add support for 8385 CompactFlash cards
dcbw at redhat.com
Tue Jun 19 11:54:07 EDT 2007
On Tue, 2007-06-19 at 17:30 +0200, Holger Schurig wrote:
> > > /** INT status Bit Definition*/
> > > -#define his_cmddnldrdy 0x01
> > > -#define his_cardevent 0x02
> > > -#define his_cmdupldrdy 0x04
> > > +#define his_cmddnldrdy 0x0004
> > > +#define his_cmdupldrdy 0x0008
> > > +#define his_cardevent 0x0010
> > Can you upper-case these?
> They are uppercase, but I posted a patch against the wrong
> branch. In the patch against libertas-2.6 they are uppercase.
> > Also, the 8686 SDIO driver defines
> > them this way; does that just mean that the SDIO part is
> > different (in which case you should probably just make them CF
> > specific?), or should they actually be the same?
> > /** INT Status Bit Definition*/
> > #define HIS_RxUpLdRdy 0x01
> > #define HIS_TxDnLdRdy 0x02
> > #define HIS_CmdDnLdRdy 0x04
> > #define HIS_CardEvent 0x08
> > #define HIS_CmdUpLdRdy 0x10
> > #define HIS_WrFifoOvrflow 0x20
> > #define HIS_RdFifoUndrflow 0x40
> > #define HIS_WlanReady 0x80
> Yes, they have other bitfields, and if you compare the 3 ones I'm
> using, you see that they have the same values as the SDIO ones.
> That's to be expected, because SDIO and CF directly access the
> registers of the WLAN chip.
+#define his_cmddnldrdy 0x0004
+#define his_cmdupldrdy 0x0008
+#define his_cardevent 0x0010
Here you've defined cardevent = 0x0010, but the SDIO driver has
cardevent = 0x0008... are they just different for each part?
> The USB driver used to work with completely different bits set,
> because here the bit's aren't used to directly access chip
> registers. USB set's them by itself, see usb_int_cause in
> if_usb, and so it doesn't care what bit values they are, as long
> as they are different.
> By changing them from the old values (1,2,4) to the new ones that
> reflect the hardware register (4,8,10) I'm saving a translation.
> BTW: I'm sending an updated version of my driver against
> branch "libertas" ASAP.
> libertas-dev mailing list
> libertas-dev at lists.infradead.org
More information about the libertas-dev