Dave,
> I still haven't had a chance to try the patch to see if it even
> compiles, but from my perusal of the code, it still looks reasonable.
I get an error when compiling:
| sound/pci/ac97/ac97_patch.c: In function 'patch_ucb1400':
| sound/pci/ac97/ac97_patch.c:590: error: invalid type argument of '->'
| sound/pci/ac97/ac97_patch.c:591: warning: implicit declaration of
function 'proc_match'
The lines in question (line 590 is the one beginning with "for"):
proc_gpio_parent = NULL;
for (proc_gpio_parent = proc_root->subdir; proc_gpio_parent;
proc_gpio_parent = proc_gpio_parent->next) {
if (proc_match(4, "gpio", proc_gpio_parent))
break;
}
I haven't used the proc stuff before, so I'll go research & try to fix this.
Steve
On Jan 31, 2008 1:15 PM, Dave Hylands <dhylands@...> wrote:
> Hi Steve,
>
> > Did you have a chance to look at this any further?
> >
> > Just want to make sure I don't duplicate any work you are doing.
>
> As best I can tell, the proc_mkdir is basically just a shortcut for
> the create_proc_entry. So I'd keep the proc_mkdir and otherwise use
> the patch. I couldn't find any functions for querying if a particular
> proc entry actually exists or not (which is what the for loop in the
> patch is essentially doing).
>
> I still haven't had a chance to try the patch to see if it even
> compiles, but from my perusal of the code, it still looks reasonable.
> If it does in fact work, it needs to be applied to both the proc_gpio
> and sound card.
>
> --
>
> Dave Hylands
> Vancouver, BC, Canada
> http://www.DaveHylands.com/
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Microsoft
> Defy all challenges. Microsoft(R) Visual Studio 2008.
> http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
> _______________________________________________
> gumstix-users mailing list
> gumstix-users@...
> https://lists.sourceforge.net/lists/listinfo/gumstix-users
>
|