radeon: writeback-issues since kernel 2.6.34 on ATI FireMV 2200 PCI

2012-01-02 Thread Helge Deller
I'm facing the problem with the radeon drm driver, that I now manually need to set the kernel module parameter radeon.no_wb to 1 at bootup, else X just hangs in average up to 8 seconds per minute without any real activity (X or the video driver just seems to wait for something..). Fedora 13

Re: radeon: writeback-issues since kernel 2.6.34 on ATI FireMV 2200 PCI

2012-01-03 Thread Helge Deller
On 01/03/2012 03:27 PM, Alex Deucher wrote: On Mon, Jan 2, 2012 at 5:07 PM, Helge Dellerdel...@gmx.de wrote: I'm facing the problem with the radeon drm driver, that I now manually need to set the kernel module parameter radeon.no_wb to 1 at bootup, else X just hangs in average up to 8 seconds

Re: radeon: writeback-issues since kernel 2.6.34 on ATI FireMV 2200 PCI

2012-01-04 Thread Helge Deller
On 01/04/2012 12:37 AM, Alex Deucher wrote: On Tue, Jan 3, 2012 at 6:12 PM, Helge Dellerdel...@gmx.de wrote: On 01/03/2012 03:27 PM, Alex Deucher wrote: On Mon, Jan 2, 2012 at 5:07 PM, Helge Dellerdel...@gmx.dewrote: I'm facing the problem with the radeon drm driver, that I now

kernel panic in drm_calc_vbltimestamp_from_scanoutpos with 3.18.0-rc2

2014-10-27 Thread Helge Deller
With kernel 3.18.0-rc2 I'm getting the following kernel panic in drm_calc_vbltimestamp_from_scanoutpos()on the parisc platform. As you can see in the attached log, the ring test fails. This is not a new bug - it has failed with older kernels too and of course it would be nice to get it fixed at

radeon: writeback-issues since kernel 2.6.34 on ATI FireMV 2200 PCI

2012-01-02 Thread Helge Deller
I'm facing the problem with the radeon drm driver, that I now manually need to set the kernel module parameter radeon.no_wb to 1 at bootup, else X just hangs in average up to 8 seconds per minute without any real activity (X or the video driver just seems to wait for something..). Fedora 13

radeon: writeback-issues since kernel 2.6.34 on ATI FireMV 2200 PCI

2012-01-04 Thread Helge Deller
On 01/03/2012 03:27 PM, Alex Deucher wrote: > On Mon, Jan 2, 2012 at 5:07 PM, Helge Deller wrote: >> I'm facing the problem with the radeon drm driver, that I now manually need >> to set the kernel module parameter radeon.no_wb to 1 at bootup, else X just >> hangs in averag

radeon: writeback-issues since kernel 2.6.34 on ATI FireMV 2200 PCI

2012-01-04 Thread Helge Deller
On 01/04/2012 12:37 AM, Alex Deucher wrote: > On Tue, Jan 3, 2012 at 6:12 PM, Helge Deller wrote: >> On 01/03/2012 03:27 PM, Alex Deucher wrote: >>> >>> On Mon, Jan 2, 2012 at 5:07 PM, Helge Dellerwrote: >>>> >>>> I'm facing the prob

Re: [PATCH 26/38] vt: use newly defined CUR_* macros

2020-06-16 Thread Helge Deller
iewicz > Cc: "James E.J. Bottomley" > Cc: Helge Deller > Cc: linux-...@vger.kernel.org > Cc: dri-devel@lists.freedesktop.org > Cc: linux-fb...@vger.kernel.org > Cc: linux-par...@vger.kernel.org Acked-by: Helge Deller Thanks! Helge > --- > drivers/tty/vt/vt.c

Re: [PATCH v2 1/1] kernel.h: Split out panic and oops helpers

2021-04-10 Thread Helge Deller
Reviewed-by: Bjorn Andersson Acked-by: Mike Rapoport Acked-by: Corey Minyard Acked-by: Christian Brauner Acked-by: Arnd Bergmann Acked-by: Kees Cook Acked-by: Wei Liu Acked-by: Rasmus Villemoes Signed-off-by: Andrew Morton Acked-by: Helge Deller # parisc Helge

Re: [PATCH] agp: define proper stubs for empty helpers

2021-09-20 Thread Helge Deller
: suggest braces around empty body in an 'if' statement [-Werror=empty-body] 1265 | unmap_page_from_agp(page); Change the definitions to a 'do { } while (0)' construct to make these more reliable. Signed-off-by: Arnd Bergmann Acked-by: Helge Deller # parisc Thanks, Helge --- arch/parisc

Re: Build regressions/improvements in v5.16-rc1

2021-11-15 Thread Helge Deller
On 11/15/21 17:12, Geert Uytterhoeven wrote: > On Mon, Nov 15, 2021 at 4:54 PM Geert Uytterhoeven > wrote: >> Below is the list of build error/warning regressions/improvements in >> v5.16-rc1[1] compared to v5.15[2]. >> >> Summarized: >> - build errors: +20/-13 >> - build warnings: +3/-28 >>

Re: Build regressions/improvements in v5.16-rc1

2021-11-17 Thread Helge Deller
On 11/17/21 03:19, Nick Terrell wrote: > > >> On Nov 16, 2021, at 6:05 PM, Randy Dunlap wrote: >> >> On 11/16/21 5:59 PM, Nick Terrell wrote: >>>> On Nov 15, 2021, at 8:44 AM, Helge Deller wrote: >>>> >>>> On 11/15/21 17:12, Geert Uyt

Re: [PATCH] agp: parisc-agp: fix section mismatch warning

2021-11-27 Thread Helge Deller
On 11/27/21 11:56, Rolf Eike Beer wrote: > Am Samstag, 27. November 2021, 05:57:57 CET schrieb Randy Dunlap: >> Fix section mismatch warning in parisc-agp: Thanks Randy! > Too late ;) > > https://lore.kernel.org/linux-parisc/20211126154754.263487-1-del...@gmx.de/ Being late doesn't matter! I'm

Re: [PATCH 0/7] agp: Various minor fixes

2021-11-30 Thread Helge Deller
On 11/30/21 11:03, Thomas Zimmermann wrote: > Ping! Any feedback on these patches? I just reviewed those patches, and sent some feedback. The others are: Acked-by: Helge Deller Helge > Am 12.11.21 um 15:16 schrieb Thomas Zimmermann: >> Fix a number of compiler warnings in the

Re: [PATCH 4/7] agp/ati: Return error from ati_create_page_map()

2021-11-30 Thread Helge Deller
On 11/12/21 15:16, Thomas Zimmermann wrote: > Fix the compiler warning > > drivers/char/agp/ati-agp.c: In function 'ati_create_page_map': > drivers/char/agp/ati-agp.c:58:16: warning: variable 'err' set but not used > [-Wunused-but-set-variable] > 58 | int i, err = 0; > > by

Re: [PATCH 5/7] agp/nvidia: Ignore value returned by readl()

2021-11-30 Thread Helge Deller
On 11/12/21 15:16, Thomas Zimmermann wrote: > Fix the compiler warning > > drivers/char/agp/nvidia-agp.c: In function 'nvidia_tlbflush': > drivers/char/agp/nvidia-agp.c:264:22: warning: variable 'temp' set but not > used [-Wunused-but-set-variable] > 264 | u32 wbc_reg, temp; > >

Re: [PATCH v2 0/7] agp: Various minor fixes

2022-01-13 Thread Helge Deller
On 1/13/22 09:51, Thomas Zimmermann wrote: > ping! > > This patchset got lost. Patches 4 and 5 still need a review. for patches 4 & 5: Acked-by: Helge Deller Helge > > Am 01.12.21 um 12:46 schrieb Thomas Zimmermann: >> Fix a number of compiler warnings in the A

Re: fbdev: Garbage collect fbdev scrolling acceleration

2022-01-13 Thread Helge Deller
I may have missed some discussions, but I'm objecting against this patch: b3ec8cdf457e5 ("fbdev: Garbage collect fbdev scrolling acceleration, part 1 (from TODO list)") Can we please (partly) revert it and restore the scrolling behaviour, where fbcon uses fb_copyarea() to copy the

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-17 Thread Helge Deller
Hi Thomas, On 1/17/22 12:16, Thomas Zimmermann wrote: > Hi > > Am 14.01.22 um 19:11 schrieb Helge Deller: >> The fbdev layer is orphaned, but seems to need some care. >> So I'd like to step up as new maintainer. >> >> Signed-off-by: Helge Deller > >

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-17 Thread Helge Deller
Hello Jani, On 1/17/22 11:49, Jani Nikula wrote: > On Mon, 17 Jan 2022, Daniel Vetter wrote: >> Hi Helge >> >> On Fri, Jan 14, 2022 at 7:18 PM Helge Deller wrote: >>> >>> The fbdev layer is orphaned, but seems to need some care. >>> So I'd like

Re: [GIT PULL] fbdev updates for v5.17-rc1

2022-01-17 Thread Helge Deller
On 1/17/22 05:33, Linus Torvalds wrote: > On Sun, Jan 16, 2022 at 9:32 PM Helge Deller wrote: >> >> This pull request contains only one single initial patch which adds >> myself to the MAINTAINERS file for the FRAMBUFFER LAYER. > > I'll pull this Thanks! > (as m

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-17 Thread Helge Deller
Hello Daniel, On 1/17/22 11:02, Daniel Vetter wrote: > Hi Helge > > On Fri, Jan 14, 2022 at 7:18 PM Helge Deller wrote: >> >> The fbdev layer is orphaned, but seems to need some care. >> So I'd like to step up as new maintainer. >> >> Signed-off-by: Helge D

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-17 Thread Helge Deller
On 1/17/22 15:10, Geert Uytterhoeven wrote: > Hi Thomas, > > On Mon, Jan 17, 2022 at 2:51 PM Thomas Zimmermann wrote: >> Am 17.01.22 um 14:29 schrieb Geert Uytterhoeven: >>> On Mon, Jan 17, 2022 at 1:57 PM Gerd Hoffmann wrote: > b) to include new drivers (for old hardware) if they arrive

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-17 Thread Helge Deller
On 1/17/22 16:00, Daniel Vetter wrote: > On Mon, Jan 17, 2022 at 1:16 PM Helge Deller wrote: >> >> Hello Daniel, >> >> On 1/17/22 11:02, Daniel Vetter wrote: >>> Hi Helge >>> >>> On Fri, Jan 14, 2022 at 7:18 PM Helge Deller wrote: >>

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-17 Thread Helge Deller
Hi Thomas, On 1/17/22 16:05, Thomas Zimmermann wrote: > Hi > > Am 17.01.22 um 15:47 schrieb Helge Deller: >> On 1/17/22 15:10, Geert Uytterhoeven wrote: >> [...] >>> Using an XRGB32 intermediate would kill the user experience on old >>> machines, due

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-17 Thread Helge Deller
On 1/17/22 17:38, Daniel Vetter wrote: > On Mon, Jan 17, 2022 at 5:22 PM Helge Deller wrote: >> >> On 1/17/22 16:58, Thomas Zimmermann wrote: >>> Hi >>> >>> Am 17.01.22 um 16:42 schrieb Helge Deller: >>>> [...] >>>>>>

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-17 Thread Helge Deller
On 1/17/22 16:58, Thomas Zimmermann wrote: > Hi > > Am 17.01.22 um 16:42 schrieb Helge Deller: >> [...] >>>> c) reintroduce the state where fbcon is fast on fbdev. This is important >>>> for non-DRM machines, >>>>     either when run on nativ

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-17 Thread Helge Deller
On 1/17/22 17:21, Helge Deller wrote: > On 1/17/22 16:58, Thomas Zimmermann wrote: >> Hi >> >> Am 17.01.22 um 16:42 schrieb Helge Deller: >>> [...] >>>>> c) reintroduce the state where fbcon is fast on fbdev. This is important >>>>&g

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-17 Thread Helge Deller
On 1/17/22 16:03, Daniel Vetter wrote: > On Mon, Jan 17, 2022 at 3:48 PM Helge Deller wrote: >> >> On 1/17/22 15:10, Geert Uytterhoeven wrote: >>> Hi Thomas, >>> >>> On Mon, Jan 17, 2022 at 2:51 PM Thomas Zimmermann >>> wrote: >>>>

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-17 Thread Helge Deller
On 1/17/22 22:40, Jani Nikula wrote: > On Mon, 17 Jan 2022, Thomas Zimmermann wrote: >> Seems like few people read linux-fbdev these days. > > How much traffic is there to linux-fbdev that is *not* Cc'd to dri-devel > also? Doesn't seem like much traffic - which IMHO is OK for such a tree with

Re: [PATCH 0/3] video: fbdev: atari: Miscellaneous fixes

2022-02-15 Thread Helge Deller
Hi Geert, On 2/15/22 12:21, Geert Uytterhoeven wrote: > This is a small series of miscellaneous fixes for the Atari frame buffer > device driver. > > The first patch has been sent before, and is untested, as I have no > access to the hardware. > The other patches have been tested on ARAnyM. > >

Re: [PATCH 22/23] video: omapfb: dss: Make use of the helper component_compare_dev

2022-02-15 Thread Helge Deller
On 2/14/22 07:08, Yong Wu wrote: > Use the common compare helper from component. > > Cc: Helge Deller > Cc: linux-o...@vger.kernel.org > Cc: linux-fb...@vger.kernel.org > Signed-off-by: Yong Wu Applied to the fbdev for-next branch. Thanks! Helge > drivers/video/fbdev/o

Re: [PATCH] omapfb: Add missing of_node_put() in dvic_probe_of

2022-03-07 Thread Helge Deller
On 3/7/22 13:38, Miaoqian Lin wrote: > The device_node pointer is returned by of_parse_phandle() with refcount > incremented. We should use of_node_put() on it when done. > > Fixes: f76ee892a99e ("omapfb: copy omapdss & displays for omapfb") > Signed-off-by: Miaoqian Lin applied to the fbdev

Re: [PATCH] video: fbdev: fix warning comparing pointer to 0

2022-03-10 Thread Helge Deller
On 3/11/22 03:56, Haowen Bai wrote: > Fix the following coccicheck warning: > drivers/video/fbdev/offb.c:415:13-14: WARNING comparing pointer to 0 > > Signed-off-by: Haowen Bai applied. Thanks! Helge > --- > drivers/video/fbdev/offb.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >

Re: [PATCH] video: fbdev: kyro: make read-only array ODValues static const

2022-03-10 Thread Helge Deller
On 3/8/22 21:23, Colin Ian King wrote: > Don't populate the read-only array ODValues on the stack but > instead make it static const. Also makes the object code a little > smaller. > > Signed-off-by: Colin Ian King > > --- > drivers/video/fbdev/kyro/STG4000InitDevice.c | 2 +- > 1 file changed,

Re: [PATCH] video: fbdev: au1100fb: Spelling s/palette/palette/

2022-02-16 Thread Helge Deller
On 2/16/22 09:40, Geert Uytterhoeven wrote: > Fix a misspelling of "palette" in a structure member. > > Signed-off-by: Geert Uytterhoeven applied. Thanks! Helge > --- > drivers/video/fbdev/au1100fb.c | 2 +- > drivers/video/fbdev/au1100fb.h | 2 +- > 2 files changed, 2 insertions(+), 2

Re: [BUG] fbdev: sm712fb: Page fault in smtcfb_read

2022-02-26 Thread Helge Deller
* Zheyu Ma : > I found a minor in the smtcfb_read() function of the driver sm712fb. > > This read function can not handle the case that the size of the > buffer is 3 and does not check for it, which may cause a page fault. > > The following log reveals it: > > [ 2432.614490] BUG: unable to handle

Re: [PATCH] video: fbdev: via: check the return value of kstrdup()

2022-02-22 Thread Helge Deller
On 2/21/22 11:37, xkernel.w...@foxmail.com wrote: > From: Xiaoke Wang > > kstrdup() is a memory allocation function which can return NULL when > some internal memory errors happen. It is better to check the return > value of it to catch the error in time during the setup of viafb. > >

Re: [PATCH] video: fbdev: sm712fb: Fix crash in smtcfb_write()

2022-03-02 Thread Helge Deller
On 3/2/22 15:33, Zheyu Ma wrote: > When the sm712fb driver writes three bytes to the framebuffer, the > driver will crash: > > BUG: unable to handle page fault for address: c90001ff > RIP: 0010:smtcfb_write+0x454/0x5b0 > Call Trace: > vfs_write+0x291/0xd60 > ?

Re: [PATCH] video: fbdev: s3c-fb: fix platform_get_irq.cocci warning

2022-03-03 Thread Helge Deller
On 3/2/22 04:44, Yihao Han wrote: > Remove dev_err() messages after platform_get_irq*() failures. > platform_get_irq() already prints an error. > > Generated by: scripts/coccinelle/api/platform_get_irq.cocci > > Signed-off-by: Yihao Han applied to the fbdev for-next git tree. Thanks! Helge >

Re: [PATCH] video: fbdev: sm712fb: Fix crash in smtcfb_write()

2022-03-03 Thread Helge Deller
On 3/3/22 02:34, Zheyu Ma wrote: > Hi, > > On Thu, Mar 3, 2022 at 12:49 AM Helge Deller wrote: >> >> On 3/2/22 15:33, Zheyu Ma wrote: >>> When the sm712fb driver writes three bytes to the framebuffer, the >>> driver will crash: >>> >

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

2022-03-22 Thread Helge Deller
roups in kobj_type Guo Zhengkui (1): video: fbdev: omapfb: Use sysfs_emit() instead of snprintf() Haowen Bai (1): video: fbdev: offb: fix warning comparing pointer to 0 Helge Deller (1): video: fbdev: sm712fb: Fix crash in smtcfb_read() Jiasheng Jiang (1): video: fbdev: im

Re: [PATCH] video: of: display_timing: Remove a redundant zeroing of memory

2022-03-24 Thread Helge Deller
On 3/22/22 21:33, Christophe JAILLET wrote: > of_parse_display_timing() already call memset(0) on its 2nd argument, so > there is no need to clear it explicitly before calling this function. > > Use kmalloc() instead of kzalloc() to save a few cycles. > > Signed-off-by: Christophe JAILLET

Re: [PATCH] fbdev: udlfb: properly check endpoint type

2022-03-24 Thread Helge Deller
On 3/22/22 21:04, Pavel Skripkin wrote: > syzbot reported warning in usb_submit_urb, which is caused by wrong > endpoint type. > > This driver uses out bulk endpoint for communication, so > let's check if this endpoint is present and bail out early if not. > > Fail log: > > usb 1-1: BOGUS urb

Re: [PATCH] linux/fb.h: Spelling s/palette/palette/

2022-04-01 Thread Helge Deller
On 2/17/22 11:16, Pekka Paalanen wrote: > On Wed, 16 Feb 2022 09:39:22 +0100 > Geert Uytterhoeven wrote: > >> Fix a misspelling of "palette" in a comment. >> >> Signed-off-by: Geert Uytterhoeven >> --- >> include/uapi/linux/fb.h | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >>

Re: [PATCH] Hvideo: fbdev: pm2fb: Fix a kernel-doc formatting issue

2022-04-01 Thread Helge Deller
On 4/1/22 10:41, Haowen Bai wrote: > This function had kernel-doc that not used a hash to separate > the function name from the one line description. > > The warning was found by running scripts/kernel-doc, which is > caused by using 'make W=1'. > > drivers/video/fbdev/pm2fb.c:1507: warning: This

Re: [PATCH] video: fbdev: sis: fix potential NULL dereference in sisfb_post_sis300()

2022-04-01 Thread Helge Deller
On 4/1/22 05:41, Haowen Bai wrote: > bios could be null without checking null and return in this function, > but still dereference bios[0xf5]. > > Signed-off-by: Haowen Bai applied. Thanks! Helge > --- > drivers/video/fbdev/sis/sis_main.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [PATCH] video: fbdev: Prepare cleanup of powerpc's asm/prom.h

2022-04-02 Thread Helge Deller
On 4/2/22 12:22, Christophe Leroy wrote: > powerpc's asm/prom.h brings some headers that it doesn't > need itself. > > In order to clean it up, first add missing headers in > users of asm/prom.h > > Signed-off-by: Christophe Leroy applied to fbdev tree. Thanks! Helge > --- >

Re: [BUG] fbdev: i740fb: Divide error when ‘var->pixclock’ is zero

2022-04-03 Thread Helge Deller
On 4/3/22 13:26, Zheyu Ma wrote: > Hi, > > I found a bug in the function i740fb_set_par(). Nice catch! > When the user calls the ioctl system call without setting the value to > 'var->pixclock', the driver will throw a divide error. > > This bug occurs because the driver uses the value of

Re: [BUG] fbdev: i740fb: Divide error when ‘var->pixclock’ is zero

2022-04-05 Thread Helge Deller
Hello Geert, On 4/4/22 13:46, Geert Uytterhoeven wrote: > Hi Helge, > > On Sun, Apr 3, 2022 at 5:41 PM Helge Deller wrote: >> On 4/3/22 13:26, Zheyu Ma wrote: >>> I found a bug in the function i740fb_set_par(). >> >> Nice catch! >> >>> When t

Re: [PATCH v3 3/3] fbcon: Add option to enable legacy hardware acceleration

2022-02-01 Thread Helge Deller
Hello Daniel, On 2/1/22 21:11, Daniel Vetter wrote: > On Tue, Feb 1, 2022 at 7:59 PM Helge Deller wrote: >> >> Add a config option CONFIG_FRAMEBUFFER_CONSOLE_LEGACY_ACCELERATION to >> enable bitblt and fillrect hardware acceleration in the framebuffer >> console. If

[PATCH v3 1/3] Revert "fbdev: Garbage collect fbdev scrolling acceleration, part 1 (from TODO list)"

2022-02-01 Thread Helge Deller
eintroduce hardware-based scrolling acceleration and fix the performance regression for fbdev drivers. There isn't any impact on DRM when reverting those patches. Signed-off-by: Helge Deller Acked-by: Geert Uytterhoeven Acked-by: Sven Schnelle Cc: sta...@vger.kernel.org # v5.16+ Signed-off-by: He

[PATCH v3 0/3] Fix regression introduced by disabling accelerated scrolling in fbcon

2022-02-01 Thread Helge Deller
driver. This series is being discussed on linux-fbdev and dri-devel mailing lists. Helge Helge Deller (3): Revert "fbdev: Garbage collect fbdev scrolling acceleration, part 1 (from TODO list)" Revert "fbcon: Disable accelerated scrolling" fbcon: Add option to en

[PATCH v3 2/3] Revert "fbcon: Disable accelerated scrolling"

2022-02-01 Thread Helge Deller
eintroduce hardware-based scrolling acceleration and fix the performance regression for fbdev drivers. There isn't any impact on DRM when reverting those patches. Signed-off-by: Helge Deller Acked-by: Geert Uytterhoeven Acked-by: Sven Schnelle Cc: sta...@vger.kernel.org # v5.10+ Signed-off-by: He

[PATCH v3 3/3] fbcon: Add option to enable legacy hardware acceleration

2022-02-01 Thread Helge Deller
patch version I additionally changed the GETVYRES() and GETVXRES() macros to take a pointer to the fbcon_display struct. This fixes the build when console rotation is enabled and helps the compiler again to optimize out code. Signed-off-by: Helge Deller Cc: sta...@vger.kernel.org # v5.10+ Signed

Re: [PATCH -next 1/2 v2] video: fbdev: pxa168fb: Remove unnecessary print function dev_err()

2022-02-01 Thread Helge Deller
On 2/1/22 13:32, Yang Li wrote: > The print function dev_err() is redundant because platform_get_irq() > already prints an error. > > Eliminate the follow coccicheck warning: > ./drivers/video/fbdev/pxa168fb.c:621:2-9: line 621 is redundant because > platform_get_irq() already prints an error > >

Re: [PATCH v3 3/3] fbcon: Add option to enable legacy hardware acceleration

2022-02-02 Thread Helge Deller
On 2/2/22 12:05, Daniel Vetter wrote: > On Tue, Feb 1, 2022 at 11:52 PM Helge Deller wrote: >> >> Hello Daniel, >> >> On 2/1/22 21:11, Daniel Vetter wrote: >>> On Tue, Feb 1, 2022 at 7:59 PM Helge Deller wrote: >>>> >>>> Add a

[PATCH v4 3/3] fbcon: Add option to enable legacy hardware acceleration

2022-02-02 Thread Helge Deller
patch version I additionally changed the GETVYRES() and GETVXRES() macros to take a pointer to the fbcon_display struct. This fixes the build when console rotation is enabled and helps the compiler again to optimize out code. Signed-off-by: Helge Deller Cc: sta...@vger.kernel.org # v5.10+ Signed

[PATCH v4 1/3] Revert "fbdev: Garbage collect fbdev scrolling acceleration, part 1 (from TODO list)"

2022-02-02 Thread Helge Deller
eintroduce hardware-based scrolling acceleration and fix the performance regression for fbdev drivers. There isn't any impact on DRM when reverting those patches. Signed-off-by: Helge Deller Acked-by: Geert Uytterhoeven Acked-by: Sven Schnelle Cc: sta...@vger.kernel.org # v5.16+ Signed-off-by: He

[PATCH v4 2/3] Revert "fbcon: Disable accelerated scrolling"

2022-02-02 Thread Helge Deller
eintroduce hardware-based scrolling acceleration and fix the performance regression for fbdev drivers. There isn't any impact on DRM when reverting those patches. Signed-off-by: Helge Deller Acked-by: Geert Uytterhoeven Acked-by: Sven Schnelle Cc: sta...@vger.kernel.org # v5.10+ Signed-off-by: He

[PATCH v4 0/3] Fix regression introduced by disabling accelerated scrolling in fbcon

2022-02-02 Thread Helge Deller
driver. This series is being discussed on linux-fbdev and dri-devel mailing lists. Updates in v3: - respin with fixes to build rotation code Updates in v4: - changed the Kconfig option and Kconfig help text Helge Deller (3): Revert "fbdev: Garbage collect fbdev scrolling acceleration, p

Re: [PATCH] fbdev: fbmem: Fix the implicit type casting

2022-02-02 Thread Helge Deller
On 2/2/22 18:27, Sam Ravnborg wrote: > Hi Helge, > > On Tue, Feb 01, 2022 at 04:02:40PM +0100, Helge Deller wrote: >> On 1/31/22 07:57, Yizhuo Zhai wrote: >>> In function do_fb_ioctl(), the "arg" is the type of unsigned long, >> >>

Re: fbdev git tree update

2022-02-02 Thread Helge Deller
Hello Sam, On 2/2/22 18:56, Sam Ravnborg wrote: > Hi Helge, > > On Sun, Jan 30, 2022 at 10:05:39PM +0100, Helge Deller wrote: >> Hello DRI and fbdev developers, >> >> I've now mostly checked all queued-up patches on the fbdev mailing list: >> https://patchwork.ke

Re: fbdev git tree update

2022-02-02 Thread Helge Deller
Hello Sam, On 2/2/22 19:39, Sam Ravnborg wrote: > Hi Helge, >> >>> How come there are backlight, dt-bindings >> >> I didn't know, and I picked up those at the beginning. >> I did agreed with the backlight maintainers that I should >> keep those which are already in my tree, and that I will ignore

Re: [PATCH v6] fbdev: fbmem: Fix the implicit type casting

2022-02-03 Thread Helge Deller
On 2/3/22 07:39, Sam Ravnborg wrote: > Hi Daniel, > > I assume you will take this. > > Patch is: > Reviewed-by: Sam Ravnborg Acked-by: Helge Deller Helge > > Sam > > On Wed, Feb 02, 2022 at 03:58:08PM -0800, Yizhuo Zhai wrote: >> In function

Re: [PATCH v4 3/3] fbcon: Add option to enable legacy hardware acceleration

2022-02-04 Thread Helge Deller
Hello Geert, On 2/4/22 09:37, Geert Uytterhoeven wrote: > On Wed, Feb 2, 2022 at 8:05 PM Helge Deller wrote: >> Add a config option CONFIG_FRAMEBUFFER_CONSOLE_LEGACY_ACCELERATION to >> enable bitblt and fillrect hardware acceleration in the framebuffer >> console. If disabl

Re: [PATCH v4 3/3] fbcon: Add option to enable legacy hardware acceleration

2022-02-04 Thread Helge Deller
Hi Geert, On 2/4/22 11:24, Geert Uytterhoeven wrote: > On Fri, Feb 4, 2022 at 11:17 AM Helge Deller wrote: >> On 2/4/22 09:37, Geert Uytterhoeven wrote: >>> On Wed, Feb 2, 2022 at 8:05 PM Helge Deller wrote: >>>> Add a config option CONFIG_FRAMEBUFFER_CONSOLE_LEGA

fbdev git tree update

2022-01-30 Thread Helge Deller
Hello DRI and fbdev developers, I've now mostly checked all queued-up patches on the fbdev mailing list: https://patchwork.kernel.org/project/linux-fbdev/list/ and applied the ones which seemed appropriate. IMHO there is nothing really critical/important/conflicting in there. Shortlog is below,

Re: [PATCH 03/21] fbcon: Restore fbcon scrolling acceleration

2022-02-01 Thread Helge Deller
On 1/31/22 22:05, Daniel Vetter wrote: > This functionally undoes 39aead8373b3 ("fbcon: Disable accelerated > scrolling"), but behind the FRAMEBUFFER_CONSOLE_LEGACY_ACCELERATION > option. you have two trivial copy-n-paste errors in this patch which still prevent the console acceleration. > diff

Re: [PATCH 01/21] MAINTAINERS: Add entry for fbdev core

2022-02-01 Thread Helge Deller
ect that to continue like > we've done before, so no new expectations that patches all go through > my hands. That would be silly. This also means I'm happy to put any > other volunteer's name in the M: line, but otherwise git log says I'm > the one who's stuck with this. Yes, agreed.

Re: [PATCH 03/21] fbcon: Restore fbcon scrolling acceleration

2022-02-01 Thread Helge Deller
On 2/1/22 11:16, Helge Deller wrote: > On 1/31/22 22:05, Daniel Vetter wrote: >> This functionally undoes 39aead8373b3 ("fbcon: Disable accelerated >> scrolling"), but behind the FRAMEBUFFER_CONSOLE_LEGACY_ACCELERATION >> option. > > you have two trivial co

Re: [PATCH 03/21] fbcon: Restore fbcon scrolling acceleration

2022-02-01 Thread Helge Deller
On 2/1/22 11:36, Daniel Vetter wrote: > On Tue, Feb 1, 2022 at 11:16 AM Helge Deller wrote: >> >> On 1/31/22 22:05, Daniel Vetter wrote: >>> This functionally undoes 39aead8373b3 ("fbcon: Disable accelerated >>> scrolling"), but behind the FRAMEBU

Re: [PATCH -next 1/2] video: fbdev: pxa168fb: Remove unnecessary print function dev_err()

2022-02-01 Thread Helge Deller
On 2/1/22 07:26, Yang Li wrote: > The print function dev_err() is redundant because platform_get_irq() > already prints an error. > > Eliminate the follow coccicheck warning: > ./drivers/video/fbdev/pxa168fb.c:621:2-9: line 621 is redundant because > platform_get_irq() already prints an error > >

Re: [PATCH] fbdev: fbmem: Fix the implicit type casting

2022-02-01 Thread Helge Deller
On 1/31/22 07:57, Yizhuo Zhai wrote: > In function do_fb_ioctl(), the "arg" is the type of unsigned long, yes, because it comes from the ioctl framework... > and in "case FBIOBLANK:" this argument is casted into an int before > passig to fb_blank(). which makes sense IMHO. > In fb_blank(), the

Re: [PATCH 03/21] fbcon: Restore fbcon scrolling acceleration

2022-02-01 Thread Helge Deller
On 2/1/22 14:45, Daniel Vetter wrote: > On Tue, Feb 1, 2022 at 12:01 PM Helge Deller wrote: >> On 2/1/22 11:36, Daniel Vetter wrote: >>> On Tue, Feb 1, 2022 at 11:16 AM Helge Deller wrote: >>>> >>>> On 1/31/22 22:05, Daniel Vetter wrote: >>&

Re: [PATCH] fbcon: Avoid 'cap' set but not used warning

2022-02-08 Thread Helge Deller
On 2/8/22 14:44, Daniel Vetter wrote: > On Mon, Feb 07, 2022 at 04:59:31PM +0100, Helge Deller wrote: >> Fix this kernel test robot warning: >> >> drivers/video/fbdev/core/fbcon.c: In function 'fbcon_init': >> drivers/video/fbdev/core/fbcon.c:1028:6:

[PATCH] fbcon: Avoid 'cap' set but not used warning

2022-02-07 Thread Helge Deller
is enabled. Drop the temporary variable and use info->flags instead. Fixes: 87ab9f6b7417 ("Revert "fbcon: Disable accelerated scrolling") Reported-by: kernel test robot Signed-off-by: Helge Deller diff --git a/drivers/video/fbdev/core/fbcon.c b/drivers/video/fbdev/core/fbcon.c

Re: [PATCH] fbcon: use min() to make code cleaner

2022-02-09 Thread Helge Deller
On 2/9/22 19:54, Sam Ravnborg wrote: > On Wed, Feb 09, 2022 at 08:48:10AM +, cgel@gmail.com wrote: >> From: Changcheng Deng >> >> Use min() in order to make code cleaner. >> >> Reported-by: Zeal Robot >> Signed-off-by: Changcheng Deng > Reviewed-by: Sam Ravnborg > > I had preferred in

Re: [PATCH] video: s3c-fb: Use platform_get_irq() to get the interrupt

2022-02-11 Thread Helge Deller
On 2/11/22 04:27, zhaoxiao wrote: > platform_get_resource(pdev, IORESOURCE_IRQ, ..) relies on static > allocation of IRQ resources in DT core code, this causes an issue > when using hierarchical interrupt domains using "interrupts" property > in the node as this bypassed the hierarchical setup and

[PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-14 Thread Helge Deller
The fbdev layer is orphaned, but seems to need some care. So I'd like to step up as new maintainer. Signed-off-by: Helge Deller diff --git a/MAINTAINERS b/MAINTAINERS index 5d0cd537803a..ce47dbc467cc 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -7583,11 +7583,12 @@ W: http

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-18 Thread Helge Deller
On 1/18/22 12:18, Daniel Vetter wrote: > On Tue, Jan 18, 2022 at 9:56 AM Helge Deller wrote: >> >> On 1/18/22 09:38, Jani Nikula wrote: >>> On Mon, 17 Jan 2022, Helge Deller wrote: >>>> On 1/17/22 22:40, Jani Nikula wrote: >>>>> On Mon,

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-18 Thread Helge Deller
On 1/18/22 10:16, Gerd Hoffmann wrote: > On Tue, Jan 18, 2022 at 09:20:43AM +0100, Helge Deller wrote: >> On 1/18/22 07:29, Gerd Hoffmann wrote: >>>> Please correct me if I'm wrong, but text-console emulation/scrolling on >>>> DRM is >>>> currently una

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-18 Thread Helge Deller
Hello Daniel, On 1/17/22 16:00, Daniel Vetter wrote: > On Mon, Jan 17, 2022 at 1:16 PM Helge Deller wrote: >> On 1/17/22 11:02, Daniel Vetter wrote: >>> On Fri, Jan 14, 2022 at 7:18 PM Helge Deller wrote: >>>> >>>> The fbdev layer is orphaned, but

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-18 Thread Helge Deller
On 1/18/22 11:13, Helge Deller wrote: > On 1/18/22 10:16, Gerd Hoffmann wrote: >> On Tue, Jan 18, 2022 at 09:20:43AM +0100, Helge Deller wrote: >>> On 1/18/22 07:29, Gerd Hoffmann wrote: >>>>> Please correct me if I'm wrong, but text-console emulation/scrolling

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-18 Thread Helge Deller
On 1/18/22 09:41, Helge Deller wrote: > Hello Daniel, > > On 1/17/22 16:00, Daniel Vetter wrote: >> On Mon, Jan 17, 2022 at 1:16 PM Helge Deller wrote: >>> On 1/17/22 11:02, Daniel Vetter wrote: >>>> On Fri, Jan 14, 2022 at 7:18 PM Helge Deller wrote: >

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-18 Thread Helge Deller
On 1/18/22 07:29, Gerd Hoffmann wrote: >> Please correct me if I'm wrong, but text-console emulation/scrolling on DRM >> is >> currently unaccelerated and bound to Truecolour modes only, > > Yes. Adding support for formats beside argb to the drm fbcon > emulation shouldn't be that much of a

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-18 Thread Helge Deller
On 1/18/22 07:11, Gerd Hoffmann wrote: > On Mon, Jan 17, 2022 at 02:29:47PM +0100, Geert Uytterhoeven wrote: >> Hi Gerd, >> >> On Mon, Jan 17, 2022 at 1:57 PM Gerd Hoffmann wrote: b) to include new drivers (for old hardware) if they arrive (probably happens rarely but there can be).

Re: [PATCH] MAINTAINERS: Add Helge as fbdev maintainer

2022-01-18 Thread Helge Deller
On 1/18/22 09:38, Jani Nikula wrote: > On Mon, 17 Jan 2022, Helge Deller wrote: >> On 1/17/22 22:40, Jani Nikula wrote: >>> On Mon, 17 Jan 2022, Thomas Zimmermann wrote: >>>> Seems like few people read linux-fbdev these days. >>> >>> How much tr

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

2022-01-21 Thread Helge Deller
On 1/21/22 10:19, Lee Jones wrote: > On Fri, 21 Jan 2022, Jani Nikula wrote: > >> On Fri, 21 Jan 2022, Daniel Vetter wrote: >>> On Wed, Jan 19, 2022 at 5:02 PM Helge Deller wrote: >>>> A first bunch of updates and fixes for the following fbdev & backlig

Re: [PATCH 2/2] Revert "fbcon: Disable accelerated scrolling"

2022-01-19 Thread Helge Deller
Hi Thomas, On 1/19/22 16:37, Thomas Zimmermann wrote: > Am 19.01.22 um 16:05 schrieb Sven Schnelle: >> Daniel Vetter writes: >> >>> On Wed, Jan 19, 2022 at 3:01 PM Linus Torvalds >>> wrote: >>> Irrespective of this code being buggy or not buggy I think the bigger >>> pictures, and really the

Re: [PATCH 2/2] Revert "fbcon: Disable accelerated scrolling"

2022-01-19 Thread Helge Deller
Hello Greg, On 1/19/22 12:47, Geert Uytterhoeven wrote: > On Wed, Jan 19, 2022 at 12:28 PM Greg Kroah-Hartman > wrote: >> On Wed, Jan 19, 2022 at 12:22:55PM +0100, Greg Kroah-Hartman wrote: >>> On Wed, Jan 19, 2022 at 12:08:39PM +0100, Helge Deller wrote: >

Re: [PATCH 2/2] Revert "fbcon: Disable accelerated scrolling"

2022-01-19 Thread Helge Deller
On 1/19/22 15:01, Linus Torvalds wrote: > On Wed, Jan 19, 2022 at 2:29 PM Helge Deller wrote: >> >>>> >>>> Ah, no, that was just the soft scrollback code I was thinking of, which >> >> Right. >> That was commit 973c096f6a85 and it was abou

Re: [PATCH 2/2] Revert "fbcon: Disable accelerated scrolling"

2022-01-19 Thread Helge Deller
On 1/19/22 15:34, Daniel Vetter wrote: > On Wed, Jan 19, 2022 at 3:01 PM Linus Torvalds > wrote: >> >> On Wed, Jan 19, 2022 at 2:29 PM Helge Deller wrote: >>> >>>>> >>>>> Ah, no, that was just the soft scrollback code I was thinking

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

2022-01-19 Thread Helge Deller
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 De

heads-up for fbdev pull request

2022-01-19 Thread Helge Deller
This is just a heads-up, that I'm planning to send a pull request for fbdev fixes to Linus later today. Nothing really important - just fixes for various fbdev drivers. It has been a few days in for-next and no DRM parts are touched. Changelog is below, and it can be pulled from:

Re: [PATCH 2/2] Revert "fbcon: Disable accelerated scrolling"

2022-01-19 Thread Helge Deller
On 1/19/22 16:42, Daniel Vetter wrote: > On Wed, Jan 19, 2022 at 4:06 PM Sven Schnelle wrote: >> >> Daniel Vetter writes: >> >>> On Wed, Jan 19, 2022 at 3:01 PM Linus Torvalds >>> wrote: >>> Irrespective of this code being buggy or not buggy I think the bigger >>> pictures, and really the

[PATCH 1/2] Revert "fbdev: Garbage collect fbdev scrolling acceleration, part 1 (from TODO list)"

2022-01-19 Thread Helge Deller
by factor of 10 and more. Reverting this commit has no impact on DRM, since none of the DRM drivers are tagged with the acceleration flags FBINFO_HWACCEL_COPYAREA, FBINFO_HWACCEL_FILLRECT or others. Signed-off-by: Helge Deller Cc: sta...@vger.kernel.org # v5.16 --- Documentation/gpu/todo.rst

[PATCH 0/2] Fix regression introduced by disabling accelerated scrolling in fbcon

2022-01-19 Thread Helge Deller
here isn't any impact on DRM when reverting those patches. Helge Deller (2): Revert "fbdev: Garbage collect fbdev scrolling acceleration, part 1 (from TODO list)" Revert "fbcon: Disable accelerated scrolling" Documentation/gpu/todo.rst | 24 -- driv

[PATCH 2/2] Revert "fbcon: Disable accelerated scrolling"

2022-01-19 Thread Helge Deller
and more. Reverting this commit has no impact on DRM, since none of the DRM drivers are tagged with the acceleration flags FBINFO_HWACCEL_COPYAREA, FBINFO_HWACCEL_FILLRECT or others. Signed-off-by: Helge Deller Cc: sta...@vger.kernel.org # v5.16 --- Documentation/gpu/todo.rst | 21

Re: [PATCH 2/2] Revert "fbcon: Disable accelerated scrolling"

2022-01-20 Thread Helge Deller
Hello Daniel, On 1/20/22 15:30, Daniel Vetter wrote: > On Wed, Jan 19, 2022 at 12:08:39PM +0100, Helge Deller wrote: >> This reverts commit 39aead8373b3c20bb5965c024dfb51a94e526151. >> >> Revert this patch. This patch started to introduce the regression that >> all ha

  1   2   3   4   5   6   >