[PATCH] backlight: lp8788: Drop support for platform data
Uwe Kleine-König
u.kleine-koenig at pengutronix.de
Thu Mar 14 10:56:25 UTC 2024
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/ |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20240314/d6b34fa2/attachment.sig>
More information about the dri-devel
mailing list