[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <0CE8B6BE3C4AD74AB97D9D29BD24E552011D6074@CORPEXCH1.na.ads.idt.com>
Date: Mon, 16 Aug 2010 11:30:39 -0700
From: "Bounine, Alexandre" <Alexandre.Bounine@....com>
To: "Micha Nelissen" <micha@...i.hopto.org>
Cc: <akpm@...ux-foundation.org>, <linux-kernel@...r.kernel.org>,
<linuxppc-dev@...abs.org>,
"Bounine, Alexandre" <Alexandre.Bounine@....com>
Subject: RE: [PATCH 9/9] RapidIO: Add support for IDT CPS Gen2 switches
Micha Nelissen wrote:
>
> Alexandre Bounine wrote:
> > + rio_mport_write_config_32(mport, destid, hopcount,
> > + LOCAL_RTE_CONF_DESTID_SEL, table);
> > +
> > + for (i = 0x80000000; i <= 0x800000ff;) {
> > + rio_mport_write_config_32(mport, destid, hopcount,
> > + RIO_STD_RTE_CONF_DESTID_SEL_CSR, i);
>
> The 0x80000000 is that an autoincrement bit? If so, it only needs to
do
> this once I think? If so please call it like that, and loop the 'i'
> variable through the destination IDs.
No. This is EXTENDED_CONFIGURATION_ENABLE bit defined in Part 3 of RIO
spec (v.2.1, sect.3.5.4).
Anyway, I should add definition for this bit. Will do now.
Alex.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists