0) Building an rpm for (vanilla) v3.14, using the origin/f20 branch of kernel.git, triggers a warning (a couple of times): .config:5466:warning: override: USB_G_DBGP_SERIAL changes choice state
1) It turns out config-armv7-generic contains both CONFIG_USB_G_DBGP_PRINTK=y
and CONFIG_USB_G_DBGP_SERIAL=y
2) But in drivers/usb/gadget/Kconfig we see (summarized): choice prompt "EHCI Debug Device mode" default USB_G_DBGP_SERIAL
config USB_G_DBGP_PRINTK bool "printk"
config USB_G_DBGP_SERIAL bool "serial"
endchoice
If I'm reading this correctly, this means that we need to choose between USB_G_DBGP_PRINTK and USB_G_DBGP_PRINTK.
3) Should we remove one of these two Kconfig macros? Or is the upstream Kconfig file incorrect and can both symbols be set simultaneously?
Paul Bolle
On Mon, Mar 31, 2014 at 05:59:41PM +0200, Paul Bolle wrote:
- Building an rpm for (vanilla) v3.14, using the origin/f20 branch of
kernel.git, triggers a warning (a couple of times): .config:5466:warning: override: USB_G_DBGP_SERIAL changes choice state
- It turns out config-armv7-generic contains both CONFIG_USB_G_DBGP_PRINTK=y
and CONFIG_USB_G_DBGP_SERIAL=y
Adding Peter. He set both of these as such.
But in drivers/usb/gadget/Kconfig we see (summarized): choice prompt "EHCI Debug Device mode" default USB_G_DBGP_SERIAL
config USB_G_DBGP_PRINTK bool "printk"
config USB_G_DBGP_SERIAL bool "serial"
endchoice
If I'm reading this correctly, this means that we need to choose between USB_G_DBGP_PRINTK and USB_G_DBGP_PRINTK.
Correct.
- Should we remove one of these two Kconfig macros? Or is the upstream
Kconfig file incorrect and can both symbols be set simultaneously?
No, looking at dbgp.c it definitely seems to be one or the other.
Peter, do you want that going to printk or serial?
josh
Josh,
On Mon, 2014-03-31 at 13:21 -0400, Josh Boyer wrote:
On Mon, Mar 31, 2014 at 05:59:41PM +0200, Paul Bolle wrote:
- Should we remove one of these two Kconfig macros? Or is the upstream
Kconfig file incorrect and can both symbols be set simultaneously?
No, looking at dbgp.c it definitely seems to be one or the other.
Peter, do you want that going to printk or serial?
Perhaps we should just drop USB_G_DBGP_PRINTK and only leave USB_G_DBGP_SERIAL (which is the default choice). Peter can always push a change if the reverse is preferable.
Should I draft the trivial patch to pick USB_G_DBGP_SERIAL?
Paul Bolle
On Fri, Apr 04, 2014 at 09:14:06AM +0200, Paul Bolle wrote:
Josh,
On Mon, 2014-03-31 at 13:21 -0400, Josh Boyer wrote:
On Mon, Mar 31, 2014 at 05:59:41PM +0200, Paul Bolle wrote:
- Should we remove one of these two Kconfig macros? Or is the upstream
Kconfig file incorrect and can both symbols be set simultaneously?
No, looking at dbgp.c it definitely seems to be one or the other.
Peter, do you want that going to printk or serial?
Perhaps we should just drop USB_G_DBGP_PRINTK and only leave USB_G_DBGP_SERIAL (which is the default choice). Peter can always push a change if the reverse is preferable.
Agreed.
Should I draft the trivial patch to pick USB_G_DBGP_SERIAL?
No need. I'll just change it with the next commit I do. Thank you for the offer.
josh
On 4 Apr 2014 20:14, "Josh Boyer" jwboyer@fedoraproject.org wrote:
On Fri, Apr 04, 2014 at 09:14:06AM +0200, Paul Bolle wrote:
Josh,
On Mon, 2014-03-31 at 13:21 -0400, Josh Boyer wrote:
On Mon, Mar 31, 2014 at 05:59:41PM +0200, Paul Bolle wrote:
- Should we remove one of these two Kconfig macros? Or is the
upstream
Kconfig file incorrect and can both symbols be set simultaneously?
No, looking at dbgp.c it definitely seems to be one or the other.
Peter, do you want that going to printk or serial?
Perhaps we should just drop USB_G_DBGP_PRINTK and only leave USB_G_DBGP_SERIAL (which is the default choice). Peter can always push a change if the reverse is preferable.
Agreed.
Fine by me, sorry been AFK, thanks for picking this up.
P
kernel@lists.fedoraproject.org
-
Josh Boyer -
Paul Bolle -
Peter Robinson