* [char-misc:char-misc-testing] warning
@ 2016-11-01 14:07 Robin van der Gracht
2016-11-01 15:33 ` Tobias Jakobi
0 siblings, 1 reply; 5+ messages in thread
From: Robin van der Gracht @ 2016-11-01 14:07 UTC (permalink / raw)
To: Tobias Jakobi; +Cc: Sean Paul, Greg Kroah-Hartman, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 1267 bytes --]
Hi Tobias,
Lately I've submitted patches to Gregs (char-misc) git repo and they got
bounced because the kbuild bot threw a warning:
warning: (PMAC_BACKLIGHT && DRM_NOUVEAU && HT16K33 && FB_TFT) selects
FB_BACKLIGHT which has unmet direct dependencies (HAS_IOMEM && FB)
My patched added a new driver named HT16K33, but..
I believe the warning is the result of a dependency issue made possible by this commit:
https://kernel.googlesource.com/pub/scm/linux/kernel/git/gregkh/char-misc/+/fc497ed8361f34e465f31e9babcb88efe38fe433
The kbuild bots configuration file stated:
CONFIG_HAS_IOMEM=y
CONFIG_DRM_NOUVEAU=y
CONFIG_DRM_NOUVEAU_BACKLIGHT=y
CONFIG_FB_BACKLIGHT=y
# CONFIG_FB is not set
PMAC_BACKLIGHT, HT16K33 and FB_TFT ware not mentioned in the config file.
I've attached the config file to this email.
FB_BACKLIGHT depends on FB like so:
(drivers/video/fbdev/Kconfig)
config FB_BACKLIGHT
bool
depends on FB
...
In your commit the 'select FB' is removed from the config DRM_NOUVEAU in
the drivers/gpu/drm/nouveau/Kconfig file but DRM_NOUVEAU still selects
FB_BACKLIGHT when using DRM_NOUVEAU_BACKLIGHT.
Is no one else selects FB, the warning mentioned above is raised.
Would you like to verify this?
Regards,
--
Robin van der Gracht
Protonic Holland
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 26724 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [char-misc:char-misc-testing] warning
2016-11-01 14:07 [char-misc:char-misc-testing] warning Robin van der Gracht
@ 2016-11-01 15:33 ` Tobias Jakobi
2016-11-01 16:02 ` Robin van der Gracht
0 siblings, 1 reply; 5+ messages in thread
From: Tobias Jakobi @ 2016-11-01 15:33 UTC (permalink / raw)
To: Robin van der Gracht; +Cc: Sean Paul, Greg Kroah-Hartman, linux-kernel
Hello Robin,
I'm afraid I can't help you with that. The series was done as a request
by Daniel Vetter, see here for reference:
http://www.spinics.net/lists/dri-devel/msg113011.html
I don't have any nouveau platform here.
With best wishes,
Tobias
Robin van der Gracht wrote:
> Hi Tobias,
>
> Lately I've submitted patches to Gregs (char-misc) git repo and they got
> bounced because the kbuild bot threw a warning:
>
> warning: (PMAC_BACKLIGHT && DRM_NOUVEAU && HT16K33 && FB_TFT) selects
> FB_BACKLIGHT which has unmet direct dependencies (HAS_IOMEM && FB)
>
> My patched added a new driver named HT16K33, but..
>
> I believe the warning is the result of a dependency issue made possible by this commit:
> https://kernel.googlesource.com/pub/scm/linux/kernel/git/gregkh/char-misc/+/fc497ed8361f34e465f31e9babcb88efe38fe433
>
> The kbuild bots configuration file stated:
> CONFIG_HAS_IOMEM=y
> CONFIG_DRM_NOUVEAU=y
> CONFIG_DRM_NOUVEAU_BACKLIGHT=y
> CONFIG_FB_BACKLIGHT=y
> # CONFIG_FB is not set
>
> PMAC_BACKLIGHT, HT16K33 and FB_TFT ware not mentioned in the config file.
> I've attached the config file to this email.
>
> FB_BACKLIGHT depends on FB like so:
>
> (drivers/video/fbdev/Kconfig)
> config FB_BACKLIGHT
> bool
> depends on FB
> ...
>
> In your commit the 'select FB' is removed from the config DRM_NOUVEAU in
> the drivers/gpu/drm/nouveau/Kconfig file but DRM_NOUVEAU still selects
> FB_BACKLIGHT when using DRM_NOUVEAU_BACKLIGHT.
>
> Is no one else selects FB, the warning mentioned above is raised.
>
> Would you like to verify this?
>
> Regards,
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [char-misc:char-misc-testing] warning
2016-11-01 15:33 ` Tobias Jakobi
@ 2016-11-01 16:02 ` Robin van der Gracht
2016-11-01 16:34 ` Randy Dunlap
0 siblings, 1 reply; 5+ messages in thread
From: Robin van der Gracht @ 2016-11-01 16:02 UTC (permalink / raw)
To: Tobias Jakobi; +Cc: Sean Paul, Greg Kroah-Hartman, linux-kernel
Hi Tobias,
On Tue, 1 Nov 2016 16:33:03 +0100
Tobias Jakobi <tjakobi@math.uni-bielefeld.de> wrote:
> Hello Robin,
>
> I'm afraid I can't help you with that. The series was done as a request
> by Daniel Vetter, see here for reference:
> http://www.spinics.net/lists/dri-devel/msg113011.html
So.. I should contact Daniel Vetter about this..?
> I don't have any nouveau platform here.
Me neither, the warning is shown when invoking Kconfig. So when I use
the config file supplied on my previous email, and start a build or a
menuconfig on my (x86_64) system its showing.
Regards,
--
Robin van der Gracht
Protonic Holland
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [char-misc:char-misc-testing] warning
2016-11-01 16:02 ` Robin van der Gracht
@ 2016-11-01 16:34 ` Randy Dunlap
2016-11-02 7:23 ` Robin van der Gracht
0 siblings, 1 reply; 5+ messages in thread
From: Randy Dunlap @ 2016-11-01 16:34 UTC (permalink / raw)
To: Robin van der Gracht, Tobias Jakobi
Cc: Sean Paul, Greg Kroah-Hartman, linux-kernel, Daniel Vetter
On 11/01/16 09:02, Robin van der Gracht wrote:
> Hi Tobias,
>
> On Tue, 1 Nov 2016 16:33:03 +0100
> Tobias Jakobi <tjakobi@math.uni-bielefeld.de> wrote:
>
>> Hello Robin,
>>
>> I'm afraid I can't help you with that. The series was done as a request
>> by Daniel Vetter, see here for reference:
>> http://www.spinics.net/lists/dri-devel/msg113011.html
> So.. I should contact Daniel Vetter about this..?
>
>> I don't have any nouveau platform here.
>
> Me neither, the warning is shown when invoking Kconfig. So when I use
> the config file supplied on my previous email, and start a build or a
> menuconfig on my (x86_64) system its showing.
The kconfig warning is in mainline now and it has nothing to do with the
new HT16K33 driver.
How is nouveau supposed to provide backlight support?
It selects FB_BACKLIGHT but FB_BACKLIGHT depends on FB, which is not
enabled in the supplied .config file.
--
~Randy
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [char-misc:char-misc-testing] warning
2016-11-01 16:34 ` Randy Dunlap
@ 2016-11-02 7:23 ` Robin van der Gracht
0 siblings, 0 replies; 5+ messages in thread
From: Robin van der Gracht @ 2016-11-02 7:23 UTC (permalink / raw)
To: Randy Dunlap
Cc: Tobias Jakobi, Sean Paul, Greg Kroah-Hartman, linux-kernel,
Daniel Vetter
Hi Randy,
On Tue, 1 Nov 2016 09:34:04 -0700
Randy Dunlap <rdunlap@infradead.org> wrote:
> On 11/01/16 09:02, Robin van der Gracht wrote:
> > Hi Tobias,
> >
> > On Tue, 1 Nov 2016 16:33:03 +0100
> > Tobias Jakobi <tjakobi@math.uni-bielefeld.de> wrote:
> >
> >> Hello Robin,
> >>
> >> I'm afraid I can't help you with that. The series was done as a request
> >> by Daniel Vetter, see here for reference:
> >> http://www.spinics.net/lists/dri-devel/msg113011.html
> > So.. I should contact Daniel Vetter about this..?
> >
> >> I don't have any nouveau platform here.
> >
> > Me neither, the warning is shown when invoking Kconfig. So when I use
> > the config file supplied on my previous email, and start a build or a
> > menuconfig on my (x86_64) system its showing.
>
>
> The kconfig warning is in mainline now and it has nothing to do with the
> new HT16K33 driver.
>
> How is nouveau supposed to provide backlight support?
> It selects FB_BACKLIGHT but FB_BACKLIGHT depends on FB, which is not
> enabled in the supplied .config file.
Yes, exactly!
I think that line should have been removed along with the other FB deps.
Regards,
--
Robin van der Gracht
Protonic Holland
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-11-02 7:23 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-01 14:07 [char-misc:char-misc-testing] warning Robin van der Gracht
2016-11-01 15:33 ` Tobias Jakobi
2016-11-01 16:02 ` Robin van der Gracht
2016-11-01 16:34 ` Randy Dunlap
2016-11-02 7:23 ` Robin van der Gracht
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.