- 28 Oct, 2020 2 commits
-
-
Peilin Ye authored
`width` and `height` are defined as unsigned in our UAPI font descriptor `struct console_font`. Make them unsigned in our kernel font descriptor `struct font_desc`, too. Also, change the corresponding printk() format identifiers from `%d` to `%u`, in sti_select_fbfont(). Signed-off-by:
Peilin Ye <yepeilin.cs@gmail.com> Signed-off-by:
Daniel Vetter <daniel.vetter@ffwll.ch> Link: https://patchwork.freedesktop.org/patch/msgid/20201028105647.1210161-1-yepeilin.cs@gmail.com
-
Peilin Ye authored
Remove 6 unused extern variables to reduce confusion. It is worth mentioning that lib/fonts/font_8x8.c and lib/fonts/font_8x16.c also declare `fontdata_8x8` and `fontdata_8x16` respectively, and this file has nothing to do with them. Signed-off-by:
-
- 27 Oct, 2020 3 commits
-
-
Arnd Bergmann authored
The -Wmissing-field-initializer warning when building with W=2 turns into an error because tilcdc is built with -Werror: drm/tilcdc/tilcdc_drv.c:431:33: error: missing field 'data' initializer [-Werror,-Wmissing-field-initializers] { "regs", tilcdc_regs_show, 0 }, drm/tilcdc/tilcdc_drv.c:432:33: error: missing field 'data' initializer [-Werror,-Wmissing-field-initializers] { "mm", tilcdc_mm_show, 0 }, Add the missing field initializers to address the warning. Signed-off-by:Arnd Bergmann <arnd@arndb.de> Signed-off-by:
Jyri Sarha <jsarha@ti.com> Link: https://patchwork.freedesktop.org/patch/msgid/20201026194110.3817470-1-arnd@kernel.org
-
Daniel Vetter authored
It's the horror and shouldn't be used. Realized we're not clear on this in a discussion with Rob about what msm is doing to better support async commits. v2: Refine existing todo item to include this (Thomas) Cc: Sean Paul <sean@poorly.run> Cc: Rob Clark <robdclark@gmail.com> Acked-by:
Rob Clark <robdclark@gmail.com> Signed-off-by:
Daniel Vetter <daniel.vetter@intel.com> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Cc: Maxime Ripard <mripard@kernel.org> Cc: Thomas Zimmermann <tzimmermann@suse.de> Cc: David Airlie <airlied@linux.ie> Cc: Daniel Vetter <daniel@ffwll.ch> Link: https://patchwork.freedesktop.org/patch/msgid/20201023123925.2374863-3-daniel.vetter@ffwll.ch
-
Arnd Bergmann authored
The open-coded list_for_each_entry() causes a harmless warning: drivers/video/fbdev/matrox/matroxfb_base.c: In function 'matroxfb_register_driver': include/linux/kernel.h:856:3: warning: array subscript -98 is outside array bounds of 'struct list_head[1]' [-Warray-bounds] Use the normal list_for_each_entry instead. Fixes: 1da177e4 ("Linux-2.6.12-rc2") Signed-off-by:
Thomas Zimmermann <tzimmermann@suse.de> Link: https://patchwork.freedesktop.org/patch/msgid/20201026194010.3817166-1-arnd@kernel.org
-
- 26 Oct, 2020 5 commits
-
-
Simon Ser authored
DRM_FORMAT_MOD_NONE is in the list of vendors, which is pretty confusing. We already have DRM_FORMAT_MOD_VENDOR_NONE. Move it down in the list of format modifiers. DRM_FORMAT_MOD_NONE is an alias for DRM_FORMAT_MOD_LINEAR, however the name is confusing: NONE doesn't mean that the modifier is implicit, instead it means that the layout is linear. Deprecate it. Signed-off-by:
Simon Ser <contact@emersion.fr> Reviewed-by:
Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> Reviewed-by:
Pekka Paalanen <pekka.paalanen@collabora.com> Acked-by:
-
Christian König authored
It makes no difference to kmalloc if the structure is 48 or 64 bytes in size. Signed-off-by:
Christian König <christian.koenig@amd.com> Reviewed-by:
Dave Airlie <airlied@redhat.com> Link: https://patchwork.freedesktop.org/patch/396950/
-
Christian König authored
We can still allocate 16TiB with that. Signed-off-by:
-
Christian König authored
This is not related to allocating the backing store in any way. Signed-off-by:
-
Christian König authored
Neither page allocation backend nor the driver should mess with that. Signed-off-by:
Madhav Chauhan <madhav.chauhan@amd.com> Link: https://patchwork.freedesktop.org/patch/396948/
-
- 23 Oct, 2020 2 commits
-
-
Qinglang Miao authored
Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code. Signed-off-by:
Qinglang Miao <miaoqinglang@huawei.com> Acked-by:
Liviu Dudau <liviu.dudau@arm.com> Tested-by:
Carsten Haitzler <carsten.haitzler@foss.arm.com> Signed-off-by:
-
Saeed Mirzamohammadi authored
This patch fixes the issue due to: [ 89.572883] divide_error: 0000 [#1] SMP KASAN PTI [ 89.572897] CPU: 3 PID: 16083 Comm: repro Not tainted 5.9.0-rc7.20200930.rc1.allarch-19-g3e32d0d.syzk #5 [ 89.572902] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 0.5.1 01/01/2011 [ 89.572934] RIP: 0010:cirrusfb_check_var+0x84/0x1260 The error happens when the pixels value is calculated before performing the sanity checks on bits_per_pixel. A bits_per_pixel set to zero causes divide by zero error. This patch moves the calculation after the sanity check. Signed-off-by:
Saeed Mirzamohammadi <saeed.mirzamohammadi@oracle.com> Tested-by:
-
- 22 Oct, 2020 5 commits
-
-
Dave Airlie authored
remove the overlapping memcp support as it's never used. Reviewed-by:
-
Maxime Ripard authored
If the DSI driver is the last to probe, component_add will try to run all the bind callbacks straight away and return the error code. However, since we depend on a power domain, we're pretty much guaranteed to be in that case on the BCM2711, and are just lucky on the previous SoCs since the v3d also depends on that power domain and is further in the probe order. In that case, the DSI host will not stick around in the system: the DSI bind callback will be executed, will not find any DSI device attached and will return EPROBE_DEFER, and we will then remove the DSI host and ask to be probed later on. But since that host doesn't stick around, DSI devices like the RaspberryPi touchscreen whose probe is not linked to the DSI host (unlike the usual DSI devices that will be probed through the call to mipi_dsi_host_register) cannot attach to the DSI host, and we thus end up in a situation where the DSI host cannot probe because the panel hasn't probed yet, and the panel cannot probe because the DSI host hasn't yet. In order to break this cycle, let's wait until there's a DSI device that attaches to the DSI host to register the component and allow to progress further. Acked-by:
Eric Anholt <eric@anholt.net> Suggested-by:
Andrzej Hajda <a.hajda@samsung.com> Signed-off-by:
Maxime Ripard <maxime@cerno.tech> Link: https://patchwork.freedesktop.org/patch/msgid/20200707101912.571531-1-maxime@cerno.tech
-
Simon Ser authored
User-space should avoid parsing EDIDs for metadata already exposed via other KMS interfaces and properties. For instance, user-space should not try to extract a list of modes from the EDID: the kernel might mutate the mode list (because of link capabilities or quirks for instance). Other metadata not exposed by KMS can be parsed by user-space. This includes for instance monitor identification (make/model/serial) and supported color-spaces. v2: add short explanation why user-space shouldn't do this (Brian) Signed-off-by:
Brian Starkey <brian.starkey@arm.com> Acked-by:
-
Dave Airlie authored
The move notify callback is only used in one place, this should be removed in the future, but for now just rename it to the use case which is to notify the driver that the GPU memory is to be deleted. Drivers can be cleaned up after this separately. Reviewed-by:
-
Dave Airlie authored
Removed unused parameter. Reviewed-by:
-
- 21 Oct, 2020 8 commits
-
-
Daniel Vetter authored
It's the only ioctl handler purely for legacy drivers that didn't have this yet. Reviewed-by:
Alex Deucher <alexander.deucher@amd.com> Signed-off-by:
-
Dave Airlie authored
This moves the call to tt binding into the driver move, and drops the driver callback. Reviewed-by:
-
Dave Airlie authored
The drivers now do this in the move callback. move_notify is still needed in the destroy path. Reviewed-by:
-
Dave Airlie authored
This show the remaining bind callback, which my next series of patches will aim to remove. Reviewed-by:
-
Dave Airlie authored
The drivers now control this, so drop unbinding. Reviewed-by:
-
Dave Airlie authored
This moves the to system move into the drivers, and moves all the unbinds in the move path under driver control Note: radeon/nouveau already wait so don't duplicate it. Reviewed-by:
-
Dave Airlie authored
resource free already sets the domain to system, and old_mem isn't really needed. Reviewed-by:
-
Dave Airlie authored
This just gives the driver control over some of the bind paths. Reviewed-by:
-
- 20 Oct, 2020 3 commits
-
-
Matteo Franchin authored
Add ABGR format with 10-bit components packed in 64-bit per pixel. This format can be used to handle VK_FORMAT_R10X6G10X6B10X6A10X6_UNORM_4PACK16 on little-endian architectures. Signed-off-by:
Matteo Franchin <matteo.franchin@arm.com> Reviewed-by:
-
Robin Murphy authored
Since commit 9495b7e9 ("driver core: platform: Initialize dma_parms for platform devices"), struct platform_device already provides a dma_parms structure, so we can save allocating another one. Signed-off-by:
Robin Murphy <robin.murphy@arm.com> Reviewed-by:
James Qian Wang <james.qian.wang@arm.com> [replaced DMA_BIT_MASK(32) with U32_MAX] Signed-off-by:
-
Daniel Vetter authored
Hooray for generic fbdev support, making this a oneliner. We just needed to fix preferred_depth fixed and the vmap support added first. This is useful for testing fbdev testcases in igt. Right now we only have a simple one in the fbdev testcases, which passes both info and mmap subtests. v2: Augment commit message as suggested by Melissa. Acked-by:
Melissa Wen <melissa.srw@gmail.com> Signed-off-by:
-
- 19 Oct, 2020 12 commits
-
-
Dave Airlie authored
The apis to move old/new are in place everywhere so this is no longer needed. Reviewed-by:
-
Dave Airlie authored
Uninline ttm_bo_move_ttm. Eventually want to unhook the unbind out. Reviewed-by:
Ben Skeggs <bskeggs@redhat.com> Reviewed-by:
-
Dave Airlie authored
In all 3 drivers there is a case where the driver knows the bo is in SYSTEM so don't call the api that checks that. Reviewed-by:
-
Dave Airlie authored
Reviewed-by:
-
Dave Airlie authored
This factors out the code to setup non-system tt. The same code was used twice in the move paths. Reviewed-by:
-
Neil Armstrong authored
This adds support for the TDO TL070WSH30 TFT-LCD panel module. The panel has a 1024×600 resolution and uses 24 bit RGB per pixel. It provides a MIPI DSI interface to the host, a built-in LED backlight and touch controller. Signed-off-by:
Neil Armstrong <narmstrong@baylibre.com> Reviewed-by:
Sam Ravnborg <sam@ravnborg.org> Link: https://patchwork.freedesktop.org/patch/msgid/20200915121912.4347-5-narmstrong@baylibre.com
-
Neil Armstrong authored
This add the bindings for the 1024x600 TFT LCD TL070WSH30 DSI panel to panel-simple-dsi. Signed-off-by:
Rob Herring <robh@kernel.org> Link: https://patchwork.freedesktop.org/patch/msgid/20200915121912.4347-4-narmstrong@baylibre.com
-
Neil Armstrong authored
Simple DSI panels can also have a reset GPIO signal in addition/instead of an enable GPIO signal. This adds an optional reset-gpios property. Signed-off-by:
-
Neil Armstrong authored
Shanghai Top Display Optolelectronics Co., Ltd is a display manufacturer from Shanghai. Web site of the company: http://www.shtdo.com/Signed-off-by:
-
Dave Airlie authored
This isn't really used anymore, if drivers needs it later, just add back an inline wrapper. Reviewed-by:
-
Tian Tao authored
Consistently Use the same style of variable type in hibmc_drm_drv.c and hibmc_drm_drv.h. Signed-off-by:
Tian Tao <tiantao6@hisilicon.com> Acked-by:
-
Tian Tao authored
Consistently Use the same style of variable type in hibmc_drm_de.c. Signed-off-by:
-
