Re: linux-next: Tree for Jun 25 [ kbuild or rapidio? ]

[Sedat Dilek]

On Tue, Jun 25, 2013 at 10:55 AM, Sedat Dilek <sedat.dilek@xxxxxxxxx> wrote:
> On Tue, Jun 25, 2013 at 10:44 AM, Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> wrote:
>> Hi all,
>>
>> Changes since 20130624:
>>
>> The vfs tree lost its build failure.
>>
>> The net-next tree gained a conflict against the net tree.
>>
>> The usb tree gained a conflict against the devicetree tree.
>>
>> The staging tree lost its build failure.
>>
>> The msm tree gained a conflict against the arm-soc tree.
>>
>> The samsung tree gained a conflict against the tip tree.
>>
>> The akpm tree lost a patch that turned up elsewhere and gained a conflict
>> against the samsung tree.
>>
>> ----------------------------------------------------------------------------
>>
>
> [ CC linux-kbuild folks ]
>
> Not sure what's going on, but I get this with today's Linux-Next.
> ( Yesterday's next-20130624 was OK. )
>
> $ LANG=C LC_ALL=C  yes "" | make oldconfig && LANG=C LC_ALL=C make
> silentoldconfig </dev/null
> ...
> #
> # configuration written to .config
> #
> scripts/kconfig/conf --silentoldconfig Kconfig
> *
> * Restart config...
> *
> *
> * Bus options (PCI etc.)
> *
> PCI support (PCI) [Y/n/?] y
>   Support mmconfig PCI config space access (PCI_MMCONFIG) [Y/n] y
>   Read CNB20LE Host Bridge Windows (PCI_CNB20LE_QUIRK) [N/y/?] n
>   PCI Express support (PCIEPORTBUS) [Y/n/?] y
>     PCI Express Hotplug driver (HOTPLUG_PCI_PCIE) [Y/n/m/?] y
>     Root Port Advanced Error Reporting support (PCIEAER) [Y/n/?] y
>       PCI Express ECRC settings control (PCIE_ECRC) [N/y/?] n
>       PCIe AER error injector support (PCIEAER_INJECT) [N/m/y/?] n
>     PCI Express ASPM control (PCIEASPM) [Y/n/?] y
>       Debug PCI Express ASPM (PCIEASPM_DEBUG) [N/y/?] n
>       Default ASPM policy
>       > 1. BIOS default (PCIEASPM_DEFAULT)
>         2. Powersave (PCIEASPM_POWERSAVE)
>         3. Performance (PCIEASPM_PERFORMANCE)
>       choice[1-3]: 1
> Message Signaled Interrupts (MSI and MSI-X) (PCI_MSI) [Y/?] y
> PCI Debugging (PCI_DEBUG) [N/y/?] n
> Enable PCI resource re-allocation detection (PCI_REALLOC_ENABLE_AUTO) [Y/n/?] y
> PCI Stub driver (PCI_STUB) [M/n/y/?] m
> Interrupts on hypertransport devices (HT_IRQ) [Y/n/?] y
> PCI IOV support (PCI_IOV) [Y/n/?] y
> PCI PRI support (PCI_PRI) [Y/?] y
> PCI PASID support (PCI_PASID) [Y/?] y
> PCI IO-APIC hotplug support (PCI_IOAPIC) [Y/n/m] y
> ISA-style DMA support (ISA_DMA_API) [Y/n/?] y
> RapidIO support (RAPIDIO) [Y/n/?] y
>   IDT Tsi721 PCI Express SRIO Controller support (RAPIDIO_TSI721) [Y/n/?] y
>   Discovery timeout duration (seconds) (RAPIDIO_DISC_TIMEOUT) [30] 30
>   Enable RapidIO Input/Output Ports (RAPIDIO_ENABLE_RX_TX_PORTS) [N/y/?] n
>   DMA Engine support for RapidIO (RAPIDIO_DMA_ENGINE) [Y/n/?] y
>   RapidIO subsystem debug messages (RAPIDIO_DEBUG) [N/y/?] n
>   Enumeration method [M/y/?] (NEW) aborted!
>
> Console input/output is redirected. Run 'make oldconfig' to update
> configuration.
>
> make[1]: *** [silentoldconfig] Error 1
> make: *** [silentoldconfig] Error 2
>

[ CC rapidio maintainers ]

Not sure if the Kconfig-logic is wrong for RAPIDIO... or
choice/endchoice has now problems with tristate...
Switching from tristate to bool makes the error go away.

diff --git a/drivers/rapidio/Kconfig b/drivers/rapidio/Kconfig
index 5ab0564..c43b413 100644
--- a/drivers/rapidio/Kconfig
+++ b/drivers/rapidio/Kconfig
@@ -60,7 +60,7 @@ choice
          If unsure, select Basic built-in.

 config RAPIDIO_ENUM_BASIC
-       tristate "Basic"
+       bool "Basic"
        help
          This option includes basic RapidIO fabric enumeration and discovery
          mechanism similar to one described in RapidIO specification Annex 1.

Please, have a look at "rapidio: make enumeration/discovery configurable" [1].

[ drivers/rapidio/Kconfig ]
...
+choice
+ prompt "Enumeration method"
+ depends on RAPIDIO
+ default RAPIDIO_ENUM_BASIC
+ help
+ There are different enumeration and discovery mechanisms offered
+ for RapidIO subsystem. You may select single built-in method or
+ or any number of methods to be built as modules.
+ Selecting a built-in method disables use of loadable methods.
+
+ If unsure, select Basic built-in.
+
+config RAPIDIO_ENUM_BASIC
+ tristate "Basic"
+ help
+ This option includes basic RapidIO fabric enumeration and discovery
+ mechanism similar to one described in RapidIO specification Annex 1.
+
+endchoice
+
...

- Sedat -

[1] http://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/?id=a11650e11093ed57dca78bf16e7836517c599098

> - Sedat -
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/