public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: mach header files
Date: Fri, 04 Apr 2014 17:30:51 +0200	[thread overview]
Message-ID: <5297747.kWEjnPz8MW@wuerfel> (raw)
In-Reply-To: <0bbbadf628a64ee5be8ede870d421a57@SIXPR06MB173.apcprd06.prod.outlook.com>

On Friday 04 April 2014 13:52:53 Phil Edworthy wrote:
> On 04 April 2014 14:24, Arnd wrote:
> > We still make the occasional exception for adding code in the mach-*
> > directory, but we are getting pretty close to the state where this
> > is not needed for new platforms, and all the existing uses are for
> > things that can eventually get cleaned up.
> > If you think you need an exception here, please explain what you
> > are doing, and we can see if there is a better way to do that already.
> 
> Ok, that's what I suspected about the mach header files.
> 
> I didn't realise about the expectation that there is no 
> arch/arm/mach-* for new platforms. At the moment I have only a
> small amount of platform code:
> 1. SMP holding pen code (as there is no power control for the second
> core), though I have recently seen patches to provide a generic solution
> for this.

Proper platforms should not be using the holding pen, that is only
done on systems that don't have real hardware support for CPU
power management. Ideally you should be implementing PSCI in the
firmware, and then you won't need any code for SMP.

> 2. Firmware calls (via struct firmware_ops) for set_cpu_boot_addr
>  and cpu_boot. This falls back to directly accessing registers in
> the system control block, if the firmware fails. This is useful when
> developing the system when no firmware exists yet.

Same thing.

> 3. A write to a system control register to switch the pl011 uart
> from DMA single requests, to DMA burst requests.

Do you need to do this at run-time, or is it something that could
be done in the bootloader before Linux starts up?

	Arnd

  reply	other threads:[~2014-04-04 15:30 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-04  8:19 mach header files Phil Edworthy
2014-04-04 10:29 ` Grant Likely
2014-04-04 12:16   ` Phil Edworthy
2014-04-04 12:22     ` Grant Likely
2014-04-04 13:14       ` Phil Edworthy
2014-04-04 13:20         ` Thomas Petazzoni
2014-04-08 14:00           ` Laurent Pinchart
2014-04-04 13:44         ` Grant Likely
2014-04-04 13:24   ` Arnd Bergmann
2014-04-04 13:32     ` Barry Song
2014-04-04 15:27       ` Arnd Bergmann
2014-04-04 13:52     ` Phil Edworthy
2014-04-04 15:30       ` Arnd Bergmann [this message]
2014-04-04 15:42         ` Phil Edworthy
2014-04-04 15:50           ` Arnd Bergmann
2014-04-04 16:02             ` Phil Edworthy
2014-04-04 16:22               ` Russell King - ARM Linux
2014-04-04 16:38                 ` Phil Edworthy
2014-04-04 18:01     ` Kent Borg
2014-04-04 18:49       ` Arnd Bergmann
2014-04-04 19:01         ` Kent Borg
2014-04-04 19:19           ` Arnd Bergmann
  -- strict thread matches above, loose matches on Subject: below --
2014-04-02 15:11 Phil Edworthy
2014-04-02 15:16 ` Arnd Bergmann
2014-04-03  7:48   ` Phil Edworthy
2014-04-02 15:20 ` Ben Dooks

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=5297747.kWEjnPz8MW@wuerfel \
    --to=arnd@arndb.de \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox