Subject: Re: A change to
To: Jason R Thorpe <thorpej@wasabisystems.com>
From: Hiroyuki Bessho <bsh@grotto.jp>
List: port-arm
Date: 09/20/2002 03:05:51
Jason R Thorpe <thorpej@wasabisystems.com> writes:

> On Thu, Sep 19, 2002 at 02:09:14AM +0900, Hiroyuki Bessho wrote:
>
>  >     Is it ok to modify sys/arch/arm/include/cpu.h as in following
>  > patch?
>  > 
>  >     I want to move prototype of atomic_{set,clear}_bit() so that I can
>  > use them in inline functions that are defined in a header file which
>  > is included from <machine/intr.h>.
>
> Does this work properly when <machine/intr.h> is included on its own?
>

    For evbarm, it's ok because <machine/cpu.h> is included before
BOARDTYPE_intr.h in <machine/intr.h>.

> Hmmmm... those atomic funcs probably ought to be put in <machine/atomic.h>
> as inlines, or something.
>

    It sounds nice to make them inline, but why not <arm/atomic.h>?  

--
bsh.