Re: [U-Boot] [PATCH] pci: Move CONFIG_PCI_PNP to Kconfig

[Tom Rini]

On Tue, Oct 25, 2016 at 09:08:04AM +0800, Bin Meng wrote:
> Hi Tom,
> 
> On Mon, Oct 17, 2016 at 7:50 PM, Tom Rini  wrote:
> > On Sun, Oct 16, 2016 at 11:35:18PM -0700, Bin Meng wrote:
> >
> >> Introduce CONFIG_PCI_PNP in Kconfig and move over boards' defconfig
> >> to use that.
> >>
> >> Signed-off-by: Bin Meng 
> >>
> >> ---
> >>
> >> Changes in v2:
> >> - Change PCI_PNP to "depends on" and "default y" if PCI || DM_PCI
> >
> > Thanks!
> >
> > [snip]
> >> diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig
> >> index 9a7c187..5252250 100644
> >> --- a/drivers/pci/Kconfig
> >> +++ b/drivers/pci/Kconfig
> >> @@ -18,6 +18,13 @@ config DM_PCI_COMPAT
> >> measure when porting a board to use driver model for PCI. Once the
> >> board is fully supported, this option should be disabled.
> >>
> >> +config PCI_PNP
> >> + bool "Enable Plug & Play support for PCI"
> >> + depends on PCI || DM_PCI
> >> + default y if PCI || DM_PCI
> >
> > This should just be default y since we depend on those two options
> > directly.  I'll just fix it up when applying.
> >
> > Reviewed-by: Tom Rini 
> >
> 
> Will this be picked up for the v2016.11 release?

PCI itself needs to be fully migrated first, it exists in
arch/sandbox/Kconfig which is why this patch doesn't totally blow up,
but it does get a lot of build failures :)

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] pci: Move CONFIG_PCI_PNP to Kconfig

[Bin Meng]

Hi Tom,

On Mon, Oct 17, 2016 at 7:50 PM, Tom Rini  wrote:
> On Sun, Oct 16, 2016 at 11:35:18PM -0700, Bin Meng wrote:
>
>> Introduce CONFIG_PCI_PNP in Kconfig and move over boards' defconfig
>> to use that.
>>
>> Signed-off-by: Bin Meng 
>>
>> ---
>>
>> Changes in v2:
>> - Change PCI_PNP to "depends on" and "default y" if PCI || DM_PCI
>
> Thanks!
>
> [snip]
>> diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig
>> index 9a7c187..5252250 100644
>> --- a/drivers/pci/Kconfig
>> +++ b/drivers/pci/Kconfig
>> @@ -18,6 +18,13 @@ config DM_PCI_COMPAT
>> measure when porting a board to use driver model for PCI. Once the
>> board is fully supported, this option should be disabled.
>>
>> +config PCI_PNP
>> + bool "Enable Plug & Play support for PCI"
>> + depends on PCI || DM_PCI
>> + default y if PCI || DM_PCI
>
> This should just be default y since we depend on those two options
> directly.  I'll just fix it up when applying.
>
> Reviewed-by: Tom Rini 
>

Will this be picked up for the v2016.11 release?

Regards,
Bin
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] pci: Move CONFIG_PCI_PNP to Kconfig

[Bin Meng]

Introduce CONFIG_PCI_PNP in Kconfig and move over boards' defconfig
to use that.

Signed-off-by: Bin Meng 

---

Changes in v2:
- Change PCI_PNP to "depends on" and "default y" if PCI || DM_PCI

 drivers/pci/Kconfig   | 7 +++
 include/configs/B4860QDS.h| 1 -
 include/configs/BSC9132QDS.h  | 2 --
 include/configs/C29XPCIE.h| 2 --
 include/configs/CPCI2DP.h | 2 --
 include/configs/CPCI4052.h| 2 --
 include/configs/M54455EVB.h   | 1 -
 include/configs/M5475EVB.h| 1 -
 include/configs/M5485EVB.h| 1 -
 include/configs/MIP405.h  | 2 --
 include/configs/MPC8308RDB.h  | 2 --
 include/configs/MPC8313ERDB.h | 1 -
 include/configs/MPC8315ERDB.h | 2 --
 include/configs/MPC8323ERDB.h | 1 -
 include/configs/MPC832XEMDS.h | 1 -
 include/configs/MPC8349EMDS.h | 1 -
 include/configs/MPC8349ITX.h  | 2 --
 include/configs/MPC837XEMDS.h | 2 --
 include/configs/MPC837XERDB.h | 1 -
 include/configs/MPC8536DS.h   | 3 ---
 include/configs/MPC8540ADS.h  | 3 ---
 include/configs/MPC8541CDS.h  | 1 -
 include/configs/MPC8544DS.h   | 2 --
 include/configs/MPC8548CDS.h  | 3 ---
 include/configs/MPC8555CDS.h  | 1 -
 include/configs/MPC8560ADS.h  | 3 ---
 include/configs/MPC8568MDS.h  | 3 ---
 include/configs/MPC8569MDS.h  | 3 ---
 include/configs/MPC8572DS.h   | 2 --
 include/configs/MPC8610HPCD.h | 1 -
 include/configs/MPC8641HPCN.h | 2 --
 include/configs/P1010RDB.h| 2 --
 include/configs/P1022DS.h | 1 -
 include/configs/P1023RDB.h| 1 -
 include/configs/P2041RDB.h| 1 -
 include/configs/PIP405.h  | 2 --
 include/configs/PLU405.h  | 2 --
 include/configs/PMC405DE.h| 1 -
 include/configs/PMC440.h  | 1 -
 include/configs/T102xQDS.h| 1 -
 include/configs/T102xRDB.h| 1 -
 include/configs/T1040QDS.h| 2 --
 include/configs/T104xRDB.h| 2 --
 include/configs/T208xQDS.h| 1 -
 include/configs/T208xRDB.h| 1 -
 include/configs/T4240RDB.h| 1 -
 include/configs/TQM5200.h | 1 -
 include/configs/TQM834x.h | 1 -
 include/configs/UCP1020.h | 1 -
 include/configs/a4m072.h  | 1 -
 include/configs/advantech_dms-ba16.h  | 1 -
 include/configs/apalis_t30.h  | 1 -
 include/configs/aria.h| 2 --
 include/configs/bamboo.h  | 1 -
 include/configs/bayleybay.h   | 2 --
 include/configs/beaver.h  | 1 -
 include/configs/boston.h  | 1 -
 include/configs/bubinga.h | 2 --
 include/configs/canyonlands.h | 1 -
 include/configs/cardhu.h  | 1 -
 include/configs/cei-tk1-som.h | 1 -
 include/configs/clearfog.h| 1 -
 include/configs/conga-qeval20-qa3-e3845.h | 2 --
 include/configs/controlcenterd.h  | 1 -
 include/configs/corenet_ds.h  | 1 -
 include/configs/cougarcanyon2.h   | 2 --
 include/configs/crownbay.h| 2 --
 include/configs/cyrus.h   | 1 -
 include/configs/db-88f6820-amc.h  | 1 -
 include/configs/db-88f6820-gp.h   | 1 -
 include/configs/db-mv784mp-gp.h   | 1 -
 include/configs/dfi-bt700.h   | 2 --
 include/configs/digsy_mtc.h   | 1 -
 include/configs/galileo.h | 2 --
 include/configs/gdppc440etx.h | 1 -
 include/configs/ge_bx50v3.h   | 1 -
 include/configs/gw_ventana.h  | 1 -
 include/configs/hrcon.h   | 2 --
 include/configs/icon.h| 1 -
 include/configs/inka4x0.h | 1 -
 include/configs/integratorap.h| 1 -
 include/configs/intip.h   | 1 -
 include/configs/ipek01.h  | 1 -
 include/configs/jetson-tk1.h  | 1 -
 include/configs/jupiter.h | 1 -
 include/configs/katmai.h  | 1 -
 include/configs/kilauea.h | 1 -
 include/configs/km/kmp204x-common.h   | 1 -
 include/configs/ls1012aqds.h  | 1 -
 include/configs/ls1012ardb.h  | 1 -
 include/configs/ls1021aqds.h  | 1 -
 include/configs/ls1021atwr.h  | 1 -
 include/configs/ls1043a_common.h  | 1 -
 include/configs/ls2080aqds.h  | 1 -
 include/configs/ls2080ardb.h  | 1 -
 include/configs/luan.h| 1 -
 include/configs/makalu.h