From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 276D3C32771 for ; Wed, 28 Sep 2022 07:59:30 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=RlpweNGWeLPOx4uQ1rkkC8MIHP2NEq/C1DnUaAZTbE0=; b=XDUMzVszaLtev6 EKIG9nUwEfJaV/F/zahQ4CNsbR9GyaDWHFB8CoJ5roqpLcQN3Nk0+rtau6uKfTZUtn5zxpw9gX/Qw sQef9gUidhVaAMINwKxTRGUIJzOlfNtxkkpTv5tvaiAIWLBS3lqzHgS0jKtZaR3ef4xK9E75GjsiG HWR5p3HDSk84n7y6jHjrNuFP8CuqRO3KSHd1ttJOFmD1mmsnFGuH4EmZnC0RaxGYh4Jij6bhFfs/t cqRpyMIk1+d7eGtzwVWjxqEhUAZcXKYPAQp1dYWqImT0I5cOUpBiT8fEv3HZelfBw7MeL16fC2xc4 R//JxobevsVEThdDFslw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1odRxd-00EilV-JW; Wed, 28 Sep 2022 07:58:29 +0000 Received: from madras.collabora.co.uk ([2a00:1098:0:82:1000:25:2eeb:e5ab]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1odRxa-00EikP-PC; Wed, 28 Sep 2022 07:58:28 +0000 Received: from [192.168.1.100] (2-237-20-237.ip236.fastwebnet.it [2.237.20.237]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) (Authenticated sender: kholk11) by madras.collabora.co.uk (Postfix) with ESMTPSA id 3A9136601EB5; Wed, 28 Sep 2022 08:58:21 +0100 (BST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1664351902; bh=e/mPVgGURMWxGYEIW4p02kZFhXBi5nXgg4F5k0bpy54=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=TYk/3bPREZ0ETnF9mVPkIiNQOTPa+nZHbNZ9uVIoH3AMlqd4wowy7Cq6bygwyXOEh Jffp9pu0jq7qJ4MUQb/0MUQTYYrMcxuFiwio7uXbmueISdOytGiLHcJeKm97yGNQv9 o5Qp5bgILa/SPkA3eQJRxIBMu+hIrHi8rClVnLbm2fq8CJB7tsCnwnJ5xmgj8ET4DQ npVFafytKPtTkcC0C6H3uDcPzpXkFpepY5O58WOI1qQ6aGC6WSvZKbCuU5D63wMtCH VJHLUZK+nuOT+S5aTH05Q2HyNxu7MWCMoxAjDYbGrzBaWnSZpBi9QPuiu3RZcaJ759 iptJdJSKqXaRA== Message-ID: <68e1c8b0-04cf-acf8-b6b6-97d9eb8a7c4a@collabora.com> Date: Wed, 28 Sep 2022 09:58:18 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.2.0 Subject: Re: [PATCH v2] arm64: dts: mt8192: Add vcodec lat and core nodes Content-Language: en-US To: Krzysztof Kozlowski , =?UTF-8?B?QWxsZW4tS0ggQ2hlbmcgKOeoi+WGoOWLsyk=?= , "matthias.bgg@gmail.com" , "robh+dt@kernel.org" Cc: "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "linux-mediatek@lists.infradead.org" , Project_Global_Chrome_Upstream_Group , "devicetree@vger.kernel.org" References: <20220926105047.19419-1-allen-kh.cheng@mediatek.com> <4d1e8600-f73d-8d2b-2e7a-1b75be7624bd@collabora.com> <05ed341b-2db3-620f-7a70-dcebfaa66f1a@collabora.com> <172e10ee-22fd-ccec-1a5a-7bd0a29dbfc4@linaro.org> From: AngeloGioacchino Del Regno In-Reply-To: <172e10ee-22fd-ccec-1a5a-7bd0a29dbfc4@linaro.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220928_005826_989840_A4914C7D X-CRM114-Status: GOOD ( 15.95 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Il 28/09/22 09:04, Krzysztof Kozlowski ha scritto: > On 27/09/2022 12:17, AngeloGioacchino Del Regno wrote: >>>> >>> >>> Sorry, my bad. I alsways run `make dtbs_check` to confirm dtb with >>> bindings. I just think we didn't limit node names in mtk-vodec >>> bindings. I will pay attention next time. >>> >>> >>> Since currently the vcodec lat and core nodes are absent from the mtk >>> dts, do you think the child node name should be changed to something >>> more general (ex: video-codec) in mediatek,vcodec-subdev-decoder >>> bindings? >> >> The video codec is mt8192-vcodec-dec, while the other nodes are describing >> the VPU instances (and/or vpu cores)... I'm not sure. >> >> Krzysztof, please, can you give your opinion on that? >> > > What's the difference between them? I understand parent device is entire > block of consisting of multiple processing units? If so, video-codec > actually could fit in both places. But feel free to call it a bit > different (video-codec-core, video-codec-lat, processing-unit, even > something less generic). Sometimes it's tricky to find nice name, so I > wouldn't worry too much in that case. Just not "mt8192-vcodec" :) > The parent device is the entire block consisting of multiple processing units and has "global" control registers; children are LAT(s) and processing cores. From my understanding, the processing cores are physical cores of one big VPU and, depending on the actual (current gen) SoC, the VPU may have one or two cores. Right now, the bindings want vcodec-latX@addr, vcodec-coreX@addr (where X is a number, like vcodec-core0, vcodec-core1) but, in my opinion, changing that to video-codec-lat@addr and video-codec-core@addr would be more descriptive. ...Or should we simply leave the bindings as they are and just go with the abbreviated "vcodec-(hwtype)" names? Regards, Angelo _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel