Re: [PATCH] drm/bridge: fix RC_CORE dependency

2019-07-19 Thread Andrzej Hajda
On 19.07.2019 10:33, Arnd Bergmann wrote: > On Fri, Jul 19, 2019 at 9:01 AM Andrzej Hajda wrote: >> On 18.07.2019 15:42, Arnd Bergmann wrote: >>> Using 'imply' causes a new problem, as it allows the case of >>> CONFIG_INPUT=m with RC_CORE=y, which fails to link: >> >> I have reviewed dependencies

Re: [PATCH] drm/bridge: fix RC_CORE dependency

2019-07-19 Thread Arnd Bergmann
On Fri, Jul 19, 2019 at 9:01 AM Andrzej Hajda wrote: > > On 18.07.2019 15:42, Arnd Bergmann wrote: > > Using 'imply' causes a new problem, as it allows the case of > > CONFIG_INPUT=m with RC_CORE=y, which fails to link: > > > I have reviewed dependencies and I wonder how such configuration is > po

Re: [PATCH] drm/bridge: fix RC_CORE dependency

2019-07-19 Thread Dmitry Torokhov
On Thu, Jul 18, 2019 at 5:55 PM Andrzej Hajda wrote: > > On 18.07.2019 16:21, Arnd Bergmann wrote: > > On Thu, Jul 18, 2019 at 4:16 PM Andrzej Hajda wrote: > >> Hi Arnd, > >> > >> On 18.07.2019 15:42, Arnd Bergmann wrote: > >>> Using 'imply' causes a new problem, as it allows the case of > >>> CO

Re: [PATCH] drm/bridge: fix RC_CORE dependency

2019-07-19 Thread Andrzej Hajda
On 18.07.2019 15:42, Arnd Bergmann wrote: > Using 'imply' causes a new problem, as it allows the case of > CONFIG_INPUT=m with RC_CORE=y, which fails to link: I have reviewed dependencies and I wonder how such configuration is possible at all. RC_CORE depends on INPUT (at least on today's next b

Re: [PATCH] drm/bridge: fix RC_CORE dependency

2019-07-18 Thread Arnd Bergmann
On Thu, Jul 18, 2019 at 5:17 PM Dmitry Torokhov wrote: > On Thu, Jul 18, 2019 at 6:13 PM Arnd Bergmann wrote: > > On Thu, Jul 18, 2019 at 4:56 PM Andrzej Hajda wrote: > > > On 18.07.2019 16:21, Arnd Bergmann wrote: > > > > On Thu, Jul 18, 2019 at 4:16 PM Andrzej Hajda > > > > wrote: > > > >> P

Re: [PATCH] drm/bridge: fix RC_CORE dependency

2019-07-18 Thread Dmitry Torokhov
On Thu, Jul 18, 2019 at 6:13 PM Arnd Bergmann wrote: > > On Thu, Jul 18, 2019 at 4:56 PM Andrzej Hajda wrote: > > On 18.07.2019 16:21, Arnd Bergmann wrote: > > > On Thu, Jul 18, 2019 at 4:16 PM Andrzej Hajda wrote: > > >> Hi Arnd, > > >> > > >> On 18.07.2019 15:42, Arnd Bergmann wrote: > > >>> U

Re: [PATCH] drm/bridge: fix RC_CORE dependency

2019-07-18 Thread Arnd Bergmann
On Thu, Jul 18, 2019 at 4:56 PM Andrzej Hajda wrote: > On 18.07.2019 16:21, Arnd Bergmann wrote: > > On Thu, Jul 18, 2019 at 4:16 PM Andrzej Hajda wrote: > >> Hi Arnd, > >> > >> On 18.07.2019 15:42, Arnd Bergmann wrote: > >>> Using 'imply' causes a new problem, as it allows the case of > >>> CONF

Re: [PATCH] drm/bridge: fix RC_CORE dependency

2019-07-18 Thread Andrzej Hajda
On 18.07.2019 16:21, Arnd Bergmann wrote: > On Thu, Jul 18, 2019 at 4:16 PM Andrzej Hajda wrote: >> Hi Arnd, >> >> On 18.07.2019 15:42, Arnd Bergmann wrote: >>> Using 'imply' causes a new problem, as it allows the case of >>> CONFIG_INPUT=m with RC_CORE=y, which fails to link: >>> >>> drivers/medi

Re: [PATCH] drm/bridge: fix RC_CORE dependency

2019-07-18 Thread Arnd Bergmann
On Thu, Jul 18, 2019 at 4:16 PM Andrzej Hajda wrote: > > Hi Arnd, > > On 18.07.2019 15:42, Arnd Bergmann wrote: > > Using 'imply' causes a new problem, as it allows the case of > > CONFIG_INPUT=m with RC_CORE=y, which fails to link: > > > > drivers/media/rc/rc-main.o: In function `ir_do_keyup': >

Re: [PATCH] drm/bridge: fix RC_CORE dependency

2019-07-18 Thread Andrzej Hajda
Hi Arnd, On 18.07.2019 15:42, Arnd Bergmann wrote: > Using 'imply' causes a new problem, as it allows the case of > CONFIG_INPUT=m with RC_CORE=y, which fails to link: > > drivers/media/rc/rc-main.o: In function `ir_do_keyup': > rc-main.c:(.text+0x2b4): undefined reference to `input_event' > drive

[PATCH] drm/bridge: fix RC_CORE dependency

2019-07-18 Thread Arnd Bergmann
Using 'imply' causes a new problem, as it allows the case of CONFIG_INPUT=m with RC_CORE=y, which fails to link: drivers/media/rc/rc-main.o: In function `ir_do_keyup': rc-main.c:(.text+0x2b4): undefined reference to `input_event' drivers/media/rc/rc-main.o: In function `rc_repeat': rc-main.c:(.tex