[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <87y4w7jwvr.wl%satoru.takeuchi@gmail.com>
Date: Sun, 06 Jul 2014 06:53:12 +0900
From: Satoru Takeuchi <satoru.takeuchi@...il.com>
To: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc: Takashi Iwai <tiwai@...e.de>,
Satoru Takeuchi <satoru.takeuchi@...il.com>,
Guenter Roeck <linux@...ck-us.net>,
linux-kernel@...r.kernel.org, torvalds@...ux-foundation.org,
akpm@...ux-foundation.org, shuah.kh@...sung.com,
stable@...r.kernel.org
Subject: Re: [PATCH 3.14 00/59] 3.14.11-stable review
At Sat, 5 Jul 2014 12:10:58 -0700,
Greg Kroah-Hartman wrote:
>
> On Sat, Jul 05, 2014 at 02:56:55PM +0200, Takashi Iwai wrote:
> > At Sat, 05 Jul 2014 18:21:04 +0900,
> > Satoru Takeuchi wrote:
> > >
> > > Hi Greg,
> > >
> > > Add Takashi Iwai to this thread.
> > >
> > > At Sat, 05 Jul 2014 16:00:41 +0900,
> > > Satoru Takeuchi wrote:
> > > >
> > > > At Fri, 04 Jul 2014 22:45:42 -0700,
> > > > Guenter Roeck wrote:
> > > > >
> > > > > On 07/04/2014 03:18 PM, Greg Kroah-Hartman wrote:
> > > > > > This is the start of the stable review cycle for the 3.14.11 release.
> > > > > > There are 59 patches in this series, all will be posted as a response
> > > > > > to this one. If anyone has any issues with these being applied, please
> > > > > > let me know.
> > > > > >
> > > > > > Responses should be made by Sun Jul 6 22:15:27 UTC 2014.
> > > > > > Anything received after that time might be too late.
> > > > > >
> > > > >
> > > > > Build results:
> > > > > total: 144 pass: 130 skipped: 4 fail: 10
> > > > > Failed builds:
> > > > > alpha:allmodconfig
> > > > > i386:allyesconfig
> > > > > i386:allmodconfig
> > > > > powerpc:allmodconfig (binutils 2.23)
> > > > > powerpc:allmodconfig (binutils 2.24)
> > > > > sparc64:allmodconfig
> > > > > unicore32:defconfig
> > > > > x86_64:allyesconfig
> > > > > x86_64:allmodconfig
> > > > > xtensa:allmodconfig
> > > > >
> > > > > Qemu test for x86 failed.
> > > > >
> > > > > Build error is the same as seen with 3.14.
> > > > >
> > > > > sound/pci/hda/patch_sigmatel.c: In function 'stac92hd95_fixup_hp_led':
> > > > > sound/pci/hda/patch_sigmatel.c:4143:3: error: implicit declaration of function 'codec_dbg' [-Werror=implicit-function-declaration]
> > > >
> > > > This kernel failed to build with the following error. Probably the root
> > > > cause is the same as Guenter. I'm now bisecting to find the problematic
> > > > patch...
> > >
> > > The following patch caused the boot failure of both this kernel and 3.10.47-rc1.
> > >
> > > alsa-hda-adjust-speaker-hpf-and-add-led-support-for-hp-spectre-13.patch:
> > > ===============================================================================
> > > From 8b3dfdaf0c25a584cb31d04d2574115cf2d422ab Mon Sep 17 00:00:00 2001
> > > From: Takashi Iwai <tiwai@...e.de>
> > > Date: Tue, 24 Jun 2014 13:55:25 +0200
> > > Subject: ALSA: hda - Adjust speaker HPF and add LED support for HP Spectre 13
> > >
> > > From: Takashi Iwai <tiwai@...e.de>
> > >
> > > commit 8b3dfdaf0c25a584cb31d04d2574115cf2d422ab upstream.
> > >
> > > HP Spectre 13 has the IDT 92HD95 codec, and BIOS seems to set the
> > > default high-pass filter in some "safer" range, which results in the
> > > very soft tone from the built-in speakers in contrast to Windows.
> > > Also, the mute LED control is missing, since 92HD95 codec still has no
> > > HP-specific fixups for GPIO setups.
> > >
> > > This patch adds these missing features: the HPF is adjusted by the
> > > vendor-specific verb, and the LED is set up from a DMI string (but
> > > with the default polarity = 0 assumption due to the incomplete BIOS on
> > > the given machine).
> > >
> > > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=74841
> > > Signed-off-by: Takashi Iwai <tiwai@...e.de>
> > > Signed-off-by: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> > > ...
> > > ===============================================================================
> > >
> > > This patch uses codec_dbg(), however, this macro is intrdocuded by the
> > > following patch and it's not applied to these stable-tree yet.
> > >
> > > ===============================================================================
> > > ommit 4e76a8833fac8dc1735aa5be7d1b3c92c65e209e
> > > Author: Takashi Iwai <tiwai@...e.de>
> > > Date: Tue Feb 25 12:21:03 2014 +0100
> > >
> > > ALSA: hda - Replace with standard printk
> > >
> > > Use dev_err() and co for messages from HD-audio controller and codec
> > > drivers. The codec drivers are mostly bound with codec objects, so
> > > some helper macros, codec_err(), codec_info(), etc, are provided.
> > > They merely wrap the corresponding dev_xxx().
> > >
> > > There are a few places still calling snd_printk() and its variants
> > > as they are called without the codec or device context.
> > >
> > > Signed-off-by: Takashi Iwai <tiwai@...e.de>
> > > ...
> > > ===============================================================================
> > >
> > > Unfortunately I failed to apply this patch to 3.14.11-rc1 with the following
> > > error log.
> > >
> > > ===============================================================================
> > > $ git apply ~/src/test-linux-stable/extra_patch.txt
> > > error: patch failed: sound/pci/hda/hda_intel.c:897
> > > error: sound/pci/hda/hda_intel.c: patch does not apply
> > > error: sound/pci/hda/hda_sysfs.c: No such file or directory
> > > ===============================================================================
> > >
> > > I'm not sure whether we should drop this patch or should apply extra patches
> > > to remove this build failure. Any idea?
> >
> > Just drop the patch from 3.14 and earlier stable kernels. If anyone
> > wants to use the laptop, they should use 3.15.x or later kernel.
> >
> > The second patch cannot be used for 3.14 at all because it requires
> > the other infrastructure change that was introduced in 3.15.
> > Alternatively, you can replace codec_dbg() with pr_debug() and drop
> > the first argument.
>
> I've now dropped this patch from the 3.10 and 3.14 patch queues, sorry
> for the build failures, I should have caught these before I put out the
> series.
No problem. It's why we review/test stable releases. The more many people
review it, the more its quality raise. It's apparently impossible to find
any bugs only by yourself.
If you provide me 3.14.11-rc2 and 3.10.47-rc2, I'll test them too.
Thanks,
Satoru
>
> greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists