drm: remove driver date from struct drm_driver and all driversWe stopped using the driver initialized date in commit 7fb8af6798e8("drm: deprecate driver date") and (eventually) started returning "
drm: remove driver date from struct drm_driver and all driversWe stopped using the driver initialized date in commit 7fb8af6798e8("drm: deprecate driver date") and (eventually) started returning "0"for drm_version ioctl instead.Finish the job, and remove the unused date member from structdrm_driver, its initialization from drivers, along with the commonDRIVER_DATE macros.v2: Also update drivers/accel (kernel test robot)Reviewed-by: Javier Martinez Canillas <[email protected]>Acked-by: Alex Deucher <[email protected]>Acked-by: Simon Ser <[email protected]>Acked-by: Jeffrey Hugo <[email protected]>Acked-by: Lucas De Marchi <[email protected]>Acked-by: Dmitry Baryshkov <[email protected]> # msmReviewed-by: Thomas Zimmermann <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/1f2bf2543aed270a06f6c707fd6ed1b78bf16712.1733322525.git.jani.nikula@intel.comSigned-off-by: Jani Nikula <[email protected]>
show more ...
Get rid of 'remove_new' relic from platform driver structThe continual trickle of small conversion patches is grating on me, andis really not helping. Just get rid of the 'remove_new' memberfunc
Get rid of 'remove_new' relic from platform driver structThe continual trickle of small conversion patches is grating on me, andis really not helping. Just get rid of the 'remove_new' memberfunction, which is just an alias for the plain 'remove', and had acomment to that effect: /* * .remove_new() is a relic from a prototype conversion of .remove(). * New drivers are supposed to implement .remove(). Once all drivers are * converted to not use .remove_new any more, it will be dropped. */This was just a tree-wide 'sed' script that replaced '.remove_new' with'.remove', with some care taken to turn a subsequent tab into two tabsto make things line up.I did do some minimal manual whitespace adjustment for places that usedspaces to line things up.Then I just removed the old (sic) .remove_new member function, and thisis the end result. No more unnecessary conversion noise.Signed-off-by: Linus Torvalds <[email protected]>
drm/client: Move public client header to clients/ subdirectoryMove the public header file drm_client_setup.h to the clients/subdirectory and update all drivers. No functional changes.Signed-off-
drm/client: Move public client header to clients/ subdirectoryMove the public header file drm_client_setup.h to the clients/subdirectory and update all drivers. No functional changes.Signed-off-by: Thomas Zimmermann <[email protected]>Reviewed-by: Jocelyn Falempe <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
drm/imx/lcdc: Run DRM default client setupCall drm_client_setup() to run the kernel's default client setupfor DRM. Set fbdev_probe in struct drm_driver, so that the clientsetup can start the comm
drm/imx/lcdc: Run DRM default client setupCall drm_client_setup() to run the kernel's default client setupfor DRM. Set fbdev_probe in struct drm_driver, so that the clientsetup can start the common fbdev client.v5:- select DRM_CLIENT_SELECTIONSigned-off-by: Thomas Zimmermann <[email protected]>Cc: Shawn Guo <[email protected]>Cc: Sascha Hauer <[email protected]>Cc: Pengutronix Kernel Team <[email protected]>Cc: Fabio Estevam <[email protected]>Acked-by: Javier Martinez Canillas <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
drm/bridge-connector: move to DRM_DISPLAY_HELPER moduledrm_bridge_connector is a "leaf" driver, belonging to the displayhelper, rather than the "CRTC" drm_kms_helper module. Move the driverto the
drm/bridge-connector: move to DRM_DISPLAY_HELPER moduledrm_bridge_connector is a "leaf" driver, belonging to the displayhelper, rather than the "CRTC" drm_kms_helper module. Move the driverto the drm/display and add necessary Kconfig selection clauses.Suggested-by: Maxime Ripard <[email protected]>Acked-by: Maxime Ripard <[email protected]>Signed-off-by: Dmitry Baryshkov <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/20240903-drm-bridge-connector-fix-hdmi-reset-v5-2-daebde6d9857@linaro.orgSigned-off-by: Maxime Ripard <[email protected]>
drm/imx/lcdc: Use fbdev-dmaImplement fbdev emulation with fbdev-dma. Fbdev-dma now supportsdamage handling, which is required by lcdc. Avoids the overhead offbdev-generic's additional shadow buff
drm/imx/lcdc: Use fbdev-dmaImplement fbdev emulation with fbdev-dma. Fbdev-dma now supportsdamage handling, which is required by lcdc. Avoids the overhead offbdev-generic's additional shadow buffering. No functional changes.Signed-off-by: Thomas Zimmermann <[email protected]>Cc: Shawn Guo <[email protected]>Cc: Sascha Hauer <[email protected]>Cc: Pengutronix Kernel Team <[email protected]>Cc: Fabio Estevam <[email protected]>Cc: NXP Linux Team <[email protected]>Reviewed-by: Javier Martinez Canillas <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
drm/imx/lcdc: Fix double-free of driver dataThe struct imx_lcdc driver data is allocated using devm_drm_dev_alloc()so it must not be explicitly kfree()d.Also drm_kms_helper_poll_fini() should no
drm/imx/lcdc: Fix double-free of driver dataThe struct imx_lcdc driver data is allocated using devm_drm_dev_alloc()so it must not be explicitly kfree()d.Also drm_kms_helper_poll_fini() should not be called as there is nomatching drm_kms_helper_poll_init(). So drop the release functioncompletely.Fixes: c87e859cdeb5 ("drm/imx/lcdc: Implement DRM driver for imx25")Signed-off-by: Uwe Kleine-König <[email protected]>Reviewed-by: Philipp Zabel <[email protected]>Signed-off-by: Philipp Zabel <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
drm/imx: lcdc: Convert to platform remove callback returning voidThe .remove() callback for a platform driver returns an int which makesmany driver authors wrongly assume it's possible to do error
drm/imx: lcdc: Convert to platform remove callback returning voidThe .remove() callback for a platform driver returns an int which makesmany driver authors wrongly assume it's possible to do error handling byreturning an error code. However the value returned is ignored (apartfrom emitting a warning) and this typically results in resource leaks.To improve here there is a quest to make the remove callback returnvoid. In the first step of this quest all drivers are converted to.remove_new(), which already returns void. Eventually after all driversare converted, .remove_new() will be renamed to .remove().Trivially convert this driver from always returning zero in the removecallback to the void returning variant.Signed-off-by: Uwe Kleine-König <[email protected]>Signed-off-by: Thomas Zimmermann <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
drm: Explicitly include correct DT includesThe DT of_device.h and of_platform.h date back to the separateof_platform_bus_type before it as merged into the regular platform bus.As part of that mer
drm: Explicitly include correct DT includesThe DT of_device.h and of_platform.h date back to the separateof_platform_bus_type before it as merged into the regular platform bus.As part of that merge prepping Arm DT support 13 years ago, they"temporarily" include each other. They also include platform_device.hand of.h. As a result, there's a pretty much random mix of those includefiles used throughout the tree. In order to detangle these headers andreplace the implicit includes with struct declarations, users need toexplicitly include the correct includes.Signed-off-by: Rob Herring <[email protected]>Acked-by: Sam Ravnborg <[email protected]>Reviewed-by: Steven Price <[email protected]>Acked-by: Liviu Dudau <[email protected]>Reviewed-by: Kieran Bingham <[email protected]>Acked-by: Robert Foss <[email protected]>Signed-off-by: Thierry Reding <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
drm/imx/lcdc: fix a NULL vs IS_ERR() bug in probeThe devm_drm_dev_alloc() function returns error pointers. It neverreturns NULL. Fix the check.Fixes: c87e859cdeb5 ("drm/imx/lcdc: Implement DRM
drm/imx/lcdc: fix a NULL vs IS_ERR() bug in probeThe devm_drm_dev_alloc() function returns error pointers. It neverreturns NULL. Fix the check.Fixes: c87e859cdeb5 ("drm/imx/lcdc: Implement DRM driver for imx25")Signed-off-by: Dan Carpenter <[email protected]>Reviewed-by: Uwe Kleine-König <[email protected]>Signed-off-by: Javier Martinez Canillas <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
Merge tag 'drm-misc-next-2023-03-16' of git://anongit.freedesktop.org/drm/drm-misc into drm-nextdrm-misc-next for v6.4-rc1:Cross-subsystem Changes:- Add drm_bridge.h to drm_bridge maintainers.
Merge tag 'drm-misc-next-2023-03-16' of git://anongit.freedesktop.org/drm/drm-misc into drm-nextdrm-misc-next for v6.4-rc1:Cross-subsystem Changes:- Add drm_bridge.h to drm_bridge maintainers.Core Changes:- Assorted fixes to TTM, tests, format-helper, accel.- Assorted Makefile fixes to drivers and accel.- Implement fbdev emulation for GEM DMA drivers, and convert a lot of drivers to use it.- Use tgid instead of pid for tracking clients.Driver Changes:- Assorted fixes in rockchip, vmwgfx, nouveau, cirrus.- Add imx25 driver.- Add Elida KD50T048A, Sony TD4353, Novatek NT36523, STARRY 2081101QFH032011-53G panels.- Add 4K mode support to rockchip.- Convert cirrus to use regular atomic helpers, and more cirrus improvements.- Add damage clipping to cirrus, virtio.[airlied: add drm_bridge.h include to imx]Signed-off-by: Dave Airlie <[email protected]>From: Maarten Lankhorst <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
drm/imx/lcdc: Implement DRM driver for imx25Add support for the LCD Controller found on i.MX21 and i.MX25.It targets to be a drop in replacement for the imx-fb driver.[ukl: Rebase to a newer ke
drm/imx/lcdc: Implement DRM driver for imx25Add support for the LCD Controller found on i.MX21 and i.MX25.It targets to be a drop in replacement for the imx-fb driver.[ukl: Rebase to a newer kernel version, various smaller fixes andimprovements]Signed-off-by: Marian Cichy <[email protected]>Reviewed-by: Philipp Zabel <[email protected]>Signed-off-by: Uwe Kleine-König <[email protected]>Link: https://lore.kernel.org/r/[email protected]Signed-off-by: Philipp Zabel <[email protected]>Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]