devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: <Eugen.Hristev@microchip.com>
To: <hverkuil@xs4all.nl>, <linux-media@vger.kernel.org>, <jacopo@jmondi.org>
Cc: <robh+dt@kernel.org>, <laurent.pinchart@ideasonboard.com>,
	<sakari.ailus@iki.fi>, <devicetree@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<Nicolas.Ferre@microchip.com>
Subject: Re: [PATCH v3 00/23] media: atmel: atmel-isc: implement media controller
Date: Wed, 12 Jan 2022 12:46:44 +0000	[thread overview]
Message-ID: <6cf11fa8-2a9c-badf-2243-a7ead42aa91c@microchip.com> (raw)
In-Reply-To: <50bb6274-b004-eff7-56d5-68ec8e10d3c4@xs4all.nl>

On 1/12/22 2:07 PM, Hans Verkuil wrote:
> Hi Eugen,
> 
> On 13/12/2021 14:49, Eugen Hristev wrote:
>> This series is the v3 of the series that attempts to support media controller
>> in the atmel ISC and XISC drivers.
>> This series also includes the csi2dc driver which was previously sent in a
>> separate series:
>> https://www.spinics.net/lists/linux-media/msg181042.html
>> https://www.spinics.net/lists/linux-media/msg181044.html
>> The driver now addresses comments received in latest v5 series from last year.
>>
>> The series includes some minor changes and fixes that improve the isc common
>> code base, like removing the enum frameintervals VIDIOC, fixing bytesperline
>> for planar formats, etc.
>>
>> Many thanks to folks from libcamera who helped a lot with understanding
>> how a media controller driver should behave.
>>
>> Feedback is welcome !
>>
>> Changes in v3:
>> - change in bindings, small fixes in csi2dc driver and conversion to mc
>> for the isc-base.
>> - removed some MAINTAINERS patches and used patterns in MAINTAINERS
>>
>> Changes in v2:
>> - integrated many changes suggested by Jacopo in the review of the v1 series.
>> - add a few new patches
> 
> Based on the current review status I think it would make sense to add
> the following patches from this series to a PR:
> 
> 1-6, 8-9, 11, 16, 18-22.
> 
> Are there any patches (esp. 11 and later) that require other patches
> not in this list? In other words, does this make sense?

Hello Hans,

Yes, this sounds good . I will rework the patches that are left next week.
Then I will send a v4 of this series with only the patches that are not 
yet applied. And I will rebase it on top of the media tree.

Thanks for reviewing !

Eugen
> 
> Regards,
> 
>          Hans
> 
>>
>>
>> Eugen Hristev (23):
>>    MAINTAINERS: add microchip csi2dc
>>    dt-bindings: media: atmel: csi2dc: add bindings for microchip csi2dc
>>    media: atmel: introduce microchip csi2dc driver
>>    media: atmel: atmel-isc: split the clock code into separate source
>>      file
>>    media: atmel: atmel-isc: replace video device name with module name
>>    media: atmel: atmel-sama7g5-isc: fix ispck leftover
>>    media: atmel: atmel-isc-base: use streaming status when queueing
>>      buffers
>>    media: atmel: atmel-isc-base: remove frameintervals VIDIOC
>>    media: atmel: atmel-isc-base: report frame sizes as full supported
>>      range
>>    media: atmel: atmel-isc-base: implement mbus_code support in enumfmt
>>    media: atmel: atmel-isc-base: fix bytesperline value for planar
>>      formats
>>    media: atmel: atmel-isc: implement media controller
>>    ARM: dts: at91: sama7g5: add nodes for video capture
>>    ARM: configs: at91: sama7: add xisc and csi2dc
>>    ARM: multi_v7_defconfig: add atmel video pipeline modules
>>    media: atmel: atmel-sama5d2-isc: fix wrong mask in YUYV format check
>>    media: atmel: atmel-isc-base: use mutex to lock awb workqueue from
>>      streaming
>>    media: atmel: atmel-isc-base: add wb debug messages
>>    media: atmel: atmel-isc-base: clamp wb gain coefficients
>>    media: atmel: atmel-sama7g5-isc: fix UYVY input format mbus_code typo
>>    media: atmel: atmel-isc: add raw Bayer 8bit 10bit output formats
>>    media: atmel: atmel-isc: compact the controller formats list
>>    media: atmel: atmel-isc: change format propagation to subdev into only
>>      verification
>>
>>   .../bindings/media/microchip,csi2dc.yaml      | 197 +++++
>>   MAINTAINERS                                   |  14 +-
>>   arch/arm/boot/dts/sama7g5.dtsi                |  48 ++
>>   arch/arm/configs/multi_v7_defconfig           |   3 +
>>   arch/arm/configs/sama7_defconfig              |   2 +
>>   drivers/media/platform/Makefile               |   1 +
>>   drivers/media/platform/atmel/Kconfig          |  15 +
>>   drivers/media/platform/atmel/Makefile         |   4 +-
>>   drivers/media/platform/atmel/atmel-isc-base.c | 790 ++++++-----------
>>   drivers/media/platform/atmel/atmel-isc-clk.c  | 311 +++++++
>>   drivers/media/platform/atmel/atmel-isc-mc.c   | 287 +++++++
>>   drivers/media/platform/atmel/atmel-isc.h      |  49 ++
>>   .../media/platform/atmel/atmel-sama5d2-isc.c  |  59 +-
>>   .../media/platform/atmel/atmel-sama7g5-isc.c  |  72 +-
>>   .../media/platform/atmel/microchip-csi2dc.c   | 806 ++++++++++++++++++
>>   15 files changed, 2085 insertions(+), 573 deletions(-)
>>   create mode 100644 Documentation/devicetree/bindings/media/microchip,csi2dc.yaml
>>   create mode 100644 drivers/media/platform/atmel/atmel-isc-clk.c
>>   create mode 100644 drivers/media/platform/atmel/atmel-isc-mc.c
>>   create mode 100644 drivers/media/platform/atmel/microchip-csi2dc.c
>>
> 


      reply	other threads:[~2022-01-12 12:46 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-13 13:49 [PATCH v3 00/23] media: atmel: atmel-isc: implement media controller Eugen Hristev
2021-12-13 13:49 ` [PATCH v3 01/23] MAINTAINERS: add microchip csi2dc Eugen Hristev
2021-12-13 13:49 ` [PATCH v3 02/23] dt-bindings: media: atmel: csi2dc: add bindings for " Eugen Hristev
2021-12-15 20:16   ` Rob Herring
2021-12-13 13:49 ` [PATCH v3 03/23] media: atmel: introduce microchip csi2dc driver Eugen Hristev
2022-01-11 15:43   ` Jacopo Mondi
2021-12-13 13:49 ` [PATCH v3 04/23] media: atmel: atmel-isc: split the clock code into separate source file Eugen Hristev
2022-01-11 15:43   ` Jacopo Mondi
2021-12-13 13:49 ` [PATCH v3 05/23] media: atmel: atmel-isc: replace video device name with module name Eugen Hristev
2021-12-13 13:49 ` [PATCH v3 06/23] media: atmel: atmel-sama7g5-isc: fix ispck leftover Eugen Hristev
2021-12-13 13:49 ` [PATCH v3 07/23] media: atmel: atmel-isc-base: use streaming status when queueing buffers Eugen Hristev
2021-12-13 13:49 ` [PATCH v3 08/23] media: atmel: atmel-isc-base: remove frameintervals VIDIOC Eugen Hristev
2021-12-13 13:49 ` [PATCH v3 09/23] media: atmel: atmel-isc-base: report frame sizes as full supported range Eugen Hristev
2021-12-13 13:49 ` [PATCH v3 10/23] media: atmel: atmel-isc-base: implement mbus_code support in enumfmt Eugen Hristev
2021-12-13 13:49 ` [PATCH v3 11/23] media: atmel: atmel-isc-base: fix bytesperline value for planar formats Eugen Hristev
2021-12-13 13:49 ` [PATCH v3 12/23] media: atmel: atmel-isc: implement media controller Eugen Hristev
2022-01-11 16:06   ` Jacopo Mondi
2022-01-11 16:30     ` Eugen.Hristev
2022-01-11 16:51       ` Jacopo Mondi
2022-01-11 17:00         ` Eugen.Hristev
2021-12-13 13:49 ` [PATCH v3 13/23] ARM: dts: at91: sama7g5: add nodes for video capture Eugen Hristev
2021-12-13 13:49 ` [PATCH v3 14/23] ARM: configs: at91: sama7: add xisc and csi2dc Eugen Hristev
2021-12-13 13:49 ` [PATCH v3 15/23] ARM: multi_v7_defconfig: add atmel video pipeline modules Eugen Hristev
2021-12-13 13:49 ` [PATCH v3 16/23] media: atmel: atmel-sama5d2-isc: fix wrong mask in YUYV format check Eugen Hristev
2021-12-13 13:49 ` [PATCH v3 17/23] media: atmel: atmel-isc-base: use mutex to lock awb workqueue from streaming Eugen Hristev
2022-01-12  8:58   ` Jacopo Mondi
2022-01-19 15:40     ` Eugen.Hristev
2021-12-13 13:49 ` [PATCH v3 18/23] media: atmel: atmel-isc-base: add wb debug messages Eugen Hristev
2022-01-12  9:00   ` Jacopo Mondi
2021-12-13 13:49 ` [PATCH v3 19/23] media: atmel: atmel-isc-base: clamp wb gain coefficients Eugen Hristev
2022-01-12  9:04   ` Jacopo Mondi
2021-12-13 13:49 ` [PATCH v3 20/23] media: atmel: atmel-sama7g5-isc: fix UYVY input format mbus_code typo Eugen Hristev
2022-01-12  9:04   ` Jacopo Mondi
2021-12-13 13:49 ` [PATCH v3 21/23] media: atmel: atmel-isc: add raw Bayer 8bit 10bit output formats Eugen Hristev
2022-01-12  9:05   ` Jacopo Mondi
2021-12-13 13:49 ` [PATCH v3 22/23] media: atmel: atmel-isc: compact the controller formats list Eugen Hristev
2022-01-12  9:06   ` Jacopo Mondi
2021-12-13 13:49 ` [PATCH v3 23/23] media: atmel: atmel-isc: change format propagation to subdev into only verification Eugen Hristev
2022-01-12  9:21   ` Jacopo Mondi
2022-01-20  8:43     ` Eugen.Hristev
2022-01-20  8:48       ` Eugen.Hristev
2022-01-20  8:58       ` Jacopo Mondi
2022-01-12 12:07 ` [PATCH v3 00/23] media: atmel: atmel-isc: implement media controller Hans Verkuil
2022-01-12 12:46   ` Eugen.Hristev [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=6cf11fa8-2a9c-badf-2243-a7ead42aa91c@microchip.com \
    --to=eugen.hristev@microchip.com \
    --cc=Nicolas.Ferre@microchip.com \
    --cc=devicetree@vger.kernel.org \
    --cc=hverkuil@xs4all.nl \
    --cc=jacopo@jmondi.org \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=robh+dt@kernel.org \
    --cc=sakari.ailus@iki.fi \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).