diff for duplicates of <af0030847bb47370f977d18a18287cdc16192cde.camel@mediatek.com> diff --git a/a/content_digest b/N1/content_digest index 0f2b44a..abf2d34 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -8,22 +8,21 @@ AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> conor+dt@kernel.org <conor+dt@kernel.org> " chunkuang.hu@kernel.org <chunkuang.hu@kernel.org>\0" - "Cc\0Sunny Shen (\346\262\210\345\247\215\345\247\215) <Sunny.Shen@mediatek.com>" - " Sirius Wang (\347\216\213\347\232\223\346\230\261) <Sirius.Wang@mediatek.com>" - " Nancy Lin (\346\236\227\346\254\243\350\236\242) <Nancy.Lin@mediatek.com>" + "Cc\0devicetree@vger.kernel.org <devicetree@vger.kernel.org>" " Xiandong Wang (\347\216\213\345\205\210\345\206\254) <Xiandong.Wang@mediatek.com>" + " Jason-JH Lin (\346\236\227\347\235\277\347\245\245) <Jason-JH.Lin@mediatek.com>" + " Singo Chang (\345\274\265\350\210\210\345\234\213) <Singo.Chang@mediatek.com>" linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> dri-devel@lists.freedesktop.org <dri-devel@lists.freedesktop.org> Project_Global_Chrome_Upstream_Group <Project_Global_Chrome_Upstream_Group@mediatek.com> + treapking@chromium.org <treapking@chromium.org> + " Nancy Lin (\346\236\227\346\254\243\350\236\242) <Nancy.Lin@mediatek.com>" linux-mediatek@lists.infradead.org <linux-mediatek@lists.infradead.org> - " Jason-JH Lin (\346\236\227\347\235\277\347\245\245) <Jason-JH.Lin@mediatek.com>" - devicetree@vger.kernel.org <devicetree@vger.kernel.org> - fshao@chromium.org <fshao@chromium.org> + " Sunny Shen (\346\262\210\345\247\215\345\247\215) <Sunny.Shen@mediatek.com>" p.zabel@pengutronix.de <p.zabel@pengutronix.de> - " Singo Chang (\345\274\265\350\210\210\345\234\213) <Singo.Chang@mediatek.com>" - linux-arm-kernel@lists.infradead.org <linux-arm-kernel@lists.infradead.org> + " Sirius Wang (\347\216\213\347\232\223\346\230\261) <Sirius.Wang@mediatek.com>" matthias.bgg@gmail.com <matthias.bgg@gmail.com> - " treapking@chromium.org <treapking@chromium.org>\0" + " linux-arm-kernel@lists.infradead.org <linux-arm-kernel@lists.infradead.org>\0" "\00:1\0" "b\0" "On Mon, 2025-01-13 at 13:50 +0100, AngeloGioacchino Del Regno wrote:\n" @@ -95,4 +94,4 @@ "\n" //snip -ec443c10f324f19ac26ed4656be837fbb690ddb381717787724075e628e51118 +574810e07f341db70a5bf87478e704baf9ad4548d42434013ff381be3b46c265
diff --git a/N2/2.bin b/N2/2.bin new file mode 100644 index 0000000..333bb46 --- /dev/null +++ b/N2/2.bin @@ -0,0 +1,86 @@ +<html><body><p> +<pre> +On Mon, 2025-01-13 at 13:50 +0100, AngeloGioacchino Del Regno wrote: +>  +> External email : Please do not click links or open attachments until +> you have verified the sender or the content. +>  +>  +> Il 10/01/25 13:33, paul-pl.chen ha scritto: +> > From: "Paul-pl.Chen" <paul-pl.chen@mediatek.com> +> >  +> > Add support multiple mmsys instances in the one mediatek-drm +> > instance. +> >  +>  +> Hello Paul, +>  +> EXDMA, BLENDER and OUTPROC seem to have many things in common with +> the current +> mtk_disp_ovl.c driver. +>  +> Please commonize code from mtk_disp_ovl to reuse in these three +> components that +> you're adding, as there is a lot of code duplication. +>  +Hi Angelo,  + +Thanks for the review and feedback. + +In this patch, we have separated the new EXDMA, BLENDER, and OUTPROC +from the original OVL hardware IP. + +While EXDMA, BLENDER, OUTPROC, and OVL share similar functionalities, +they are fundamentally independent hardware components with distinct +register bases and offsets. +Merging them into a single driver would introduce unnecessary +complexity due to these differences, +making maintenance and future development more challenging.  For +eample: + + +1. Register Differences: +Taking EXDMA and OVL as an example: +OVL:// OVL Reset Register #define DISP_REG_OVL_RST 0x0014 +EXDMA:// EXDMA Reset Register #define DISP_REG_OVL_RST 0x0024 +Although the macro names are similar, the register addresses differ. +This means that any operation involving these registers must account +for different base addresses and offsets. + + +2. Function Implementation Variations: +These register differences impact function implementations: +OVL Function: +    mtk_ovl_config(); +EXDMA Function: +    mtk_disp_exdma_stop(); + +Integrating these functions would necessitate conditional logic to +handle the different registers and behaviors, +leading to less readable and maintainable code. + + +Best, Paul  + + +> Thanks, +> Angelo + + +//snip + + +</pre> +</p></body></html><!--type:text--><!--{--><pre>************* 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! +</pre><!--}--> diff --git a/N2/2.hdr b/N2/2.hdr new file mode 100644 index 0000000..da1f671 --- /dev/null +++ b/N2/2.hdr @@ -0,0 +1,3 @@ +Content-Type: text/html; + charset="utf-8" +Content-Transfer-Encoding: base64 diff --git a/a/content_digest b/N2/content_digest index 0f2b44a..73d1733 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -24,7 +24,7 @@ linux-arm-kernel@lists.infradead.org <linux-arm-kernel@lists.infradead.org> matthias.bgg@gmail.com <matthias.bgg@gmail.com> " treapking@chromium.org <treapking@chromium.org>\0" - "\00:1\0" + "\01:1\0" "b\0" "On Mon, 2025-01-13 at 13:50 +0100, AngeloGioacchino Del Regno wrote:\n" "> \n" @@ -94,5 +94,93 @@ "\n" "\n" //snip + "\01:2\0" + "b\0" + "<html><body><p>\r\n" + "<pre>\r\n" + "On Mon, 2025-01-13 at 13:50 +0100, AngeloGioacchino Del Regno wrote:\r\n" + "> \r\n" + "> External email : Please do not click links or open attachments until\r\n" + "> you have verified the sender or the content.\r\n" + "> \r\n" + "> \r\n" + "> Il 10/01/25 13:33, paul-pl.chen ha scritto:\r\n" + "> > From: "Paul-pl.Chen" <paul-pl.chen@mediatek.com>\r\n" + "> > \r\n" + "> > Add support multiple mmsys instances in the one mediatek-drm\r\n" + "> > instance.\r\n" + "> > \r\n" + "> \r\n" + "> Hello Paul,\r\n" + "> \r\n" + "> EXDMA, BLENDER and OUTPROC seem to have many things in common with\r\n" + "> the current\r\n" + "> mtk_disp_ovl.c driver.\r\n" + "> \r\n" + "> Please commonize code from mtk_disp_ovl to reuse in these three\r\n" + "> components that\r\n" + "> you're adding, as there is a lot of code duplication.\r\n" + "> \r\n" + "Hi Angelo, \r\n" + "\r\n" + "Thanks for the review and feedback.\r\n" + "\r\n" + "In this patch, we have separated the new EXDMA, BLENDER, and OUTPROC\r\n" + "from the original OVL hardware IP.\r\n" + "\r\n" + "While EXDMA, BLENDER, OUTPROC, and OVL share similar functionalities,\r\n" + "they are fundamentally independent hardware components with distinct\r\n" + "register bases and offsets.\r\n" + "Merging them into a single driver would introduce unnecessary\r\n" + "complexity due to these differences,\r\n" + "making maintenance and future development more challenging.  For\r\n" + "eample:\r\n" + "\r\n" + "\r\n" + "1. Register Differences:\r\n" + "Taking EXDMA and OVL as an example:\r\n" + "OVL:// OVL Reset Register #define DISP_REG_OVL_RST 0x0014\r\n" + "EXDMA:// EXDMA Reset Register #define DISP_REG_OVL_RST 0x0024\r\n" + "Although the macro names are similar, the register addresses differ.\r\n" + "This means that any operation involving these registers must account\r\n" + "for different base addresses and offsets.\r\n" + "\r\n" + "\r\n" + "2. Function Implementation Variations:\r\n" + "These register differences impact function implementations:\r\n" + "OVL Function:\r\n" + "    mtk_ovl_config();\r\n" + "EXDMA Function:\r\n" + "    mtk_disp_exdma_stop();\r\n" + "\r\n" + "Integrating these functions would necessitate conditional logic to\r\n" + "handle the different registers and behaviors,\r\n" + "leading to less readable and maintainable code.\r\n" + "\r\n" + "\r\n" + "Best, Paul \r\n" + "\r\n" + "\r\n" + "> Thanks,\r\n" + "> Angelo\r\n" + "\r\n" + "\r\n" + "//snip\r\n" + "\r\n" + "\r\n" + "</pre>\r\n" + "</p></body></html><!--type:text--><!--{--><pre>************* MEDIATEK Confidentiality Notice ********************\r\n" + "The information contained in this e-mail message (including any \r\n" + "attachments) may be confidential, proprietary, privileged, or otherwise\r\n" + "exempt from disclosure under applicable laws. It is intended to be \r\n" + "conveyed only to the designated recipient(s). Any use, dissemination, \r\n" + "distribution, printing, retaining or copying of this e-mail (including its \r\n" + "attachments) by unintended recipient(s) is strictly prohibited and may \r\n" + "be unlawful. If you are not an intended recipient of this e-mail, or believe \r\n" + "that you have received this e-mail in error, please notify the sender \r\n" + "immediately (by replying to this e-mail), delete any and all copies of \r\n" + "this e-mail (including any attachments) from your system, and do not\r\n" + "disclose the content of this e-mail to any other person. Thank you!\r\n" + </pre><!--}--> -ec443c10f324f19ac26ed4656be837fbb690ddb381717787724075e628e51118 +cc60b4099dc87559d3f22590110a3dde4bf8c3bf5e7a4dae287f95e26e4c3dc2
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.