From: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
To: Jingchang Lu <jingchang.lu-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
Cc: "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
Shawn Guo <Shawn.Guo-KZfg59tc24xl57MIdRCFDg@public.gmane.org>,
"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: [PATCHv3 5/6] ARM: imx: Add initial support for Freescale LS1021A
Date: Wed, 10 Sep 2014 09:42:02 +0200 [thread overview]
Message-ID: <20140402.o60F9Z4YVe@wuerfel> (raw)
In-Reply-To: <daf5ec5b70164b3e9a810def41909c48-AZ66ij2kwab4MB1ZSnT4iOO6mTEJWrR4XA4E9RH9d+qIuWR1G4zioA@public.gmane.org>
On Wednesday 10 September 2014 03:31:19 Jingchang Lu wrote:
> >> +DT_MACHINE_START(LS1021A, "Freescale LS1021A")
> >> + #ifdef CONFIG_ZONE_DMA
> >> + .dma_zone_size = SZ_128M,
> >> +#endif
> >> + .init_machine = ls1021a_init_machine,
> >> + .dt_compat = ls1021a_dt_compat,
> >> + .restart = mxc_restart,
> >> +MACHINE_END
> >
> >I believe someone recently posted a patch to derive the dma_zone_size from
> >device tree. Can yo try to find that and see if that will work for you?
> >
> >Can you explain what the reason is for needing a DMA zone?
>
> With LPAE enabled on our SoC, we meet the system complaint of
> "coherent DMA mask 0xffffffff is smaller than system GFP_DMA mask 0xffffffffffffffff",
> and I notice that CONFIG_ZONE_DMA and dma_zone_size is a common resolve for this.
Ok, I see. The actual point of dma_zone_size however is slightly
different, and I think you should not use it like this. We normally
only use ZONE_DMA if there are devices that have a limitation smaller
than 4GB, and that appear to be the case for your system.
The message you quote is only present in arch/powerpc, so I'm not sure
what symptoms you are actually seeing. Please try removing the
dma_zone_size setting for your platform and report if it works or
what the symptom is if it does not work with the latest kernel.
We definitely need to get this to work out of the box without
a dma_zone_size hack.
Can you describe what the memory layout is of your platform? Can you
have RAM installed above the 4GB physical address boundary? If you
can, are there any devices that are unable to perform DMA into that
memory without the use of an IOMMU?
Arnd
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2014-09-10 7:42 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-09 9:12 [PATCHv3 0/6] ARM: imx: Add Freescale LS1021A SoC and board support Jingchang Lu
2014-09-09 9:12 ` [PATCHv3 3/6] ARM: dts: Add initial LS1021A TWR board dts support Jingchang Lu
[not found] ` <1410253952-15631-1-git-send-email-jingchang.lu-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
2014-09-09 9:12 ` [PATCHv3 1/6] ARM: dts: Add SoC level device tree support for LS1021A Jingchang Lu
2014-09-11 8:41 ` suresh.gupta
[not found] ` <fa256d13e4304175b14369b9d3b55729-AZ66ij2kwaYNjdDjpj1h++O6mTEJWrR4XA4E9RH9d+qIuWR1G4zioA@public.gmane.org>
2014-09-11 8:58 ` Jingchang Lu
[not found] ` <1410253952-15631-2-git-send-email-jingchang.lu-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
2014-09-09 11:50 ` Arnd Bergmann
2014-09-11 8:21 ` Jingchang Lu
2014-09-09 11:53 ` Arnd Bergmann
2014-09-11 8:21 ` Jingchang Lu
[not found] ` <cd69af536cd248b8add94a468fa97095-AZ66ij2kwab4MB1ZSnT4iOO6mTEJWrR4XA4E9RH9d+qIuWR1G4zioA@public.gmane.org>
2014-09-11 10:36 ` Arnd Bergmann
2014-09-11 11:12 ` Sascha Hauer
[not found] ` <20140911111202.GA4958-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2014-09-12 9:59 ` Jingchang Lu
2014-09-11 10:10 ` nikhil.badola-KZfg59tc24xl57MIdRCFDg
[not found] ` <1444adbcf7cd4b578a131db11e662490-RQSpjbwlmjRJV8q+uXLxw5wN6zqB+hSMnBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org>
2014-09-12 1:46 ` Jingchang Lu
2014-09-09 9:12 ` [PATCHv3 2/6] ARM: dts: Add initial LS1021A QDS board dts support Jingchang Lu
2014-09-09 9:12 ` [PATCHv3 4/6] dt-bindings: arm: add Freescale LS1021A SoC device tree binding Jingchang Lu
2014-09-09 9:12 ` [PATCHv3 5/6] ARM: imx: Add initial support for Freescale LS1021A Jingchang Lu
[not found] ` <1410253952-15631-6-git-send-email-jingchang.lu-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
2014-09-09 11:41 ` Arnd Bergmann
2014-09-10 3:31 ` Jingchang Lu
[not found] ` <daf5ec5b70164b3e9a810def41909c48-AZ66ij2kwab4MB1ZSnT4iOO6mTEJWrR4XA4E9RH9d+qIuWR1G4zioA@public.gmane.org>
2014-09-10 7:42 ` Arnd Bergmann [this message]
2014-09-11 9:53 ` Jingchang Lu
[not found] ` <517628172fb3416ea7b12b4ae24e098a-AZ66ij2kwab4MB1ZSnT4iOO6mTEJWrR4XA4E9RH9d+qIuWR1G4zioA@public.gmane.org>
2014-09-11 10:44 ` Arnd Bergmann
2014-09-12 3:17 ` Jingchang Lu
2014-09-11 10:05 ` Jingchang Lu
2014-09-09 9:12 ` [PATCHv3 6/6] ARM: imx: Add Freescale LS1021A SMP support Jingchang Lu
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=20140402.o60F9Z4YVe@wuerfel \
--to=arnd-r2ngtmty4d4@public.gmane.org \
--cc=Shawn.Guo-KZfg59tc24xl57MIdRCFDg@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=jingchang.lu-KZfg59tc24xl57MIdRCFDg@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.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).