[RFC] How to rename SSB_TMSLOW_*, B43_TMSLOW_*?

Michael Büsch mb at bu3sch.de
Wed Feb 16 17:17:37 EST 2011


On Wed, 2011-02-16 at 14:13 +0100, Rafał Miłecki wrote: 
> Except for following 3 defines:
> #define  SSB_TMSLOW_RESET	0x00000001 /* Reset */
> #define  SSB_TMSLOW_REJECT_22	0x00000002 /* Reject (Backplane rev 2.2) */
> #define  SSB_TMSLOW_REJECT_23	0x00000004 /* Reject (Backplane rev 2.3) */
> 
> All our SSB_TMSLOW_* and B43_TMSLOW_* defines are some core control
> bits. As we now know, core control bits are not SSB specific or TMSLOW
> specific.
> 
> Should we (and how) define that names in this situation?
> 
> For b43 I propose (quite obvious?) B43_CORE_CTL_*.
> 
> However what about SSB_TMSLOW_*? George proposed SSB_CORECTL_*, but it
> contains "SSB", while that bits are not SSB specific. Same bits are
> used on AI bus. Should we use some SSBAI_CORE_CTL_* then? Any other
> ideas? Some better maybe?
> 
> P.S.
> Personally I prefer CORE_CTL over CORECTL (George). Which one should we use?

Let's simply put those bits into the drivers and call them
DRIVERNAME_TMSLOW_FOOBAR

The "TMSLOW" part seems rather important to me, because that makes it
obvious what register these bits belong to. Note that's there's also
TMSHIGH. It also follows current naming convention.

-- 
Greetings Michael.




More information about the b43-dev mailing list