linux-kernel - Re: [PATCH 00/11] media: exynos4-is: Improve support for s5pv210 and parallel ports

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: <07fb9000-ae00-efcd-e91a-48765ff3d4bf@xs4all.nl>
Date:   Wed, 24 Jun 2020 13:39:50 +0200
From:   Hans Verkuil <hverkuil@...all.nl>
To:     Jonathan Bakker <xc-racer2@...e.ca>, kyungmin.park@...sung.com,
        s.nawrocki@...sung.com, mchehab@...nel.org, kgene@...nel.org,
        krzk@...nel.org, linux-media@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org,
        linux-samsung-soc@...r.kernel.org, linux-kernel@...r.kernel.org,
        Tomasz Figa <tfiga@...omium.org>
Subject: Re: [PATCH 00/11] media: exynos4-is: Improve support for s5pv210 and
 parallel ports

Can someone from Samsung or someone who knows this SoC take a look at this series?

This series looks sane to me, so I'll probably merge this if nobody replies
in the next two weeks or so.

Regards,

	Hans

On 26/04/2020 04:26, Jonathan Bakker wrote:
> This patchset improves support in the exynos4-is driver for s5pv210
> platforms as well as for sensors connected to the parallel ports
> and not using the CSIS device.
> 
> Fixes range from nullptr exceptions to probe fixes to JPEG fixes.
> 
> Notably, it also changes the fwnode probing of parallel port sensors
> to match the binding doc.  The binding doc said port A = reg 0 and
> port B = reg 1 but the driver implemented A=1 and B=2.  The only in-tree
> user of parallel ports is currently Goni, which notes that it uses port
> A but has reg = 1.  Also note that the sensor driver is not mainlined
> despite being present.  I have left the DTS as-is.
> 
> Some patches cleaning up non-DT support are also included.
> 
> The patches have been tested on a first-gen Galaxy S GT-i900M device based
> on S5PV210.  Two sensors (CE147 and S5KA3DFX, both with non-mainline
> drivers), both attached to port A, were succesfully used.
> 
> Jonathan Bakker (10):
>   media: exynos4-is: Remove static driver data for S5PV210 FIMC variants
>   media: exynos4-is: Fix nullptr when no CSIS device present
>   media: exynos4-is: Correct missing entity function initialization
>   media: exynos4-is: Improve support for sensors with multiple pads
>   media: exynos4-is: Properly set JPEG options when not using CSIS
>   media: exynos4-is: Add support for multiple sensors on one port
>   media: exynos4-is: Remove inh_sensor_ctrls
>   media: exynos4-is: Remove unused struct member input_index
>   media: exynos4-is: Prevent duplicate call to media_pipeline_stop
>   media: exynos4-is: Correct parallel port probing
> 
> Tomasz Figa (1):
>   media: exynos4-is: Request syscon only if ISP writeback is present
> 
>  .../media/platform/exynos4-is/fimc-capture.c  | 22 ++----
>  drivers/media/platform/exynos4-is/fimc-core.c | 67 ++----------------
>  drivers/media/platform/exynos4-is/fimc-core.h |  5 --
>  drivers/media/platform/exynos4-is/fimc-reg.c  |  7 ++
>  drivers/media/platform/exynos4-is/media-dev.c | 68 ++++++++++++++-----
>  drivers/media/platform/exynos4-is/media-dev.h |  1 +
>  include/media/drv-intf/exynos-fimc.h          |  2 +-
>  7 files changed, 72 insertions(+), 100 deletions(-)
>