Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <20200107203231.920256-1-arnd@arndb.de> <87zheyqnla.fsf@intel.com>
 <20200108100831.GA23308@ravnborg.org> <CAK8P3a1FKOV=1No7Q0g1vF_NQmVHK+g0VOqzPL499Pxbbt1aPQ@mail.gmail.com>
 <20200108164618.GA28588@ravnborg.org>
In-Reply-To: <20200108164618.GA28588@ravnborg.org>
From:   Arnd Bergmann <arnd@arndb.de>
Date:   Wed, 8 Jan 2020 21:17:32 +0100
Message-ID: <CAK8P3a1TmfqXA3gK0YwbPBSeydEs=b8Tae=JWvju4OZaSMUscA@mail.gmail.com>
Subject: Re: [PATCH] drm/drm_panel: fix export of drm_panel_of_backlight, try #3
To:     Sam Ravnborg <sam@ravnborg.org>
Cc:     Jani Nikula <jani.nikula@intel.com>,
        Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
        Maxime Ripard <mripard@kernel.org>,
        Thierry Reding <thierry.reding@gmail.com>,
        David Airlie <airlied@linux.ie>,
        Daniel Vetter <daniel@ffwll.ch>,
        Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
        Randy Dunlap <rdunlap@infradead.org>,
        Linus Walleij <linus.walleij@linaro.org>,
        Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
        Sean Paul <sean@poorly.run>,
        dri-devel <dri-devel@lists.freedesktop.org>,
        Maxime Ripard <maxime.ripard@bootlin.com>,
        Emil Velikov <emil.velikov@collabora.com>,
        Boris Brezillon <boris.brezillon@collabora.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Wed, Jan 8, 2020 at 5:46 PM Sam Ravnborg <sam@ravnborg.org> wrote:
>
> Hi Arnd.
>
> > > > All of this is just another hack until the backlight config usage is
> > > > fixed for good. Do we really want to make this the example to copy paste
> > > > wherever we hit the issue next?
> > > >
> > > > I'm not naking, but I'm not acking either.
> > >
> > > I will try to take a look at your old BACKLIGHT_CLASS_DEVICE patch this
> > > weekend. I think we need that one fixed - and then we can have this mess
> > > with "drm_panel_of_backlight" fixed in the right way.
> >
> > I had also attempted to fix the larger mess around 'select' statements in DRM/FB
> > around BACKLIGHT_CLASS_DEVICE  several times in the past, and even at
> > some point sent a patch that was acked but never merged and later broke because
> > of other changes.
>
> Any chance you have the patch around or can dig up a pointer?
> My google foo did not turn up anything.

I found it now:
https://lore.kernel.org/lkml/20170726135312.2214309-1-arnd@arndb.de/

      Arnd