All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael Ellerman <patch-notifications@ellerman.id.au>
To: Michael Jeanson <mjeanson@efficios.com>,
Cc: stable@vger.kernel.org, Michael Jeanson <mjeanson@efficios.com>,
	Paul Mackerras <paulus@samba.org>,
	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>,
	Vakul Garg <vakul.garg@nxp.com>,
	linuxppc-dev@lists.ozlabs.org, Scott Wood <swood@redhat.com>,
	linux-kernel@vger.kernel.org
Subject: Re: powerpc/e500mc: Set assembler machine type to e500mc
Date: Wed, 20 Jun 2018 09:21:12 +1000 (AEST)	[thread overview]
Message-ID: <419P944XMVz9s3C@ozlabs.org> (raw)
In-Reply-To: <20180614152742.21075-1-mjeanson@efficios.com>

On Thu, 2018-06-14 at 15:27:42 UTC, Michael Jeanson wrote:
> In binutils 2.26 a new opcode for the "wait" instruction was added for the
> POWER9 and has precedence over the one specific to the e500mc. Commit
> ebf714ff3756 ("powerpc/e500mc: Add support for the wait instruction in
> e500_idle") uses this instruction specifically on the e500mc to work around
> an erratum.
> 
> This results in an invalid instruction in idle_e500 when we build for the
> e500mc on bintutils >= 2.26 with the default assembler machine type.
> 
> Since multiplatform between e500 and non-e500 is not supported, set the
> assembler machine type globaly when CONFIG_PPC_E500MC=y.
> 
> Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
> Reviewed-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
> CC: Benjamin Herrenschmidt <benh@kernel.crashing.org>
> CC: Paul Mackerras <paulus@samba.org>
> CC: Michael Ellerman <mpe@ellerman.id.au>
> CC: Kumar Gala <galak@kernel.crashing.org>
> CC: Vakul Garg <vakul.garg@nxp.com>
> CC: Scott Wood <swood@redhat.com>
> CC: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
> CC: linuxppc-dev@lists.ozlabs.org
> CC: linux-kernel@vger.kernel.org
> CC: stable@vger.kernel.org

Applied to powerpc fixes, thanks.

https://git.kernel.org/powerpc/c/69a8405999aa1c489de4b8d349468f

cheers

      reply	other threads:[~2018-06-19 23:21 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-14 15:27 [PATCH] powerpc/e500mc: Set assembler machine type to e500mc Michael Jeanson
2018-06-14 15:27 ` Michael Jeanson
2018-06-19 23:21 ` Michael Ellerman [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=419P944XMVz9s3C@ozlabs.org \
    --to=patch-notifications@ellerman.id.au \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=mathieu.desnoyers@efficios.com \
    --cc=mjeanson@efficios.com \
    --cc=paulus@samba.org \
    --cc=stable@vger.kernel.org \
    --cc=swood@redhat.com \
    --cc=vakul.garg@nxp.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.