[alsa-devel] [PATCH 4/7] ice1724 - adding GPIO routines for mask and direction
tiwai at suse.de
Wed Sep 16 09:48:22 CEST 2009
At Wed, 16 Sep 2009 09:15:56 +0200,
Pavel Hofman wrote:
> Takashi Iwai wrote:
> > At Wed, 16 Sep 2009 00:39:53 +0200,
> > pavel.hofman at ivitera.com wrote:
> >> From: pavel <pavel at nahore.(none)>
> >> * get/set routines for GPIO mask and direction
> >> Signed-off-by: Pavel Hofman <pavel.hofman at ivitera.com>
> > The get_mask callback is still undefined for ice1724...
> That is weird. I split the original commit using interactive git rebase
> + edit + add interactive into three new commits - get-mask,
> pro-rate-locked and all the external-clock. I commited all of them,
> rechecked git status before continuing with rebase - nothing uncommitted
> was left, no outstanding changes. But the get_mask commit did not show
> up in the subsequent interactive rebase list of commits. I was looking
> for it, I thought I found it in the main gpio-methods commit but
> overlooked that it is for ice1712.c only.
> The method and its callback has disappeared from my master after the
> rebase. Do you have any idea what could have gone wrong?
But rebasing is a bit dangerous operation, and I often forget to finish
the rebase until the last commit, getting incomplete rebase in the
end. So, I usually copy the original branch first before rebase
explicitly, and check the difference from the original branch after
> I will fix and repost in the evening.
> >> --- a/pci/ice1712/ice1712.h
> >> +++ b/pci/ice1712/ice1712.h
> >> @@ -28,6 +28,7 @@
> >> #include <sound/i2c.h>
> >> #include <sound/ak4xxx-adda.h>
> >> #include <sound/ak4114.h>
> >> +#include <sound/ak4113.h>
> > We don't need this header here.
> > (Ditto for ak4114.h and pt2258.h. They were needed because the
> > structure were referred in ice1712.h. But they are now gone to the
> > local structure, so no need in the common header.)
> Yesterday I moved it from ice1712.h to quartet.h. The build succeeded
> but when loading the new snd-ice1724 module I got dmesg messages about
> unknown references to ak4113 methods used in the quartet code. That is
> why I left the include in ice1712.h which works fine.
> Unfortunately my knowledge of the build/load infrastructure is very
> limited, I can do just copy/paste in this area :)
More information about the Alsa-devel