From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bjorn Andersson Subject: Re: [PATCH v2 5/5] venus: register separate driver for firmware device Date: Wed, 6 Jun 2018 09:46:28 -0700 Message-ID: <20180606164628.GF510@tuxbook-pro> References: <1527884768-22392-1-git-send-email-vgarodia@codeaurora.org> <1527884768-22392-6-git-send-email-vgarodia@codeaurora.org> <2cf4f7e8-f9e6-d62b-45a8-2c348af4aafe@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <2cf4f7e8-f9e6-d62b-45a8-2c348af4aafe@linaro.org> Sender: linux-kernel-owner@vger.kernel.org To: Stanimir Varbanov Cc: Tomasz Figa , vgarodia@codeaurora.org, Hans Verkuil , Mauro Carvalho Chehab , Rob Herring , Mark Rutland , andy.gross@linaro.org, Linux Media Mailing List , Linux Kernel Mailing List , linux-arm-msm , linux-soc@vger.kernel.org, devicetree@vger.kernel.org, Alexandre Courbot List-Id: linux-arm-msm@vger.kernel.org On Mon 04 Jun 06:56 PDT 2018, Stanimir Varbanov wrote: > On 06/04/2018 04:18 PM, Tomasz Figa wrote: > > On Sat, Jun 2, 2018 at 5:27 AM Vikash Garodia wrote: [..] > >> + venus-firmware { > >> + compatible = "qcom,venus-firmware-no-tz"; > > > > I don't think "-no-tz" should be mentioned here in DT, since it's a > > firmware/software detail. > > I have to agree with Tomasz, non-tz or tz is a software detail and it > shouldn't be reflected in compatible string. > While it is software, the alternative boot and security configuration does imply different requirements on how the driver deals with the hardware. I'm not sure how you expect the kernel to be informed about the abilities of the boot/security capabilities if it's not passed through DT. In the other cases of firmware loading for co-processors this means that a number of additional resources (clocks, resets) needs to be specified in the DT node; something it seems like Venus doesn't have to do. > Also I'm not sure but what will happen if this video-firmware subnode is > not added, do you expect that backward compatibility is satisfied for > older venus versions? > I do expect that the driver should be possible to run on a 845 with the normal TZ based security model we've seen on e.g. 820. I don't know the details of Venus well enough to see if this differentiation would be sufficient. Regards, Bjorn