Linux MIPS Architecture development
 help / color / mirror / Atom feed
From: David Daney <ddaney.cavm@gmail.com>
To: David Daney <ddaney.cavm@gmail.com>, Jonas Gorski <jogo@openwrt.org>
Cc: linux-mips@linux-mips.org, Ralf Baechle <ralf@linux-mips.org>,
	Gabor Juhos <juhosg@openwrt.org>,
	"Steven J. Hill" <sjhill@mips.com>,
	David Daney <david.daney@cavium.com>,
	John Crispin <blogic@openwrt.org>,
	Lars-Peter Clausen <lars@metafoo.de>,
	Manuel Lauss <manuel.lauss@googlemail.com>,
	Huacai Chen <chenhc@lemote.com>,
	Jayachandran C <jchandra@broadcom.com>,
	Florian Fainelli <florian@openwrt.org>
Subject: Re: [PATCH] MIPS: define cpu_has_mmips where appropriate
Date: Fri, 24 May 2013 14:46:02 -0700	[thread overview]
Message-ID: <519FDF9A.6080204@gmail.com> (raw)
In-Reply-To: <519F933A.6020407@gmail.com>

On 05/24/2013 09:20 AM, David Daney wrote:
> On 05/23/2013 02:42 PM, Jonas Gorski wrote:
>> Disable cpu_has_mmips for everything but SEAD3 and MALTA. Most of these
>> platforms are from before the micromips introduction, so they are very
>> unlikely to implement it.
>>
>> Reduces an -Os compiled, uncompressed kernel image by 8KiB for BCM63XX.
>>
>> Signed-off-by: Jonas Gorski <jogo@openwrt.org>
>
> Acked-by: David Daney <david.daney@cavium.com>

I changed my mind:  NAK.

I will send a smaller, but equivalent patch.


>> ---
>>
>> Position chosen is based on asm/cpu-features.h. Missing hardware and/or
>> data sheets for most platforms I obviously couldn't verify its
>> non-existence, so maintainer acks are appreciated.
>>
>>   arch/mips/include/asm/mach-ath79/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-au1x00/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-bcm63xx/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-cavium-octeon/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-cobalt/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-ip22/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-ip27/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-ip28/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-ip32/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-jz4740/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-loongson/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-netlogic/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-pmcs-msp71xx/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-powertv/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-ralink/rt288x/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-ralink/rt305x/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-ralink/rt3883/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-rc32434/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-rm/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-sibyte/cpu-feature-overrides.h
>> |    1 +
>>   arch/mips/include/asm/mach-tx49xx/cpu-feature-overrides.h
>> |    1 +
>>   21 files changed, 21 insertions(+)
>>
> [...]
>

  reply	other threads:[~2013-05-24 21:46 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-23 21:42 [PATCH] MIPS: define cpu_has_mmips where appropriate Jonas Gorski
2013-05-24 16:20 ` David Daney
2013-05-24 21:46   ` David Daney [this message]
2013-06-26 23:24     ` Ralf Baechle
2013-06-26 23:46       ` Ralf Baechle

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=519FDF9A.6080204@gmail.com \
    --to=ddaney.cavm@gmail.com \
    --cc=blogic@openwrt.org \
    --cc=chenhc@lemote.com \
    --cc=david.daney@cavium.com \
    --cc=florian@openwrt.org \
    --cc=jchandra@broadcom.com \
    --cc=jogo@openwrt.org \
    --cc=juhosg@openwrt.org \
    --cc=lars@metafoo.de \
    --cc=linux-mips@linux-mips.org \
    --cc=manuel.lauss@googlemail.com \
    --cc=ralf@linux-mips.org \
    --cc=sjhill@mips.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox