linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: linux@arm.linux.org.uk (Russell King - ARM Linux)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] clkdev: add support to lookup for early platform device
Date: Thu, 28 Apr 2011 09:33:12 +0100	[thread overview]
Message-ID: <20110428083312.GL17290@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <20110428032906.GN29103@game.jcrosoft.org>

On Thu, Apr 28, 2011 at 05:29:06AM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> I use it for gpio drivers that are init during map_io

Why are you initializing hardware during map_io?  That's totally crazy.
Don't do that - the new mappings won't be flushed out of the cache/tlb
at that point, so it's going to be hit and miss whether you can access
the device without locking up the kernel.

Instead, use the early initialization hook which I've recently added for
this stuff, and keep the map_io for _only_ doing what the name says.

  parent reply	other threads:[~2011-04-28  8:33 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-20 14:05 [PATCH] clkdev: add support to lookup for early platform device Jean-Christophe PLAGNIOL-VILLARD
2011-04-27  8:42 ` Russell King - ARM Linux
2011-04-27 10:31   ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-27 10:47     ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-27 15:00       ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-27 22:12         ` Greg KH
2011-04-28  2:09         ` Paul Mundt
2011-04-28  2:45           ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28  3:08             ` Paul Mundt
2011-04-28  3:17               ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28  3:20                 ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28  3:35                   ` Magnus Damm
2011-04-28  3:29                     ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28  3:44                       ` Magnus Damm
2011-04-28  8:33                       ` Russell King - ARM Linux [this message]
2011-04-28  9:28                         ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28  3:38                 ` Paul Mundt
2011-04-28  4:31                   ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28  4:46                     ` Saravana Kannan
2011-04-28  4:50                       ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28  5:12                   ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28  5:47                     ` Magnus Damm
2011-04-28  8:35                     ` Russell King - ARM Linux
2011-04-28  9:26                       ` Jean-Christophe PLAGNIOL-VILLARD
2011-04-28 10:07                         ` Russell King - ARM Linux
2011-04-28 11:19                           ` Jean-Christophe PLAGNIOL-VILLARD

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=20110428083312.GL17290@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).