* linux 2.6.34 bug
2010-07-08 16:42 ` linux 2.6.34 bug Marek Vasut
@ 2010-07-08 19:22 ` Uwe Kleine-König
2010-07-13 1:50 ` Eric Miao
1 sibling, 0 replies; 4+ messages in thread
From: Uwe Kleine-König @ 2010-07-08 19:22 UTC (permalink / raw)
To: linux-arm-kernel
Hello,
On Thu, Jul 08, 2010 at 06:42:16PM +0200, Marek Vasut wrote:
> Dne ?t 8. ?ervence 2010 17:36:21 dylan cristiani napsal(a):
> > Hi Marek,
> > we were in touch few moths ago about a ucb1400 touchscreen, issue if you
> > remember: you sent me a patch to pass the platform specific codec irq,
> > through ucb1400_pdata; after that i went from 3.6.32 to 2.6.34 and
> > i noticed that your patch was into mainline; in this kernel, related to
> > the ucb1400 codec again, i noticed one bug about the ac97 codec reset
> > that is bulverde platform specific: i don't know hot to submit a patch
> > and to whom to submit it, so i bother, you sorry in advance....; in
> > particular: in new kernel the function
> > "void pxa27x_assert_ac97reset(int reset_gpio, int on)"
> > has been moved to arch/arm/mach-pxa/pxa27x.c file:
>
> CCing linux-arm-kernel and Eric.
>
> btw. to submit patch :
>
> git add <list of changed files>
>
> git commit -s
> <editor will pop up, type the commit message>
>
> git log # here check the hash of the last commit before yours
>
> git send-email --to="linux-arm-kernel at lists.infradead.org" --
> cc="eric.y.miao at gmail.com" --cc="marek.vasut at gmail.com" <the hash from previous
> step>
>
> That's probably the easiest way :)
No. Instead of checking the hash of the last commit via git log, you
can simply pass "HEAD~1" instead of the actual hash.
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-K?nig |
Industrial Linux Solutions | http://www.pengutronix.de/ |
^ permalink raw reply [flat|nested] 4+ messages in thread
* linux 2.6.34 bug
2010-07-08 16:42 ` linux 2.6.34 bug Marek Vasut
2010-07-08 19:22 ` Uwe Kleine-König
@ 2010-07-13 1:50 ` Eric Miao
2010-07-13 10:44 ` Dylan Cristiani
1 sibling, 1 reply; 4+ messages in thread
From: Eric Miao @ 2010-07-13 1:50 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Jul 9, 2010 at 12:42 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
> Dne ?t 8. ?ervence 2010 17:36:21 dylan cristiani napsal(a):
>> Hi Marek,
>> we were in touch few moths ago about a ucb1400 touchscreen, issue if you
>> remember: you sent me a patch to pass the platform specific codec irq,
>> through ucb1400_pdata; after that i went from 3.6.32 to 2.6.34 and
>> i noticed that your patch was into mainline; in this kernel, related to
>> the ucb1400 codec again, i noticed one bug about the ac97 codec reset
>> that is bulverde platform specific: i don't know hot to submit a patch
>> and to whom to submit it, so i bother, you sorry in advance....; in
>> particular: in new kernel the function
>> "void pxa27x_assert_ac97reset(int reset_gpio, int on)"
>> has been moved to arch/arm/mach-pxa/pxa27x.c file:
>
> CCing linux-arm-kernel and Eric.
>
> btw. to submit patch :
>
> git add <list of changed files>
>
> git commit -s
> <editor will pop up, type the commit message>
>
> git log # here check the hash of the last commit before yours
>
> git send-email --to="linux-arm-kernel at lists.infradead.org" --
> cc="eric.y.miao at gmail.com" --cc="marek.vasut at gmail.com" <the hash from previous
> step>
>
> That's probably the easiest way :)
>>
>> ......
>> static unsigned long ac97_reset_config[] = {
>> ? ? ? GPIO95_AC97_nRESET,
>> ? ? ? GPIO95_GPIO,
>> ? ? ? GPIO113_AC97_nRESET,
>> ? ? ? GPIO113_GPIO,
>> };
>>
>> void pxa27x_assert_ac97reset(int reset_gpio, int on)
>> {
>> ? ? ? if (reset_gpio == 113)
>> ? ? ? ? ? ? ? pxa2xx_mfp_config(on ? &ac97_reset_config[0] :
>> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?&ac97_reset_config[1], 1);
>>
>> ? ? ? if (reset_gpio == 95)
>> ? ? ? ? ? ? ? pxa2xx_mfp_config(on ? &ac97_reset_config[2] :
>> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?&ac97_reset_config[3], 1);
>> }
>> EXPORT_SYMBOL_GPL(pxa27x_assert_ac97reset);
>> .......
>>
>> but, as it seems to me, there are two bugs: the first is the cross
>> exchange of the position of the
>> ? ? ? 'if (reset_gpio == 113)'
>> with position of
>> ? ? ? 'if (reset_gpio == 95)'
>> with regard to the position of the members of the array
>> "static unsigned long ac97_reset_config[]"
>
> Seems like a good catch, Eric ?
>>
>> The 2nd one regards the "pxa2xx_mfp_config(..)" function's call at
>> sound/arm/pxa2xx-ac97-lib.c line 340:
>>
>> ? ? ? if (cpu_is_pxa27x()) {
>> ? ? ? ? ? ? ? /* Use GPIO 113 as AC97 Reset on Bulverde */
>> ? ? ? ? ? ? ? pxa27x_assert_ac97reset(reset_gpio, 0);
>> ? ? ? ? ? ? ? ac97conf_clk = clk_get(&dev->dev, "AC97CONFCLK");
>> ? ? ? ? ? ? ? .....
>>
>> so, if the second param is 0, (as the comment /**/ states) you want to
>> use gpio 113 as AC97 reset (that is the Alternative Function 2 of the
>> pin), so the function call into arch/arm/mach-pxa/pxa27x.c has to be:
>>
>> ? ? ? if (reset_gpio == 113)
>> ? ? ? ? ? ? ? pxa2xx_mfp_config(on ? &ac97_reset_config[3] :
>> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?&ac97_reset_config[2], 1);
>>
>> instead the current:
>>
>> ? ? ? if (reset_gpio == 113)
>> ? ? ? ? ? ? ? pxa2xx_mfp_config(on ? &ac97_reset_config[2] :
>> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?&ac97_reset_config[3], 1);
>>
>> i noticed that you can solve the whole bugs just moving the array
>> members of "ac97_reset_config[]" into this:
>>
>> static unsigned long ac97_reset_config[] = {
>> ? ? ? GPIO113_GPIO,
>> ? ? ? GPIO113_AC97_nRESET,
>> ? ? ? GPIO95_GPIO,
>> ? ? ? GPIO95_AC97_nRESET,
>> };
>>
>> i tryed this and for me it works; i'm sorry again to bother you but i
>> don't really know how to submit a bug report so maybe you can do this
>> if it sounds good to your expert ears
>>
>> tks and sorry for the noise and don't hate me if i wrote some
>> garbage stuff...
>
> Eric, care to explain? I see a valid point here.
It was my bad. Fixed updated as below. Dylan, please help review:
[ARM] pxa: fix incorrect order of AC97 reset pin configs
Reported-by: Dylan Cristiani <d.cristiani@idem-tech.it>
Signed-off-by: Eric Miao <eric.y.miao@gmail.com>
diff --git a/arch/arm/mach-pxa/pxa27x.c b/arch/arm/mach-pxa/pxa27x.c
index 0af3617..c059dac 100644
--- a/arch/arm/mach-pxa/pxa27x.c
+++ b/arch/arm/mach-pxa/pxa27x.c
@@ -41,10 +41,10 @@ void pxa27x_clear_otgph(void)
EXPORT_SYMBOL(pxa27x_clear_otgph);
static unsigned long ac97_reset_config[] = {
- GPIO95_AC97_nRESET,
- GPIO95_GPIO,
- GPIO113_AC97_nRESET,
GPIO113_GPIO,
+ GPIO113_AC97_nRESET,
+ GPIO95_GPIO,
+ GPIO95_AC97_nRESET,
};
void pxa27x_assert_ac97reset(int reset_gpio, int on)
^ permalink raw reply related [flat|nested] 4+ messages in thread