RE: [PATCH 4/9] RapidIO: Add relation links between RIO device structures

2010-08-17 Thread Bounine, Alexandre
Micha Nelissen wrote: > > I agree it's desirable to have this information. Notes: > 1) is rio_dev->prev used anywhere? (maybe I missed it) It is used to scan route back when servicing an orphaned PW message. I also see its future use when invalidating route(s) in case of device removal. > 2) i

Re: [PATCH 4/9] RapidIO: Add relation links between RIO device structures

2010-08-17 Thread Micha Nelissen
Bounine, Alexandre wrote: As RapidIO is a switched network, the concept of 'previous' and 'next' devices is invalid. Perhaps it's just the way they were discovered/enumerated, but that does not matter any more at runtime. Or at least, should not matter. Yes, the "previous" and "next" have to b

RE: [PATCH 4/9] RapidIO: Add relation links between RIO device structures

2010-08-16 Thread Bounine, Alexandre
Micha Nelissen wrote: > > Alexandre Bounine wrote: > > Create back and forward links between RIO devices. These links are intended for > > use by error management and hot-plug extensions. > > As RapidIO is a switched network, the concept of 'previous' and 'next' > devices is invalid. Perhaps it's

Re: [PATCH 4/9] RapidIO: Add relation links between RIO device structures

2010-08-16 Thread Micha Nelissen
Alexandre Bounine wrote: Create back and forward links between RIO devices. These links are intended for use by error management and hot-plug extensions. As RapidIO is a switched network, the concept of 'previous' and 'next' devices is invalid. Perhaps it's just the way they were discovered/e

[PATCH 4/9] RapidIO: Add relation links between RIO device structures

2010-08-13 Thread Alexandre Bounine
Create back and forward links between RIO devices. These links are intended for use by error management and hot-plug extensions. Signed-off-by: Alexandre Bounine Reviewed-by: Thomas Moll Cc: Matt Porter Cc: Li Yang Cc: Kumar Gala --- drivers/rapidio/rio-scan.c | 55 +---