All of lore.kernel.org
 help / color / mirror / Atom feed
From: marc.zyngier@arm.com (Marc Zyngier)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/9] ARM Versatile multi-platform support
Date: Mon, 05 Jan 2015 11:19:00 +0000	[thread overview]
Message-ID: <87y4ph79wb.fsf@approximate.cambridge.arm.com> (raw)
In-Reply-To: <CAFEAcA8Nq6BPOerN7EhHpT_oa9W3+2Y_=5+zBuQD_c5fh0Yb1A@mail.gmail.com> (Peter Maydell's message of "Mon, 5 Jan 2015 10:08:18 +0000")

On Mon, Jan 05 2015 at 10:08:18 am GMT, Peter Maydell <peter.maydell@linaro.org> wrote:
> On 5 January 2015 at 09:50, Marc Zyngier <marc.zyngier@arm.com> wrote:
>> On 31/12/14 09:25, Peter Maydell wrote:
>>> Will Deacon and Marc Zyngier are currently in possession of that h/w.
>>> I'm sure they'd be delighted to pull it out of the cupboard and
>>> give it a spin :-)
>>
>> I'll try to find some time (or a victim to do the testing... ;-)
>
> To save you some time in trying to debug things which didn't work
> before this patchset, the status last time I tested on h/w was:
>  * PCI MMIO accesses work
>  * PCI IO accesses work
>  * PCI bus-master (DMA) doesn't work
>
> with the failure symptom for DMA being that the card wrote to
> the correct area of memory but only every other word, so you
> got strings like "HellXXXXrld" with the Xs being garbage.
> This feels to me like a h/w issue rather than a driver problem.

Looks like some 64bit address decoding getting in the way. Or
something.... Do you remember the exact platform you tried this on (I
have PB926, PB1176, and a bunch of RealViews with an odd mix of
tiles...)?

	M.
-- 
Jazz is not dead. It just smells funny.

WARNING: multiple messages have this Message-ID (diff)
From: Marc Zyngier <marc.zyngier-5wv7dgnIgG8@public.gmane.org>
To: Peter Maydell <peter.maydell-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: Rob Herring <robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	arm-mail-list
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
	"devicetree@vger.kernel.org"
	<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Russell King <linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>,
	Linus Walleij
	<linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	"arm@kernel.org" <arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	Bjorn Helgaas <bhelgaas-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>,
	Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	Will Deacon <Will.Deacon-5wv7dgnIgG8@public.gmane.org>
Subject: Re: [PATCH 0/9] ARM Versatile multi-platform support
Date: Mon, 05 Jan 2015 11:19:00 +0000	[thread overview]
Message-ID: <87y4ph79wb.fsf@approximate.cambridge.arm.com> (raw)
In-Reply-To: <CAFEAcA8Nq6BPOerN7EhHpT_oa9W3+2Y_=5+zBuQD_c5fh0Yb1A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> (Peter Maydell's message of "Mon, 5 Jan 2015 10:08:18 +0000")

On Mon, Jan 05 2015 at 10:08:18 am GMT, Peter Maydell <peter.maydell-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> wrote:
> On 5 January 2015 at 09:50, Marc Zyngier <marc.zyngier-5wv7dgnIgG8@public.gmane.org> wrote:
>> On 31/12/14 09:25, Peter Maydell wrote:
>>> Will Deacon and Marc Zyngier are currently in possession of that h/w.
>>> I'm sure they'd be delighted to pull it out of the cupboard and
>>> give it a spin :-)
>>
>> I'll try to find some time (or a victim to do the testing... ;-)
>
> To save you some time in trying to debug things which didn't work
> before this patchset, the status last time I tested on h/w was:
>  * PCI MMIO accesses work
>  * PCI IO accesses work
>  * PCI bus-master (DMA) doesn't work
>
> with the failure symptom for DMA being that the card wrote to
> the correct area of memory but only every other word, so you
> got strings like "HellXXXXrld" with the Xs being garbage.
> This feels to me like a h/w issue rather than a driver problem.

Looks like some 64bit address decoding getting in the way. Or
something.... Do you remember the exact platform you tried this on (I
have PB926, PB1176, and a bunch of RealViews with an odd mix of
tiles...)?

	M.
-- 
Jazz is not dead. It just smells funny.
--
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

  reply	other threads:[~2015-01-05 11:19 UTC|newest]

Thread overview: 86+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-30 19:28 [PATCH 0/9] ARM Versatile multi-platform support Rob Herring
2014-12-30 19:28 ` Rob Herring
2014-12-30 19:28 ` [PATCH 1/9] dt/bindings: add versatile PCI binding Rob Herring
2014-12-30 19:28   ` Rob Herring
2014-12-30 19:28 ` [PATCH 2/9] dts: versatile: add PCI controller binding Rob Herring
2014-12-30 19:28   ` Rob Herring
2014-12-30 19:28 ` [PATCH 3/9] ARM: versatile: add DT based PCI detection Rob Herring
2014-12-30 19:28   ` Rob Herring
2014-12-30 21:37   ` Arnd Bergmann
2014-12-30 21:37     ` Arnd Bergmann
2014-12-30 23:05     ` Rob Herring
2014-12-30 23:05       ` Rob Herring
2014-12-31 15:23       ` Arnd Bergmann
2014-12-31 15:23         ` Arnd Bergmann
2014-12-31 16:13         ` Peter Maydell
2014-12-31 16:13           ` Peter Maydell
2014-12-31 19:22           ` Rob Herring
2014-12-31 19:22             ` Rob Herring
2014-12-31 21:07             ` Peter Maydell
2014-12-31 21:07               ` Peter Maydell
2015-01-01 15:35               ` Arnd Bergmann
2015-01-01 15:35                 ` Arnd Bergmann
2015-01-01 15:52                 ` Peter Maydell
2015-01-01 15:52                   ` Peter Maydell
2015-01-08 19:37   ` Linus Walleij
2015-01-08 19:37     ` Linus Walleij
2015-01-08 21:34     ` Rob Herring
2015-01-08 21:34       ` Rob Herring
2014-12-30 19:28 ` [PATCH 4/9] pci: add DT based ARM Versatile PCI host driver Rob Herring
2014-12-30 19:28   ` Rob Herring
2014-12-30 21:58   ` Arnd Bergmann
2014-12-30 21:58     ` Arnd Bergmann
2015-01-02 18:14     ` Rob Herring
2015-01-02 18:14       ` Rob Herring
2015-01-02 20:52       ` Arnd Bergmann
2015-01-02 20:52         ` Arnd Bergmann
2015-01-02 23:13         ` Rob Herring
2015-01-02 23:13           ` Rob Herring
2015-01-05  9:35           ` Arnd Bergmann
2015-01-05  9:35             ` Arnd Bergmann
2015-01-24  1:01     ` Bjorn Helgaas
2015-01-24  1:01       ` Bjorn Helgaas
2015-01-24  0:54   ` Bjorn Helgaas
2015-01-24  0:54     ` Bjorn Helgaas
2014-12-30 19:28 ` [PATCH 5/9] dts: versatile: add sysregs nodes Rob Herring
2014-12-30 19:28   ` Rob Herring
2015-01-08 19:44   ` Linus Walleij
2015-01-08 19:44     ` Linus Walleij
2015-01-08 23:53     ` Rob Herring
2015-01-08 23:53       ` Rob Herring
2015-01-09  7:10       ` Linus Walleij
2015-01-09  7:10         ` Linus Walleij
2015-01-09 11:53         ` Lorenzo Pieralisi
2015-01-09 11:53           ` Lorenzo Pieralisi
2015-01-15 16:06       ` Lorenzo Pieralisi
2015-01-15 16:06         ` Lorenzo Pieralisi
2015-01-19 10:25         ` Linus Walleij
2015-01-19 10:25           ` Linus Walleij
2014-12-30 19:28 ` [PATCH 6/9] ARM: versatile: switch to DT only booting and remove legacy code Rob Herring
2014-12-30 19:28   ` Rob Herring
2014-12-30 19:28 ` [PATCH 7/9] ARM: versatile: move mach includes into mach directory Rob Herring
2014-12-30 19:28   ` Rob Herring
2014-12-30 22:05   ` Arnd Bergmann
2014-12-30 22:05     ` Arnd Bergmann
2014-12-30 19:28 ` [PATCH 8/9] ARM: versatile: convert to multi-platform Rob Herring
2014-12-30 19:28   ` Rob Herring
2014-12-30 19:28 ` [PATCH 9/9] ARM: versatile: consolidate code to single file Rob Herring
2014-12-30 19:28   ` Rob Herring
2014-12-30 22:08 ` [PATCH 0/9] ARM Versatile multi-platform support Arnd Bergmann
2014-12-30 22:08   ` Arnd Bergmann
2014-12-31  9:25 ` Peter Maydell
2014-12-31  9:25   ` Peter Maydell
2015-01-05  9:50   ` Marc Zyngier
2015-01-05  9:50     ` Marc Zyngier
2015-01-05 10:08     ` Peter Maydell
2015-01-05 10:08       ` Peter Maydell
2015-01-05 11:19       ` Marc Zyngier [this message]
2015-01-05 11:19         ` Marc Zyngier
2015-01-05 17:41         ` Peter Maydell
2015-01-05 17:41           ` Peter Maydell
2015-01-08 19:47 ` Linus Walleij
2015-01-08 19:47   ` Linus Walleij
2015-01-08 21:38   ` Rob Herring
2015-01-08 21:38     ` Rob Herring
     [not found]     ` <CAL_JsqKLNPVDCUELaZU8JW0roT3RcyqcxtJbvbYQrjzxjt3FeQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-01-09  8:34       ` Linus Walleij
2015-01-09  8:34         ` Linus Walleij

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=87y4ph79wb.fsf@approximate.cambridge.arm.com \
    --to=marc.zyngier@arm.com \
    --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 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.