Re: [PATCH 16/38] arm64: dts: mediatek: mt6331: Fix pmic, regulators, rtc, keys node names

2025-07-25 Thread Fei Shao
dts: mediatek: Add MT6331 PMIC devicetree") > Signed-off-by: AngeloGioacchino Del Regno > Reviewed-by: Fei Shao > --- > arch/arm64/boot/dts/mediatek/mt6331.dtsi | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/arch/arm64/boot/dts/m

Re: [PATCH 17/38] arm64: dts: mediatek: mt6797: Fix pinctrl node names

2025-07-25 Thread Fei Shao
On Thu, Jul 24, 2025 at 5:48 PM AngeloGioacchino Del Regno wrote: > > Change the pinctrl node names to adhere to the binding: the main > nodes are now named like "uart0-pins" and the children "pins-bus". > > Signed-off-by: AngeloGioacchino Del Regno >

Re: [PATCH 26/38] arm64: dts: mediatek: acelink-ew-7886cax: Remove unnecessary cells in spi-nand

2025-07-25 Thread Fei Shao
;partitions" subnode. > > Remove those to suppress an avoid_unnecessary_addr_size warning. > > Signed-off-by: AngeloGioacchino Del Regno > Reviewed-by: Fei Shao > --- > arch/arm64/boot/dts/mediatek/mt7986a-acelink-ew-7886cax.dts | 2 -- > 1 file changed, 2 deletions(-) >

Re: [PATCH 19/38] arm64: dts: mediatek: mt6795: Add mediatek,infracfg to iommu node

2025-07-25 Thread Fei Shao
On Thu, Jul 24, 2025 at 5:49 PM AngeloGioacchino Del Regno wrote: > > The "M4U" IOMMU requires a handle to the infracfg to switch to > the 4gb/pae addressing mode: add it. > > Signed-off-by: AngeloGioacchino Del Regno > Reviewed-by: Fei Shao > --- > arch/a

Re: [PATCH 18/38] arm64: dts: mediatek: mt6797: Remove bogus id property in i2c nodes

2025-07-25 Thread Fei Shao
dtbs_check warnings: remove it from all i2c nodes. > > Signed-off-by: AngeloGioacchino Del Regno > Reviewed-by: Fei Shao > --- > arch/arm64/boot/dts/mediatek/mt6797.dtsi | 10 -- > 1 file changed, 10 deletions(-) > > diff --git a/arch/arm64/boot/dts/mediatek/

Re: [PATCH 37/38] arm64: dts: mediatek: mt8395-kontron-i1200: Fix MT6360 regulator nodes

2025-07-25 Thread Fei Shao
ase to get working regulators. > > Fixes: 94aaf79a6af5 ("arm64: dts: mediatek: add Kontron 3.5"-SBC-i1200") > Signed-off-by: AngeloGioacchino Del Regno > Reviewed-by: Fei Shao > --- > .../mediatek/mt8395-kontron-3-5-sbc-i1200.dts| 16 > 1

Re: [PATCH 38/38] arm64: dts: mediatek: mt8516-pumpkin: Fix machine compatible

2025-07-25 Thread Fei Shao
("arm64: dts: mediatek: add pumpkin board dts") > Signed-off-by: AngeloGioacchino Del Regno > Reviewed-by: Fei Shao > --- > arch/arm64/boot/dts/mediatek/mt8516-pumpkin.dts | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/bo

Re: [PATCH 24/38] arm64: dts: mediatek: mt7986a-bpi-r3: Fix SFP I2C node names

2025-07-25 Thread Fei Shao
he binding and suppress > dtbs_check warnings. > > Signed-off-by: AngeloGioacchino Del Regno > It'd be nice to mention in v2 (if needed) that this patch also drops redundant #address-cells and #size-cells, but it's minor. Reviewed-by: Fei Shao > --- > arch/arm64/boot/d

Re: [PATCH 23/38] arm64: dts: mediatek: mt7986a: Fix PCI-Express T-PHY node address

2025-07-25 Thread Fei Shao
the main > node to silence a dtbs_check warning, and fix the children to > use the MMIO range of t-phy. > > Fixes: 963c3b0c47ec ("arm64: dts: mediatek: fix t-phy unit name") > Fixes: 918aed7abd2d ("arm64: dts: mt7986: add pcie related device nodes") > Signed-

Re: [PATCH 29/38] arm64: dts: mediatek: mt8183: Fix pinctrl node names

2025-07-25 Thread Fei Shao
oGioacchino Del Regno > Reviewed-by: Fei Shao > --- > .../mediatek/mt8183-kukui-audio-da7219.dtsi | 4 +- > .../mediatek/mt8183-kukui-audio-ts3a227e.dtsi | 2 +- > .../dts/mediatek/mt8183-kukui-jacuzzi.dtsi| 22 +++--- > .../dts/mediatek/mt8183-kukui-kakadu.dtsi

Re: [PATCH 34/38] arm64: dts: mediatek: mt8195: Fix ranges for jpeg enc/decoder nodes

2025-07-25 Thread Fei Shao
coder isn't sorted in order, I guess we all agree it's much easier to read in this way, so Reviewed-by: Fei Shao > --- > arch/arm64/boot/dts/mediatek/mt8195.dtsi | 30 +--- > 1 file changed, 16 insertions(+), 14 deletions(-) > > diff --git a/arch/arm64

Re: [PATCH 30/38] arm64: dts: mediatek: pumpkin-common: Fix pinctrl node names

2025-07-25 Thread Fei Shao
oGioacchino Del Regno > Reviewed-by: Fei Shao > --- > .../boot/dts/mediatek/pumpkin-common.dtsi | 18 +- > 1 file changed, 9 insertions(+), 9 deletions(-) > > diff --git a/arch/arm64/boot/dts/mediatek/pumpkin-common.dtsi > b/arch/arm64/boot/dts/medi

Re: [PATCH] drm/bridge: anx7625: Simplify INTERFACE_CHANGE_INT reg clear write

2025-03-27 Thread Fei Shao
On Tue, Mar 25, 2025 at 4:14 PM Fei Shao wrote: > > Simply clear the INTERFACE_CHANGE_INT register with 0 and replace the > indirect `x & (~x)` pattern, since they are logically the same. > No functional change. > > Signed-off-by: Fei Shao > --- Hi, I noticed that Xin

[PATCH] drm/bridge: anx7625: Simplify INTERFACE_CHANGE_INT reg clear write

2025-03-25 Thread Fei Shao
Simply clear the INTERFACE_CHANGE_INT register with 0 and replace the indirect `x & (~x)` pattern, since they are logically the same. No functional change. Signed-off-by: Fei Shao --- drivers/gpu/drm/bridge/analogix/anx7625.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff -

Re: [PATCH 01/12] dt-bindings: arm: mediatek: mmsys: add compatible for MT8196

2025-01-13 Thread Fei Shao
On Fri, Jan 10, 2025 at 8:58 PM Krzysztof Kozlowski wrote: > > On 10/01/2025 13:33, paul-pl.chen wrote: > > From: "Paul-pl.Chen" > > > > Add compatible for mmsys yaml of MT8196 > > > > Signed-off-by: Paul-pl.Chen > > > This is a wide pattern now in Mediatek: login name is used as family > name.

[PATCH v2] drm/mediatek: dp: Support flexible length of DP calibration data

2024-12-04 Thread Fei Shao
indicates an error in either the efuse data length described in DT or the efuse format within the driver. Signed-off-by: Fei Shao --- Changes in v2: - use %zu identifier for size_t in dev_warn() drivers/gpu/drm/mediatek/mtk_dp.c | 18 +- 1 file changed, 13 insertions(+), 5 deletions

[PATCH] drm/mediatek: dp: Support flexible length of DP calibration data

2024-12-03 Thread Fei Shao
error in either the efuse data length described in DT or the efuse format within the driver. Signed-off-by: Fei Shao --- drivers/gpu/drm/mediatek/mtk_dp.c | 18 +- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/mediatek/mtk_dp.c b/drivers/gpu/drm

[PATCH v2] dt-bindings: display: mediatek: dp: Reference common DAI properties

2024-11-05 Thread Fei Shao
ding doesn't support audio. This fixes dtbs_check error: '#sound-dai-cells' does not match any of the regexes: 'pinctrl-[0-9]+' Signed-off-by: Fei Shao --- Changes in v2: - reference to dai-common.yaml since the hardware exposes DAI and update to `unevalua

Re: [RFC PATCH] drm/bridge: panel: Use devm_drm_bridge_add()

2024-10-29 Thread Fei Shao
On Thu, Oct 24, 2024 at 8:36 PM Maxime Ripard wrote: > > On Wed, Oct 09, 2024 at 01:23:31PM +0800, Fei Shao wrote: > > In the mtk_dsi driver, its DSI host attach callback calls > > devm_drm_of_get_bridge() to get the next bridge. If that next bridge is > > a panel bridge,

Re: [PATCH] dt-bindings: display: mediatek: dp: Add #sound-dai-cells property

2024-10-28 Thread Fei Shao
On Mon, Oct 28, 2024 at 4:52 AM Krzysztof Kozlowski wrote: > > On Fri, Oct 25, 2024 at 06:42:45PM +0800, Fei Shao wrote: > > The MediaTek DP hardware supports audio, and the "#sound-dai-cells" > > property is required to describe DAI links and audio routing. &

[PATCH] dt-bindings: display: mediatek: dp: Add #sound-dai-cells property

2024-10-25 Thread Fei Shao
The MediaTek DP hardware supports audio, and the "#sound-dai-cells" property is required to describe DAI links and audio routing. Add "#sound-dai-cells" property to the binding and filter out non-DP compatibles, as MediaTek eDP doesn't support audio. Signed-off-by: Fei

[PATCH] dt-bindings: display: mediatek: dpi: Update device list with power-domains

2024-10-18 Thread Fei Shao
This list is expected to be fixed, and it encourages new MTK DPI devices to describe their power domain whenever possible; if not, those should be listed with proper rationale. Signed-off-by: Fei Shao --- .../bindings/display/mediatek/mediatek,dpi.yaml | 15 +++ 1 file chang

Re: [PATCH v2] drm/mediatek: Fix color format MACROs in OVL

2024-10-16 Thread Fei Shao
On Wed, Oct 16, 2024 at 7:10 PM Hsin-Te Yuan wrote: > > In commit 9f428b95ac89 ("drm/mediatek: Add new color format MACROs in > OVL"), some new color formats are defined in the MACROs to make the > switch statement more concise. That commit was intended to be a no-op > cleanup. However, there are

Re: [PATCH v5 1/1] drm/mediatek: Fix get efuse issue for MT8188 DPTX

2024-10-14 Thread Fei Shao
ort MT8188 dp/edp function") > Reviewed-by: Matthias Brugger > Reviewed-by: AngeloGioacchino Del Regno > > Signed-off-by: Liankun Yang Tested on MT8188-based Lenovo Chromebook Duet 11. Reviewed-by: Fei Shao Tested-by: Fei Shao > --- > Changes in V5: > - No change

[RFC PATCH] drm/bridge: panel: Use devm_drm_bridge_add()

2024-10-08 Thread Fei Shao
ff80c4e9e100: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb ^ ff80c4e9e180: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb ff80c4e9e200: fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb fb ======= Signe

[PATCH] drm/mediatek: Use spin_lock_irqsave() for CRTC event lock

2024-08-28 Thread Fei Shao
Use the state-aware spin_lock_irqsave() and spin_unlock_irqrestore() to avoid unconditionally re-enabling the local interrupts. Fixes: 411f5c1eacfe ("drm/mediatek: handle events when enabling/disabling crtc") Signed-off-by: Fei Shao --- drivers/gpu/drm/mediatek/mtk_crtc.c | 5 +++

Re: [PATCH v3 2/2] drm/mediatek: change config_lock from spin_lock to spin_lock_irqsave

2024-08-28 Thread Fei Shao
hen picking up the patch. But the change itself is valid, so after fixing that: Reviewed-by: Fei Shao Regards, Fei > Signed-off-by: Jason-JH.Lin

Re: [PATCH] drm/mediatek: Declare Z Position for all planes

2024-07-18 Thread Fei Shao
the Z position for all planes with an immutable property > at least for now, so that the userspace can take its decisions > accordingly. > > Signed-off-by: AngeloGioacchino Del Regno > Reviewed-by: Fei Shao > --- > drivers/gpu/drm/mediatek/mtk_crtc.c | 2 +- > drivers/gpu/d

Re: [PATCH] drm/mediatek: Set sensible cursor width/height values to fix crash

2024-07-18 Thread Fei Shao
On Thu, Jul 18, 2024 at 4:49 PM Chen-Yu Tsai wrote: > > (CC-ed Fei Shao) > > On Thu, Jul 18, 2024 at 4:24 PM AngeloGioacchino Del Regno > wrote: > > > > Hardware-speaking, there is no feature-reduced cursor specific > > plane, so this driver reserves the last

Re: [PATCH v7 05/18] drm/mediatek: Set DRM mode configs accordingly

2024-05-26 Thread Fei Shao
On Mon, May 27, 2024 at 12:38 PM Chen-Yu Tsai wrote: > > On Mon, May 27, 2024 at 12:34 PM Fei Shao wrote: > > > > Hi Shawn, > > > > On Thu, May 2, 2024 at 6:39 PM Shawn Sung wrote: > > > > > > From: Hsiao Chien Sung > > > > &

Re: [PATCH v7 05/18] drm/mediatek: Set DRM mode configs accordingly

2024-05-26 Thread Fei Shao
Hi Shawn, On Thu, May 2, 2024 at 6:39 PM Shawn Sung wrote: > > From: Hsiao Chien Sung > > Set DRM mode configs limitation according to the hardware capabilities > and pass the IGT checks as below: > > - The test "graphics.IgtKms.kms_plane" requires a frame buffer with > width of 4512 pixels (>

Re: [PATCH] drm/mediatek: Support MT8195 OVL compatible in mtk_drm_drv

2024-02-26 Thread Fei Shao
On Mon, Feb 26, 2024 at 4:07 PM Fei Shao wrote: > > Specify the component type for mediatek,mt8195-disp-ovl in the MediaTek > DRM driver on top of commit 76cdcb87d391 ("drm/mediatek: Add MT8195 ovl > driver support"). > > With this, the compatible can function as

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

2024-02-26 Thread Fei Shao
On Mon, Feb 26, 2024 at 4:43 PM AngeloGioacchino Del Regno wrote: > > 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: > >>> Ad

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

2024-02-26 Thread Fei Shao
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 ChromeOS UI on MT8188 and usi

[PATCH] drm/mediatek: Support MT8195 OVL compatible in mtk_drm_drv

2024-02-26 Thread Fei Shao
ned-off-by: Fei Shao --- drivers/gpu/drm/mediatek/mtk_drm_drv.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/mediatek/mtk_drm_drv.c b/drivers/gpu/drm/mediatek/mtk_drm_drv.c index 14a1e0157cc4..703caba48420 100644 --- a/drivers/gpu/drm/mediatek/mtk_drm_drv.c +++ b/drive

Re: [PATCH v3 1/3] dt-bindings: mailbox: Add mediatek,gce-props.yaml

2024-01-21 Thread Fei Shao
On Fri, Jan 19, 2024 at 2:32 PM Jason-JH.Lin wrote: Hi Jason, Just few nitpicks about typo: > > Add mediatek,gce-props.yaml for common GCE properties that is used for > both mailbox providers and consumers. We place the common property > "mediatek,gce-events" in this binding currently. > > The pr

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

2023-12-26 Thread Fei Shao
o add another patch and integrate those usages. Just a thought. :) > > + > + ps_wc = vm->hactive * dsi_buf_bpp; I noticed the "& DSI_PS_WC" part was dropped (but perhaps with awareness?). While the outcome seems to always fall within the range of DSI_PS_WC so we sho

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

2023-12-26 Thread Fei Shao
Hi Angelo, On Wed, Dec 20, 2023 at 9:57 PM AngeloGioacchino Del Regno wrote: > > Change magic numerical masks with usage of the GENMASK() macro > to improve readability. > > This commit brings no functional changes. > > Signed-off-by: AngeloGioacchino Del Regno > > --- > drivers/gpu/drm/mediat

Re: [PATCH] drm/mediatek: Correctly free sg_table in gem prime vmap

2023-10-03 Thread Fei Shao
Hi, On Mon, Oct 2, 2023 at 5:21 PM Chen-Yu Tsai wrote: > > The MediaTek DRM driver implements GEM PRIME vmap by fetching the > sg_table for the object, iterating through the pages, and then > vmapping them. In essence, unlike the GEM DMA helpers which vmap > when the object is first created or im

Re: [PATCH] drm/mediatek: Add spinlock for setting vblank event in atomic_begin

2023-09-18 Thread Fei Shao
Hi Angelo, On Wed, Sep 13, 2023 at 4:35 PM AngeloGioacchino Del Regno wrote: > > Il 22/08/23 15:26, Jason-JH.Lin ha scritto: > > Add spinlock protection to avoid race condition on vblank event > > between mtk_drm_crtc_atomic_begin() and mtk_drm_finish_page_flip(). > > > > Hello Jason, > > Can you

Re: [PATCH] drm/mediatek: Add spinlock for setting vblank event in atomic_begin

2023-09-13 Thread Fei Shao
On Thu, Aug 31, 2023 at 3:12 PM Fei Shao wrote: > > On Tue, Aug 22, 2023 at 10:27 PM Jason-JH.Lin > wrote: > > > > Add spinlock protection to avoid race condition on vblank event > > between mtk_drm_crtc_atomic_begin() and mtk_drm_finish_page_flip(). > > >

Re: [RFT PATCH 05/15] drm/mediatek: Call drm_atomic_helper_shutdown() at shutdown time

2023-09-11 Thread Fei Shao
On Tue, Sep 12, 2023 at 12:11 AM Doug Anderson wrote: > [...] > > That makes sense. I had based this series on drm-misc-next which > didn't have those, but now that a new -rc1 is out it then > drm-misc-next should rebase shortly. I'll make sure that the next > version includes Uwe's changes as muc

Re: [RFT PATCH 05/15] drm/mediatek: Call drm_atomic_helper_shutdown() at shutdown time

2023-09-08 Thread Fei Shao
see issues during boot / reboot, so Reviewed-by: Fei Shao Tested-by: Fei Shao [1]: https://git.kernel.org/pub/scm/linux/kernel/git/chunkuang.hu/linux.git/commit/?h=mediatek-drm-next&id=b3af12a0b46888340e024ba8b231605bcf2d0ab3 > + .shutdown = mtk_drm_shutdown, > .dr

Re: [PATCH v4 3/4] drm/mediatek: Fix unintentional integer overflow in multiplying different types

2023-08-31 Thread Fei Shao
On Fri, Jul 14, 2023 at 6:59 PM Jason-JH.Lin wrote: > > 1. Instead of multiplying 2 variable of different types. Change to > assign a value of one variable and then multiply the other variable. > > 2. Add a int variable for multiplier calculation instead of calculating > different types multiplier

Re: [PATCH] drm/mediatek: Add spinlock for setting vblank event in atomic_begin

2023-08-31 Thread Fei Shao
;) > Signed-off-by: Jason-JH.Lin Reviewed-by: Fei Shao

Re: [PATCH v4,2/3] drm/mediatek: dp: Add the audio packet flag to mtk_dp_data struct

2023-08-14 Thread Fei Shao
g Li > Signed-off-by: Jitao Shi Reviewed-by: Fei Shao

Re: [PATCH v4,3/3] drm/mediatek: dp: Add the audio divider to mtk_dp_data struct

2023-08-14 Thread Fei Shao
On Mon, Aug 14, 2023 at 3:28 PM Shuijing Li wrote: > > Due to the difference of HW, different dividers need to be set. > > Signed-off-by: Shuijing Li > Signed-off-by: Jitao Shi Reviewed-by: Fei Shao

Re: [PATCH v9 7/7] drm/mediatek: Support DSI on MT8188 VDOSYS0

2023-08-09 Thread Fei Shao
On Thu, Aug 10, 2023 at 2:15 AM Jason-JH.Lin wrote: > > Add DSI as a main display output selection on MT8188 VDOSYS0. > > Signed-off-by: Nathan Lu > Reviewed-by: Matthias Brugger > Signed-off-by: Jason-JH.Lin Reviewed-by: Fei Shao Tested-by: Fei Shao

Re: [PATCH v9 6/7] drm/mediatek: dsi: Support dynamic connector selection

2023-08-09 Thread Fei Shao
On Thu, Aug 10, 2023 at 2:16 AM Jason-JH.Lin wrote: > > Add implementation of mtk_dsi_encoder_index to mtk_ddp_comp_func > to make mtk_dsi support dynamic connector selection. > > Signed-off-by: Jason-JH.Lin Reviewed-by: Fei Shao Tested-by: Fei Shao

Re: [PATCH v9 5/7] drm/mediatek: Add connector dynamic selection capability for mt8188

2023-08-09 Thread Fei Shao
array to a > connector routes array called mt8188_mtk_ddp_main_routes to support > dynamic selection capability for mt8188. > > Signed-off-by: Nancy Lin > Signed-off-by: Nathan Lu > Signed-off-by: Jason-JH.Lin Tested-by: Fei Shao

Re: [PATCH v9 4/7] drm/mediatek: Add encoder_index function to mtk_ddp_comp_funcs for dpi

2023-08-09 Thread Fei Shao
ementation > of mtk_dpi_encoder_index to get its encoder_index. > > Signed-off-by: Jason-JH.Lin Reviewed-by: Fei Shao Tested-by: Fei Shao

Re: [PATCH v9 3/7] drm/mediatek: Fix using wrong drm private data to bind mediatek-drm

2023-08-09 Thread Fei Shao
ned-off-by: Jason-JH.Lin > Reviewed-by: AngeloGioacchino Del Regno > > Reviewed-by: CK Hu Tested-by: Fei Shao

Re: [PATCH v9 2/7] drm/mediatek: Add crtc path enum for all_drm_priv array

2023-08-09 Thread Fei Shao
d with MT8188 and the internal display is up and functioning. Reviewed-by: Fei Shao Tested-by: Fei Shao

Re: [PATCH v9 1/7] drm/mediatek: Add mmsys_dev_num to mt8188 vdosys0 driver data

2023-08-09 Thread Fei Shao
Reviewed-by: AngeloGioacchino Del Regno > Tested with MT8188 and the internal display is up and functioning. Reviewed-by: Fei Shao Tested-by: Fei Shao

Re: [PATCH RESEND] drm/mediatek: Add valid modifier check

2023-08-03 Thread Fei Shao
On Thu, Aug 3, 2023 at 10:57 PM Daniel Vetter wrote: > > On Thu, Aug 03, 2023 at 08:48:56AM -0400, Justin Green wrote: > > > See c91acda3a380 ("drm/gem: Check for valid formats") and the related gem > > fb helper functions to see how this is supposed to be done. Thanks for shedding the light, Sim

Re: [PATCH RESEND] drm/mediatek: Add valid modifier check

2023-07-25 Thread Fei Shao
hromeOS kms_addfb_basic binary, which has a test called > "addfb25-bad-modifier" that attempts to create a framebuffer with the > modifier DRM_FORMAT_MOD_INVALID and verifies the ADDFB2 ioctl returns > EINVAL. > > Signed-off-by: Justin Green Tested-by: Fei Shao Tested the U

Re: [PATCH v2 6/6] drm/mediatek: mtk_dpi: Compress struct of_device_id entries

2023-07-19 Thread Fei Shao
> > Signed-off-by: AngeloGioacchino Del Regno > Reviewed-by: Fei Shao

Re: [PATCH v2 5/6] drm/mediatek: mtk_dpi: Use devm_platform_ioremap_resource()

2023-07-19 Thread Fei Shao
: AngeloGioacchino Del Regno > Reviewed-by: Fei Shao

Re: [PATCH v2 4/6] drm/mediatek: mtk_dpi: Switch to .remove_new() void callback

2023-07-19 Thread Fei Shao
On Wed, Jul 19, 2023 at 3:51 PM AngeloGioacchino Del Regno wrote: > > The .remove() callback cannot fail: switch to .remove_new() and > change mtk_dpi_remove() to void. > > Signed-off-by: AngeloGioacchino Del Regno > Reviewed-by: Fei Shao

Re: [PATCH v2 1/6] drm/mediatek: mtk_dpi: Simplify with devm_drm_bridge_add()

2023-07-19 Thread Fei Shao
On Wed, Jul 19, 2023 at 3:51 PM AngeloGioacchino Del Regno wrote: > > Change drm_bridge_add() to its devm variant to slightly simplify the > probe function. > > Signed-off-by: AngeloGioacchino Del Regno > Reviewed-by: Fei Shao

Re: [PATCH v2] drm/mediatek: fix uninitialized symbol

2023-07-16 Thread Fei Shao
Hi CK, On Fri, Jul 14, 2023 at 5:27 PM CK Hu (胡俊光) wrote: > > Hi, Nancy: > > On Fri, 2023-04-21 at 10:16 +0800, Nancy.Lin wrote: snip > > In line 566, the statement is > > if (nodo) { > comp_pdev = ... > } > > The comment says that only ovl_adaptoer has no device node, so the > checking s

Re: [PATCH 4/4] drm/mediatek: mtk_dpi: Use devm_platform_get_and_ioremap_resource()

2023-07-13 Thread Fei Shao
Hi Angelo, On Wed, Apr 12, 2023 at 7:53 PM AngeloGioacchino Del Regno wrote: > > Instead of the open-coded platform_get_resource, devm_ioremap_resource > switch to devm_platform_get_and_ioremap_resource(), doing exactly the > same. > > Signed-off-by: AngeloGioacchino Del Regno > > --- > driver

Re: [PATCH 3/4] drm/mediatek: mtk_dpi: Switch to devm_drm_of_get_bridge()

2023-07-13 Thread Fei Shao
rm_of_find_panel_or_bridge(). Furthermore, that will make the code visually align with your previous patch. But that's just optional, and since this patch works anyway, Reviewed-by: Fei Shao > > dev_info(dev, "Found bridge node: %pOF\n", dpi->next_bridge->of_node); > > -- > 2.40.0 > >

Re: [PATCH 2/4] drm/mediatek: mtk_dpi: Simplify with dev_err_probe()

2023-07-13 Thread Fei Shao
On Wed, Apr 12, 2023 at 7:53 PM AngeloGioacchino Del Regno wrote: > > Use dev_err_probe() across the entire probe function of this driver > to shrink the size. > > Signed-off-by: AngeloGioacchino Del Regno > Reviewed-by: Fei Shao

Re: [PATCH 1/4] drm/mediatek: mtk_dpi: Simplify with devm_drm_bridge_add()

2023-07-13 Thread Fei Shao
Hi Angelo, On Wed, Apr 12, 2023 at 7:53 PM AngeloGioacchino Del Regno wrote: > > Change drm_bridge_add() to its devm variant to slightly simplify the > probe function. > > Signed-off-by: AngeloGioacchino Del Regno > > --- > drivers/gpu/drm/mediatek/mtk_dpi.c | 5 +++-- > 1 file changed, 3 inse

Re: [PATCH v2] drm/mediatek: fix uninitialized symbol

2023-07-12 Thread Fei Shao
On Fri, Apr 21, 2023 at 10:16 AM Nancy.Lin wrote: > > fix Smatch static checker warning > - uninitialized symbol comp_pdev in mtk_ddp_comp_init. > > Fixes: 0d9eee9118b7 ("drm/mediatek: Add drm ovl_adaptor sub driver for > MT8195") > Signed-off-by: Nancy.Lin Rev

Re: [PATCH v2 2/2] drm/mediatek: Fix iommu fault during crtc enabling

2023-07-09 Thread Fei Shao
Hi Jason, On Mon, Jul 10, 2023 at 12:27 AM Jason-JH.Lin wrote: > > OVL layer should not be enabled before crtc is enabled. > The plane_state of drm_atomic_state is not sync to > the plane_state stored in mtk_crtc during crtc enabling, > so just set all planes to disabled. > Please add the "Fixes"

Re: [PATCH v6 2/2] arm64: dts: mt8192: Add node for the Mali GPU

2022-04-17 Thread Fei Shao
On Thu, Apr 14, 2022 at 10:53 AM Nick Fan wrote: > > Add a basic GPU node for mt8192. > > Signed-off-by: Nick Fan Reviewed-by: Fei Shao

Re: [PATCH v6 3/6] drm/mediatek: Detect CMDQ execution timeout

2021-10-28 Thread Fei Shao
cmdq done. If it fail to execute in next 3 vblank, > tiemout happen. > > Signed-off-by: Chun-Kuang Hu > Signed-off-by: jason-jh.lin Reviewed-by: Fei Shao > --- > drivers/gpu/drm/mediatek/mtk_drm_crtc.c | 20 ++-- > 1 file changed, 18 insertions(+), 2 del

Re: [PATCH v5 3/6] drm/mediatek: Detect CMDQ execution timeout

2021-10-27 Thread Fei Shao
Hi Chun-Kuang, On Thu, Oct 28, 2021 at 7:47 AM Chun-Kuang Hu wrote: > > Hi, Fei: > > Fei Shao 於 2021年10月27日 週三 下午5:32寫道: > > > > Hi Jason, > > > > On Wed, Oct 27, 2021 at 10:19 AM jason-jh.lin > > wrote: > > > > > > From: Chun-Kuan

Re: [PATCH v5 3/6] drm/mediatek: Detect CMDQ execution timeout

2021-10-27 Thread Fei Shao
Hi Jason, On Wed, Oct 27, 2021 at 10:19 AM jason-jh.lin wrote: > > From: Chun-Kuang Hu > > CMDQ is used to update display register in vblank period, so > it should be execute in next 2 vblank. One vblank interrupt > before send message (occasionally) and one vblank interrupt > after cmdq done. I

Re: [PATCH v11 09/16] soc: mediatek: add mtk-mmsys support for mt8195 vdosys0

2021-10-25 Thread Fei Shao
On Fri, Oct 22, 2021 at 6:13 PM Jason-JH Lin wrote: > > Hi Angelo, > > Thanks for the reviews. > > > On Thu, 2021-10-14 at 16:05 +0200, AngeloGioacchino Del Regno wrote: > > > Add mt8195 vdosys0 clock driver name and routing table to > > > the driver data of mtk-mmsys. > > > > > [snip] > > > > > >