On Sun, 2007-06-10 15:11:22 +0200, Karsten Keil <[email protected]> wrote: > > --- a/drivers/isdn/hisax/callc.c > > +++ b/drivers/isdn/hisax/callc.c > > @@ -834,7 +834,7 @@ static struct FsmNode fnlist[] __initdata = > > }; > > /* *INDENT-ON* */ > > > > -#define FNCOUNT (sizeof(fnlist)/sizeof(struct FsmNode)) > > +#define FNCOUNT ARRAY_SIZE(fnlist) > > > > int __init > > CallcNew(void) > > This is OK, thanks for changing. I'd suggest to not use another define, but use ARRAY_SIZE(foo) _instead of_ eg. FNCOUNT. MfG, JBG -- Jan-Benedict Glaw [email protected] +49-172-7608481 Signature of: Ich hatte in letzter Zeit ein bißchen viel Realitycheck. the second : Langsam möchte ich mal wieder weiterträumen können. -- Maximilian Wilhelm (18. Mai 2006, #lug-owl.de)
Attachment:
signature.asc
Description: Digital signature
- Follow-Ups:
- Re: [PATCH] drivers/isdn/hisax: ARRAY_SIZE instead of sizeof
- From: Andi Drebes <[email protected]>
- Re: [PATCH] drivers/isdn/hisax: ARRAY_SIZE instead of sizeof
- References:
- [PATCH] drivers/isdn/hisax: ARRAY_SIZE instead of sizeof
- From: Andi Drebes <[email protected]>
- Re: [PATCH] drivers/isdn/hisax: ARRAY_SIZE instead of sizeof
- From: Karsten Keil <[email protected]>
- [PATCH] drivers/isdn/hisax: ARRAY_SIZE instead of sizeof
- Prev by Date: Re: [PATCH] drivers/isdn/hisax: ARRAY_SIZE instead of sizeof
- Next by Date: Re: [patch-mm 06/23] clockevents: Fix resume logic
- Previous by thread: Re: [PATCH] drivers/isdn/hisax: ARRAY_SIZE instead of sizeof
- Next by thread: Re: [PATCH] drivers/isdn/hisax: ARRAY_SIZE instead of sizeof
- Index(es):
 |