All of lore.kernel.org
 help / color / mirror / Atom feed
From: linux@arm.linux.org.uk (Russell King - ARM Linux)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/9] arm: select different compiler flags for ARM CortexM3
Date: Tue, 23 Jun 2015 22:48:18 +0100	[thread overview]
Message-ID: <20150623214818.GF7557@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <1435094387-20146-2-git-send-email-pawelo@king.net.pl>

On Tue, Jun 23, 2015 at 11:19:39PM +0200, Paul Osmialowski wrote:
> This one is inspired by two commits published on Emcraft git repo:
> 
> https://github.com/EmcraftSystems/linux-emcraft.git
> 
> 1) 6302b692f570ff9d5645a6e72c11f87b0c1aa409
>  RT #62654. Fixed kernel crashes while running httpd by enabling
>  "-mfix-cortex-m3-ldrd" compiler option, which prevents compiler from
>  generating code like 'ldrd Ra, Rb, [Ra, #Imm]' - according to the 602117
>  Cortex-M3 Errata it may result in incorrect base register when interrupted
>  or faulted.
> 
> by: Yuri Tikhonov <yur@emcraft.com>
> 
> 2) 359d3cda84c01c0f3fae1a519b97a31f318f57ab
>  RT #62654. Removed "--march=..." leaving only "-mcpu=cortex-m3" to make
>  sure only the correct instructions will be generated.
> 
> by: Sergei Poselenov <sposelenov@emcraft.com>
> 
> I reworked these patches to make them less intrusive.
> 
> Signed-off-by: Paul Osmialowski <pawelo@king.net.pl>

NAK.

The EFM32 code already added core support for Cortex-M3 cores, under
the symbol CPU_V7M.  Rather than implementing a whole new set of
Cortex-M3 support alongside the existing code, please work with the
EFM32 maintainer (Uwe Kleine-K?nig) to come up with a common set of
core changes that you can all agree on for Cortex-M3.

Please also try to be a little smarter with whom you're sending your
patches to, the Cc list looks to be excessively long - having soo many
recipients on your message is a good way to get your message rejected
by mailing lists as spam.

-- 
FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up
according to speedtest.net.

WARNING: multiple messages have this Message-ID (diff)
From: Russell King - ARM Linux <linux@arm.linux.org.uk>
To: Paul Osmialowski <pawelo@king.net.pl>
Cc: Arnd Bergmann <arnd@arndb.de>, Olof Johansson <olof@lixom.net>,
	Uwe Kleine-Koenig <u.kleine-koenig@pengutronix.de>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Dave Martin <Dave.Martin@arm.com>,
	Mark Rutland <mark.rutland@arm.com>,
	Pawel Moll <pawel.moll@arm.com>,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	Yuri Tikhonov <yur@emcraft.com>,
	Sergei Poselenov <sposelenov@emcraft.com>,
	Dmitry Cherkassov <d_cherkasov@emcraft.com>,
	Alexander Potashev <aspotashev@emcraft.com>
Subject: Re: [PATCH 1/9] arm: select different compiler flags for ARM CortexM3
Date: Tue, 23 Jun 2015 22:48:18 +0100	[thread overview]
Message-ID: <20150623214818.GF7557@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <1435094387-20146-2-git-send-email-pawelo@king.net.pl>

On Tue, Jun 23, 2015 at 11:19:39PM +0200, Paul Osmialowski wrote:
> This one is inspired by two commits published on Emcraft git repo:
> 
> https://github.com/EmcraftSystems/linux-emcraft.git
> 
> 1) 6302b692f570ff9d5645a6e72c11f87b0c1aa409
>  RT #62654. Fixed kernel crashes while running httpd by enabling
>  "-mfix-cortex-m3-ldrd" compiler option, which prevents compiler from
>  generating code like 'ldrd Ra, Rb, [Ra, #Imm]' - according to the 602117
>  Cortex-M3 Errata it may result in incorrect base register when interrupted
>  or faulted.
> 
> by: Yuri Tikhonov <yur@emcraft.com>
> 
> 2) 359d3cda84c01c0f3fae1a519b97a31f318f57ab
>  RT #62654. Removed "--march=..." leaving only "-mcpu=cortex-m3" to make
>  sure only the correct instructions will be generated.
> 
> by: Sergei Poselenov <sposelenov@emcraft.com>
> 
> I reworked these patches to make them less intrusive.
> 
> Signed-off-by: Paul Osmialowski <pawelo@king.net.pl>

NAK.

The EFM32 code already added core support for Cortex-M3 cores, under
the symbol CPU_V7M.  Rather than implementing a whole new set of
Cortex-M3 support alongside the existing code, please work with the
EFM32 maintainer (Uwe Kleine-König) to come up with a common set of
core changes that you can all agree on for Cortex-M3.

Please also try to be a little smarter with whom you're sending your
patches to, the Cc list looks to be excessively long - having soo many
recipients on your message is a good way to get your message rejected
by mailing lists as spam.

-- 
FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up
according to speedtest.net.

  reply	other threads:[~2015-06-23 21:48 UTC|newest]

Thread overview: 83+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-23 21:19 [PATCH 0/9] [New BSP] Add initial support for Freescale Kinetis TWR-K70F120M development kit Paul Osmialowski
2015-06-23 21:19 ` Paul Osmialowski
2015-06-23 21:19 ` [PATCH 1/9] arm: select different compiler flags for ARM CortexM3 Paul Osmialowski
2015-06-23 21:19   ` Paul Osmialowski
2015-06-23 21:48   ` Russell King - ARM Linux [this message]
2015-06-23 21:48     ` Russell King - ARM Linux
2015-06-24  4:22     ` Paul Osmialowski
2015-06-24  4:22       ` Paul Osmialowski
     [not found] ` <1435094387-20146-1-git-send-email-pawelo-rhuoMcPwk82rDJvtcaxF/A@public.gmane.org>
2015-06-23 21:19   ` Paul Osmialowski
2015-06-23 21:19   ` [PATCH 2/9] arm: do not place huge encoder tables on stack when it is too small Paul Osmialowski
2015-06-23 21:19   ` [PATCH 3/9] arm: add call to CPU idle quirks handler Paul Osmialowski
2015-06-23 21:19   ` [PATCH 4/9] arm: allow copying of vector table to internal SRAM memory Paul Osmialowski
2015-06-23 21:19   ` [PATCH 6/9] arm: twr-k70f120m: clock source drivers for Kinetis SoC Paul Osmialowski
2015-06-23 21:19   ` [PATCH 7/9] arm: twr-k70f120m: IOMUX driver " Paul Osmialowski
2015-06-23 21:19 ` [PATCH 2/9] arm: do not place huge encoder tables on stack when it is too small Paul Osmialowski
2015-06-23 21:19   ` Paul Osmialowski
2015-06-24  7:10   ` Geert Uytterhoeven
2015-06-24  7:17     ` Paul Osmialowski
2015-06-23 21:19 ` [PATCH 3/9] arm: add call to CPU idle quirks handler Paul Osmialowski
2015-06-23 21:51   ` Russell King - ARM Linux
2015-06-23 21:51     ` Russell King - ARM Linux
2015-06-23 21:59   ` Arnd Bergmann
2015-06-23 21:59     ` Arnd Bergmann
2015-06-25 16:42     ` Nicolas Pitre
2015-06-25 16:42       ` Nicolas Pitre
2015-06-26  5:30       ` Paul Osmialowski
2015-06-26  5:30         ` Paul Osmialowski
2015-06-26  7:40         ` Arnd Bergmann
2015-06-26  7:40           ` Arnd Bergmann
2015-06-26 21:52           ` Paul Osmialowski
2015-06-26 21:52             ` Paul Osmialowski
2015-06-26 22:27             ` Russell King - ARM Linux
2015-06-26 22:27               ` Russell King - ARM Linux
2015-06-23 21:19 ` Paul Osmialowski
2015-06-23 21:19 ` [PATCH 4/9] arm: allow copying of vector table to internal SRAM memory Paul Osmialowski
2015-06-23 21:19   ` Paul Osmialowski
2015-06-23 21:19 ` [PATCH 5/9] arm: twr-k70f120m: basic support for Kinetis TWR-K70F120M Paul Osmialowski
2015-06-23 21:19   ` Paul Osmialowski
2015-06-23 22:05   ` Arnd Bergmann
2015-06-23 22:05     ` Arnd Bergmann
2015-06-23 22:33     ` Russell King - ARM Linux
2015-06-23 22:33       ` Russell King - ARM Linux
2015-06-24  4:42     ` Paul Osmialowski
2015-06-24  4:42       ` Paul Osmialowski
2015-06-23 21:19 ` Paul Osmialowski
2015-06-23 21:19 ` [PATCH 6/9] arm: twr-k70f120m: clock source drivers for Kinetis SoC Paul Osmialowski
2015-06-23 21:19   ` Paul Osmialowski
2015-06-23 22:24   ` Stephen Boyd
2015-06-23 22:24   ` Stephen Boyd
2015-06-23 22:24     ` Stephen Boyd
2015-06-24  5:09     ` Paul Osmialowski
2015-06-24  5:09       ` Paul Osmialowski
2015-06-23 22:25   ` Arnd Bergmann
2015-06-23 22:25     ` Arnd Bergmann
2015-06-24  7:53   ` Thomas Gleixner
2015-06-24  7:53     ` Thomas Gleixner
2015-06-23 21:19 ` [PATCH 7/9] arm: twr-k70f120m: IOMUX driver " Paul Osmialowski
     [not found]   ` <1435094387-20146-8-git-send-email-pawelo-rhuoMcPwk82rDJvtcaxF/A@public.gmane.org>
2015-06-24 10:21     ` Paul Bolle
2015-06-24 10:21   ` Paul Bolle
2015-06-24 10:21     ` Paul Bolle
2015-06-24 17:44     ` Paul Osmialowski
2015-06-24 17:44       ` Paul Osmialowski
2015-07-14  8:53   ` Linus Walleij
2015-07-14  8:53     ` Linus Walleij
2015-07-14  8:53     ` Linus Walleij
2015-09-08  8:04     ` Paul Osmialowski
2015-09-08  8:04       ` Paul Osmialowski
2015-09-08  8:04       ` Paul Osmialowski
2015-09-08  8:04       ` Paul Osmialowski
2015-09-08 14:28       ` Linus Walleij
2015-09-08 14:28         ` Linus Walleij
2015-09-08 14:28         ` Linus Walleij
2015-06-23 21:19 ` Paul Osmialowski
2015-06-23 21:19 ` [PATCH 8/9] arm: twr-k70f120m: extend Freescale eDMA driver with ability to support " Paul Osmialowski
2015-06-23 21:19   ` Paul Osmialowski
2015-06-24 16:14   ` Vinod Koul
2015-06-24 16:14     ` Vinod Koul
2015-06-24 17:43     ` Paul Osmialowski
2015-06-24 17:43       ` Paul Osmialowski
2015-06-23 21:19 ` Paul Osmialowski
2015-06-23 21:19 ` [PATCH 9/9] arm: twr-k70f120m: extend Freescale lpuart " Paul Osmialowski
2015-06-23 21:19 ` Paul Osmialowski
2015-06-23 21:19   ` Paul Osmialowski

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=20150623214818.GF7557@n2100.arm.linux.org.uk \
    --to=linux@arm.linux.org.uk \
    --cc=linux-arm-kernel@lists.infradead.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 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.