mbox series

[GIT,PULL] fbdev updates & fixes for v5.17-rc1

Message ID Yeg11pHSqx29yg/T@ls3530 (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL] fbdev updates & fixes for v5.17-rc1 | expand

Pull-request

http://git.kernel.org/pub/scm/linux/kernel/git/deller/linux-fbdev.git tags/fbdev-5.17-2

Message

Helge Deller Jan. 19, 2022, 4:01 p.m. UTC
The following changes since commit 0c947b893d69231a9add855939da7c66237ab44f:

  Merge tag '5.17-rc-part1-smb3-fixes' of git://git.samba.org/sfrench/cifs-2.6 (2022-01-17 09:53:21 +0200)

are available in the Git repository at:

  http://git.kernel.org/pub/scm/linux/kernel/git/deller/linux-fbdev.git tags/fbdev-5.17-2

for you to fetch changes up to 842086bc7262a36f002e0361f9dc351556cae3f3:

  video: fbdev: controlfb: Fix COMPILE_TEST build (2022-01-17 22:39:37 +0100)

----------------------------------------------------------------
fbdev updates for v5.17-rc1

A first bunch of updates and fixes for the following fbdev & backlight drivers:
ocfb, aty128fb, mb862xx, omapfb, qcom-wled, dt-bindings, hyperv_fb,
lm3630a_bl, omap2, controlfb, matroxfb

Nothing really important, mostly cleanups, const conversions, added null
pointer/boundary checks and build fixes.

Signed-off-by: Helge Deller <deller@gmx.de>

----------------------------------------------------------------
Chunyang Zhong (1):
      video: ocfb: add const to of_device_id 

Colin Ian King (2):
      fbdev: aty128fb: make some arrays static const
      video: fbdev: mb862xx: remove redundant assignment to pointer ptr

Greg Kroah-Hartman (1):
      omapfb: use default_groups in kobj_type

Jiasheng Jiang (1):
      video: fbdev: Check for null res pointer

Luca Weiss (2):
      backlight: qcom-wled: Add PM6150L compatible
      dt-bindings: simple-framebuffer: allow standalone compatible

Michael Kelley (1):
      video: hyperv_fb: Fix validation of screen resolution

Minghao Chi (1):
      drivers/video: remove redundant res variable

Xu Wang (2):
      backlight: lm3630a_bl: Remove redundant 'flush_workqueue()' calls
      fbdev: omap2: omapfb: Remove redundant 'flush_workqueue()' calls

Yang Guang (1):
      video: fbdev: use swap() to make code cleaner

YueHaibing (1):
      video: fbdev: controlfb: Fix COMPILE_TEST build

Z. Liu (1):
      matroxfb: set maxvram of vbG200eW to the same as vbG200 to avoid black screen

 .../devicetree/bindings/display/simple-framebuffer.yaml  | 12 +++++++-----
 drivers/video/backlight/lm3630a_bl.c                     |  1 -
 drivers/video/backlight/qcom-wled.c                      |  1 +
 drivers/video/fbdev/aty/aty128fb.c                       | 10 ++++++----
 drivers/video/fbdev/aty/mach64_ct.c                      |  4 +---
 drivers/video/fbdev/controlfb.c                          |  2 ++
 drivers/video/fbdev/hyperv_fb.c                          | 16 +++-------------
 drivers/video/fbdev/imxfb.c                              |  2 ++
 drivers/video/fbdev/matrox/matroxfb_base.c               |  2 +-
 drivers/video/fbdev/mb862xx/mb862xxfb_accel.c            |  2 +-
 drivers/video/fbdev/ocfb.c                               |  2 +-
 drivers/video/fbdev/omap2/omapfb/dss/display-sysfs.c     |  3 ++-
 drivers/video/fbdev/omap2/omapfb/dss/manager-sysfs.c     |  3 ++-
 drivers/video/fbdev/omap2/omapfb/dss/overlay-sysfs.c     |  3 ++-
 drivers/video/fbdev/omap2/omapfb/omapfb-main.c           |  1 -
 drivers/video/fbdev/sis/sis_main.c                       |  2 +-
 16 files changed, 32 insertions(+), 34 deletions(-)

Comments

Daniel Vetter Jan. 21, 2022, 8:47 a.m. UTC | #1
On Wed, Jan 19, 2022 at 5:02 PM Helge Deller <deller@gmx.de> wrote:
>
> The following changes since commit 0c947b893d69231a9add855939da7c66237ab44f:
>
>   Merge tag '5.17-rc-part1-smb3-fixes' of git://git.samba.org/sfrench/cifs-2.6 (2022-01-17 09:53:21 +0200)
>
> are available in the Git repository at:
>
>   http://git.kernel.org/pub/scm/linux/kernel/git/deller/linux-fbdev.git tags/fbdev-5.17-2
>
> for you to fetch changes up to 842086bc7262a36f002e0361f9dc351556cae3f3:
>
>   video: fbdev: controlfb: Fix COMPILE_TEST build (2022-01-17 22:39:37 +0100)
>
> ----------------------------------------------------------------
> fbdev updates for v5.17-rc1
>
> A first bunch of updates and fixes for the following fbdev & backlight drivers:
> ocfb, aty128fb, mb862xx, omapfb, qcom-wled, dt-bindings, hyperv_fb,
> lm3630a_bl, omap2, controlfb, matroxfb
>
> Nothing really important, mostly cleanups, const conversions, added null
> pointer/boundary checks and build fixes.
>
> Signed-off-by: Helge Deller <deller@gmx.de>

Not sure whether Linus missed this or just wanted to let the
discussion settle first. But since this is all random patches for
drivers that many distros don't even enable anymore there's no issues
here, and I very much welcome someone volunteering to pick these up.
I'd expect there's a pile more since it's been 1-2 years since Bart
took care of these and merged them consistently.

Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Cheers, Daniel

>
> ----------------------------------------------------------------
> Chunyang Zhong (1):
>       video: ocfb: add const to of_device_id
>
> Colin Ian King (2):
>       fbdev: aty128fb: make some arrays static const
>       video: fbdev: mb862xx: remove redundant assignment to pointer ptr
>
> Greg Kroah-Hartman (1):
>       omapfb: use default_groups in kobj_type
>
> Jiasheng Jiang (1):
>       video: fbdev: Check for null res pointer
>
> Luca Weiss (2):
>       backlight: qcom-wled: Add PM6150L compatible
>       dt-bindings: simple-framebuffer: allow standalone compatible
>
> Michael Kelley (1):
>       video: hyperv_fb: Fix validation of screen resolution
>
> Minghao Chi (1):
>       drivers/video: remove redundant res variable
>
> Xu Wang (2):
>       backlight: lm3630a_bl: Remove redundant 'flush_workqueue()' calls
>       fbdev: omap2: omapfb: Remove redundant 'flush_workqueue()' calls
>
> Yang Guang (1):
>       video: fbdev: use swap() to make code cleaner
>
> YueHaibing (1):
>       video: fbdev: controlfb: Fix COMPILE_TEST build
>
> Z. Liu (1):
>       matroxfb: set maxvram of vbG200eW to the same as vbG200 to avoid black screen
>
>  .../devicetree/bindings/display/simple-framebuffer.yaml  | 12 +++++++-----
>  drivers/video/backlight/lm3630a_bl.c                     |  1 -
>  drivers/video/backlight/qcom-wled.c                      |  1 +
>  drivers/video/fbdev/aty/aty128fb.c                       | 10 ++++++----
>  drivers/video/fbdev/aty/mach64_ct.c                      |  4 +---
>  drivers/video/fbdev/controlfb.c                          |  2 ++
>  drivers/video/fbdev/hyperv_fb.c                          | 16 +++-------------
>  drivers/video/fbdev/imxfb.c                              |  2 ++
>  drivers/video/fbdev/matrox/matroxfb_base.c               |  2 +-
>  drivers/video/fbdev/mb862xx/mb862xxfb_accel.c            |  2 +-
>  drivers/video/fbdev/ocfb.c                               |  2 +-
>  drivers/video/fbdev/omap2/omapfb/dss/display-sysfs.c     |  3 ++-
>  drivers/video/fbdev/omap2/omapfb/dss/manager-sysfs.c     |  3 ++-
>  drivers/video/fbdev/omap2/omapfb/dss/overlay-sysfs.c     |  3 ++-
>  drivers/video/fbdev/omap2/omapfb/omapfb-main.c           |  1 -
>  drivers/video/fbdev/sis/sis_main.c                       |  2 +-
>  16 files changed, 32 insertions(+), 34 deletions(-)
Jani Nikula Jan. 21, 2022, 9:09 a.m. UTC | #2
On Fri, 21 Jan 2022, Daniel Vetter <daniel@ffwll.ch> wrote:
> On Wed, Jan 19, 2022 at 5:02 PM Helge Deller <deller@gmx.de> wrote:
>> A first bunch of updates and fixes for the following fbdev & backlight drivers:
>> ocfb, aty128fb, mb862xx, omapfb, qcom-wled, dt-bindings, hyperv_fb,
>> lm3630a_bl, omap2, controlfb, matroxfb
>>
>> Nothing really important, mostly cleanups, const conversions, added null
>> pointer/boundary checks and build fixes.
>>
>> Signed-off-by: Helge Deller <deller@gmx.de>
>
> Not sure whether Linus missed this or just wanted to let the
> discussion settle first. But since this is all random patches for
> drivers that many distros don't even enable anymore there's no issues
> here, and I very much welcome someone volunteering to pick these up.
> I'd expect there's a pile more since it's been 1-2 years since Bart
> took care of these and merged them consistently.
>
> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>

...

>>  drivers/video/backlight/lm3630a_bl.c                     |  1 -
>>  drivers/video/backlight/qcom-wled.c                      |  1 +

Backlight changes usually go through the backlight tree.


BR,
Jani.
Lee Jones Jan. 21, 2022, 9:19 a.m. UTC | #3
On Fri, 21 Jan 2022, Jani Nikula wrote:

> On Fri, 21 Jan 2022, Daniel Vetter <daniel@ffwll.ch> wrote:
> > On Wed, Jan 19, 2022 at 5:02 PM Helge Deller <deller@gmx.de> wrote:
> >> A first bunch of updates and fixes for the following fbdev & backlight drivers:
> >> ocfb, aty128fb, mb862xx, omapfb, qcom-wled, dt-bindings, hyperv_fb,
> >> lm3630a_bl, omap2, controlfb, matroxfb
> >>
> >> Nothing really important, mostly cleanups, const conversions, added null
> >> pointer/boundary checks and build fixes.
> >>
> >> Signed-off-by: Helge Deller <deller@gmx.de>
> >
> > Not sure whether Linus missed this or just wanted to let the
> > discussion settle first. But since this is all random patches for
> > drivers that many distros don't even enable anymore there's no issues
> > here, and I very much welcome someone volunteering to pick these up.
> > I'd expect there's a pile more since it's been 1-2 years since Bart
> > took care of these and merged them consistently.
> >
> > Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> 
> ...
> 
> >>  drivers/video/backlight/lm3630a_bl.c                     |  1 -
> >>  drivers/video/backlight/qcom-wled.c                      |  1 +
> 
> Backlight changes usually go through the backlight tree.

Yes, they do.  How were these applied to the DRM tree?

I don't see any mails about them being applied:

  Luca Weiss (2):
        backlight: qcom-wled: Add PM6150L compatible

  https://lore.kernel.org/all/20211229170358.2457006-2-luca.weiss@fairphone.com/

  Xu Wang (2):
        backlight: lm3630a_bl: Remove redundant 'flush_workqueue()' calls

  https://lore.kernel.org/all/20220113084806.13822-1-vulab@iscas.ac.cn/
Helge Deller Jan. 21, 2022, 9:53 a.m. UTC | #4
On 1/21/22 10:19, Lee Jones wrote:
> On Fri, 21 Jan 2022, Jani Nikula wrote:
>
>> On Fri, 21 Jan 2022, Daniel Vetter <daniel@ffwll.ch> wrote:
>>> On Wed, Jan 19, 2022 at 5:02 PM Helge Deller <deller@gmx.de> wrote:
>>>> A first bunch of updates and fixes for the following fbdev & backlight drivers:
>>>> ocfb, aty128fb, mb862xx, omapfb, qcom-wled, dt-bindings, hyperv_fb,
>>>> lm3630a_bl, omap2, controlfb, matroxfb
>>>>
>>>> Nothing really important, mostly cleanups, const conversions, added null
>>>> pointer/boundary checks and build fixes.
>>>>
>>>> Signed-off-by: Helge Deller <deller@gmx.de>
>>>
>>> Not sure whether Linus missed this or just wanted to let the
>>> discussion settle first. But since this is all random patches for
>>> drivers that many distros don't even enable anymore there's no issues
>>> here, and I very much welcome someone volunteering to pick these up.
>>> I'd expect there's a pile more since it's been 1-2 years since Bart
>>> took care of these and merged them consistently.
>>>
>>> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
>>
>> ...
>>
>>>>  drivers/video/backlight/lm3630a_bl.c                     |  1 -
>>>>  drivers/video/backlight/qcom-wled.c                      |  1 +
>>
>> Backlight changes usually go through the backlight tree.

Sorry, I didn't know yet.

> Yes, they do.  How were these applied to the DRM tree?

I did applied them.
Shall I drop them and you decide about them?
Next I round I'll take care not to pick backlight patches.

Helge


> I don't see any mails about them being applied:
>
>   Luca Weiss (2):
>         backlight: qcom-wled: Add PM6150L compatible
>
>   https://lore.kernel.org/all/20211229170358.2457006-2-luca.weiss@fairphone.com/
>
>   Xu Wang (2):
>         backlight: lm3630a_bl: Remove redundant 'flush_workqueue()' calls
>
>   https://lore.kernel.org/all/20220113084806.13822-1-vulab@iscas.ac.cn/
>
Daniel Vetter Jan. 21, 2022, 9:53 a.m. UTC | #5
On Fri, Jan 21, 2022 at 10:19 AM Lee Jones <lee.jones@linaro.org> wrote:
>
> On Fri, 21 Jan 2022, Jani Nikula wrote:
>
> > On Fri, 21 Jan 2022, Daniel Vetter <daniel@ffwll.ch> wrote:
> > > On Wed, Jan 19, 2022 at 5:02 PM Helge Deller <deller@gmx.de> wrote:
> > >> A first bunch of updates and fixes for the following fbdev & backlight drivers:
> > >> ocfb, aty128fb, mb862xx, omapfb, qcom-wled, dt-bindings, hyperv_fb,
> > >> lm3630a_bl, omap2, controlfb, matroxfb
> > >>
> > >> Nothing really important, mostly cleanups, const conversions, added null
> > >> pointer/boundary checks and build fixes.
> > >>
> > >> Signed-off-by: Helge Deller <deller@gmx.de>
> > >
> > > Not sure whether Linus missed this or just wanted to let the
> > > discussion settle first. But since this is all random patches for
> > > drivers that many distros don't even enable anymore there's no issues
> > > here, and I very much welcome someone volunteering to pick these up.
> > > I'd expect there's a pile more since it's been 1-2 years since Bart
> > > took care of these and merged them consistently.
> > >
> > > Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> >
> > ...
> >
> > >>  drivers/video/backlight/lm3630a_bl.c                     |  1 -
> > >>  drivers/video/backlight/qcom-wled.c                      |  1 +
> >
> > Backlight changes usually go through the backlight tree.
>
> Yes, they do.  How were these applied to the DRM tree?

They are not applied to any drm trees, Helge jumped in last week to
take over drivers/video maintainership.
-Daniel

>
> I don't see any mails about them being applied:
>
>   Luca Weiss (2):
>         backlight: qcom-wled: Add PM6150L compatible
>
>   https://lore.kernel.org/all/20211229170358.2457006-2-luca.weiss@fairphone.com/
>
>   Xu Wang (2):
>         backlight: lm3630a_bl: Remove redundant 'flush_workqueue()' calls
>
>   https://lore.kernel.org/all/20220113084806.13822-1-vulab@iscas.ac.cn/
>
> --
> Lee Jones [李琼斯]
> Principal Technical Lead - Developer Services
> Linaro.org │ Open source software for Arm SoCs
> Follow Linaro: Facebook | Twitter | Blog
Lee Jones Jan. 21, 2022, 10:16 a.m. UTC | #6
On Fri, 21 Jan 2022, Daniel Vetter wrote:

> On Fri, Jan 21, 2022 at 10:19 AM Lee Jones <lee.jones@linaro.org> wrote:
> >
> > On Fri, 21 Jan 2022, Jani Nikula wrote:
> >
> > > On Fri, 21 Jan 2022, Daniel Vetter <daniel@ffwll.ch> wrote:
> > > > On Wed, Jan 19, 2022 at 5:02 PM Helge Deller <deller@gmx.de> wrote:
> > > >> A first bunch of updates and fixes for the following fbdev & backlight drivers:
> > > >> ocfb, aty128fb, mb862xx, omapfb, qcom-wled, dt-bindings, hyperv_fb,
> > > >> lm3630a_bl, omap2, controlfb, matroxfb
> > > >>
> > > >> Nothing really important, mostly cleanups, const conversions, added null
> > > >> pointer/boundary checks and build fixes.
> > > >>
> > > >> Signed-off-by: Helge Deller <deller@gmx.de>
> > > >
> > > > Not sure whether Linus missed this or just wanted to let the
> > > > discussion settle first. But since this is all random patches for
> > > > drivers that many distros don't even enable anymore there's no issues
> > > > here, and I very much welcome someone volunteering to pick these up.
> > > > I'd expect there's a pile more since it's been 1-2 years since Bart
> > > > took care of these and merged them consistently.
> > > >
> > > > Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> > >
> > > ...
> > >
> > > >>  drivers/video/backlight/lm3630a_bl.c                     |  1 -
> > > >>  drivers/video/backlight/qcom-wled.c                      |  1 +
> > >
> > > Backlight changes usually go through the backlight tree.
> >
> > Yes, they do.  How were these applied to the DRM tree?
> 
> They are not applied to any drm trees, Helge jumped in last week to
> take over drivers/video maintainership.
> -Daniel

Sorry s/DRM/fbdev/

> > I don't see any mails about them being applied:
> >
> >   Luca Weiss (2):
> >         backlight: qcom-wled: Add PM6150L compatible
> >
> >   https://lore.kernel.org/all/20211229170358.2457006-2-luca.weiss@fairphone.com/
> >
> >   Xu Wang (2):
> >         backlight: lm3630a_bl: Remove redundant 'flush_workqueue()' calls
> >
> >   https://lore.kernel.org/all/20220113084806.13822-1-vulab@iscas.ac.cn/
> >
> 
> 
>
Lee Jones Jan. 21, 2022, 10:17 a.m. UTC | #7
On Fri, 21 Jan 2022, Helge Deller wrote:

> On 1/21/22 10:19, Lee Jones wrote:
> > On Fri, 21 Jan 2022, Jani Nikula wrote:
> >
> >> On Fri, 21 Jan 2022, Daniel Vetter <daniel@ffwll.ch> wrote:
> >>> On Wed, Jan 19, 2022 at 5:02 PM Helge Deller <deller@gmx.de> wrote:
> >>>> A first bunch of updates and fixes for the following fbdev & backlight drivers:
> >>>> ocfb, aty128fb, mb862xx, omapfb, qcom-wled, dt-bindings, hyperv_fb,
> >>>> lm3630a_bl, omap2, controlfb, matroxfb
> >>>>
> >>>> Nothing really important, mostly cleanups, const conversions, added null
> >>>> pointer/boundary checks and build fixes.
> >>>>
> >>>> Signed-off-by: Helge Deller <deller@gmx.de>
> >>>
> >>> Not sure whether Linus missed this or just wanted to let the
> >>> discussion settle first. But since this is all random patches for
> >>> drivers that many distros don't even enable anymore there's no issues
> >>> here, and I very much welcome someone volunteering to pick these up.
> >>> I'd expect there's a pile more since it's been 1-2 years since Bart
> >>> took care of these and merged them consistently.
> >>>
> >>> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> >>
> >> ...
> >>
> >>>>  drivers/video/backlight/lm3630a_bl.c                     |  1 -
> >>>>  drivers/video/backlight/qcom-wled.c                      |  1 +
> >>
> >> Backlight changes usually go through the backlight tree.
> 
> Sorry, I didn't know yet.
> 
> > Yes, they do.  How were these applied to the DRM tree?
> 
> I did applied them.

When you apply patches, you should reply to let the contributor know.

> Shall I drop them and you decide about them?

No, there's no need for that.

> Next I round I'll take care not to pick backlight patches.

Thank you.