From: Enric Balletbo i Serra <enric.balletbo@collabora.com>
To: Hans Verkuil <hverkuil@xs4all.nl>,
Eizan Miyamoto <eizan@chromium.org>,
LKML <linux-kernel@vger.kernel.org>
Cc: Andrew-CT Chen <andrew-ct.chen@mediatek.com>,
Minghsiu Tsai <minghsiu.tsai@mediatek.com>,
Houlong Wei <houlong.wei@mediatek.com>,
linux-mediatek@lists.infradead.org,
Matthias Brugger <matthias.bgg@gmail.com>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
linux-arm-kernel@lists.infradead.org,
linux-media@vger.kernel.org
Subject: Re: [PATCH v2 0/2] Refactor MDP driver and add dummy component driver
Date: Fri, 26 Jun 2020 11:37:31 +0200 [thread overview]
Message-ID: <177292b5-ebe9-6e10-c93e-0106e312ba83@collabora.com> (raw)
In-Reply-To: <25cbac25-035f-9c2b-24ec-f59403b56424@xs4all.nl>
Hi Eizan and Hans,
On 24/6/20 16:05, Hans Verkuil wrote:
> On 06/05/2020 10:40, Eizan Miyamoto wrote:
>>
>> This series depends on all changes in the series:
>> https://patchwork.kernel.org/patch/11530275/
>
> I plan on merging the v3 of this series.
>
>>
>> We are adding a dummy MDP component driver so that all the components
>> are properly configured with IOMMUs and LARBs. This is required for
>> us to get hardware video decode working in 4.19, and possibly newer
>> kernels.
>
> What is the status of this series?
>
First of all, sorry, I should have had answer before but for some reason I
missed the track of this patchset.
> There was some discussion with Enric, but that didn't come to a conclusion,
> I think.
>
Yes, my main concern is how this driver is instantiated, it is using one of the
rdma nodes, see the rdma0 node vs rdma1, to instantiate the driver,
mdp_rdma0: rdma@14001000 {
compatible = "mediatek,mt8173-mdp-rdma";
"mediatek,mt8173-mdp";
reg = <0 0x14001000 0 0x1000>;
clocks = <&mmsys CLK_MM_MDP_RDMA0>,
<&mmsys CLK_MM_MUTEX_32K>;
power-domains = <&scpsys MT8173_POWER_DOMAIN_MM>;
iommus = <&iommu M4U_PORT_MDP_RDMA0>;
mediatek,larb = <&larb0>;
mediatek,vpu = <&vpu>;
};
mdp_rdma1: rdma@14002000 {
compatible = "mediatek,mt8173-mdp-rdma";
reg = <0 0x14002000 0 0x1000>;
clocks = <&mmsys CLK_MM_MDP_RDMA1>,
<&mmsys CLK_MM_MUTEX_32K>;
power-domains = <&scpsys MT8173_POWER_DOMAIN_MM>;
iommus = <&iommu M4U_PORT_MDP_RDMA1>;
mediatek,larb = <&larb4>;
};
My point is that DT is to describe hardware not a trick to instantiate drivers,
so the "mediatek,mt8173-mdp" is completely unnecessary. We had the same issue
with the MMSYS driver and the mediatek DRM driver, and we solved by having the
mmsys driver instantiating the drm driver. I think we should apply the same
solution here, see [1] for reference. If you want to continue the discussion
maybe would be better
Thanks,
Enric
[1]
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/soc/mediatek/mtk-mmsys.c?id=667c769246b01c53ad0925d603d2a2531abd3ef2
> Regards,
>
> Hans
>
>>
>> Changes in v2:
>> - remove empty mtk_mdp_comp_init
>> - update documentation for enum mtk_mdp_comp_type
>> - remove comma after last element of mtk_mdp_comp_driver_dt_match
>>
>> Eizan Miyamoto (2):
>> [media] mtk-mdp: add driver to probe mdp components
>> [media] mtk-mdp: use pm_runtime in MDP component driver
>>
>> drivers/media/platform/mtk-mdp/mtk_mdp_comp.c | 172 ++++++++++++++---
>> drivers/media/platform/mtk-mdp/mtk_mdp_comp.h | 32 +--
>> drivers/media/platform/mtk-mdp/mtk_mdp_core.c | 182 ++++++++++++------
>> drivers/media/platform/mtk-mdp/mtk_mdp_core.h | 1 +
>> 4 files changed, 286 insertions(+), 101 deletions(-)
>>
>
_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek
prev parent reply other threads:[~2020-06-26 9:59 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-06 8:40 [PATCH v2 0/2] Refactor MDP driver and add dummy component driver Eizan Miyamoto
2020-05-06 8:40 ` [PATCH v2 1/2] [media] mtk-mdp: add driver to probe mdp components Eizan Miyamoto
2020-05-06 16:54 ` Enric Balletbo Serra
2020-05-07 11:11 ` Eizan Miyamoto
2020-05-07 13:44 ` Enric Balletbo i Serra
2020-05-15 5:21 ` Eizan Miyamoto
2020-07-20 16:09 ` Chun-Kuang Hu
2020-05-06 8:40 ` [PATCH v2 2/2] [media] mtk-mdp: use pm_runtime in MDP component driver Eizan Miyamoto
2020-05-06 17:06 ` Enric Balletbo Serra
2020-05-21 1:36 ` Eizan Miyamoto
2020-06-24 14:05 ` [PATCH v2 0/2] Refactor MDP driver and add dummy " Hans Verkuil
2020-06-26 9:37 ` Enric Balletbo i Serra [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=177292b5-ebe9-6e10-c93e-0106e312ba83@collabora.com \
--to=enric.balletbo@collabora.com \
--cc=andrew-ct.chen@mediatek.com \
--cc=eizan@chromium.org \
--cc=houlong.wei@mediatek.com \
--cc=hverkuil@xs4all.nl \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=matthias.bgg@gmail.com \
--cc=mchehab@kernel.org \
--cc=minghsiu.tsai@mediatek.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).