[PATCH] PCI Domains Support

[Simon Horman]

On Sat, Dec 26, 2009 at 05:58:05AM +0800, Richard Liu wrote:
> Simon Horman wrote:
> >> Actually, most ARM platforms didn't need PCI domains.
> >>
> >> So, original patch is
> >>
> >> +config PCI_DOMAINS
> >> +        def_bool y
> >> +        depends on PCI && ARCH_CXXXXXX
> >> +
> >>
> >> But I removed the "&& ARCH_CXXXXXXX" before I provided the patch.
> >> Because ARCH_CXXXXXXX is not exist in ARM Linux kernel now (maybe it
> >> would be committed at someday)
> >>     
> >
> > Ok, now I am completely confused. Nothing currently in-tree should
> > use PCI_DOMAINS, but it defaults to y and nothing switches it to n?
> >
> > Perhaps the following would be more appropriate:
> >
> > config PCI_DOMAINS
> >         bool
> >
> > And then platforms can "select PCI_DOMAINS" as needed.
> >
> >   
> You are right,
> Don't use def_bool is right choice.
> 
> I only maintain one arch, so I didn't think too much.
> 
> Should I regenerate a new patch, or you will handle it ?
> 
> 

I think it would be best if you regenerated your patch.