Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1487835 > unrolled thread

staging/greybus: undefined Kconfig symbols

Started byValentin Rothberg <valentinrothberg@gmail.com>
First post2016-09-21 08:40 +0200
Last post2016-09-21 12:40 +0200
Articles 4 — 2 participants

Back to article view | Back to linux.kernel

Contents

  staging/greybus: undefined Kconfig symbols Valentin Rothberg <valentinrothberg@gmail.com> - 2016-09-21 08:40 +0200
    Re: staging/greybus: undefined Kconfig symbols Greg KH <gregkh@linuxfoundation.org> - 2016-09-21 09:50 +0200
      Re: staging/greybus: undefined Kconfig symbols Valentin Rothberg <valentinrothberg@gmail.com> - 2016-09-21 12:30 +0200
        Re: staging/greybus: undefined Kconfig symbols Greg KH <gregkh@linuxfoundation.org> - 2016-09-21 12:40 +0200

#1487835 — staging/greybus: undefined Kconfig symbols

FromValentin Rothberg <valentinrothberg@gmail.com>
Date2016-09-21 08:40 +0200
Subjectstaging/greybus: undefined Kconfig symbols
Message-ID<sjJmh-5vr-1@gated-at.bofh.it>
Hi Greg,

checkkconfigsymbols.py found some undefined Kconfig symbols (see
below) in the Makefile and Kconfig file.  I could not find any patch
adding those symbols.  Are there patches queued somewhere else?

Best regards,
 Valentin

GREYBUS_AUDIO_MSM8994
Referencing files: drivers/staging/greybus/Makefile
Similar symbols: GREYBUS_AUDIO, GREYBUS_GPIO, GREYBUS_SDIO, GREYBUS_USB
Commits changing symbol:
- d4f56b47a8fa ("staging: greybus: Add drivers/staging/greybus to the build")

MEDIA
Referencing files: drivers/staging/greybus/Kconfig
Similar symbols: COMEDI, EDAC, EISA, HID_CMEDIA, LANMEDIA, MDIO,
MEDIA_CEC, MEDIA_TUNER, MEMDMA0, MEMDMA1
Commits changing symbol:
- d4f56b47a8fa ("staging: greybus: Add drivers/staging/greybus to the build")
- 847175e8e660 ("greybus: audio: Fetch jack_mask, button_mask from
module's topology data")
- 64a7e2cceb75 ("greybus: audio: Added jack support to audio module")

USB_HSIC_USB3613
Referencing files: drivers/staging/greybus/Makefile
Similar symbols: USB_FSL_USB2, USB_GSPCA_SPCA561, USB_GSPCA_STK1135,
USB_GSPCA_T613, USB_HCD_SSB, USB_HSIC_USB3503, USB_HSIC_USB4604,
USB_RENESAS_USB3, USB_SI4713, USB_SISUSBVGA
Commits changing symbol:
- d4f56b47a8fa ("staging: greybus: Add drivers/staging/greybus to the build")
- 7d0493d191ab ("greybus: only build arche platform driver if usb3613
is enabled")

[toc] | [next] | [standalone]

#1487871

FromGreg KH <gregkh@linuxfoundation.org>
Date2016-09-21 09:50 +0200
Message-ID<sjKs2-68g-3@gated-at.bofh.it>
In reply to#1487835
On Wed, Sep 21, 2016 at 08:30:01AM +0200, Valentin Rothberg wrote:
> Hi Greg,

<fixing odd email address you used for me...>

> checkkconfigsymbols.py found some undefined Kconfig symbols (see
> below) in the Makefile and Kconfig file.  I could not find any patch
> adding those symbols.  Are there patches queued somewhere else?
> 
> Best regards,
>  Valentin
> 
> GREYBUS_AUDIO_MSM8994
> Referencing files: drivers/staging/greybus/Makefile
> Similar symbols: GREYBUS_AUDIO, GREYBUS_GPIO, GREYBUS_SDIO, GREYBUS_USB
> Commits changing symbol:
> - d4f56b47a8fa ("staging: greybus: Add drivers/staging/greybus to the build")

This one is tricky, as parts of this audio driver depended on an
out-of-tree qualcomm audio driver.  It will be unwound over time to not
depend on that mess.

> MEDIA
> Referencing files: drivers/staging/greybus/Kconfig
> Similar symbols: COMEDI, EDAC, EISA, HID_CMEDIA, LANMEDIA, MDIO,
> MEDIA_CEC, MEDIA_TUNER, MEMDMA0, MEMDMA1
> Commits changing symbol:
> - d4f56b47a8fa ("staging: greybus: Add drivers/staging/greybus to the build")
> - 847175e8e660 ("greybus: audio: Fetch jack_mask, button_mask from
> module's topology data")
> - 64a7e2cceb75 ("greybus: audio: Added jack support to audio module")

Ugh, this should be CONFIG_MEDIA_SUPPORT, right?  If so, I'll make a
patch to change this, that was my fault.

> USB_HSIC_USB3613
> Referencing files: drivers/staging/greybus/Makefile
> Similar symbols: USB_FSL_USB2, USB_GSPCA_SPCA561, USB_GSPCA_STK1135,
> USB_GSPCA_T613, USB_HCD_SSB, USB_HSIC_USB3503, USB_HSIC_USB4604,
> USB_RENESAS_USB3, USB_SI4713, USB_SISUSBVGA
> Commits changing symbol:
> - d4f56b47a8fa ("staging: greybus: Add drivers/staging/greybus to the build")
> - 7d0493d191ab ("greybus: only build arche platform driver if usb3613
> is enabled")

This is a driver that has yet to be queued up, give us a few weeks to
add it, and then it will probably be merged into the arche-platform
driver as they both depend on each other.

thanks,

greg k-h

[toc] | [prev] | [next] | [standalone]

#1488023

FromValentin Rothberg <valentinrothberg@gmail.com>
Date2016-09-21 12:30 +0200
Message-ID<sjMWS-7Q9-41@gated-at.bofh.it>
In reply to#1487871
On Wed, Sep 21, 2016 at 9:39 AM, Greg KH <gregkh@linuxfoundation.org> wrote:
> On Wed, Sep 21, 2016 at 08:30:01AM +0200, Valentin Rothberg wrote:
>> Hi Greg,
>
> <fixing odd email address you used for me...>

I was a little surprised, too, but that's the one in the signed-off
field in the greybus commits, so I used it.

>> checkkconfigsymbols.py found some undefined Kconfig symbols (see
>> below) in the Makefile and Kconfig file.  I could not find any patch
>> adding those symbols.  Are there patches queued somewhere else?
>>
>> Best regards,
>>  Valentin
>>
>> GREYBUS_AUDIO_MSM8994
>> Referencing files: drivers/staging/greybus/Makefile
>> Similar symbols: GREYBUS_AUDIO, GREYBUS_GPIO, GREYBUS_SDIO, GREYBUS_USB
>> Commits changing symbol:
>> - d4f56b47a8fa ("staging: greybus: Add drivers/staging/greybus to the build")
>
> This one is tricky, as parts of this audio driver depended on an
> out-of-tree qualcomm audio driver.  It will be unwound over time to not
> depend on that mess.
>
>> MEDIA
>> Referencing files: drivers/staging/greybus/Kconfig
>> Similar symbols: COMEDI, EDAC, EISA, HID_CMEDIA, LANMEDIA, MDIO,
>> MEDIA_CEC, MEDIA_TUNER, MEMDMA0, MEMDMA1
>> Commits changing symbol:
>> - d4f56b47a8fa ("staging: greybus: Add drivers/staging/greybus to the build")
>> - 847175e8e660 ("greybus: audio: Fetch jack_mask, button_mask from
>> module's topology data")
>> - 64a7e2cceb75 ("greybus: audio: Added jack support to audio module")
>
> Ugh, this should be CONFIG_MEDIA_SUPPORT, right?  If so, I'll make a
> patch to change this, that was my fault.

Yes, since it's a camera driver, MEDIA_SUPPORT seems to be the good dependency.

>> USB_HSIC_USB3613
>> Referencing files: drivers/staging/greybus/Makefile
>> Similar symbols: USB_FSL_USB2, USB_GSPCA_SPCA561, USB_GSPCA_STK1135,
>> USB_GSPCA_T613, USB_HCD_SSB, USB_HSIC_USB3503, USB_HSIC_USB4604,
>> USB_RENESAS_USB3, USB_SI4713, USB_SISUSBVGA
>> Commits changing symbol:
>> - d4f56b47a8fa ("staging: greybus: Add drivers/staging/greybus to the build")
>> - 7d0493d191ab ("greybus: only build arche platform driver if usb3613
>> is enabled")
>
> This is a driver that has yet to be queued up, give us a few weeks to
> add it, and then it will probably be merged into the arche-platform
> driver as they both depend on each other.

Okay.  I won't report these symbols in the future :)

Thanks,
 Valentin

[toc] | [prev] | [next] | [standalone]

#1488031

FromGreg KH <gregkh@linuxfoundation.org>
Date2016-09-21 12:40 +0200
Message-ID<sjN6x-7Tm-1@gated-at.bofh.it>
In reply to#1488023
On Wed, Sep 21, 2016 at 12:25:54PM +0200, Valentin Rothberg wrote:
> On Wed, Sep 21, 2016 at 9:39 AM, Greg KH <gregkh@linuxfoundation.org> wrote:
> > On Wed, Sep 21, 2016 at 08:30:01AM +0200, Valentin Rothberg wrote:
> >> Hi Greg,
> >
> > <fixing odd email address you used for me...>
> 
> I was a little surprised, too, but that's the one in the signed-off
> field in the greybus commits, so I used it.

Not a problem, but the MAINTAINERS file should have the correct one to
use for this subsystem (it's not just me maintaining it...)

> >> checkkconfigsymbols.py found some undefined Kconfig symbols (see
> >> below) in the Makefile and Kconfig file.  I could not find any patch
> >> adding those symbols.  Are there patches queued somewhere else?
> >>
> >> Best regards,
> >>  Valentin
> >>
> >> GREYBUS_AUDIO_MSM8994
> >> Referencing files: drivers/staging/greybus/Makefile
> >> Similar symbols: GREYBUS_AUDIO, GREYBUS_GPIO, GREYBUS_SDIO, GREYBUS_USB
> >> Commits changing symbol:
> >> - d4f56b47a8fa ("staging: greybus: Add drivers/staging/greybus to the build")
> >
> > This one is tricky, as parts of this audio driver depended on an
> > out-of-tree qualcomm audio driver.  It will be unwound over time to not
> > depend on that mess.
> >
> >> MEDIA
> >> Referencing files: drivers/staging/greybus/Kconfig
> >> Similar symbols: COMEDI, EDAC, EISA, HID_CMEDIA, LANMEDIA, MDIO,
> >> MEDIA_CEC, MEDIA_TUNER, MEMDMA0, MEMDMA1
> >> Commits changing symbol:
> >> - d4f56b47a8fa ("staging: greybus: Add drivers/staging/greybus to the build")
> >> - 847175e8e660 ("greybus: audio: Fetch jack_mask, button_mask from
> >> module's topology data")
> >> - 64a7e2cceb75 ("greybus: audio: Added jack support to audio module")
> >
> > Ugh, this should be CONFIG_MEDIA_SUPPORT, right?  If so, I'll make a
> > patch to change this, that was my fault.
> 
> Yes, since it's a camera driver, MEDIA_SUPPORT seems to be the good dependency.

Great, I'll make up a patch for this, thanks.

greg k-h

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web