lkml.org 
[lkml]   [2018]   [Nov]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH anybus v3 1/6] misc: support the Arcx anybus bridge
On Tue, Nov 6, 2018 at 1:31 PM Rob Herring <robh@kernel.org> wrote:

> If the host is not a h/w component, but just a s/w protocol then it
> doesn't belong in DT. Perhaps it could be a library which the bridge
> driver can call into.

Anybus cards have an id register, which identifies what they are, so
that the appropriate client driver may be instantiated.
In that sense anybus is very suited to the bus/client abstraction of
pci/usb/etc.

> What are the resets connected to? The slots? Maybe you should model
> the slots in DT.
>

Yes, the resets are ultimately connected to the slots.
I'm happy to model the slots in DT. It makes sense, they are physical,
hardware components.

>
> A block diagram would help. Something that shows the host SoC, your
> CPLD, reset, irq, etc.
>

+------------------------------------------------------------------------+
| SOC (i.MX6) |
|------------------------------------------------------------------------|
| i.MX WEIM bus | i.MX GPIO |
+------------------------------------------------------------------------+
^ ^ ^
| parallel bus | | irq x2
v | |
+------------------------------------------------------------------------+
| CPLD |
|------------------------------------------------------------------------|
| anybus slot 1 || anybus slot 2 |
|------------------------------------------------------------------------|
| memory bus | irq| reset || memory bus | irq| reset |
+------------------------------------------------------------------------+
^ ^
| |
v v
+--------------------------------------+ +-------------------------------+
| anybus card | | anybus card |
+--------------------------------------+ +-------------------------------+

\
 
 \ /
  Last update: 2018-11-06 21:07    [W:0.136 / U:0.628 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site