Linux MIPS Architecture development
 help / color / mirror / Atom feed
From: Jiaxun Yang <jiaxun.yang@flygoat.com>
To: James Hogan <james.hogan@mips.com>
Cc: Ralf Baechle <ralf@linux-mips.org>,
	Huacai CHen <chenhc@lemote.com>,
	linux-mips@linux-mips.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] MIPS: Loongson64: Drop 32-bit support for Loongson 2E/2F devices
Date: Tue, 02 Jan 2018 21:29:46 +0800	[thread overview]
Message-ID: <1514899786.1694.6.camel@flygoat.com> (raw)
In-Reply-To: <20180102084759.GL5027@jhogan-linux.mipstec.com>

[-- Attachment #1: Type: text/plain, Size: 1601 bytes --]

On 2018-01-02 Tue 08:48 +0000,James Hogan Wrote:
> On Tue, Dec 26, 2017 at 12:21:38PM +0800, Jiaxun Yang wrote:
> > Make loongson64 a pure 64-bit mach.
> 
> Please expand to provide some rationale behind the change. Was 32-bit
> support broken at runtime, or broken at build time, or are we simply
> no

The 32-bit support was broken at runtime, it doesn't boot anymore,
witch is hard to debug because even early printk isn't working, also
there are some build warnings. Some newer bootloader may not support
32-bit ELF. So we decide to drop 32-bit support.

Jiaxun

> longer interested in supporting it?
> 
> Cheers
> James
> 
> > 
> > Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
> > ---
> >  arch/mips/loongson64/Kconfig | 2 --
> >  1 file changed, 2 deletions(-)
> > 
> > diff --git a/arch/mips/loongson64/Kconfig
> > b/arch/mips/loongson64/Kconfig
> > index 0d249fc3cfe9..a7d9a9241ac4 100644
> > --- a/arch/mips/loongson64/Kconfig
> > +++ b/arch/mips/loongson64/Kconfig
> > @@ -17,7 +17,6 @@ config LEMOTE_FULOONG2E
> >  	select I8259
> >  	select ISA
> >  	select IRQ_MIPS_CPU
> > -	select SYS_SUPPORTS_32BIT_KERNEL
> >  	select SYS_SUPPORTS_64BIT_KERNEL
> >  	select SYS_SUPPORTS_LITTLE_ENDIAN
> >  	select SYS_SUPPORTS_HIGHMEM
> > @@ -49,7 +48,6 @@ config LEMOTE_MACH2F
> >  	select ISA
> >  	select SYS_HAS_CPU_LOONGSON2F
> >  	select SYS_HAS_EARLY_PRINTK
> > -	select SYS_SUPPORTS_32BIT_KERNEL
> >  	select SYS_SUPPORTS_64BIT_KERNEL
> >  	select SYS_SUPPORTS_HIGHMEM
> >  	select SYS_SUPPORTS_LITTLE_ENDIAN
> > -- 
> > 2.15.1
> > 

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  parent reply	other threads:[~2018-01-02 13:31 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-26  4:21 [PATCH] MIPS: Loongson64: Drop 32-bit support for Loongson 2E/2F devices Jiaxun Yang
2018-01-02  8:48 ` James Hogan
2018-01-02  8:48   ` James Hogan
2018-01-02 13:29   ` Jiaxun Yang [this message]
2018-01-02 13:39     ` James Hogan
2018-01-02 13:39       ` James Hogan

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=1514899786.1694.6.camel@flygoat.com \
    --to=jiaxun.yang@flygoat.com \
    --cc=chenhc@lemote.com \
    --cc=james.hogan@mips.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mips@linux-mips.org \
    --cc=ralf@linux-mips.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox