linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Wade Farnsworth <wfarnsworth@mvista.com>
To: Zhang Wei-r63237 <Wei.Zhang@freescale.com>
Cc: linuxppc-dev <linuxppc-dev@ozlabs.org>
Subject: RE: [PATCH 5/5] PCI fixes for the MPC8641 Rev 2.0 silicon and Rev 1.02hardware
Date: Thu, 17 May 2007 09:03:33 -0700	[thread overview]
Message-ID: <1179417813.8132.250.camel@rhino> (raw)
In-Reply-To: <46B96294322F7D458F9648B60E15112C23441B@zch01exm26.fsl.freescale.net>

On Thu, 2007-05-17 at 10:50 +0800, Zhang Wei-r63237 wrote:
> Hi, 
> 
> If you add below sector to pci@8000, please remove the interrupt-map
> sector from pci@8000.

Correct me if I'm wrong, but I think if the interrupt-map for pci@8000
is removed then the pci devices on the revision 1.0 board will not get
the correct interrupts.  I don't currently have a rev 1.0 to test
though.  Can someone with a rev 1.0 HPCN board confirm this?

--Wade

> > @@ -286,6 +286,125 @@
> >  				f800 0 0 4 &i8259 0 0
> >  				>;
> >  
> > +			pci@00 {
> > +				device_type = "pci";
> > +				#interrupt-cells = <1>;
> > +				#size-cells = <2>;
> > +				#address-cells = <3>;
> > +				reg = <0000 0 0 0 0>;
> > +				bus-range = <0 fe>;
> > +				ranges = <02000000 0 80000000 80000000 0
> > +					  20000000
> > +					  01000000 0 00000000 00000000 0
> > +					  00100000>;
> > +
> > +				pci@00 {
> > +					device_type = "pci";
> > +					#interrupt-cells = <1>;
> > +					#size-cells = <2>;
> > +					#address-cells = <3>;
> > +					reg = <0000 0 0 0 0>;
> > +					bus-range = <1 fe>;
> > +					ranges = <02000000 0 
> > 80000000 80000000
> > +						  0 20000000
> > +						  01000000 0 
> > 00000000 00000000
> > +						  0 00100000>;
> > +					interrupt-map-mask = 
> > <3f800 0 0 7>;
> > +					interrupt-map = <
> > +						/* IDSEL 0x11 */
> > +						28800 0 0 1 &i8259 3 2
> > +						28800 0 0 2 &i8259 4 2
> > +						28800 0 0 3 &i8259 5 2
> > +						28800 0 0 4 &i8259 6 2
> > +
> > +						/* IDSEL 0x12 */
> > +						29000 0 0 1 &i8259 4 2
> > +						29000 0 0 2 &i8259 5 2
> > +						29000 0 0 3 &i8259 6 2
> > +						29000 0 0 4 &i8259 3 2
> > +
> > +						/* IDSEL 0x13 */
> > +						29800 0 0 1 &i8259 0 0
> > +						29800 0 0 2 &i8259 0 0
> > +						29800 0 0 3 &i8259 0 0
> > +						29800 0 0 4 &i8259 0 0
> > +
> > +						/* IDSEL 0x14 */
> > +						2a000 0 0 1 &i8259 0 0
> > +						2a000 0 0 2 &i8259 0 0
> > +						2a000 0 0 3 &i8259 0 0
> > +						2a000 0 0 4 &i8259 0 0
> > +
> > +						/* IDSEL 0x15 */
> > +						2a800 0 0 1 &i8259 0 0
> > +						2a800 0 0 2 &i8259 0 0
> > +						2a800 0 0 3 &i8259 0 0
> > +						2a800 0 0 4 &i8259 0 0
> > +
> > +						/* IDSEL 0x16 */
> > +						2b000 0 0 1 &i8259 0 0
> > +						2b000 0 0 2 &i8259 0 0
> > +						2b000 0 0 3 &i8259 0 0
> > +						2b000 0 0 4 &i8259 0 0
> > +
> > +						/* IDSEL 0x17 */
> > +						2b800 0 0 1 &i8259 0 0
> > +						2b800 0 0 2 &i8259 0 0
> > +						2b800 0 0 3 &i8259 0 0
> > +						2b800 0 0 4 &i8259 0 0
> > +
> > +						/* IDSEL 0x18 */
> > +						2c000 0 0 1 &i8259 0 0
> > +						2c000 0 0 2 &i8259 0 0
> > +						2c000 0 0 3 &i8259 0 0
> > +						2c000 0 0 4 &i8259 0 0
> > +
> > +						/* IDSEL 0x19 */
> > +						2c800 0 0 1 &i8259 0 0
> > +						2c800 0 0 2 &i8259 0 0
> > +						2c800 0 0 3 &i8259 0 0
> > +						2c800 0 0 4 &i8259 0 0
> > +
> > +						/* IDSEL 0x1a */
> > +						2d000 0 0 1 &i8259 6 2
> > +						2d000 0 0 2 &i8259 3 2
> > +						2d000 0 0 3 &i8259 4 2
> > +						2d000 0 0 4 &i8259 5 2
> > +
> > +						/* IDSEL 0x1b */
> > +						2d800 0 0 1 &i8259 5 2
> > +						2d800 0 0 2 &i8259 0 0
> > +						2d800 0 0 3 &i8259 0 0
> > +						2d800 0 0 4 &i8259 0 0
> > +
> > +						/* IDSEL 0x1c */
> > +						2e000 0 0 1 &i8259 9 2
> > +						2e000 0 0 2 &i8259 a 2
> > +						2e000 0 0 3 &i8259 b 2
> > +						2e000 0 0 4 &i8259 7 2
> > +
> > +						/* IDSEL 0x1d */
> > +						2e800 0 0 1 &i8259 9 2
> > +						2e800 0 0 2 &i8259 a 2
> > +						2e800 0 0 3 &i8259 b 2
> > +						2e800 0 0 4 &i8259 0 0
> > +
> > +						/* IDSEL 0x1e */
> > +						2f000 0 0 1 &i8259 b 2
> > +						2f000 0 0 2 &i8259 0 0
> > +						2f000 0 0 3 &i8259 0 0
> > +						2f000 0 0 4 &i8259 0 0
> > +
> > +						/* IDSEL 0x1f */
> > +						2f800 0 0 1 &i8259 6 2
> > +						2f800 0 0 2 &i8259 0 0
> > +						2f800 0 0 3 &i8259 0 0
> > +						2f800 0 0 4 &i8259 0 0
> > +						>;
> > +					};
> > +				};
> > +
> > +
> >  			isa@f0 {
> >  				device_type = "isa";
> >  				#interrupt-cells = <2>;
> > @@ -335,7 +454,7 @@
> >  			#size-cells = <2>;
> >  			#address-cells = <3>;
> >  			reg = <9000 1000>;
> > -			bus-range = <0 ff>;
> > +			bus-range = <3 ff>;
> >  			ranges = <02000000 0 a0000000 a0000000 
> > 0 20000000
> >  				  01000000 0 00000000 e3000000 
> > 0 00100000>;
> >  			clock-frequency = <1fca055>;
> 
> Best Regards,
> Zhang Wei

  reply	other threads:[~2007-05-17 16:03 UTC|newest]

Thread overview: 67+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-15 16:17 [PATCH 0/5] MPC8641 HPCN fixes Wade Farnsworth
2007-05-15 16:23 ` [PATCH 1/5] Add MTD support to mpc8641_hpcn.dts Wade Farnsworth
2007-05-15 16:27 ` [PATCH 2/5] Add legacy devices " Wade Farnsworth
2007-05-16 16:37   ` Jon Loeliger
2007-05-16 17:59     ` Wade Farnsworth
2007-05-16 19:59       ` Segher Boessenkool
2007-05-16 19:55     ` Segher Boessenkool
2007-05-16 20:03       ` Sergei Shtylyov
2007-05-16 20:17       ` Wade Farnsworth
2007-05-16 23:40         ` Segher Boessenkool
2007-05-16 23:47           ` Segher Boessenkool
2007-05-17  0:11             ` Wade Farnsworth
2007-05-17  0:09           ` Wade Farnsworth
2007-05-18 16:44           ` Jon Loeliger
2007-05-21 16:04             ` Wade Farnsworth
2007-05-21 16:10               ` Jon Loeliger
2007-05-21 16:51                 ` Wade Farnsworth
2007-05-16 20:29       ` Wade Farnsworth
2007-07-07 18:29       ` Sergei Shtylyov
2007-07-07 18:39         ` Arnd Bergmann
2007-07-08 13:13         ` Segher Boessenkool
2007-07-08 20:36           ` Arnd Bergmann
2007-07-08 23:42             ` Segher Boessenkool
2007-07-09 12:43           ` Sergei Shtylyov
2007-07-09 14:43             ` Segher Boessenkool
2007-07-07 18:59   ` Sergei Shtylyov
2007-07-07 19:23     ` Arnd Bergmann
2007-05-15 16:32 ` [PATCH 3/5] Support for the LPC47M192 SuperIO on the MPC8641 HPCN Wade Farnsworth
2007-05-15 16:36 ` [PATCH 4/5] Add RTC CMOS support for " Wade Farnsworth
2007-05-15 16:50 ` [PATCH 5/5] PCI fixes for the MPC8641 Rev 2.0 silicon and Rev 1.02 hardware Wade Farnsworth
2007-05-16 16:44   ` Jon Loeliger
2007-05-16 18:06     ` Wade Farnsworth
2007-05-18  6:52       ` [PATCH 5/5] PCI fixes for the MPC8641 Rev 2.0 silicon and Rev1.02 hardware Jin Zhengxiong-R64188
2007-05-21 15:59         ` Wade Farnsworth
2007-05-17  2:50   ` [PATCH 5/5] PCI fixes for the MPC8641 Rev 2.0 silicon and Rev 1.02hardware Zhang Wei-r63237
2007-05-17 16:03     ` Wade Farnsworth [this message]
2007-05-17 16:14       ` Kumar Gala
2007-05-17 16:58         ` Wade Farnsworth
2007-05-17 18:46           ` Timur Tabi
2007-05-17 18:48             ` Timur Tabi
2007-05-17 18:53               ` Scott Wood
2007-05-17 18:59                 ` Timur Tabi
2007-05-17 19:16                   ` Jerry Van Baren
2007-05-18 13:48                   ` Matt Sealey
2007-05-17 22:34             ` Wade Farnsworth
2007-05-18  0:56             ` David Gibson
2007-05-18  3:49               ` Jerry Van Baren
2007-05-18 14:34               ` Timur Tabi
2007-05-18 15:22                 ` Jerry Van Baren
2007-05-18 16:46                 ` Scott Wood
2007-05-18 17:30                   ` Jerry Van Baren
2007-05-18 17:39                     ` Timur Tabi
2007-05-18 17:43                       ` Scott Wood
2007-05-18 17:47                         ` Timur Tabi
2007-05-18 17:52                           ` Scott Wood
2007-05-18 17:56                             ` Timur Tabi
2007-05-18 18:02                               ` Scott Wood
2007-05-18 18:03                               ` Jerry Van Baren
2007-05-18 18:19                               ` Jon Loeliger
2007-05-18 18:23                                 ` Scott Wood
2007-05-18 19:05                                   ` Jon Loeliger
2007-05-18 19:09                                     ` Scott Wood
2007-05-18 18:30                                 ` Timur Tabi
2007-05-18 17:59                       ` Jerry Van Baren
2007-05-18 10:20           ` [PATCH 5/5] PCI fixes for the MPC8641 Rev 2.0 silicon and Rev1.02hardware Zhang Wei-r63237
2007-05-21 16:01             ` Wade Farnsworth
2007-05-22  2:53               ` [PATCH 5/5] PCI fixes for the MPC8641 Rev 2.0 silicon andRev1.02hardware Zhang Wei-r63237

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=1179417813.8132.250.camel@rhino \
    --to=wfarnsworth@mvista.com \
    --cc=Wei.Zhang@freescale.com \
    --cc=linuxppc-dev@ozlabs.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).