Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1560070 > unrolled thread

Re: Nokia N900: mixers changed between 4.9 and 4.10-rc3, no longer can use in-call speaker

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: Nokia N900: mixers changed between 4.9 and 4.10-rc3, no longer  can use in-call speaker Pavel Machek <pavel@ucw.cz> - 2017-01-16 22:30 +0100
    Re: Nokia N900: mixers changed between 4.9 and 4.10-rc3, no longer  can use in-call speaker Pavel Machek <pavel@ucw.cz> - 2017-01-18 22:00 +0100
      Re: Nokia N900: mixers changed between 4.9 and 4.10-rc3, no longer  can use in-call speaker Pavel Machek <pavel@ucw.cz> - 2017-01-18 23:10 +0100

#1560070 — Re: Nokia N900: mixers changed between 4.9 and 4.10-rc3, no longer can use in-call speaker

[Multipart message — attachments visible in raw view] — view raw

Hi!

> > So regression seems to be between v4.9 and v4.10. Any ideas?
> 
> Interesting... seems there are no sound relevant changes after v4.9.
> 
> Looks like there are only three commits after v4.9 for sound/soc which 
> are built for Nokia N900:

I'm at ce38207f161513ee3d2bd3860489f07ebe65bc78, which is already
broken, and first broken commit by Linus.

> e411b0b5eb9b65257a050eac333d181d6e00e2c6

Can't revert that easily.

> e7aa450fe17890e59db7d3c2d8eff5b6b41fc531

This can be reverted. Lets try. Did not help.

> 63c3194b82530bd71fd49db84eb7ab656b8d404a

Commit older than this one fails, IIRC.

> Maybe something not related to sound/soc could broke it?

Its definitely caused by something merged here:

commit ce38207f161513ee3d2bd3860489f07ebe65bc78
Merge: a9042de 995c6a7
Author: Linus Torvalds <torvalds@linux-foundation.org>
Date:   Wed Dec 14 11:14:28 2016 -0800

So it should be easy to find out..

Best regards,
								Pavel



-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[toc] | [next] | [standalone]

#1562171

[Multipart message — attachments visible in raw view] — view raw

Hi!

> > > So regression seems to be between v4.9 and v4.10. Any ideas?
> > 
> > Interesting... seems there are no sound relevant changes after v4.9.
> > 
> > Looks like there are only three commits after v4.9 for sound/soc which 
> > are built for Nokia N900:
...
> Its definitely caused by something merged here:
> 
> commit ce38207f161513ee3d2bd3860489f07ebe65bc78
> Merge: a9042de 995c6a7
> Author: Linus Torvalds <torvalds@linux-foundation.org>
> Date:   Wed Dec 14 11:14:28 2016 -0800
> 
> So it should be easy to find out..

Doing git bisect now. Hmmm.

pavel@amd:/data/l/linux-n900$ git bisect bad
1a653aa44725668590b36bbe2d7fe4736a69f055 is the first bad commit
commit 1a653aa44725668590b36bbe2d7fe4736a69f055
Author: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Date:   Wed Nov 30 06:22:55 2016 +0000

    ASoC: core: replace aux_comp_list to component_dev_list

    Now, Card has component_dev_list, we can replace aux_comp_list
        to component_dev_list with new auxiliary flags

    Signed-off-by: Kuninori Morimoto
    <kuninori.morimoto.gx@renesas.com>
        Signed-off-by: Mark Brown <broonie@kernel.org>

:040000 040000 a571fa05e5ce8e00b73bbc3f1e30e26775d619e8
e9336929cacf2d7bc94da99acfc10c9a7f513b8b M	include
:040000 040000 87976e9406d452e791870cc12e27cb67ba04fd12
60d36f685642621c0e3ebb61ba7a14186d8b3eb0 M	sound
pavel@amd:/data/l/linux-n900$

I'll try reverting this patch on top of 4.10-rc, next. Any other ideas?

Best regards,
									Pavel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[toc] | [prev] | [next] | [standalone]

#1562219

[Multipart message — attachments visible in raw view] — view raw

Hi!

> > > > So regression seems to be between v4.9 and v4.10. Any ideas?
> > > 
> > > Interesting... seems there are no sound relevant changes after v4.9.
> > > 
> > > Looks like there are only three commits after v4.9 for sound/soc which 
> > > are built for Nokia N900:
> ...
> > Its definitely caused by something merged here:
> > 
> > commit ce38207f161513ee3d2bd3860489f07ebe65bc78
> > Merge: a9042de 995c6a7
> > Author: Linus Torvalds <torvalds@linux-foundation.org>
> > Date:   Wed Dec 14 11:14:28 2016 -0800
> > 
> > So it should be easy to find out..
> 
> Doing git bisect now. Hmmm.
> 
> pavel@amd:/data/l/linux-n900$ git bisect bad
> 1a653aa44725668590b36bbe2d7fe4736a69f055 is the first bad commit
> commit 1a653aa44725668590b36bbe2d7fe4736a69f055
> Author: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> Date:   Wed Nov 30 06:22:55 2016 +0000
> 
>     ASoC: core: replace aux_comp_list to component_dev_list
> 
>     Now, Card has component_dev_list, we can replace aux_comp_list
>         to component_dev_list with new auxiliary flags
> 
>     Signed-off-by: Kuninori Morimoto
>     <kuninori.morimoto.gx@renesas.com>
>         Signed-off-by: Mark Brown <broonie@kernel.org>
> 
> :040000 040000 a571fa05e5ce8e00b73bbc3f1e30e26775d619e8
> e9336929cacf2d7bc94da99acfc10c9a7f513b8b M	include
> :040000 040000 87976e9406d452e791870cc12e27cb67ba04fd12
> 60d36f685642621c0e3ebb61ba7a14186d8b3eb0 M	sound
> pavel@amd:/data/l/linux-n900$
> 
> I'll try reverting this patch on top of 4.10-rc, next. Any other ideas?

Reverting that fixed the issue, but when I re-checked current -rc4, it
seems it has similar fix there, too. GOod.

								Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web