From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from e06smtp12.uk.ibm.com ([195.75.94.108]:53627 "EHLO e06smtp12.uk.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751395AbbGaGa7 (ORCPT ); Fri, 31 Jul 2015 02:30:59 -0400 Received: from /spool/local by e06smtp12.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Fri, 31 Jul 2015 07:30:58 +0100 Date: Fri, 31 Jul 2015 08:30:52 +0200 From: Heiko Carstens To: Sasha Levin Cc: stable@vger.kernel.org, stable-commits@vger.kernel.org, "J.D. Schroeder" , Roger Quadros , Marc Kleine-Budde Subject: Re: [added to the 3.18 stable tree] can: c_can: Fix default pinmux glitch at init Message-ID: <20150731063052.GA3908@osiris> References: <1438308644-15635-1-git-send-email-sasha.levin@oracle.com> <1438308644-15635-35-git-send-email-sasha.levin@oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1438308644-15635-35-git-send-email-sasha.levin@oracle.com> Sender: stable-owner@vger.kernel.org List-ID: On Thu, Jul 30, 2015 at 10:10:35PM -0400, Sasha Levin wrote: > From: Heiko Carstens > > This patch has been added to the 3.18 stable tree. If you have any > objections, please let us know. > > =============== > > [ Upstream commit 033365191136c97f88c81b7bd0011414db28bb4e ] > > The previous change 3973c526ae9c (net: can: c_can: Disable pins when CAN > interface is down) causes a slight glitch on the pinctrl settings when used. > Since commit ab78029 (drivers/pinctrl: grab default handles from device core), > the device core will automatically set the default pins. This causes the pins > to be momentarily set to the default and then to the sleep state in > register_c_can_dev(). By adding an optional "enable" state, boards can set the > default pin state to be disabled and avoid the glitch when the switch from > default to sleep first occurs. If the "enable" state is not available > c_can_pinctrl_select_state() falls back to using the "default" pinctrl state. > > [Roger Q] - Forward port to v4.2 and use pinctrl_get_select(). > > Signed-off-by: J.D. Schroeder > Signed-off-by: Roger Quadros > Reviewed-by: Grygorii Strashko > Cc: linux-stable > Signed-off-by: Marc Kleine-Budde > Signed-off-by: Sasha Levin > --- > arch/s390/kernel/process.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/s390/kernel/process.c b/arch/s390/kernel/process.c > index ed84cc2..9b72e68 100644 > --- a/arch/s390/kernel/process.c > +++ b/arch/s390/kernel/process.c > @@ -171,7 +171,7 @@ asmlinkage void execve_tail(void) > { > current->thread.fp_regs.fpc = 0; > if (MACHINE_HAS_IEEE) > - asm volatile("sfpc %0,%0" : : "d" (0)); > + asm volatile("sfpc %0" : : "d" (0)); > } Hi Sasha, I think the commit message for this patch got screwed up. The original upstream commit is: commit e47994dd44bcb4a77b4152bd0eada585934703c0 Author: Heiko Carstens Date: Mon Jul 6 15:02:37 2015 +0200 s390/process: fix sfpc inline assembly The sfpc inline assembly within execve_tail() may incorrectly set bits 28-31 of the sfpc instruction to a value which is not zero. These bits however are currently unused and therefore should be zero so we won't get surprised if these bits will be used in the future. Therefore remove the second operand from the inline assembly. Cc: Signed-off-by: Heiko Carstens Signed-off-by: Martin Schwidefsky