[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <8502e61e-7040-0f73-9f27-60985fb2283d@gmail.com>
Date: Wed, 9 Nov 2022 12:17:22 +0100
From: Matthias Brugger <matthias.bgg@...il.com>
To: Xinlei Lee (李昕磊)
<Xinlei.Lee@...iatek.com>,
Jason-JH Lin (林睿祥)
<Jason-JH.Lin@...iatek.com>,
"chunkuang.hu@...nel.org" <chunkuang.hu@...nel.org>,
"daniel@...ll.ch" <daniel@...ll.ch>,
"p.zabel@...gutronix.de" <p.zabel@...gutronix.de>,
"airlied@...ux.ie" <airlied@...ux.ie>,
Rex-BC Chen (陳柏辰)
<Rex-BC.Chen@...iatek.com>,
"angelogioacchino.delregno@...labora.com"
<angelogioacchino.delregno@...labora.com>,
"nfraprado@...labora.com" <nfraprado@...labora.com>
Cc: "dri-devel@...ts.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
"linux-mediatek@...ts.infradead.org"
<linux-mediatek@...ts.infradead.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Project_Global_Chrome_Upstream_Group
<Project_Global_Chrome_Upstream_Group@...iatek.com>
Subject: Re: [PATCH v13,0/3] Add dpi output format control for MT8186
Hi Xinlei,
On 09/11/2022 03:40, Xinlei Lee (李昕磊) wrote:
> On Tue, 2022-11-08 at 19:27 +0100, Matthias Brugger wrote:
>> Hi Xinlei,
>>
>> Somehow b4 broke with your thread but I was able to apply patch 1 and
>> 2 by hand.
>>
>> Thanks
>> Matthias
>>
>> On 24/10/2022 04:04, xinlei.lee@...iatek.com wrote:
>> > From: Xinlei Lee <xinlei.lee@...iatek.com>
>> >
>> > Base on the branch of linus/master v6.1 rc1.
>> >
>> > Change since v12:
>> > 1. Add MT8186_ prefix to variables added in mt8186-mmsys.h file.
>> >
>> > Change since v11:
>> > 1. Rebase on v6.1-rc1. Change nothing.
>> >
>> > Change since v10:
>> > 1. Modify patch title and add review tag.
>> >
>> > Change since v9:
>> > 1. Modify the location of the mmsys_dev member variable.
>> >
>> > Change since v8:
>> > 1. Modified the title and some description information.
>> >
>> > Changes since v7:
>> > 1. This series is based on the following patch:
>> > [1] soc: mediatek: Add mmsys func to adapt to dpi output for
>> > MT8186
>> >
>> > https://urldefense.com/v3/__https://patchwork.kernel.org/project/linux-mediatek/patch/1663161662-1598-2-git-send-email-xinlei.lee@mediatek.com/__;!!CTRNKA9wMg0ARbw!3tXTL3P6SgcP8Q_rcyCro64dxIXE6VuVbcNftU0ZnX6TNtU1akXwd96YfnoJs_fEig$
>> >
>> > 2. Modify the DPI_FORMAT_MASK macro definition to GENMASK(1, 0);
>> > 3. Add all settings to mtk_mmsys_ddp_dpi_fmt_config;
>> > 4. Modify the commit title to Add mt8186 dpi compatibles and
>> > platform
>> > data.
>> >
>> > Changes since v6:
>> > 1. Different from other ICs, when mt8186 DPI changes the output
>> > format,
>> > the mmsys_base+400 register needs to be set to be valid at the same
>> > time.
>> > In this series, all the situations that mmsys need to be set up
>> > are
>> > perfected (not necessarily used in practice).
>> > 2. Put the value that controls the mmsys function in mtk-mmsys.h.
>> > 3. Encountered the sink ic switched between dual edge and single
>> > edge,
>> > perfected setting and clearing mmsys bit operations in mtk_dpi.c.
>> >
>> > Changes since v5:
>> > 1. Separate the patch that adds edge_cfg_in_mmsys from the patch
>> > that
>> > adds mt8186 dpi support.
>> > 2. Move the mmsys register definition to mmsys driver.
>> >
>> > Changes since v4:
>> > 1. This series of cancellations is based on the following patches:
>> > [1] Add MediaTek SoC(vdosys1) support for mt8195
>> >
>> > https://urldefense.com/v3/__https://patchwork.kernel.org/project/linux-mediatek/cover/20220711075245.10492-1-nancy.lin@mediatek.com/__;!!CTRNKA9wMg0ARbw!3tXTL3P6SgcP8Q_rcyCro64dxIXE6VuVbcNftU0ZnX6TNtU1akXwd96Yfnqv0_QYpg$
>> >
>> > [2] Add MediaTek SoC DRM (vdosys1) support for mt8195
>> >
>> > https://urldefense.com/v3/__https://patchwork.kernel.org/project/linux-mediatek/cover/20220804072827.22383-1-nancy.lin@mediatek.com/__;!!CTRNKA9wMg0ARbw!3tXTL3P6SgcP8Q_rcyCro64dxIXE6VuVbcNftU0ZnX6TNtU1akXwd96YfnoZMZ_peA$
>> >
>> > 2. Added mtk_mmsys_update_bits function in mtk-mmsys.c;
>> > 3. MMSYS 0x400 register is modified to
>> > MT8186_MMSYS_DPI_OUTPUT_FORMAT;
>> > 4. Fix formatting issues.
>> >
>> > Changes since v3:
>> > 1. Fix formatting issues;
>> > 2. Modify the edge output control name & description;
>> > 3. Fix the threading problem.
>> >
>> > Changes since v2:
>> > 1. Modify key nouns in the description;
>> > 2. Add the label of jitao to Co-developed-by;
>> > 3. Macro definition address lowercase problem and function naming;
>> > 4. Add missing a description of this property in the mtk_dpi_conf.
>> >
>> > Change since v1:
>> > 1. Modify mt8186 compatiable location.
>> > 2. Modify MT8186_DPI_OUTPUT_FORMAT name.
>> >
>> > When MT8186 outputs dpi signal, it is necessary to add dual edge
>> > output
>> > format control in mmsys.
>> >
>> > Xinlei Lee (3):
>> > soc: mediatek: Add all settings to mtk_mmsys_ddp_dpi_fmt_config
>> > func
>> > drm: mediatek: Set dpi format in mmsys
>> > drm: mediatek: Add mt8186 dpi compatibles and platform data
>> >
>> > drivers/gpu/drm/mediatek/mtk_dpi.c | 32
>> > ++++++++++++++++++++++++++
>> > drivers/gpu/drm/mediatek/mtk_drm_drv.c | 2 ++
>> > drivers/soc/mediatek/mt8186-mmsys.h | 8 ++++---
>> > drivers/soc/mediatek/mtk-mmsys.c | 27 +++++++++++++++++--
>> > ---
>> > include/linux/soc/mediatek/mtk-mmsys.h | 7 ++++++
>> > 5 files changed, 67 insertions(+), 9 deletions(-)
>> >
>
> Hi Matthias:
>
> Is your problem solved?
>
> I tried to pull the series in link[1] to the kernel 6.1-rc1 version,
> the patches are all successfully applied, and the patch in link[1]
> looks complete.
>
> If you still think there is something wrong, please let me know and I
> can send a new version to try to fix it.
Everything is fine, no worries.
Thanks for asking.
Matthias
>
> [1]:
> https://patchwork.kernel.org/project/linux-mediatek/cover/1666577099-3859-1-git-send-email-xinlei.lee@mediatek.com/
>
> Best Regards!
> xinlei
>
> ************* MEDIATEK Confidentiality Notice
> ********************
> The information contained in this e-mail message (including any
> attachments) may be confidential, proprietary, privileged, or otherwise
> exempt from disclosure under applicable laws. It is intended to be
> conveyed only to the designated recipient(s). Any use, dissemination,
> distribution, printing, retaining or copying of this e-mail (including its
> attachments) by unintended recipient(s) is strictly prohibited and may
> be unlawful. If you are not an intended recipient of this e-mail, or believe
>
> that you have received this e-mail in error, please notify the sender
> immediately (by replying to this e-mail), delete any and all copies of
> this e-mail (including any attachments) from your system, and do not
> disclose the content of this e-mail to any other person. Thank you!
Powered by blists - more mailing lists