linux-kernel - Re: [PATCH v9 2/3] media: rockchip: Add a driver for Rockchip's camera interface

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: <2afa69d0-93f6-4033-ad87-c3bf01588ba9@wolfvision.net>
Date:   Wed, 8 Nov 2023 09:50:25 +0100
From:   Michael Riesch <michael.riesch@...fvision.net>
To:     Mehdi Djait <mehdi.djait@...tlin.com>
Cc:     mchehab@...nel.org, heiko@...ech.de, hverkuil-cisco@...all.nl,
        krzysztof.kozlowski+dt@...aro.org, robh+dt@...nel.org,
        conor+dt@...nel.org, linux-media@...r.kernel.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        thomas.petazzoni@...tlin.com, alexandre.belloni@...tlin.com,
        maxime.chevallier@...tlin.com, paul.kocialkowski@...tlin.com
Subject: Re: [PATCH v9 2/3] media: rockchip: Add a driver for Rockchip's
 camera interface

Hi Mehdi,

On 11/3/23 09:23, Mehdi Djait wrote:
> Hello Michael,
> 
> On Tue, Oct 31, 2023 at 11:23:17PM +0100, Michael Riesch wrote:
>> Hi Mehdi,
>>
>> Thanks for your work! Please find a few comments inline:
>>
> 
> thank you for the comments! 
> 
> Have you been able to test this driver ? That would be really nice.

I applied your patches and added some modifications (= crude hacks) on
top of them to bring up the RK3568 VICAP (note that I don't have any
PX30 hardware). This setup is not yet able to capture the input stream
(from a HDMI receiver chip), but I am on it.

One question popped up: to get the cif driver to probe with my device
tree I had to request the IRQ with IRQF_SHARED. This is due to the
approach that the CIF block has one IRQ but is represented by two
drivers: the cif driver and the rockchip-iommu driver.

Subsequently I was surprised that you are not using the MMU at all,
although the PX30 VIP features one. Is there any particular reason for that?

Can we request the IRQ with IRQF_SHARED anyway?

Thanks and best regards,
Michael