3.0.11 backport 11b8fc6ae54bf18a48c94e181c37ca135b858b42 is broken, please revert

November 30th, 2011 - 05:30 pm ET by Koen Kooi | Report spam

Hi,

I noticed that audio stopped working after 3.0.10 which I narrowed down to:

11b8fc6ae54bf18a48c94e181c37ca135b858b42 mfd: Fix twl4030 dependencies for audio codec

The commit message has this bit:

"In commit 57fe7251f5bfc4332f24479376de48a1e8ca6211
the CONFIG_TWL4030_CODEC was renamed
into CONFIG_MFD_TWL4030_AUDIO, thatswhy the codec
was not detected."

Which is this one: http://git.kernel.org/?p=linux/kern...a1e8ca6211

Note the 4th of July date

koen@dominion:/OE/tentacle/build/tmp-angstrom_2010_x-eglibc/work/beagleboard-angstrom-linux-gnueabi/linux-3.0.12-r113a/git/drivers/mfd$ git log --oneline Kconfig
8509e14 mfd: Add Makefile and Kconfig Entries for tps65911 comparator
66c500f mfd: Fix build breakage caused by tps65910 gpio directory move
83545d8 GPIO: TPS65910: Move driver to drivers/gpio/
c01e36d linux-next: build failure after merge of the voltage tree
aec519b linux-next: build failure after merge of the voltage tree
27c6750 MFD: TPS65910: Add new mfd device for TPS65910
c013f0a mfd: Add pm8xxx irq support
cbdb53e mfd: Add Qualcomm PMIC 8921 core driver
1305134 mfd: Group NXP PCF50633* drivers together
37df003 mfd: Group TPSxxxxx power management chips together
8317797 mfd: add DB5500 PRCMU driver
650c2a2 mach-ux500: move the DB8500 PRCMU driver to MFD

Let's look at the top commit:

koen@dominion:/OE/tentacle/build/tmp-angstrom_2010_x-eglibc/work/beagleboard-angstrom-linux-gnueabi/linux-3.0.12-r113a/git/drivers/mfd$ git show 8509e14
commit 8509e142567d69e170a184ecf27d7a6a3239fd72
Author: Axel Lin <axel.lin@gmail.com>
Date: Thu Jun 23 10:15:51 2011 +0800

So the commit that is referenced is not actually in 3.0.x! Long story short:

git revert 11b8fc6ae54bf18a48c94e181c37ca135b858b42 is broken ; make ; scp uImage beagleboard

root@beagleboard:~# cat /proc/asound/cards
0 [omap3beagle ]: - omap3beagle
omap3beagle

So please revert 11b8fc6ae54bf18a48c94e181c37ca135b858b42 for 3.0.13, cherrypicking the TWL codec rename is a world of hurt since it needs other fixes to get pulled in as well.

regards,

Koen
filename=signature.asc
name=signature.asc


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
email Follow the discussionReplies 1 replyReplies Make a reply

Replies

#1 Greg KH
December 02nd, 2011 - 06:30 pm ET | Report spam
On Wed, Nov 30, 2011 at 11:26:41PM +0100, Koen Kooi wrote:
Hi,

I noticed that audio stopped working after 3.0.10 which I narrowed down to:

11b8fc6ae54bf18a48c94e181c37ca135b858b42 mfd: Fix twl4030 dependencies for audio codec

The commit message has this bit:

"In commit 57fe7251f5bfc4332f24479376de48a1e8ca6211
the CONFIG_TWL4030_CODEC was renamed
into CONFIG_MFD_TWL4030_AUDIO, thatswhy the codec
was not detected."

Which is this one: http://git.kernel.org/?p=linux/kern...a1e8ca6211

Note the 4th of July date

:/OE/tentacle/build/tmp-angstrom_2010_x-eglibc/work/beagleboard-angstrom-linux-gnueabi/linux-3.0.12-r113a/git/drivers/mfd$ git log --oneline Kconfig
8509e14 mfd: Add Makefile and Kconfig Entries for tps65911 comparator
66c500f mfd: Fix build breakage caused by tps65910 gpio directory move
83545d8 GPIO: TPS65910: Move driver to drivers/gpio/
c01e36d linux-next: build failure after merge of the voltage tree
aec519b linux-next: build failure after merge of the voltage tree
27c6750 MFD: TPS65910: Add new mfd device for TPS65910
c013f0a mfd: Add pm8xxx irq support
cbdb53e mfd: Add Qualcomm PMIC 8921 core driver
1305134 mfd: Group NXP PCF50633* drivers together
37df003 mfd: Group TPSxxxxx power management chips together
8317797 mfd: add DB5500 PRCMU driver
650c2a2 mach-ux500: move the DB8500 PRCMU driver to MFD

Let's look at the top commit:

:/OE/tentacle/build/tmp-angstrom_2010_x-eglibc/work/beagleboard-angstrom-linux-gnueabi/linux-3.0.12-r113a/git/drivers/mfd$ git show 8509e14
commit 8509e142567d69e170a184ecf27d7a6a3239fd72
Author: Axel Lin
Date: Thu Jun 23 10:15:51 2011 +0800

So the commit that is referenced is not actually in 3.0.x! Long story short:

git revert 11b8fc6ae54bf18a48c94e181c37ca135b858b42 is broken ; make ; scp uImage beagleboard

:~# cat /proc/asound/cards
0 [omap3beagle ]: - omap3beagle
omap3beagle

So please revert 11b8fc6ae54bf18a48c94e181c37ca135b858b42 for 3.0.13,
cherrypicking the TWL codec rename is a world of hurt since it needs
other fixes to get pulled in as well.



Now reverted, thanks.

greg k-h
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Similar topics