Re: [PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Arnd Bergmann
On Mon, Apr 22, 2024, at 21:42, Masahiro Yamada wrote: > On Tue, Apr 23, 2024 at 3:24 AM Arnd Bergmann wrote: >> On Mon, Apr 22, 2024, at 18:58, Geert Uytterhoeven wrote: >> > On Mon, Apr 22, 2024 at 3:55 PM Arnd Bergmann wrote: >> >> On Mon, Apr 22, 2024, at 15:28, Jani Nikula wrote: >> >> I thi

Re: [PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Masahiro Yamada
On Tue, Apr 23, 2024 at 3:24 AM Arnd Bergmann wrote: > > On Mon, Apr 22, 2024, at 18:58, Geert Uytterhoeven wrote: > > On Mon, Apr 22, 2024 at 3:55 PM Arnd Bergmann wrote: > >> On Mon, Apr 22, 2024, at 15:28, Jani Nikula wrote: > >> Whereas this one is broken: > >> > >> config FEATURE_A > >>

Re: [PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Arnd Bergmann
On Mon, Apr 22, 2024, at 18:58, Geert Uytterhoeven wrote: > On Mon, Apr 22, 2024 at 3:55 PM Arnd Bergmann wrote: >> On Mon, Apr 22, 2024, at 15:28, Jani Nikula wrote: >> Whereas this one is broken: >> >> config FEATURE_A >>tristate "user visible if I2C is enabled" >>depends on I2C

Re: [PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Geert Uytterhoeven
Hi Jani, CC kbuild On Mon, Apr 22, 2024 at 7:00 PM Jani Nikula wrote: > On Mon, 22 Apr 2024, "Arnd Bergmann" wrote: > > I'm not sure where this misunderstanding comes from, as you > > seem to be repeating the same incorrect assumption about > > how select works that Maxime wrote in his changelo

Re: [PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Geert Uytterhoeven
Hi Jani, On Mon, Apr 22, 2024 at 7:15 PM Jani Nikula wrote: > On Mon, 22 Apr 2024, Geert Uytterhoeven wrote: > > On Mon, Apr 22, 2024 at 3:55 PM Arnd Bergmann wrote: > >> I'm not sure where this misunderstanding comes from, as you > >> seem to be repeating the same incorrect assumption about >

Re: [PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Jani Nikula
On Mon, 22 Apr 2024, Geert Uytterhoeven wrote: > Hi Arnd, > > CC kbuild > > On Mon, Apr 22, 2024 at 3:55 PM Arnd Bergmann wrote: >> I'm not sure where this misunderstanding comes from, as you >> seem to be repeating the same incorrect assumption about >> how select works that Maxime wrote in his

Re: [PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Jani Nikula
On Mon, 22 Apr 2024, "Arnd Bergmann" wrote: > On Mon, Apr 22, 2024, at 15:28, Jani Nikula wrote: >> On Mon, 22 Apr 2024, "Arnd Bergmann" wrote: >>> On Mon, Apr 22, 2024, at 13:50, Jani Nikula wrote: >>> I still disagree with this, because fundamentally the source symbol really should no

Re: [PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Geert Uytterhoeven
Hi Arnd, CC kbuild On Mon, Apr 22, 2024 at 3:55 PM Arnd Bergmann wrote: > On Mon, Apr 22, 2024, at 15:28, Jani Nikula wrote: > > On Mon, 22 Apr 2024, "Arnd Bergmann" wrote: > >> 2. Using "select" on user visible symbols that have dependencies > >>is a common source for bugs, and this is is

Re: [PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Arnd Bergmann
On Mon, Apr 22, 2024, at 15:28, Jani Nikula wrote: > On Mon, 22 Apr 2024, "Arnd Bergmann" wrote: >> On Mon, Apr 22, 2024, at 13:50, Jani Nikula wrote: >> >>> I still disagree with this, because fundamentally the source symbol >>> really should not have to care about the dependencies of the target

Re: [PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Jani Nikula
On Mon, 22 Apr 2024, "Arnd Bergmann" wrote: > On Mon, Apr 22, 2024, at 13:50, Jani Nikula wrote: >> On Mon, 22 Apr 2024, Geert Uytterhoeven wrote: >>> Hi all, >>> >>> As discussed on IRC with Maxime and Arnd, this series reverts the >>> conversion of select to depends for various DRM helpers

Re: [PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Arnd Bergmann
On Mon, Apr 22, 2024, at 13:50, Jani Nikula wrote: > On Mon, 22 Apr 2024, Geert Uytterhoeven wrote: >> Hi all, >> >> As discussed on IRC with Maxime and Arnd, this series reverts the >> conversion of select to depends for various DRM helpers in series >> "[PATCH v3 00/13] drm/display: Convert

Re: [PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Dmitry Baryshkov
On Mon, Apr 22, 2024 at 02:50:09PM +0300, Jani Nikula wrote: > On Mon, 22 Apr 2024, Geert Uytterhoeven wrote: > > Hi all, > > > > As discussed on IRC with Maxime and Arnd, this series reverts the > > conversion of select to depends for various DRM helpers in series > > "[PATCH v3 00/13] drm/di

Re: [PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Jani Nikula
On Mon, 22 Apr 2024, Geert Uytterhoeven wrote: > Hi all, > > As discussed on IRC with Maxime and Arnd, this series reverts the > conversion of select to depends for various DRM helpers in series > "[PATCH v3 00/13] drm/display: Convert helpers Kconfig symbols to > depends on"[1], and various

[PATCH 00/11] drm: Restore helper usability

2024-04-22 Thread Geert Uytterhoeven
Hi all, As discussed on IRC with Maxime and Arnd, this series reverts the conversion of select to depends for various DRM helpers in series "[PATCH v3 00/13] drm/display: Convert helpers Kconfig symbols to depends on"[1], and various fixes for it. This conversion introduced a big usabilit