From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] pci: Move CONFIG_PCI_PNP to Kconfig
Date: Mon, 24 Oct 2016 21:09:11 -0400 [thread overview]
Message-ID: <20161025010911.GR18591@bill-the-cat> (raw)
In-Reply-To: <CAEUhbmW6aeTbxNUB4fgfjBxdO08tx8wd2r3c+F-vzanRe-571Q@mail.gmail.com>
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 <trini@konsulko.com> 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 <bmeng.cn@gmail.com>
> >>
> >> ---
> >>
> >> 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 <trini@konsulko.com>
> >
>
> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161024/05df52eb/attachment.sig>
next prev parent reply other threads:[~2016-10-25 1:09 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-10-17 6:35 [U-Boot] [PATCH] pci: Move CONFIG_PCI_PNP to Kconfig Bin Meng
2016-10-17 11:50 ` Tom Rini
2016-10-25 1:08 ` Bin Meng
2016-10-25 1:09 ` Tom Rini [this message]
2016-10-28 11:13 ` [U-Boot] [PATCH v3] " Tom Rini
2016-10-28 13:07 ` Tom Rini
-- strict thread matches above, loose matches on Subject: below --
2016-10-10 5:04 [U-Boot] [PATCH] " Bin Meng
2016-10-12 12:46 ` Tom Rini
2016-10-14 6:28 ` Bin Meng
2016-10-14 13:07 ` Tom Rini
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=20161025010911.GR18591@bill-the-cat \
--to=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
/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