mbox series

[resend,v3,0/3] backlight: mp3309c: Allow to use on non-OF platforms

Message ID 20240208184313.2224579-1-andriy.shevchenko@linux.intel.com
Headers show
Series backlight: mp3309c: Allow to use on non-OF platforms | expand

Message

Andy Shevchenko Feb. 8, 2024, 6:42 p.m. UTC 
Allow to use driver on non-OF platforms and other cleanups.

Changelog v3:
- rebased on top of the last changes against this driver (Lee)
- added tags to patch 2 (Daniel, Flavio)

Changelog v2:
- rename pm3309c_parse_dt_node() --> mp3309c_parse_fwnode() (Daniel)                                             - add tags (Daniel, Flavio)
- new patch 2

Andy Shevchenko (3):
  backlight: mp3309c: Make use of device properties
  backlight: mp3309c: use dev_err_probe() instead of dev_err()
  backlight: mp3309c: Utilise temporary variable for struct device

 drivers/video/backlight/mp3309c.c | 88 ++++++++++++-------------------
 1 file changed, 35 insertions(+), 53 deletions(-)

Comments

Lee Jones Feb. 23, 2024, 1:49 p.m. UTC | #1 
On Thu, 08 Feb 2024, Andy Shevchenko wrote:

> Allow to use driver on non-OF platforms and other cleanups.
> 
> Changelog v3:
> - rebased on top of the last changes against this driver (Lee)
> - added tags to patch 2 (Daniel, Flavio)
> 
> Changelog v2:
> - rename pm3309c_parse_dt_node() --> mp3309c_parse_fwnode() (Daniel)                                             - add tags (Daniel, Flavio)
> - new patch 2
> 
> Andy Shevchenko (3):
>   backlight: mp3309c: Make use of device properties
>   backlight: mp3309c: use dev_err_probe() instead of dev_err()
>   backlight: mp3309c: Utilise temporary variable for struct device
> 
>  drivers/video/backlight/mp3309c.c | 88 ++++++++++++-------------------
>  1 file changed, 35 insertions(+), 53 deletions(-)

Sorry for the delay, I was on vacation and I've been fighting fires this
week.

This set still doesn't apply?
Lee Jones Feb. 23, 2024, 1:51 p.m. UTC | #2 
On Thu, 08 Feb 2024 20:42:25 +0200, Andy Shevchenko wrote:
> Allow to use driver on non-OF platforms and other cleanups.
> 
> Changelog v3:
> - rebased on top of the last changes against this driver (Lee)
> - added tags to patch 2 (Daniel, Flavio)
> 
> Changelog v2:
> - rename pm3309c_parse_dt_node() --> mp3309c_parse_fwnode() (Daniel)                                             - add tags (Daniel, Flavio)
> - new patch 2
> 
> [...]

Applied, thanks!

[1/3] backlight: mp3309c: Make use of device properties
      commit: b6fc7e62a7afc3709b31f6779665c7fc4a7755d7
[2/3] backlight: mp3309c: use dev_err_probe() instead of dev_err()
      commit: 73e209b9ef8743071e8a04aad48db6644f3c8ce0
[3/3] backlight: mp3309c: Utilise temporary variable for struct device
      commit: 2a93258e20bb03016347a2e27e5d21ead3bfd2e4

--
Lee Jones [李琼斯]
Lee Jones Feb. 23, 2024, 1:52 p.m. UTC | #3 
On Fri, 23 Feb 2024, Lee Jones wrote:

> On Thu, 08 Feb 2024, Andy Shevchenko wrote:
> 
> > Allow to use driver on non-OF platforms and other cleanups.
> > 
> > Changelog v3:
> > - rebased on top of the last changes against this driver (Lee)
> > - added tags to patch 2 (Daniel, Flavio)
> > 
> > Changelog v2:
> > - rename pm3309c_parse_dt_node() --> mp3309c_parse_fwnode() (Daniel)                                             - add tags (Daniel, Flavio)
> > - new patch 2
> > 
> > Andy Shevchenko (3):
> >   backlight: mp3309c: Make use of device properties
> >   backlight: mp3309c: use dev_err_probe() instead of dev_err()
> >   backlight: mp3309c: Utilise temporary variable for struct device
> > 
> >  drivers/video/backlight/mp3309c.c | 88 ++++++++++++-------------------
> >  1 file changed, 35 insertions(+), 53 deletions(-)
> 
> Sorry for the delay, I was on vacation and I've been fighting fires this
> week.
> 
> This set still doesn't apply?

Disregard, my fault.  Applied now, thanks.