linux-acpi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Brandon Anderson <brandon.anderson@amd.com>
To: ssg.sos.patches@amd.com, linaro-acpi@lists.linaro.org,
	linux-acpi@vger.kernel.org
Cc: Brandon Anderson <brandon.anderson@amd.com>
Subject: [PATCH 1/4] ACPI/ARM: Load fixed-clk module early
Date: Mon, 11 Nov 2013 11:16:07 -0600	[thread overview]
Message-ID: <1384190170-8707-2-git-send-email-brandon.anderson@amd.com> (raw)
In-Reply-To: <1384190170-8707-1-git-send-email-brandon.anderson@amd.com>

An AMBA device will be probed once two things have occured: 1) ACPI has
registered a device, and 2) the driver has registered itself as an AMBA
driver. Since several drivers register themselves very early on, the
probe will happen as soon as ACPI registers the device. If a device
depends on a clock, the clock must be probed before ACPI registers the
device. This means that the clock definition must be encountered first in the
DSDT file, and the clock driver must already be loaded when this happens.

There are other potential solutions, with this solution registering the clock
driver very early in the boot process to avoid changes to other drivers.


Signed-off-by: Brandon Anderson <brandon.anderson@amd.com>
---
 drivers/clk/clk-fixed-rate.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/clk-fixed-rate.c b/drivers/clk/clk-fixed-rate.c
index 226cefb..778291e 100644
--- a/drivers/clk/clk-fixed-rate.c
+++ b/drivers/clk/clk-fixed-rate.c
@@ -174,5 +174,5 @@ static int __init fixed_clk_init(void)
  * fixed clock will used for AMBA bus, UART and etc, so it should be
  * initialized early enough.
  */
-subsys_initcall(fixed_clk_init);
+postcore_initcall(fixed_clk_init);
 #endif
-- 
1.7.9.5



  reply	other threads:[~2013-11-11 17:17 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-11 17:16 [PATCH 0/4] ACPI/ARM: AMBA bus ACPI module Brandon Anderson
2013-11-11 17:16 ` Brandon Anderson [this message]
2013-11-11 17:16 ` [PATCH 2/4] ACPI/ARM: Add " Brandon Anderson
2013-11-12 10:40   ` Mika Westerberg
     [not found]     ` <CE40542A5D952D47989966E71788B1840157C0F8@satlexdag05.amd.com>
2013-11-12 19:13       ` Mika Westerberg
2013-11-21 15:09   ` Tomasz Nowicki
     [not found]     ` <CE40542A5D952D47989966E71788B184037DE61F@satlexdag05.amd.com>
2013-11-22  9:35       ` Tomasz Nowicki
2013-11-11 17:16 ` [PATCH 3/4] ACPI/ARM: Add ACPI to AMBA SPI driver Brandon Anderson
2013-11-12 10:43   ` Mika Westerberg
     [not found]     ` <CE40542A5D952D47989966E71788B1840157C119@satlexdag05.amd.com>
2013-11-12 19:14       ` Mika Westerberg
2013-11-17 21:42         ` Rafael J. Wysocki
2013-11-11 17:16 ` [PATCH 4/4] ACPI/ARM: Remove sections of DTS definition Brandon Anderson

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=1384190170-8707-2-git-send-email-brandon.anderson@amd.com \
    --to=brandon.anderson@amd.com \
    --cc=linaro-acpi@lists.linaro.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=ssg.sos.patches@amd.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;
as well as URLs for NNTP newsgroup(s).