Re: [PATCH] dt-bindings: display: Drop unneeded quotes

2023-03-18 Thread Laurent Pinchart
Hi Rob, Thank you for the patch. On Fri, Mar 17, 2023 at 06:36:24PM -0500, Rob Herring wrote: > Cleanup bindings dropping unneeded quotes. Once all these are fixed, > checking for this can be enabled in yamllint. > > Signed-off-by: Rob Herring Reviewed-by: Laurent Pinchart > --- > .../bindi

[PATCH] drm/kmb: remove unused set_test_mode_src_osc_freq_target_low, hi_bits functions

2023-03-18 Thread Tom Rix
clang with W=1 reports drivers/gpu/drm/kmb/kmb_dsi.c:822:2: error: unused function 'set_test_mode_src_osc_freq_target_low_bits' [-Werror,-Wunused-function] set_test_mode_src_osc_freq_target_low_bits(struct kmb_dsi *kmb_dsi, ^ drivers/gpu/drm/kmb/kmb_dsi.c:834:2: error: unused func

[PATCH v6 0/9] Fix DSI host idx detection on HW revision clash

2023-03-18 Thread Konrad Dybcio
v5 -> v6: - Squash both fixes that concerned the deprecated QCM2290 compatible to avoid warnings v5: https://lore.kernel.org/r/20230307-topic-dsi_qcm-v5-0-9d4235b77...@linaro.org v4 -> v5: - Drop superfluous items: level in [8/10] - Remove the header define for the qcm2290 config in [6/10] ins

[PATCH v6 1/9] dt-bindings: display/msm: dsi-controller-main: Fix deprecated QCM2290 compatible

2023-03-18 Thread Konrad Dybcio
The point of the previous cleanup was to disallow "qcom,mdss-dsi-ctrl" alone. This however didn't quite work out and the property became undocumented instead of deprecated. Fix that. Additionally, the "qcom," prefix was missed previously. Fix it. Fixes: 0c0f65c6dd44 ("dt-bindings: msm: dsi-contro

[PATCH v6 4/9] drm/msm/dsi: dsi_cfg: Deduplicate identical structs

2023-03-18 Thread Konrad Dybcio
Some structs were defined multiple times for no apparent reason. Deduplicate them. Reviewed-by: Dmitry Baryshkov Reviewed-by: Marijn Suijten Signed-off-by: Konrad Dybcio --- drivers/gpu/drm/msm/dsi/dsi_cfg.c | 93 +-- 1 file changed, 30 insertions(+), 63 del

[PATCH v6 2/9] drm/msm/dsi: Get rid of msm_dsi_config::num_dsi

2023-03-18 Thread Konrad Dybcio
In preparation for supporting multiple sets of possible base registers, remove the num_dsi variable. We're comparing the io_start array contents with the reg value from the DTS, so it will either match one of the expected values or don't match against a zero (which we get from partial array initial

[PATCH v6 7/9] drm/msm/dsi: Remove custom DSI config handling

2023-03-18 Thread Konrad Dybcio
Now that the only user is handled by common code, remove the option to specify custom handlers through match data. This is effectively a revert of commit: 5ae15e76271 ("drm/msm/dsi: Allow to specify dsi config as pdata") Reviewed-by: Dmitry Baryshkov Reviewed-by: Marijn Suijten Signed-off-by: K

[PATCH v6 6/9] drm/msm/dsi: Switch the QCM2290-specific compatible to index autodetection

2023-03-18 Thread Konrad Dybcio
Now that the logic can handle multiple sets of registers, move the QCM2290 to the common logic and mark it deprecated. This allows us to remove a couple of structs, saving some memory. Reviewed-by: Dmitry Baryshkov Reviewed-by: Marijn Suijten Signed-off-by: Konrad Dybcio --- drivers/gpu/drm/ms

[PATCH v6 3/9] drm/msm/dsi: Fix DSI index detection when version clash occurs

2023-03-18 Thread Konrad Dybcio
Currently, we allow for MAX_DSI entries in io_start to facilitate for MAX_DSI number of DSI hosts at different addresses. The configuration is matched against the DSI CTRL hardware revision read back from the component. We need a way to resolve situations where multiple SoCs with different register

[PATCH v6 5/9] drm/msm/dsi: dsi_cfg: Merge SC7180 config into SDM845

2023-03-18 Thread Konrad Dybcio
The configs are identical, other than the number of *maximum* DSI hosts allowed. This isn't an issue, unless somebody deliberately tries to access the inexistent host by adding a dt node for it. Remove the SC7180 struct and point the hw revision match to the SDM845's one. On a note, this could hav

[PATCH v6 8/9] dt-bindings: display/msm: dsi-controller-main: Add SM6115

2023-03-18 Thread Konrad Dybcio
Add a compatible for the DSI on SM6115. Acked-by: Rob Herring Reviewed-by: Marijn Suijten Signed-off-by: Konrad Dybcio --- .../devicetree/bindings/display/msm/dsi-controller-main.yaml | 2 ++ .../devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml | 10 -- 2 files changed, 10

[PATCH v6 9/9] arm64: dts: qcom: sm6115: Use the correct DSI compatible

2023-03-18 Thread Konrad Dybcio
Use the non-deprecated, SoC-specific DSI compatible. Reviewed-by: Dmitry Baryshkov Reviewed-by: Marijn Suijten Signed-off-by: Konrad Dybcio --- arch/arm64/boot/dts/qcom/sm6115.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/boot/dts/qcom/sm6115.dtsi b/arch/

[PATCH] Revert "drm/i915/hwmon: Enable PL1 power limit"

2023-03-18 Thread Ashutosh Dixit
This reverts commit ee892ea83d99610fa33bea612de058e0955eec3a. 0349c41b0596 ("drm/i915/hwmon: Enable PL1 power limit") was reverted in 05d5562e401e ("Revert "drm/i915/hwmon: Enable PL1 power limit"") but has appeared again as ee892ea83d99 ("drm/i915/hwmon: Enable PL1 power limit"). Revert it again.

Re: Reverted patch reappears!

2023-03-18 Thread Dixit, Ashutosh
On Fri, 17 Mar 2023 20:28:58 -0700, Dixit, Ashutosh wrote: > > Jani/Rodrigo, > > Original Subject: Re: [Intel-gfx] [PATCH] Revert "drm/i915/hwmon: Enable PL1 > power limit" > > On Wed, 15 Feb 2023 09:19:07 -0800, Rodrigo Vivi wrote: > > > > On Wed, Feb 15, 2023 at 08:24:51AM -0800, Dixit, Ashutosh

Re: [Intel-gfx] [PATCH v10 09/15] drm/syncobj: Add deadline support for syncobj waits

2023-03-18 Thread Rob Clark
On Fri, Mar 17, 2023 at 12:08 PM Faith Ekstrand wrote: > > > On Wed, Mar 8, 2023 at 9:54 AM Rob Clark wrote: >> >> From: Rob Clark >> >> Add a new flag to let userspace provide a deadline as a hint for syncobj >> and timeline waits. This gives a hint to the driver signaling the >> backing fence

Re: [REGRESSION] Missing nouveau backlight control on 6.2.x kernel

2023-03-18 Thread Hans de Goede
Hi Takashi, On 3/17/23 11:04, Takashi Iwai wrote: > Hi, > > we've received a regression report on openSUSE Bugzilla about the > missing backlight control of nouveau device: > https://bugzilla.opensuse.org/show_bug.cgi?id=1209296 > > On 6.1, with acpi_video=native option, the system provided >

[PATCH 04/19] drm/bridge: display-connector: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 11/19] drm/bridge: lvds-codec: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 12/19] drm/bridge: nwl-dsi: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 15/19] drm/bridge: dw-hdmi-cec: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 09/19] drm/bridge: imx8qxp-pixel-link: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 02/19] drm/bridge: cdns-dsi: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 14/19] drm/bridge: dw-hdmi-ahb-audio: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 13/19] drm/bridge: simple-bridg: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 08/19] drm/bridge: imx8qxp-pixel-combiner: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 19/19] drm/bridge: ti-tfp410: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 17/19] drm/bridge: dw-hdmi-i2s-audio: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 05/19] drm/bridge: fsl-ldb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 03/19] drm/bridge: cdns-mhdp8546: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 18/19] drm/bridge: thc63lvd1024: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 10/19] drm/bridge: imx8qxp-pxl2dpi: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 16/19] drm/bridge: dw-hdmi-gp-audio: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 06/19] drm/bridge: imx8qm-ldb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 01/19] drm/bridge: cdns-mhdp8546: Improve error reporting in remove callback

2023-03-18 Thread Uwe Kleine-König
Replace the generic error message issued by the driver core when the remove callback returns non-zero ("remove callback returned a non-zero value. This will be ignored.") by a message that tells the actual problem. Also simplify a bit by checking the return value of wait_event_timeout a bit later.

[PATCH 00/19] drm/bridge: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
Hello, this series adapts the platform drivers below drivers/gpu/drm/bridge to use the .remove_new() callback. Compared to the traditional .remove() callback .remove_new() returns no value. This is a good thing because the driver core doesn't (and cannot) cope for errors during remove. The only ef

[PATCH 07/19] drm/bridge: imx8qxp-ldb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH v3 0/2] Some debugfs refactoring and improvements

2023-03-18 Thread Andi Shyti
Hi, These two patches aim to enhance the multi-GT capabilities of the debugfs. The first patch reorganizes the file structure, while the second patch extends the functionality of the original files in the upper directories to operate on all tiles with a single write, providing an or'ed value amon

[PATCH v3 1/2] drm/i915/gt: Create per-gt debugfs files

2023-03-18 Thread Andi Shyti
To support multi-GT configurations, we need to generate independent debug files for each GT. To achieve this create a separate directory for each GT under the debugfs directory. For instance, in a system with two GTs, the debugfs structure would look like this: /sys/kernel/debug/dri

[PATCH v3 1/2] drm/i915/gt: Create per-tile debugfs files

2023-03-18 Thread Andi Shyti
To support multi-GT configurations, we need to generate independent debug files for each GT. To achieve this create a separate directory for each GT under the debugfs directory. For instance, in a system with two tiles, the debugfs structure would look like this: /sys/kernel/debug/dri

[PATCH v3 2/2] drm/i915/debugfs: Enable upper layer interfaces to act on all gt's

2023-03-18 Thread Andi Shyti
The commit 82a149a62b6b5 ('drm/i915/gt: move remaining debugfs interfaces into gt') moved gt-related debugfs files in the gtX/ directories to operate on individual gt's. However, the original files were only functioning on the root GT (GT 0) and have been left in the same location to maintain comp

[GIT PULL] fbdev fixes for v6.3-rc3

2023-03-18 Thread Helge Deller
Hi Linus, please pull the latest fbdev updates and fixes. The majority of lines changed is due to a code style cleanup in the pnmtologo helper program. Arnd removed the omap1 osk driver and the SIS fb driver is now orphaned. Other than that it's the usual bunch of small fixes and cleanups, e.g.

Re: [GIT PULL] fbdev fixes for v6.3-rc3

2023-03-18 Thread pr-tracker-bot
The pull request you sent on Sat, 18 Mar 2023 21:47:06 +0100: > http://git.kernel.org/pub/scm/linux/kernel/git/deller/linux-fbdev.git > tags/fbdev-for-6.3-rc3 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/a3671bd86a9770e34969522d29bb30a1b66fd88a Thank you! -- Deet

[PATCH 02/51] video: fbdev: arcfb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 04/51] video: fbdev: au1200fb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 09/51] video: fbdev: cg6: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 07/51] video: fbdev: cg14: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 01/51] video: fbdev: au1100fb: Drop if with an always false condition

2023-03-18 Thread Uwe Kleine-König
The driver core never calls a remove callback with the platform_device pointer being NULL. So the check for this condition can just be dropped. Signed-off-by: Uwe Kleine-König --- drivers/video/fbdev/au1100fb.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/video/fbdev/au1100fb.c

[PATCH 18/51] video: fbdev: goldfishfb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 32/51] video: fbdev: platinumfb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 10/51] video: fbdev: clps711x-fb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 27/51] video: fbdev: mx3fb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 25/51] video: fbdev: mb862xx: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 03/51] video: fbdev: au1100fb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 24/51] video: fbdev: leo: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 33/51] video: fbdev: pxa168fb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 16/51] video: fbdev: fsl-diu-fb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 45/51] video: fbdev: vfb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 12/51] video: fbdev: da8xx-fb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 17/51] video: fbdev: gbefb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 31/51] video: fbdev: p9100: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 08/51] video: fbdev: cg3: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 37/51] video: fbdev: s3c-fb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 43/51] video: fbdev: uvesafb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 51/51] video: fbdev: xilinxfb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 40/51] video: fbdev: simplefb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 28/51] video: fbdev: ocfb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 44/51] video: fbdev: vesafb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 05/51] video: fbdev: broadsheetfb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 26/51] video: fbdev: metronomefb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 20/51] video: fbdev: hecubafb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 35/51] video: fbdev: pxafb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 42/51] video: fbdev: tcx: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 49/51] video: fbdev: wm8505fb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 46/51] video: fbdev: vga16fb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 48/51] video: fbdev: vt8500lcdfb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 41/51] video: fbdev: sm501fb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 38/51] video: fbdev: sh7760fb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 34/51] video: fbdev: pxa3xx-gcu: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 11/51] video: fbdev: cobalt_lcdfb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 19/51] video: fbdev: grvga: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 14/51] video: fbdev: ep93xx-fb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 06/51] video: fbdev: bw2: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 29/51] video: fbdev: offb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 39/51] video: fbdev: sh_mobile_lcdcfb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 23/51] video: fbdev: imxfb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 13/51] video: fbdev: efifb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 47/51] video: fbdev: via: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 50/51] video: fbdev: wmt_ge_rops: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 15/51] video: fbdev: ffb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 36/51] video: fbdev: s1d13xxxfb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 22/51] video: fbdev: hitfb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 30/51] video: fbdev: omapfb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 21/51] video: fbdev: hgafb: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ignored and this typically results in resource leaks. To improve here there is a quest to

[PATCH 00/51] video: fbdev: Convert to platform remove callback returning void

2023-03-18 Thread Uwe Kleine-König
Hello, this series adapts the platform drivers below drivers/video/fbdev to use the .remove_new() callback. Compared to the traditional .remove() callback .remove_new() returns no value. This is a good thing because the driver core doesn't (and cannot) cope for errors during remove. The only effec

Re: [PATCH v4 2/8] kunit: drm/tests: move generic helpers

2023-03-18 Thread Matti Vaittinen
Hi Maxime & All First of all - I am sorry. During the last minute rebase I accidentally dropped the header file from this series. Will fix that for v5. (Also the build bot pointed this mistake). On 3/17/23 17:09, Maxime Ripard wrote: Hi Matti, On Fri, Mar 17, 2023 at 04:42:25PM +0200, Matti