All of lore.kernel.org
 help / color / mirror / Atom feed
From: arno@natisbad.org (Arnaud Ebalard)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/2] ARM: mvebu: fix DT def. of PCIe units for mv78230 and mv78260
Date: Wed, 06 Nov 2013 22:32:57 +0100	[thread overview]
Message-ID: <87ob5xqkqe.fsf@natisbad.org> (raw)
In-Reply-To: <20131106205556.GH8308@titan.lakedaemon.net> (Jason Cooper's message of "Wed, 6 Nov 2013 15:55:56 -0500")

Hi Jason,

Jason Cooper <jason@lakedaemon.net> writes:

> On Wed, Nov 06, 2013 at 11:37:24AM -0700, Jason Gunthorpe wrote:
> ...
>> This PCI stuff would be 100% more readable with some macros ..
>> 
>> #define PCI_REG_BDF(bus,device,function) (....)
>> #define PCI_MEM_BAR_BDF(bus,device,function,bar) \
>>    (0x82000000 | PCI_REG_BDF(bus,device,function) + bar*XX) 0)
>> 
>> So that:
>>  reg = <PCI_REG_BDF(0,1,0) 0 0>
>>  assigned-addresses = <PCI_MEM_BAR_BDF(0,1,0,0)  0x40000  0 0x2000>
>
> Do I smell a patch brewing?  :)
>
> Arnaud, would you like to take a crack at it?

My understanding of the topic being in the end limited to what I learnt
from recent discussions, I'll pass on that one but I'll be happy to test
on a mv78230.

Cheers,

a+

  reply	other threads:[~2013-11-06 21:32 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-05 20:45 [PATCH 0/2] ARM: mvebu: fix DT def. of PCIe units for mv78230 and mv78260 Arnaud Ebalard
2013-11-05 20:45 ` [PATCH 1/2] ARM: mvebu: second PCIe unit of Armada XP mv78230 is only x1 capable Arnaud Ebalard
2013-11-06 14:43   ` Thomas Petazzoni
2013-11-06 18:08     ` Arnaud Ebalard
2013-11-06 18:12       ` Jason Cooper
2013-11-06 18:20         ` Thomas Petazzoni
2013-11-21 23:28         ` Arnaud Ebalard
2013-11-22 13:44           ` Jason Cooper
2013-11-22 14:28             ` Thomas Petazzoni
2013-11-05 20:46 ` [PATCH 2/2] ARM: mvebu: fix second and third PCIe unit of Armada XP mv78260 Arnaud Ebalard
2013-11-06 14:54   ` Thomas Petazzoni
2013-11-06 18:14 ` [PATCH 0/2] ARM: mvebu: fix DT def. of PCIe units for mv78230 and mv78260 Arnaud Ebalard
2013-11-06 18:22   ` Thomas Petazzoni
2013-11-06 18:37     ` Jason Gunthorpe
2013-11-06 20:08       ` Arnaud Ebalard
2013-11-06 20:55       ` Jason Cooper
2013-11-06 21:32         ` Arnaud Ebalard [this message]
2013-11-22 15:04 ` Jason Cooper
2013-11-22 15:29   ` Arnaud Ebalard
2013-11-23 15:36 ` Jason Cooper

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=87ob5xqkqe.fsf@natisbad.org \
    --to=arno@natisbad.org \
    --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.