public inbox for linux-sh@vger.kernel.org
 help / color / mirror / Atom feed
From: Paul Mundt <lethal@linux-sh.org>
To: linux-sh@vger.kernel.org
Subject: Re: Discussion about register definition header
Date: Thu, 06 May 2010 05:17:08 +0000	[thread overview]
Message-ID: <20100506051708.GC25493@linux-sh.org> (raw)
In-Reply-To: <201005021110.22111.fabio.giovagnini@aurion-tech.com>

On Sun, May 02, 2010 at 11:10:21AM +0200, Fabio Giovagnini wrote:
> I think that into include/asm dir shoud be a file containing ALL the mpu 
> registers address definitions, for avoiding to have same definitions across 
> different files and an high probability to write something wrong each time.
> I think this has not already done beacuse it is a long work for testing. Is it 
> the main reason?
> 
The main reason is that there's simply no need for it. Many of the CPUs
in question have similar register definitions but just position the
blocks at different locations. This is the sort of change that should be
managed between the driver and the CPU setup code in charge of
registering the SoC's platform devices.

For the most part we've evolved beyond simple hardcoded register
definitions, and the few cases where we still have to put up with them
now are mostly for blocks that aren't handled through the driver model.
Also note that these days almost everything needs to be remapped (either
via the TLB or via the PMB) so it's no longer possible to simply permit
drivers to do absurd P1/P2 abuses.

If you want to see what a total mess having these sorts of headers
quickly turns in to, you don't have to dig very far through the
architecture tree to find them. ARM used to do this via an
all-encompassing asm/hardware.h and has likewise moved on.

      reply	other threads:[~2010-05-06  5:17 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-02  9:10 Discussion about register definition header Fabio Giovagnini
2010-05-06  5:17 ` Paul Mundt [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=20100506051708.GC25493@linux-sh.org \
    --to=lethal@linux-sh.org \
    --cc=linux-sh@vger.kernel.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