From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1757293AbdELJiu (ORCPT <rfc822;w@1wt.eu>);
        Fri, 12 May 2017 05:38:50 -0400
Received: from mail.linuxfoundation.org ([140.211.169.12]:51116 "EHLO
        mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1756298AbdELJh7 (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Fri, 12 May 2017 05:37:59 -0400
Date: Fri, 12 May 2017 11:37:48 +0200
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Tony Lindgren <tony@atomide.com>, Arnd Bergmann <arnd@arndb.de>
Cc: Guenter Roeck <linux@roeck-us.net>, linux-kernel@vger.kernel.org,
        torvalds@linux-foundation.org, akpm@linux-foundation.org,
        shuahkh@osg.samsung.com, patches@kernelci.org,
        ben.hutchings@codethink.co.uk, stable@vger.kernel.org
Subject: Re: [PATCH 3.18 00/39] 3.18.53-stable review
Message-ID: <20170512093748.GA32014@kroah.com>
References: <20170511130238.045434679@linuxfoundation.org>
 <20170511211607.GC12362@roeck-us.net>
 <20170511213122.gemj5buwrct2aemd@squirrel.local>
 <20170511214636.GC3489@atomide.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20170511214636.GC3489@atomide.com>
User-Agent: Mutt/1.8.2 (2017-04-18)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Thu, May 11, 2017 at 02:46:37PM -0700, Tony Lindgren wrote:
> * Matthijs van Duin <matthijsvanduin@gmail.com> [170511 14:34]:
> > On Thu, May 11, 2017 at 02:16:07PM -0700, Guenter Roeck wrote:
> > > arch/arm/mach-omap2/omap-headsmp.S:60: Error: bad instruction `badr r0,hyp_boot'
> > > 
> > > I see "badr" used in later kernels, but not in v3.18. Does this possibly
> > > require some secondary patches ?
> > 
> > It was introduced in kernel 4.2 by
> > 	14327c662822 "ARM: replace BSYM() with badr assembly macro"
> > 
> > The correct backport would therefore just be:
> > 
> > -	adr	r0, hyp_boot
> > +	adr	r0, BSYM(hyp_boot)
> > 
> > Right?
> 
> Or just skip this for v3.18 until somebody actually needs thumb
> kernel with hypervisor and can provide a Tested-by.

Ok, I'll drop this patch for now, it was added to fix a build warning
that Arnd found.  I'll wait for a proper backport if people really get
annoyed by it :)

thanks,

greg k-h