linux-kernel - Re: [PATCH V3 7/7] arm64: dts: imx8mp: add interconnect for hsio blk ctrl

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <792028b9-cd4c-4ff4-a7cb-e60c518aa573@kernel.org>
Date:   Fri, 31 Mar 2023 15:55:20 +1000
From:   Greg Ungerer <gerg@...nel.org>
To:     Marco Felsch <m.felsch@...gutronix.de>
Cc:     peng.fan@....com,
        Alexander Stein <alexander.stein@...tq-group.com>,
        abailon@...libre.com, krzysztof.kozlowski+dt@...aro.org,
        festevam@...il.com, abelvesa@...nel.org, marex@...x.de,
        Markus.Niebel@...tq-group.com,
        Ahmad Fatoum <a.fatoum@...gutronix.de>,
        paul.elder@...asonboard.com, linux-imx@....com,
        devicetree@...r.kernel.org,
        "Peng Fan (OSS)" <peng.fan@....nxp.com>, linux-pm@...r.kernel.org,
        s.hauer@...gutronix.de, robh+dt@...nel.org, aford173@...il.com,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        laurent.pinchart@...asonboard.com, linux-kernel@...r.kernel.org,
        kernel@...gutronix.de, djakov@...nel.org, shawnguo@...nel.org,
        l.stach@...gutronix.de
Subject: Re: [PATCH V3 7/7] arm64: dts: imx8mp: add interconnect for hsio blk
 ctrl

Hi Marco,

On 29/3/23 01:11, Marco Felsch wrote:
> Hi Greg,
> 
> On 23-03-29, Greg Ungerer wrote:
>> Hi Marco,
> 
> ...
> 
>>> I forgot to ask: Does your i.MX8MP have a VPU? There are i.MX8MP devices
>>> (don't know the name) which don't have support for certain IPs. If this
>>
>> The hardware platform I have is using the MIMX8ML4CVNKZAB "i.MX 8M Plus QuadLite"
>> (https://www.nxp.com/part/MIMX8ML4CVNKZAB#/) which does not have the hardware
>> video encode/decoder module (like the "i.MX 8M Plus Quad" parts).
> 
> and that's the problem :) You need to update your bootloader to a
> version which support disabling the VPU nodes else you will always see
> the errors.

I agree this is the problem, I don't agree that the boot loader is the
only place to fix this :-)  I should be able to generate a working devicetree
blob from the kernel that is good, and ready to use no runtime changes
required I figure.

It is not overly difficult to break out the vpu nodes and have them
only included when you have a board that has the iMX8MP-quad with the
VPU hardware blocks.

Example patch attached.

Regards
Greg
View attachment "0001-arm64-dts-imx8mp-separate-out-VPU-nodes.patch" of type "text/x-patch" (11867 bytes)