From: Laura Abbott <lauraa@codeaurora.org>
To: "arnd@arndb.de" <arnd@arndb.de>,
Will Deacon <will.deacon@arm.com>,
Robin Murphy <robin.murphy@arm.com>
Cc: devicetree@vger.kernel.org,
Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>,
Joreg Roedel <joro@8bytes.org>,
linux-kernel@vger.kernel.org, iommu@lists.linux-foundation.org,
Grant Likely <grant.likely@linaro.org>,
Mitchel Humpherys <mitchelh@codeaurora.org>,
linux-arm-kernel@lists.infradead.org,
Marek Szyprowski <m.szyprowski@samsung.com>
Subject: Re: [PATCH/RFC 0/4] Probe deferral for IOMMU DT integration
Date: Mon, 09 Feb 2015 13:27:58 -0800 [thread overview]
Message-ID: <54D9265E.1030403@codeaurora.org> (raw)
In-Reply-To: <2115835102.210232.1423348884955.JavaMail.open-xchange@oxbaltgw04.schlund.de>
On 2/7/2015 2:41 PM, arnd@arndb.de wrote:
> Laura Abbott <lauraa@codeaurora.org> hat am 6. Februar 2015 um 01:31
> geschrieben:
>>
>> The requirement for this is based on a previous patch to add clock
>> support to the ARM SMMU driver[2]. Once we have clock support, it's
>> possible that the driver itself may need to be defered which breaks
>> a bunch of assumptions about how SMMU probing is supposed to work.
>
> Hi Laura,
>
> I was hoping that we would not need this, and instead treat the iommu in
> the same way as timers and SMP initialization, both
> of which need to be run early at boot time but may rely on clock controllers
> to be initialized first.
>
> Is there a specific requirement that makes this impossible here, or is your
> intention to solve the problem more nicely by allowing deferred probing
> over forcing the input clocks of the iommu to be early?
>
> Arnd
>
The current clock driver for qcom targets doesn't support the early
initialization needed for timers and SMP because neither of those depend
on the clocksources. I discussed this with Stephen some and adding the
early support would not mesh well with the device/driver design of the
current clock driver so that's not really an option right now.
I do think the deferred probing design is cleaner. Even cleaner would
be a proper bus type but that's a different can of worms.
Thanks,
Laura
--
Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
WARNING: multiple messages have this Message-ID (diff)
From: lauraa@codeaurora.org (Laura Abbott)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH/RFC 0/4] Probe deferral for IOMMU DT integration
Date: Mon, 09 Feb 2015 13:27:58 -0800 [thread overview]
Message-ID: <54D9265E.1030403@codeaurora.org> (raw)
In-Reply-To: <2115835102.210232.1423348884955.JavaMail.open-xchange@oxbaltgw04.schlund.de>
On 2/7/2015 2:41 PM, arnd at arndb.de wrote:
> Laura Abbott <lauraa@codeaurora.org> hat am 6. Februar 2015 um 01:31
> geschrieben:
>>
>> The requirement for this is based on a previous patch to add clock
>> support to the ARM SMMU driver[2]. Once we have clock support, it's
>> possible that the driver itself may need to be defered which breaks
>> a bunch of assumptions about how SMMU probing is supposed to work.
>
> Hi Laura,
>
> I was hoping that we would not need this, and instead treat the iommu in
> the same way as timers and SMP initialization, both
> of which need to be run early at boot time but may rely on clock controllers
> to be initialized first.
>
> Is there a specific requirement that makes this impossible here, or is your
> intention to solve the problem more nicely by allowing deferred probing
> over forcing the input clocks of the iommu to be early?
>
> Arnd
>
The current clock driver for qcom targets doesn't support the early
initialization needed for timers and SMP because neither of those depend
on the clocksources. I discussed this with Stephen some and adding the
early support would not mesh well with the device/driver design of the
current clock driver so that's not really an option right now.
I do think the deferred probing design is cleaner. Even cleaner would
be a proper bus type but that's a different can of worms.
Thanks,
Laura
--
Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
next prev parent reply other threads:[~2015-02-09 21:27 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-06 0:31 [PATCH/RFC 0/4] Probe deferral for IOMMU DT integration Laura Abbott
2015-02-06 0:31 ` Laura Abbott
2015-02-06 0:31 ` Laura Abbott
[not found] ` <1423182722-16646-1-git-send-email-lauraa-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2015-02-06 0:31 ` [PATCH/RFC 1/4] dma-mapping: Make arch_setup_dma_ops return an error code Laura Abbott
2015-02-06 0:31 ` Laura Abbott
2015-02-06 0:31 ` Laura Abbott
2015-02-06 0:32 ` [PATCH/RFC 2/4] of: Return error codes from of_dma_configure Laura Abbott
2015-02-06 0:32 ` Laura Abbott
2015-02-06 0:32 ` Laura Abbott
2015-02-06 0:32 ` [PATCH/RFC 3/4] iommu/arm-smmu: add support for specifying clocks Laura Abbott
2015-02-06 0:32 ` Laura Abbott
2015-02-06 0:32 ` Laura Abbott
2015-02-06 0:32 ` [PATCH/RFC 4/4] iommu/arm-smmu: Support deferred probing Laura Abbott
2015-02-06 0:32 ` Laura Abbott
2015-02-06 0:32 ` Laura Abbott
[not found] ` <1423182722-16646-5-git-send-email-lauraa-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2015-02-06 18:31 ` Robin Murphy
2015-02-06 18:31 ` Robin Murphy
2015-02-06 18:31 ` Robin Murphy
2015-02-07 22:41 ` [PATCH/RFC 0/4] Probe deferral for IOMMU DT integration arnd-r2nGTMty4D4
2015-02-07 22:41 ` arnd
2015-02-07 22:41 ` arnd at arndb.de
2015-02-09 21:27 ` Laura Abbott [this message]
2015-02-09 21:27 ` Laura Abbott
[not found] ` <2115835102.210232.1423348884955.JavaMail.open-xchange-h4m1HHXQYNGZU4JK52HgGMgmgJlYmuWJ@public.gmane.org>
2015-02-11 9:37 ` Marek Szyprowski
2015-02-11 9:37 ` Marek Szyprowski
2015-02-11 9:37 ` Marek Szyprowski
2015-02-16 16:14 ` Laurent Pinchart
2015-02-16 16:14 ` Laurent Pinchart
2015-02-16 16:14 ` Laurent Pinchart
2015-03-03 23:54 ` Laurent Pinchart
2015-03-03 23:54 ` Laurent Pinchart
2015-03-03 23:54 ` Laurent Pinchart
2015-03-04 15:25 ` Will Deacon
2015-03-04 15:25 ` Will Deacon
2015-03-04 15:25 ` Will Deacon
[not found] ` <20150304152505.GE17250-5wv7dgnIgG8@public.gmane.org>
2015-03-04 15:41 ` Laurent Pinchart
2015-03-04 15:41 ` Laurent Pinchart
2015-03-04 15:41 ` Laurent Pinchart
2015-03-04 9:20 ` Marek Szyprowski
2015-03-04 9:20 ` Marek Szyprowski
2015-03-04 9:20 ` Marek Szyprowski
[not found] ` <54F6CE64.8040503-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2015-05-12 10:01 ` Laurent Pinchart
2015-05-12 10:01 ` Laurent Pinchart
2015-05-12 10:01 ` Laurent Pinchart
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=54D9265E.1030403@codeaurora.org \
--to=lauraa@codeaurora.org \
--cc=arnd@arndb.de \
--cc=devicetree@vger.kernel.org \
--cc=grant.likely@linaro.org \
--cc=iommu@lists.linux-foundation.org \
--cc=joro@8bytes.org \
--cc=laurent.pinchart+renesas@ideasonboard.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=m.szyprowski@samsung.com \
--cc=mitchelh@codeaurora.org \
--cc=robin.murphy@arm.com \
--cc=will.deacon@arm.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 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.