[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Openvortex-dev] build patches, other patches

From: Jeff Muizelaar
Subject: Re: [Openvortex-dev] build patches, other patches
Date: Thu, 24 Apr 2003 19:46:47 -0400 (EDT)

On Thu, 24 Apr 2003, Will DeRousse wrote:

> Hi, 4 more patches for review.
> The first two fix building on new checkouts. The last patch for some reason 
> only did half the changes.
> The third is an error handling cleanup in au88x0.c. It seems fairly common in 
> kernel code but did not save on module size (oddly).

gcc will drop the common code out to the end like you did by hand. (this
is a guess whithout actually looking at the generated asm)

> The fourth is my second attempt at reindenting header files (au8810.h only). 
> I also removed some duplicate defines.

seems to be already there.




I am not going to commit this yet. This is basically a case of alsa style
vs. kernel style. The main reason for doing stuff like this is to
make unwinding/deallocation stuff sane. However, in the case of
snd_card_free, alsa does this for us. The cleanup for snd_vortex_free is
actually a buggy as it will try and release things that we might not have.
I'll probably clean/fix this tonight.


I'll wait for Manuel to say something on this one.

Will, if you are looking for more things to do, one thing that I have been
thinking about is a script/program that takes au8810.h, au8820.h and


#define VORTEX_IRQ_CTRL 0x2a004

#define VORTEX_IRQ_CTRL 0x12804

#define VORTEX_IRQ_CTRL 0x2a004

and produces something like:

struct reg_defn{
        char *name;
        int au8810;
        int au8820;
        int au8830;

struct reg_defn vortex_regs[] = {
        {"VORTEX_IRQ_CTRL", 0x2a000, 0x12804, 0x2a000 },
        { and so on for each of the registers}


reply via email to

[Prev in Thread] Current Thread [Next in Thread]