similar to: [PATCH] drm/nouveau: remove unused variable from nouveau_bios_shadow_of

From: Andrea Tacconi <tacconet at libero.it> Subject: [PATCH] drm/nouveau: fix vbios load and check functions on PowerPC On PowerPC the Bios signature reports a wrong lenght of video rom. Fix this by reading the correct size from Open Firmware. Set Pramin as primary vbios searching method, because it's the only working method on PowerPC. The nv_cksum function always fails. Fix this

Some archs defconfigs have CONFIG_FRAME_WARN set to 1024, which lead to this warning: drivers/gpu/drm/nouveau/core/engine/graph/ctxnv40.c: warning: the frame size of 1184 bytes is larger than 1024 bytes Reported-by: Geert Uytterhoeven <geert at linux-m68k.org> Signed-off-by: Marcin Slusarz <marcin.slusarz at gmail.com> --- drivers/gpu/drm/nouveau/core/engine/graph/ctxnv40.c | 12

On Mon, Feb 15, 2010 at 03:40:56PM +0300, Dan Carpenter wrote: > This is the results from: > make C=1 CHECK="/path/to/smatch -p=kernel" bzImage modules | tee warns.txt > grep -w overflow warns.txt | uniq -f 3 | tee err-list > > I hacked on the buffer overflow check last weekend and these are the > results. It has way more false positives than the other bug

Remove unused vairable, return 0 explicitly. Signed-off-by: Muhammad Falak R Wani <falakreyaz at gmail.com> --- drivers/gpu/drm/nouveau/nouveau_bios.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_bios.c b/drivers/gpu/drm/nouveau/nouveau_bios.c index 4dca65a..5449838 100644 --- a/drivers/gpu/drm/nouveau/nouveau_bios.c +++

On 5/18/24 01:26, linux at treblig.org wrote: > From: "Dr. David Alan Gilbert" <linux at treblig.org> > > 'init_exec' is unused since > commit cb75d97e9c77 ("drm/nouveau: implement devinit subdev, and new > init table parser") > Remove it. > > Signed-off-by: Dr. David Alan Gilbert <linux at treblig.org> Acked-by: Danilo Krummrich

On 6/11/24 12:37, Dr. David Alan Gilbert wrote: > * Danilo Krummrich (dakr at redhat.com) wrote: >> On 5/18/24 01:26, linux at treblig.org wrote: >>> From: "Dr. David Alan Gilbert" <linux at treblig.org> >>> >>> 'init_exec' is unused since >>> commit cb75d97e9c77 ("drm/nouveau: implement devinit subdev, and new >>>

Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu> --- Tested on a PowerMac7,3 with a NV34 AGP adapter. drm/nouveau/nvkm/subdev/bios/priv.h | 3 +++ drm/nouveau/nvkm/subdev/bios/shadow.c | 27 ++++++++++++++++++--------- drm/nouveau/nvkm/subdev/bios/shadowof.c | 18 ++++++++++++++++-- 3 files changed, 37 insertions(+), 11 deletions(-) diff --git

On Fri, Oct 2, 2015 at 3:18 AM, Hans de Goede <hdegoede at redhat.com> wrote: > Hi, > > On 02-10-15 05:41, Ilia Mirkin wrote: > > <nothing> > > As someone who has recently started following nouveau I must say that > it would greatly help me (and likely others) if patches likes this would > come with a somewhat more descriptive commit message. Duly noted. I

Signed-off-by: Marcin Slusarz <marcin.slusarz at gmail.com> --- drivers/gpu/drm/nouveau/core/core/parent.c | 2 ++ drivers/gpu/drm/nouveau/core/include/core/parent.h | 3 +++ 2 files changed, 5 insertions(+) diff --git a/drivers/gpu/drm/nouveau/core/core/parent.c b/drivers/gpu/drm/nouveau/core/core/parent.c index a1ea034..0e7733c 100644 ---

Upcoming mobile Kepler GPUs (such as GK20A) use the platform bus instead of PCI to which Nouveau is tightly dependent. This patch allows Nouveau to handle platform devices by: - abstracting PCI-dependent functions that were typically used for resource querying and page mapping, - introducing a nv_device_is_pci() function that allows to make PCI-dependent code conditional, - providing a

Signed-off-by: Roy Spliet <rspliet at eclipso.eu> --- drivers/gpu/drm/nouveau/core/include/subdev/pwr.h | 2 + drivers/gpu/drm/nouveau/core/subdev/fb/ramfuc.h | 16 ++ drivers/gpu/drm/nouveau/core/subdev/fb/ramnva3.c | 318 +++++++++++++++++++-- .../gpu/drm/nouveau/core/subdev/pwr/fuc/memx.fuc | 111 +++++++ drivers/gpu/drm/nouveau/core/subdev/pwr/fuc/os.h | 5 +

From: Colin Ian King <colin.king at canonical.com> Don't populate arrays hwsq_signature and edid_sig on the stack but instead make them static. Makes the object code smaller by over 190 bytes: Before: text data bss dec hex filename 35676 3312 64 39052 988c nouveau_bios.o After: text data bss dec hex filename 35319 3472 64

Absence of a TMDS Info Table is common on Optimus setups where the NVIDIA gpu is not connected directly to any outputs. Reporting an error in this scenario is too harsh. Accordingly, change the error message to an info message. By default the error message also causes a boot flicker for these sytems. Signed-off-by: Rhys Kidd <rhyskidd at gmail.com> ---

Optimus cards with no outputs trigger this message (NVD7, for example) This commit changes the warning into an informational message. The return value of the function is not used, so I left it at -EINVAL. Signed-off-by: Albert Pool <albertpool at solcon.nl> Cc: Emil Velikov <emil.l.velikov at gmail.com> --- drivers/gpu/drm/nouveau/nouveau_bios.c | 3 ++- 1 file changed, 2

A 2 minute google search shows that this error is prevalent with mobile Nvidia GPUs. Since we can assume mobile parts have no fp table, demote the error to desktop cards. Also reword the comment since the error is not restricted to Apple hardware. Signed-off by: Rosen Penev <rosenp at gmail.com> --- drm/nouveau/nouveau_bios.c | 10 +++++----- 1 file changed, 5 insertions(+), 5

The nv31m in bug 23212 claims its TV-out and LVDS are in the same connector. Ignore it completely as it's otherwise useless. Signed-off-by: Francisco Jerez <currojerez at riseup.net> --- drivers/gpu/drm/nouveau/nouveau_bios.c | 11 +++++++++++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_bios.c

It looks like Ryan's Nouveau patch [1,2] to get the I2C working for EDID (thus modelines) is crucial for avoiding the Nvidia binary drivers on MacbookPro 2012s. Any plans/chance for it to be upstreamed to eg 3.6-rc3? Many thanks, Daniel --- [1] diff -urNp a/drivers/gpu/drm/nouveau/nouveau_bios.c b/drivers/gpu/drm/nouveau/nouveau_bios.c --- a/drivers/gpu/drm/nouveau/nouveau_bios.c

On Thu, Mar 05, 2020 at 04:59:29PM +0100, Thomas Zimmermann wrote: > The arc driver uses empty implementations for its encoders. Replace > the code with the generic simple encoder. We should , as a follow-up patch, embed the encoder in arcgpu_drm_private. Then we drop the kzalloc() and avoid that life-time challenge. This patch looks good for what it does. Acked-by: Sam Ravnborg <sam

The arc driver uses empty implementations for its encoders. Replace the code with the generic simple encoder. Signed-off-by: Thomas Zimmermann <tzimmermann at suse.de> --- drivers/gpu/drm/arc/arcpgu_hdmi.c | 10 +++------- drivers/gpu/drm/arc/arcpgu_sim.c | 8 ++------ 2 files changed, 5 insertions(+), 13 deletions(-) diff --git a/drivers/gpu/drm/arc/arcpgu_hdmi.c

The nouveau driver makes a number of unaligned accesses via the ROM16(), ROM32() and ROM64() macros which fault on SPARC (but would be transparently handled by x86 hardware). Making use of get_unaligned() macro fixes the problem for me. diff --git a/drivers/gpu/drm/nouveau/nouveau_bios.h b/drivers/gpu/drm/nouveau/nouveau_bios.h index 7ccd28f..92031f6 100644 ---