Hey Lee,
On Thu, Mar 14, 2024 at 10:20:35AM +0000, Lee Jones wrote:
> On Wed, 13 Mar 2024, Uwe Kleine-König wrote:
> > On Wed, Mar 13, 2024 at 01:48:27PM +0100, Uwe Kleine-König wrote:
> > > struct lp8788_bl {
> > > struct lp8788 *lp;
> > > struct backlight_device *bl_dev;
> > > - struct lp8788_backlight_platform_data *pdata;
> > > - enum lp8788_bl_ctrl_mode mode;
> > > struct pwm_device *pwm;
> >
> > Actually this pwm_device member should be dropped, too. I wonder why
> > this even passes a W=1 build without a warning ...
> >
> > @Lee: Feel free to fix this up while applying, or tell me if you prefer
> > an incremental fixup or a complete v2.
>
> Either of the last 2 options would be fine.I'd prefert a v2 then. Added to my today's todo list. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ |
signature.asc
Description: PGP signature
