Re: [PATCH] drm/mediatek: Init `ddp_comp` with devm_kcalloc()

2024-04-02 Thread AngeloGioacchino Del Regno
-initting alloc functions if you really need to. Let's switch the function to use an allocation function that zeros the memory. For me, this avoids the crash. Fixes: 01389b324c97 ("drm/mediatek: Add connector dynamic selection capability") Signed-off-by: Douglas Anderson Reviewed-

Re: [PATCH 00/11] drm/mediatek: drop driver owner initialization

2024-04-02 Thread AngeloGioacchino Del Regno
Il 30/03/24 21:43, Krzysztof Kozlowski ha scritto: Simplify the code by dropping unnecessary .owner initialization in the driver. For the entire series: Reviewed-by: AngeloGioacchino Del Regno Best regards, Krzysztof --- Krzysztof Kozlowski (11): drm/mediatek: aal: drop driver

Re: [PATCH] drm/panfrost: fix power transition timeout warnings

2024-03-25 Thread AngeloGioacchino Del Regno
d it, but not since the values were increased. As long as it’s acceptable I won’t over-think it but if more testing is needed I can look at it more. Thanks for clarifying. Reviewed-by: AngeloGioacchino Del Regno

Re: [PATCH] drm/panfrost: fix power transition timeout warnings

2024-03-25 Thread AngeloGioacchino Del Regno
Il 22/03/24 17:45, Christian Hewitt ha scritto: Increase the timeout value to prevent system logs on Amlogic boards flooding with power transition warnings: [ 13.047638] panfrost ffe4.gpu: shader power transition timeout [ 13.048674] panfrost ffe4.gpu: l2 power transition timeout [

Re: [PATCH v4 03/14] drm/mediatek: Rename "mtk_drm_plane" to "mtk_plane"

2024-03-22 Thread AngeloGioacchino Del Regno
Il 22/03/24 02:27, Shawn Sung ha scritto: From: Hsiao Chien Sung Rename all "mtk_drm_plane" to "mtk_plane": - To align the naming rule - To reduce the code size Reviewed-by: AngeloGiaocchino Del Regno Shawn - please - can you fix my typo'ed name also here and on all of the patches of this

Re: [PATCH 0/3] drm/mediatek: Fixes for DDP component search/destroy

2024-03-21 Thread AngeloGioacchino Del Regno
Il 01/02/24 13:53, AngeloGioacchino Del Regno ha scritto: This series performs some cleanups for DDP component CRTC search and correctly iounmaps the previously of_iomap() calls from drm_ddp_comp. Tested on MT8195 Cherry Tomato Hello CK, gentle ping for this series. Cheers, Angelo

Re: [PATCH v3 09/14] drm/mediatek: Rename files "mtk_drm_ddp_comp.c" to "mtk_ddp_comp.c"

2024-03-20 Thread AngeloGioacchino Del Regno
Il 20/03/24 03:42, Shawn Sung ha scritto: From: Hsiao Chien Sung Rename files mtk_drm_ddp_comp.c to mtk_ddp_comp.c and modify the Makefile accordingly. Reviewed-by: CK Hu Signed-off-by: Hsiao Chien Sung Reviewed-by: AngeloGioacchino Del Regno

Re: [PATCH v3 07/14] drm/mediatek: Rename files "mtk_drm_crtc.c" to "mtk_crtc.c"

2024-03-20 Thread AngeloGioacchino Del Regno
Il 20/03/24 03:42, Shawn Sung ha scritto: From: Hsiao Chien Sung Rename files mtk_drm_crtc.c to mtk_crtc.c and modify the Makefile accordingly. Reviewed-by: CK Hu Signed-off-by: Hsiao Chien Sung Reviewed-by: AngeloGioacchino Del Regno

Re: [PATCH v2 02/14] drm/mediatek: Rename "mtk_drm_ddp_comp" to "mtk_ddp_comp"

2024-03-19 Thread AngeloGioacchino Del Regno
ctually possible, since I write the tags by hand), or what actually happened to my Reviewed-by tags on the entire series. Can you please fix the typo in the tag? Reviewed-by: AngeloGioacchino Del Regno Use this one, please. Thanks, Angelo Reviewed-by: CK Hu Signed-off-by: Hsiao Chien Sung

Re: [PATCH v2 11/14] drm/mediatek: Rename files "mtk_drm_plane.c" to "mtk_plane.c"

2024-03-19 Thread AngeloGioacchino Del Regno
Il 19/03/24 08:02, Shawn Sung ha scritto: From: Hsiao Chien Sung Rename files mtk_drm_plane.c to mtk_plane.c and modify the Makefile accordingly. Signed-off-by: Hsiao Chien Sung Reviewed-by: AngeloGioacchino Del Regno

Re: [PATCH v2] drm/panthor: Fix the CONFIG_PM=n case

2024-03-18 Thread AngeloGioacchino Del Regno
p the #ifdef CONFIG_PM section around panthor_pm_ops's definition Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202403031944.eoimq8wk-...@intel.com/ Signed-off-by: Boris Brezillon Reviewed-by: AngeloGioacchino Del Regno --- Tested by faking CONFIG_PM=n in

Re: [PATCH v1 1/1] drm/mediatek/ dp: Adjust bandwidth limit for DP

2024-03-18 Thread AngeloGioacchino Del Regno
Il 15/03/24 02:52, Liankun Yang ha scritto: By adjusting the order of link training and relocating it to HPD, link training can identify the usability of each lane in the current link. It also supports handling signal instability and weakness due to environmental issues, enabling the acquisition

Re: [PATCH] mediatek: dsi: Correct calculation formula of PHY Timing

2024-03-15 Thread AngeloGioacchino Del Regno
Il 15/03/24 08:29, Shuijing Li ha scritto: This patch correct calculation formula of PHY timing. Make actual phy timing more accurate. More accurate in which cases? By how much? On which SoC(s)? I agree about those changes if those are improving the PHY timing, but can you please document wha

Re: [PATCH] mediatek: dsi: Add dsi per-frame lp code for mt8188

2024-03-14 Thread AngeloGioacchino Del Regno
Il 14/03/24 10:41, Shuijing Li ha scritto: Adding the per-frame lp function of mt8188, which can keep HFP in HS and reduce the time required for each line to enter and exit low power. Per Frame LP: |<--One Active Frame>| --___

Re: [PATCH 12/18] ASoC: codecs: mt6357: add MT6357 codec

2024-03-12 Thread AngeloGioacchino Del Regno
Il 12/03/24 15:50, Alexandre Mergnat ha scritto: On 26/02/2024 16:25, AngeloGioacchino Del Regno wrote: +    if (enable) { +    /* set gpio mosi mode */ +    regmap_write(priv->regmap, MT6357_GPIO_MODE2_CLR, GPIO_MODE2_CLEAR_ALL); +    regmap_write(priv->

Re: [PATCH] drm/mediatek: Add 0 size check to mtk_drm_gem_obj

2024-03-07 Thread AngeloGioacchino Del Regno
buffer on an MT8188 and verifying that we now return EINVAL. Signed-off-by: Justin green You need a Fixes tag for this one. Please add the right one and resend. After adding the correct Fixes tag, Reviewed-by: AngeloGioacchino Del Regno Cheers, Angelo --- drivers/gpu/drm/mediatek

Re: [PATCH v2 9/9] drm/panel: truly-nt35597: Don't log an error when DSI host can't be found

2024-03-01 Thread AngeloGioacchino Del Regno
Il 01/03/24 09:56, Laurent Pinchart ha scritto: On Fri, Mar 01, 2024 at 09:44:36AM +0100, AngeloGioacchino Del Regno wrote: Il 01/03/24 07:30, Laurent Pinchart ha scritto: On Thu, Feb 29, 2024 at 07:12:15PM -0500, Nícolas F. R. A. Prado wrote: Given that failing to find a DSI host causes the

Re: [PATCH v2 0/9] drm: Switch from dev_err to dev_err_probe for missing DSI host error path

2024-03-01 Thread AngeloGioacchino Del Regno
Apart from patch [8/9], where you're going over 100 cols, this series looks good. After fixing that, on v3, please feel free to add my Reviewed-by: AngeloGioacchino Del Regno ...to all of the patches in this series (and the one that you'll fix as well). Cheers! Angelo

Re: [PATCH v2 8/9] drm/panel: novatek-nt35950: Don't log an error when DSI host can't be found

2024-03-01 Thread AngeloGioacchino Del Regno
. Fixes: 623a3531e9cf ("drm/panel: Add driver for Novatek NT35950 DSI DriverIC panels") Suggested-by: AngeloGioacchino Del Regno Signed-off-by: Nícolas F. R. A. Prado --- drivers/gpu/drm/panel/panel-novatek-nt35950.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) di

Re: [PATCH v2 9/9] drm/panel: truly-nt35597: Don't log an error when DSI host can't be found

2024-03-01 Thread AngeloGioacchino Del Regno
defer probe reason available and avoids alerting userspace about something that is not necessarily an error. Suggested-by: AngeloGioacchino Del Regno Signed-off-by: Nícolas F. R. A. Prado --- drivers/gpu/drm/panel/panel-truly-nt35597.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions

Re: [PATCH v2 1/3] dt-bindings: display: mediatek: gamma: Change MT8195 to single enum group

2024-02-29 Thread AngeloGioacchino Del Regno
Acked-by: Krzysztof Kozlowski Reviewed-by: AngeloGioacchino Del Regno

Re: [PATCH 3/3] drm/mediatek: Add gamma support for MT8195

2024-02-29 Thread AngeloGioacchino Del Regno
Il 29/02/24 03:35, Jason-JH.Lin ha scritto: Since MT8195 compatible is in the single enum group, we have to add its compatible into mediatek-drm component binding table to ensure that it can be bound as a ddp_comp. Reviewed-by: AngeloGioacchino Del Regno

Re: [PATCH 2/3] dt-bindings: display: mediatek: gamma: Add support for MT8188

2024-02-29 Thread AngeloGioacchino Del Regno
Il 29/02/24 03:35, Jason-JH.Lin ha scritto: The gamma LUT setting of MT8188 and MT8195 are the same, so we create a one of items for MT8188 to reuse the driver data settings of MT8195. Signed-off-by: Jason-JH.Lin Reviewed-by: AngeloGioacchino Del Regno

Re: [PATCH 1/3] dt-bindings: display: mediatek: gamma: Change MT8195 to single enum group

2024-02-29 Thread AngeloGioacchino Del Regno
Il 29/02/24 03:35, Jason-JH.Lin ha scritto: Since MT8195 gamma has multiple bank for 12 bits LUT and it is different from any other SoC LUT setting. So we move MT8195 compatible from the one of items to the single enum group. Signed-off-by: Jason-JH.Lin --- .../devicetree/bindings/display/me

Re: [PATCH 01/18] ASoC: dt-bindings: mediatek,mt8365-afe: Add audio afe document

2024-02-28 Thread AngeloGioacchino Del Regno
Il 28/02/24 10:57, Alexandre Mergnat ha scritto: I think I got it. - mediatek,i2s-shared-clock: will be remove from DT - mediatek,dmic-iir-on: will be remove from DT - mediatek,dmic-irr-mode: will be remove from DT - mediatek,dmic-two-wire-mode: rephrase description needed I've did abstraction

Re: [PATCH] drm/bridge: anx7625: Don't log an error when DSI host can't be found

2024-02-28 Thread AngeloGioacchino Del Regno
t noticed that, in this driver, of_node_put() is never called for the `mipi_host_node`, so the reference is never dropped... but that can come later and not necessarily in the eventual series that fixes the err message strategy. In the meanwhile, for this commit Reviewed-

Re: [PATCH 02/18] ASoC: dt-bindings: mediatek,mt8365-mt6357: Add audio sound card document

2024-02-27 Thread AngeloGioacchino Del Regno
Il 27/02/24 11:23, Alexandre Mergnat ha scritto: On 26/02/2024 16:30, AngeloGioacchino Del Regno wrote: Il 26/02/24 15:01, Alexandre Mergnat ha scritto: Add soundcard bindings for the MT8365 SoC with the MT6357 audio codec. Signed-off-by: Alexandre Mergnat ---   .../bindings/sound/mediatek

Re: [PATCH 02/18] ASoC: dt-bindings: mediatek,mt8365-mt6357: Add audio sound card document

2024-02-26 Thread AngeloGioacchino Del Regno
Il 26/02/24 15:01, Alexandre Mergnat ha scritto: Add soundcard bindings for the MT8365 SoC with the MT6357 audio codec. Signed-off-by: Alexandre Mergnat --- .../bindings/sound/mediatek,mt8365-mt6357.yaml | 127 + 1 file changed, 127 insertions(+) diff --git a/Docume

Re: [PATCH 13/18] mfd: mt6397-core: register mt6357 sound codec

2024-02-26 Thread AngeloGioacchino Del Regno
Il 26/02/24 15:01, amerg...@baylibre.com ha scritto: From: Fabien Parent Add MT6357 codec entry in the MFD driver. Signed-off-by: Fabien Parent Signed-off-by: Alexandre Mergnat Reviewed-by: AngeloGioacchino Del Regno --- drivers/mfd/mt6397-core.c | 3 +++ 1 file changed, 3

Re: [PATCH 12/18] ASoC: codecs: mt6357: add MT6357 codec

2024-02-26 Thread AngeloGioacchino Del Regno
Il 26/02/24 15:01, amerg...@baylibre.com ha scritto: From: Nicolas Belin Add the support of MT6357 PMIC audio codec. Signed-off-by: Nicolas Belin Signed-off-by: Alexandre Mergnat --- sound/soc/codecs/Kconfig |7 + sound/soc/codecs/Makefile |2 + sound/soc/codecs/mt6357.c | 1805

Re: [PATCH 10/18] ASoc: mediatek: mt8365: Add a specific soundcard for EVK

2024-02-26 Thread AngeloGioacchino Del Regno
Il 26/02/24 15:01, amerg...@baylibre.com ha scritto: From: Nicolas Belin Add a specific soundcard for mt8365-evk. It supports audio jack in/out, dmics, the amic and lineout. Signed-off-by: Nicolas Belin Signed-off-by: Alexandre Mergnat --- sound/soc/mediatek/mt8365/mt8365-mt6357.c | 379 ++

Re: [PATCH 00/18] Add audio support for the MediaTek Genio 350-evk board

2024-02-26 Thread AngeloGioacchino Del Regno
Il 26/02/24 15:01, Alexandre Mergnat ha scritto: This serie aim to add the following audio support for the Genio 350-evk: - Playback - 2ch Headset Jack (Earphone) - 1ch Line-out Jack (Speaker) - 8ch HDMI Tx - Capture - 1ch DMIC (On-board Digital Microphone) - 1ch AMIC (On-board Ana

Re: [PATCH 01/11] drm/mediatek: Rename "mtk_drm_crtc" to "mtk_crtc"

2024-02-26 Thread AngeloGioacchino Del Regno
Il 26/02/24 09:50, Shawn Sung ha scritto: From: Hsiao Chien Sung Rename all "mtk_drm_crtc" to "mtk_crtc" due to the following benefits: - Lower the matches when searching the native drm_crtc* codes - Reduce the code size Signed-off-by: Hsiao Chien Sung Reviewed-by: AngeloGiaocchino Del Regn

Re: [PATCH 05/11] drm/mediatek: Rename "mtk_drm_hdmi" to "mtk_hdmi"

2024-02-26 Thread AngeloGioacchino Del Regno
Il 26/02/24 09:50, Shawn Sung ha scritto: From: Hsiao Chien Sung Rename all "mtk_drm_hdmi" to "mtk_hdmi": - To align the naming rule - To reduce the code size Signed-off-by: Hsiao Chien Sung Reviewed-by: AngeloGiaocchino Del Regno

Re: [PATCH 11/11] drm/mediatek: Rename "pending_needs_vblank" to "needs_vblank"

2024-02-26 Thread AngeloGioacchino Del Regno
Il 26/02/24 09:50, Shawn Sung ha scritto: From: Hsiao Chien Sung Rename "pending_needs_vblank" to "needs_vblank" to reduce the code size. This decreases human readability though, so I would prefer that we omit this one. Regards, Angelo Signed-off-by: Hsiao Chien Sung --- drivers/gpu/d

Re: [PATCH 04/11] drm/mediatek: Rename "mtk_drm_gem" to "mtk_gem"

2024-02-26 Thread AngeloGioacchino Del Regno
Il 26/02/24 09:50, Shawn Sung ha scritto: From: Hsiao Chien Sung Rename all "mtk_drm_gem" to "mtk_gem": - To align the naming rule - To reduce the code size Signed-off-by: Hsiao Chien Sung Reviewed-by: AngeloGiaocchino Del Regno

Re: [PATCH 02/11] drm/mediatek: Rename "mtk_drm_ddp_comp" to "mtk_ddp_comp"

2024-02-26 Thread AngeloGioacchino Del Regno
Il 26/02/24 09:50, Shawn Sung ha scritto: From: Hsiao Chien Sung Rename all "mtk_drm_ddp_comp" to "mtk_ddp_comp": - To align the naming rule - To reduce the code size Signed-off-by: Hsiao Chien Sung Reviewed-by: AngeloGiaocchino Del Regno

Re: [PATCH 03/11] drm/mediatek: Rename "mtk_drm_plane" to "mtk_plane"

2024-02-26 Thread AngeloGioacchino Del Regno
Il 26/02/24 09:50, Shawn Sung ha scritto: From: Hsiao Chien Sung Rename all "mtk_drm_plane" to "mtk_plane": - To align the naming rule - To reduce the code size Signed-off-by: Hsiao Chien Sung Reviewed-by: AngeloGiaocchino Del Regno

Re: [PATCH 06/11] drm/mediatek: Rename files "mtk_drm_crtc.*" to "mtk_crtc.*"

2024-02-26 Thread AngeloGioacchino Del Regno
Il 26/02/24 09:50, Shawn Sung ha scritto: From: Hsiao Chien Sung Rename files mtk_drm_crtc.* to mtk_crtc.*. Signed-off-by: Hsiao Chien Sung Reviewed-by: AngeloGiaocchino Del Regno

Re: [PATCH 09/11] drm/mediatek: Rename files "mtk_drm_gem.*" to "mtk_gem.*"

2024-02-26 Thread AngeloGioacchino Del Regno
Il 26/02/24 09:50, Shawn Sung ha scritto: From: Hsiao Chien Sung Rename files mtk_drm_gem.* to mtk_gem.*. Signed-off-by: Hsiao Chien Sung Reviewed-by: AngeloGiaocchino Del Regno

Re: [PATCH 08/11] drm/mediatek: Rename files "mtk_drm_plane.*" to "mtk_plane.*"

2024-02-26 Thread AngeloGioacchino Del Regno
Il 26/02/24 09:50, Shawn Sung ha scritto: From: Hsiao Chien Sung Rename files mtk_drm_plane.* to mtk_plane.*. Signed-off-by: Hsiao Chien Sung Reviewed-by: AngeloGiaocchino Del Regno

Re: [PATCH 07/11] drm/mediatek: Rename files "mtk_drm_ddp_comp.*" to "mtk_ddp_comp.*"

2024-02-26 Thread AngeloGioacchino Del Regno
Il 26/02/24 09:50, Shawn Sung ha scritto: From: Hsiao Chien Sung Rename files mtk_drm_ddp_comp.* to mtk_ddp_comp.*. Signed-off-by: Hsiao Chien Sung Reviewed-by: AngeloGiaocchino Del Regno

Re: [PATCH 10/11] drm/mediatek: Rename mtk_ddp_comp functions

2024-02-26 Thread AngeloGioacchino Del Regno
Il 26/02/24 09:50, Shawn Sung ha scritto: From: Hsiao Chien Sung Rename functions of mtk_ddp_comp: - To align the naming rule - To reduce the code size Signed-off-by: Hsiao Chien Sung Reviewed-by: AngeloGiaocchino Del Regno

Re: [PATCH] drm/mediatek: Add MT8188 Overlay Driver Data

2024-02-26 Thread AngeloGioacchino Del Regno
Il 26/02/24 09:21, Fei Shao ha scritto: On Thu, Feb 22, 2024 at 4:43 PM AngeloGioacchino Del Regno wrote: Il 21/02/24 17:56, Justin Green ha scritto: Add MT8188 overlay driver configuration data. This change consequently enables 10-bit overlay support on MT8188 devices. Tested by running

Re: [PATCH v2 04/12] soc: mediatek: cmdq: Add cmdq_pkt_jump_rel() helper function

2024-02-23 Thread AngeloGioacchino Del Regno
Il 22/02/24 16:41, Chun-Kuang Hu ha scritto: cmdq_pkt_jump_rel() append relative jump command to the packet. Relative jump change PC to the target address with offset from current PC. Signed-off-by: Chun-Kuang Hu Reviewed-by: AngeloGioacchino Del Regno

Re: [PATCH v2 01/12] soc: mediatek: cmdq: Fix typo of CMDQ_JUMP_RELATIVE

2024-02-23 Thread AngeloGioacchino Del Regno
t;) Signed-off-by: Chun-Kuang Hu Reviewed-by: AngeloGioacchino Del Regno

Re: [PATCH v2 03/12] soc: mediatek: cmdq: Rename cmdq_pkt_jump() to cmdq_pkt_jump_abs()

2024-02-23 Thread AngeloGioacchino Del Regno
Il 22/02/24 16:41, Chun-Kuang Hu ha scritto: In order to distinguish absolute jump and relative jump, cmdq_pkt_jump() append absolute jump command, so rename it to cmdq_pkt_jump_abs(). Signed-off-by: Chun-Kuang Hu Reviewed-by: AngeloGioacchino Del Regno

Re: [PATCH] drm/mediatek: Add MT8188 Overlay Driver Data

2024-02-22 Thread AngeloGioacchino Del Regno
Il 21/02/24 17:56, Justin Green ha scritto: Add MT8188 overlay driver configuration data. This change consequently enables 10-bit overlay support on MT8188 devices. Tested by running ChromeOS UI on MT8188 and using modetest -P. AR30 and BA30 overlays are confirmed to work from modetest. Signed-

Re: [PATCH v5 10/13] drm/mediatek: Support CRC in display driver

2024-02-15 Thread AngeloGioacchino Del Regno
Il 15/02/24 11:11, Hsiao Chien Sung ha scritto: Register CRC related function pointers to support CRC retrieval. Signed-off-by: Hsiao Chien Sung --- drivers/gpu/drm/mediatek/mtk_drm_crtc.c | 239 drivers/gpu/drm/mediatek/mtk_drm_crtc.h | 39 drivers/gpu/dr

Re: [PATCH v5 08/13] drm/mediatek: Support alpha blending in OVL

2024-02-15 Thread AngeloGioacchino Del Regno
Il 15/02/24 11:11, Hsiao Chien Sung ha scritto: Support "Pre-multiplied" and "None" blend mode on MediaTek's chips. Before this patch, only the "Coverage" mode is supported. Please refer to the description of the commit "drm/mediatek: Support alpha blending in display driver" for more informatio

Re: [PATCH v5 07/13] drm/mediatek: Support alpha blending in display driver

2024-02-15 Thread AngeloGioacchino Del Regno
pha for the composited layer will take effects. Signed-off-by: Hsiao Chien Sung Reviewed-by: CK Hu Reviewed-by: AngeloGioacchino Del Regno

Re: [PATCH v5 06/13] drm/mediatek: Turn off the layers with zero width or height

2024-02-15 Thread AngeloGioacchino Del Regno
n any case, it's not mandatory as that depends on what the maintainer prefers, so it's CK's call anyway. Besides that, Reviewed-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_disp_ovl_adaptor.c | 2 +- drivers/gpu/drm/mediatek/mtk_ethdr.c| 7

[PATCH v6 6/9] drm/mediatek: dsi: Register DSI host after acquiring clocks and PHY

2024-02-15 Thread AngeloGioacchino Del Regno
Register the DSI host after getting clocks, iospace and PHY. This wil also allow to simplify the error paths in a later commit. Reviewed-by: Alexandre Mergnat Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 28 ++-- 1 file changed, 14 in

[PATCH v6 9/9] drm/mediatek: dsi: Use mipi_dsi_pixel_format_to_bpp() helper function

2024-02-15 Thread AngeloGioacchino Del Regno
e any extra-spec format support and/or because the data rate (hence, the HS clock) will be wrong. Reviewed-by: Alexandre Mergnat Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 26 +- 1 file changed, 9 insertions(+), 17 deletions(-) diff

[PATCH v6 8/9] drm/mediatek: dsi: Compress of_device_id entries and add sentinel

2024-02-15 Thread AngeloGioacchino Del Regno
All entries fit in 82 columns, which is acceptable: compress all of the mtk_dsi_of_match[] entries to a single line for each. While at it, also add the usual sentinel comment to the last entry. Reviewed-by: Alexandre Mergnat Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm

[PATCH v6 4/9] drm/mediatek: dsi: Use bitfield macros where useful

2024-02-15 Thread AngeloGioacchino Del Regno
Reviewed-by: CK Hu Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 95 -- 1 file changed, 50 insertions(+), 45 deletions(-) diff --git a/drivers/gpu/drm/mediatek/mtk_dsi.c b/drivers/gpu/drm/mediatek/mtk_dsi.c index 0a83875ec1ba

[PATCH v6 5/9] drm/mediatek: dsi: Replace open-coded instance of HZ_PER_MHZ

2024-02-15 Thread AngeloGioacchino Del Regno
In mtk_dsi_phy_timconfig(), we're dividing the `data_rate` variable, expressed in Hz to retrieve a value in MHz: instead of open-coding, use the HZ_PER_MHZ definition, available in linux/units.h. Reviewed-by: Alexandre Mergnat Reviewed-by: CK Hu Signed-off-by: AngeloGioacchino Del

[PATCH v6 1/9] drm/mediatek: dsi: Use GENMASK() for register mask definitions

2024-02-15 Thread AngeloGioacchino Del Regno
Change magic numerical masks with usage of the GENMASK() macro to improve readability. This commit brings no functional changes. Reviewed-by: Alexandre Mergnat Reviewed-by: CK Hu Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 45

[PATCH v6 3/9] drm/mediatek: dsi: Cleanup functions mtk_dsi_ps_control{_vact}()

2024-02-15 Thread AngeloGioacchino Del Regno
Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 78 ++ 1 file changed, 25 insertions(+), 53 deletions(-) diff --git a/drivers/gpu/drm/mediatek/mtk_dsi.c b/drivers/gpu/drm/mediatek/mtk_dsi.c index 8af0afbe9e3d..0a83875ec1ba 100644

[PATCH v6 7/9] drm/mediatek: dsi: Simplify with dev_err_probe and remove gotos

2024-02-15 Thread AngeloGioacchino Del Regno
ister() function in the devm_request_irq() error path, allowing to also remove the same label. Reviewed-by: Alexandre Mergnat Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 60 +- 1 file changed, 18 insertions(+), 42 deletions(-) diff

[PATCH v6 2/9] drm/mediatek: dsi: Fix DSI RGB666 formats and definitions

2024-02-15 Thread AngeloGioacchino Del Regno
d, 18-bits for MIPI_DSI_FMT_RGB666_PACKED Fixes: 2e54c14e310f ("drm/mediatek: Add DSI sub driver") Reviewed-by: Alexandre Mergnat Reviewed-by: CK Hu Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 10 +- 1 file changed, 5 insertions(+), 5 dele

[PATCH v6 0/9] MediaTek DRM - DSI driver cleanups

2024-02-15 Thread AngeloGioacchino Del Regno
panel). AngeloGioacchino Del Regno (9): drm/mediatek: dsi: Use GENMASK() for register mask definitions drm/mediatek: dsi: Fix DSI RGB666 formats and definitions drm/mediatek: dsi: Cleanup functions mtk_dsi_ps_control{_vact}() drm/mediatek: dsi: Use bitfield macros where useful drm/mediate

[PATCH v5 8/9] drm/mediatek: dsi: Compress of_device_id entries and add sentinel

2024-02-07 Thread AngeloGioacchino Del Regno
All entries fit in 82 columns, which is acceptable: compress all of the mtk_dsi_of_match[] entries to a single line for each. While at it, also add the usual sentinel comment to the last entry. Reviewed-by: Alexandre Mergnat Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm

[PATCH v5 6/9] drm/mediatek: dsi: Register DSI host after acquiring clocks and PHY

2024-02-07 Thread AngeloGioacchino Del Regno
Register the DSI host after getting clocks, iospace and PHY. This wil also allow to simplify the error paths in a later commit. Reviewed-by: Alexandre Mergnat Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 28 ++-- 1 file changed, 14 in

[PATCH v5 5/9] drm/mediatek: dsi: Replace open-coded instance of HZ_PER_MHZ

2024-02-07 Thread AngeloGioacchino Del Regno
In mtk_dsi_phy_timconfig(), we're dividing the `data_rate` variable, expressed in Hz to retrieve a value in MHz: instead of open-coding, use the HZ_PER_MHZ definition, available in linux/units.h. Reviewed-by: Alexandre Mergnat Signed-off-by: AngeloGioacchino Del Regno --- drivers/gp

[PATCH v5 9/9] drm/mediatek: dsi: Use mipi_dsi_pixel_format_to_bpp() helper function

2024-02-07 Thread AngeloGioacchino Del Regno
e any extra-spec format support and/or because the data rate (hence, the HS clock) will be wrong. Reviewed-by: Alexandre Mergnat Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 26 +- 1 file changed, 9 insertions(+), 17 deletions(-) diff

[PATCH v5 7/9] drm/mediatek: dsi: Simplify with dev_err_probe and remove gotos

2024-02-07 Thread AngeloGioacchino Del Regno
ister() function in the devm_request_irq() error path, allowing to also remove the same label. Reviewed-by: Alexandre Mergnat Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 60 +- 1 file changed, 18 insertions(+), 42 deletions(-) diff

[PATCH v5 4/9] drm/mediatek: dsi: Use bitfield macros where useful

2024-02-07 Thread AngeloGioacchino Del Regno
Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 97 -- 1 file changed, 52 insertions(+), 45 deletions(-) diff --git a/drivers/gpu/drm/mediatek/mtk_dsi.c b/drivers/gpu/drm/mediatek/mtk_dsi.c index 7d38e9500700..a330bb94c44a 100644

[PATCH v5 1/9] drm/mediatek: dsi: Use GENMASK() for register mask definitions

2024-02-07 Thread AngeloGioacchino Del Regno
Change magic numerical masks with usage of the GENMASK() macro to improve readability. This commit brings no functional changes. Reviewed-by: Alexandre Mergnat Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 45 +++--- 1 file changed

[PATCH v5 3/9] drm/mediatek: dsi: Cleanup functions mtk_dsi_ps_control{_vact}()

2024-02-07 Thread AngeloGioacchino Del Regno
: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 76 +- 1 file changed, 23 insertions(+), 53 deletions(-) diff --git a/drivers/gpu/drm/mediatek/mtk_dsi.c b/drivers/gpu/drm/mediatek/mtk_dsi.c index 8af0afbe9e3d..7d38e9500700 100644 --- a/drivers/gpu/drm

[PATCH v5 2/9] drm/mediatek: dsi: Fix DSI RGB666 formats and definitions

2024-02-07 Thread AngeloGioacchino Del Regno
d, 18-bits for MIPI_DSI_FMT_RGB666_PACKED Fixes: 2e54c14e310f ("drm/mediatek: Add DSI sub driver") Reviewed-by: Alexandre Mergnat Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --g

[PATCH v5 0/9] MediaTek DRM - DSI driver cleanups

2024-02-07 Thread AngeloGioacchino Del Regno
, enhancing human readability, using kernel provided macros where possible and also reducing code size. Tested on MT8173 and MT8192 Chromebooks (using a DSI<->DP bridge) and on MT6795 Sony Xperia M5 (DSI video mode panel). AngeloGioacchino Del Regno (9): drm/mediatek: dsi: Use GENMASK

Re: [PATCH v4 1/9] drm/mediatek: dsi: Use GENMASK() for register mask definitions

2024-02-06 Thread AngeloGioacchino Del Regno
Il 06/02/24 15:47, Alexandre Mergnat ha scritto: On 06/02/2024 13:07, AngeloGioacchino Del Regno wrote: Change magic numerical masks with usage of the GENMASK() macro to improve readability. While at it, also fix the DSI_PS_SEL mask to include all bits instead of just a subset of them. This

Re: [PATCH v2 2/3] Subject: [PATCH] drm/mediatek/dp: Add HDCP2.x feature for DisplayPort

2024-02-06 Thread AngeloGioacchino Del Regno
Il 05/02/24 06:50, mac.shen ha scritto: Add HDCP2.x feature for DisplayPort. When userspace request the kernel protect future content communicated over the link with Content_Protection property, the feature will do HDCP2.x authentication if the sink support HDCP2.X. Changes in v2: - remove switc

Re: [PATCH v2 1/3] Subject: [PATCH] drm/mediatek/dp: Add tee client application for HDCP feature

2024-02-06 Thread AngeloGioacchino Del Regno
Il 05/02/24 06:50, mac.shen ha scritto: Add tee client application which will be used for HDCP 1.x and 2.x authentication in DisplayPort. Changes in v2: - remove ca folder, and change file name with lower case - refine the tci_t structure to make the data to tee can through this structure - r

Re: [PATCH 1/3] drm/mediatek: Dynamically allocate CMDQ and use helper functions

2024-02-06 Thread AngeloGioacchino Del Regno
Il 03/08/23 10:37, CK Hu (胡俊光) ha scritto: Hi, Angelo: On Thu, 2023-08-03 at 10:25 +0200, AngeloGioacchino Del Regno wrote: Il 03/08/23 08:28, CK Hu (胡俊光) ha scritto: Hi, Angelo: On Wed, 2023-08-02 at 12:41 +0200, AngeloGioacchino Del Regno wrote: Il 02/08/23 08:24, CK Hu (胡俊光) ha scritto

Re: [PATCH v3 1/7] drm/mediatek: dsi: Use GENMASK() for register mask definitions

2024-02-06 Thread AngeloGioacchino Del Regno
Il 06/02/24 09:57, CK Hu (胡俊光) ha scritto: Hi, Angelo: On Wed, 2024-01-31 at 12:34 +0100, AngeloGioacchino Del Regno wrote: Change magic numerical masks with usage of the GENMASK() macro to improve readability. While at it, also fix the DSI_PS_SEL mask to include all bits instead of just a

[PATCH v4 9/9] drm/mediatek: dsi: Use mipi_dsi_pixel_format_to_bpp() helper function

2024-02-06 Thread AngeloGioacchino Del Regno
e any extra-spec format support and/or because the data rate (hence, the HS clock) will be wrong. Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 26 +- 1 file changed, 9 insertions(+), 17 deletions(-) diff --git a/drivers/gpu/drm/med

[PATCH v4 8/9] drm/mediatek: dsi: Compress of_device_id entries and add sentinel

2024-02-06 Thread AngeloGioacchino Del Regno
All entries fit in 82 columns, which is acceptable: compress all of the mtk_dsi_of_match[] entries to a single line for each. While at it, also add the usual sentinel comment to the last entry. Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 17

[PATCH v4 3/9] drm/mediatek: dsi: Cleanup functions mtk_dsi_ps_control{_vact}()

2024-02-06 Thread AngeloGioacchino Del Regno
Function mtk_dsi_ps_control() is a subset of mtk_dsi_ps_control_vact(): merge the two in one mtk_dsi_ps_control() function by adding one function parameter `config_vact` which, when true, writes the VACT related registers. Reviewed-by: Fei Shao Signed-off-by: AngeloGioacchino Del Regno

[PATCH v4 0/9] MediaTek DRM - DSI driver cleanups

2024-02-06 Thread AngeloGioacchino Del Regno
also reducing code size. Tested on MT8173 and MT8192 Chromebooks (using a DSI<->DP bridge) and on MT6795 Sony Xperia M5 (DSI video mode panel). AngeloGioacchino Del Regno (9): drm/mediatek: dsi: Use GENMASK() for register mask definitions drm/mediatek: dsi: Fix DSI RGB666 formats and defin

[PATCH v4 1/9] drm/mediatek: dsi: Use GENMASK() for register mask definitions

2024-02-06 Thread AngeloGioacchino Del Regno
Change magic numerical masks with usage of the GENMASK() macro to improve readability. While at it, also fix the DSI_PS_SEL mask to include all bits instead of just a subset of them. This commit brings no functional changes. Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm

[PATCH v4 6/9] drm/mediatek: dsi: Register DSI host after acquiring clocks and PHY

2024-02-06 Thread AngeloGioacchino Del Regno
Register the DSI host after getting clocks, iospace and PHY. This wil also allow to simplify the error paths in a later commit. Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) di

[PATCH v4 5/9] drm/mediatek: dsi: Replace open-coded instance of HZ_PER_MHZ

2024-02-06 Thread AngeloGioacchino Del Regno
In mtk_dsi_phy_timconfig(), we're dividing the `data_rate` variable, expressed in Hz to retrieve a value in MHz: instead of open-coding, use the HZ_PER_MHZ definition, available in linux/units.h. Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 3 ++- 1

[PATCH v4 2/9] drm/mediatek: dsi: Fix DSI RGB666 formats and definitions

2024-02-06 Thread AngeloGioacchino Del Regno
d, 18-bits for MIPI_DSI_FMT_RGB666_PACKED Fixes: 2e54c14e310f ("drm/mediatek: Add DSI sub driver") Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/mediatek/mtk_

[PATCH v4 7/9] drm/mediatek: dsi: Simplify with dev_err_probe and remove gotos

2024-02-06 Thread AngeloGioacchino Del Regno
ister() function in the devm_request_irq() error path, allowing to also remove the same label. Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 60 +- 1 file changed, 18 insertions(+), 42 deletions(-) diff --git a/drivers/gpu/drm/med

[PATCH v4 4/9] drm/mediatek: dsi: Use bitfield macros where useful

2024-02-06 Thread AngeloGioacchino Del Regno
Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 97 -- 1 file changed, 52 insertions(+), 45 deletions(-) diff --git a/drivers/gpu/drm/mediatek/mtk_dsi.c b/drivers/gpu/drm/mediatek/mtk_dsi.c index b025886be680..26c221737387 100644 --- a/drivers/gpu/drm/mediatek

Re: [PATCH v3 2/7] drm/mediatek: dsi: Cleanup functions mtk_dsi_ps_control{_vact}()

2024-02-06 Thread AngeloGioacchino Del Regno
Il 06/02/24 10:50, CK Hu (胡俊光) ha scritto: Hi, Angelo: On Wed, 2024-01-31 at 12:34 +0100, AngeloGioacchino Del Regno wrote: Function mtk_dsi_ps_control() is a subset of mtk_dsi_ps_control_vact(): merge the two in one mtk_dsi_ps_control() function by adding one function parameter `config_vact

[PATCH 3/3] drm/mediatek: drm_ddp_comp: Add mtk_ddp_is_simple_comp() internal helper

2024-02-01 Thread AngeloGioacchino Del Regno
Move the simple component check to a new mtk_ddp_is_simple_comp() internal helper to reduce code duplication. Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c | 57 +++-- 1 file changed, 31 insertions(+), 26 deletions(-) diff --git a

[PATCH 1/3] drm/mediatek: drm_ddp_comp: Fix and cleanup DDP component CRTC search

2024-02-01 Thread AngeloGioacchino Del Regno
tmask of the found possible CRTC; and - mtk_drm_find_possible_crtc_by_comp() always returns either a bitmask of the possible CRTC, or zero if none available. Fixes: 01389b324c97 ("drm/mediatek: Add connector dynamic selection capability") Signed-off-by: AngeloGioacchino Del Regno -

[PATCH 2/3] drm/mediatek: Perform iounmap on simple DDP component destruction

2024-02-01 Thread AngeloGioacchino Del Regno
from sub driver to DRM driver") Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c | 38 - drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.h | 1 + drivers/gpu/drm/mediatek/mtk_drm_drv.c | 4 ++- 3 files changed, 41 insertions(+), 2

[PATCH 0/3] drm/mediatek: Fixes for DDP component search/destroy

2024-02-01 Thread AngeloGioacchino Del Regno
This series performs some cleanups for DDP component CRTC search and correctly iounmaps the previously of_iomap() calls from drm_ddp_comp. Tested on MT8195 Cherry Tomato AngeloGioacchino Del Regno (3): drm/mediatek: drm_ddp_comp: Fix and cleanup DDP component CRTC search drm/mediatek

Re: [PATCH v4 00/17] Support IGT in display driver

2024-02-01 Thread AngeloGioacchino Del Regno
Il 12/12/23 13:19, Hsiao Chien Sung ha scritto: This series is based on mediatek-drm-next. This series adds support for running IGT (Intel GPU Tool) tests with MediaTek display driver. The following changes will be applied: 1. Add a new API for creating GCE thread loop to retrieve CRCs from

Re: [v3 2/3] ASoC: mediatek: mt8186: correct the HDMI widgets

2024-01-31 Thread AngeloGioacchino Del Regno
Il 31/01/24 13:25, Jiaxin Yu (俞家鑫) ha scritto: On Wed, 2024-01-31 at 12:42 +0100, AngeloGioacchino Del Regno wrote: Il 03/08/23 21:33, Mark Brown ha scritto: On Thu, Aug 03, 2023 at 07:20:15AM +, Jiaxin Yu (俞家鑫) wrote: I agree with you, in fact the speaker is indeed doing this way. But

Re: [v3 2/3] ASoC: mediatek: mt8186: correct the HDMI widgets

2024-01-31 Thread AngeloGioacchino Del Regno
Il 03/08/23 21:33, Mark Brown ha scritto: On Thu, Aug 03, 2023 at 07:20:15AM +, Jiaxin Yu (俞家鑫) wrote: I agree with you, in fact the speaker is indeed doing this way. But about the hdmi that on the board, I did not find a defination link snd_soc_dapm_hdmi, so I use snd_soc_dapm_line to repl

[PATCH v3 3/7] drm/mediatek: dsi: Use bitfield macros where useful

2024-01-31 Thread AngeloGioacchino Del Regno
Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 97 -- 1 file changed, 52 insertions(+), 45 deletions(-) diff --git a/drivers/gpu/drm/mediatek/mtk_dsi.c b/drivers/gpu/drm/mediatek/mtk_dsi.c index 8414ce73ce9f..cba63de5092d 100644 --- a/drivers/gpu/drm/mediatek

[PATCH v3 6/7] drm/mediatek: dsi: Simplify with dev_err_probe and remove gotos

2024-01-31 Thread AngeloGioacchino Del Regno
ister() function in the devm_request_irq() error path, allowing to also remove the same label. Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 60 +- 1 file changed, 18 insertions(+), 42 deletions(-) diff --git a/drivers/gpu/drm/med

[PATCH v3 4/7] drm/mediatek: dsi: Replace open-coded instance of HZ_PER_MHZ

2024-01-31 Thread AngeloGioacchino Del Regno
In mtk_dsi_phy_timconfig(), we're dividing the `data_rate` variable, expressed in Hz to retrieve a value in MHz: instead of open-coding, use the HZ_PER_MHZ definition, available in linux/units.h. Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm/mediatek/mtk_dsi.c | 3 ++- 1

[PATCH v3 2/7] drm/mediatek: dsi: Cleanup functions mtk_dsi_ps_control{_vact}()

2024-01-31 Thread AngeloGioacchino Del Regno
Function mtk_dsi_ps_control() is a subset of mtk_dsi_ps_control_vact(): merge the two in one mtk_dsi_ps_control() function by adding one function parameter `config_vact` which, when true, writes the VACT related registers. Reviewed-by: Fei Shao Signed-off-by: AngeloGioacchino Del Regno

[PATCH v3 1/7] drm/mediatek: dsi: Use GENMASK() for register mask definitions

2024-01-31 Thread AngeloGioacchino Del Regno
Change magic numerical masks with usage of the GENMASK() macro to improve readability. While at it, also fix the DSI_PS_SEL mask to include all bits instead of just a subset of them. This commit brings no functional changes. Signed-off-by: AngeloGioacchino Del Regno --- drivers/gpu/drm

<    1   2   3   4   5   6   7   8   9   10   >