All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jacopo Mondi <jacopo+renesas@jmondi.org>
To: kieran.bingham+renesas@ideasonboard.com,
	laurent.pinchart+renesas@ideasonboard.com,
	niklas.soderlund+renesas@ragnatech.se, geert@linux-m68k.org
Cc: Jacopo Mondi <jacopo+renesas@jmondi.org>,
	Mauro Carvalho Chehab <mchehab@kernel.org>,
	linux-media@vger.kernel.org, linux-renesas-soc@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: [PATCH v2 00/18] media: gmsl: Reliability improvement
Date: Mon, 15 Mar 2021 14:14:54 +0100	[thread overview]
Message-ID: <20210315131512.133720-1-jacopo+renesas@jmondi.org> (raw)

Hello,
  this series follows
https://patchwork.linuxtv.org/project/linux-media/list/?series=4650

Compared to the previous iteration the most substantial changes are
- max9271: Add a wakeup() function
- max9271: Add a patch to report errors from max9271_write()
- rdacm21: Re-work ov10640 initialization
  Laurent spotted a mis-use of the SPWDN gpio signal. Fixing it an re-applying
  the v1 patch that adds the correct delays to the power-up sequence fixes all
  the ov10640 sporadic initialization errors \o/
  Details in the commit message
- rdacm21: Increase OV490 firmware boot timeout. Details in the commit message.
- media: Propose de-depreaction of subdev init() core operation
- Squash max9286 and rdacm20/21 initialization sequence rework to maintain
  bisectability as suggestd by Kieran and Laurent
- Drop i2c speed adjustment as it seems not to have any impact on reliability

Run more than 300 boot tests on the in-development version of the series.
The previous iteration fixed the failure rate down to 13% from the 25% of the
current mainline version.

This new iteration on which I run 80 boot tests gave me a single failure when
tested with RDACM21 and R8A77970 Eagle board \o/

Thanks
   j

Jacopo Mondi (18):
  media: i2c: rdamc21: Fix warning on u8 cast
  media: i2c: rdacm20: Enable noise immunity
  media: i2c: rdacm20: Embedded 'serializer' field
  media: i2c: rdacm20: Replace goto with a loop
  media: i2c: rdacm20: Report camera module name
  media: i2c: max9271: Check max9271_write() return
  media: i2c: rdacm20: Check return values
  media: i2c: rdacm20: Re-work ov10635 reset
  media: i2c: max9271: Introduce wake_up() function
  media: i2c: max9286: Adjust parameters indent
  media: i2c: rdacm21: Fix OV10640 powerdown
  media: i2c: rdacm21: Give more time to OV490 to boot
  media: i2c: max9286: Rename reverse_channel_mv
  media: i2c: max9286: Cache channel amplitude
  media: i2c: max9286: Define high channel amplitude
  media: v4l2-subdev: De-deprecate init() subdev op
  media: gmsl: Reimplement initialization sequence
  media: i2c: max9286: Rework comments in .bound()

 drivers/media/i2c/max9271.c |  37 +++++++--
 drivers/media/i2c/max9271.h |   9 ++
 drivers/media/i2c/max9286.c |  61 ++++++++------
 drivers/media/i2c/rdacm20.c | 160 ++++++++++++++++++++----------------
 drivers/media/i2c/rdacm21.c |  74 ++++++++++-------
 include/media/v4l2-subdev.h |  15 +++-
 6 files changed, 223 insertions(+), 133 deletions(-)

--
2.30.0


             reply	other threads:[~2021-03-15 13:15 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-15 13:14 Jacopo Mondi [this message]
2021-03-15 13:14 ` [PATCH v2 01/18] media: i2c: rdamc21: Fix warning on u8 cast Jacopo Mondi
2021-03-15 15:27   ` Kieran Bingham
2021-03-15 21:35   ` Laurent Pinchart
2021-03-15 13:14 ` [PATCH v2 02/18] media: i2c: rdacm20: Enable noise immunity Jacopo Mondi
2021-03-15 21:37   ` Laurent Pinchart
2021-03-16 12:56     ` Jacopo Mondi
2021-03-16 19:24       ` Laurent Pinchart
2021-03-15 13:14 ` [PATCH v2 03/18] media: i2c: rdacm20: Embedded 'serializer' field Jacopo Mondi
2021-03-15 15:29   ` Kieran Bingham
2021-03-15 13:14 ` [PATCH v2 04/18] media: i2c: rdacm20: Replace goto with a loop Jacopo Mondi
2021-03-15 15:36   ` Kieran Bingham
2021-03-15 13:14 ` [PATCH v2 05/18] media: i2c: rdacm20: Report camera module name Jacopo Mondi
2021-03-15 13:15 ` [PATCH v2 06/18] media: i2c: max9271: Check max9271_write() return Jacopo Mondi
2021-03-15 15:46   ` Kieran Bingham
2021-03-15 21:38   ` Laurent Pinchart
2021-03-15 13:15 ` [PATCH v2 07/18] media: i2c: rdacm20: Check return values Jacopo Mondi
2021-03-15 13:15 ` [PATCH v2 08/18] media: i2c: rdacm20: Re-work ov10635 reset Jacopo Mondi
2021-03-15 13:15 ` [PATCH v2 09/18] media: i2c: max9271: Introduce wake_up() function Jacopo Mondi
2021-03-15 17:14   ` Kieran Bingham
2021-03-15 21:43   ` Laurent Pinchart
2021-03-15 13:15 ` [PATCH v2 10/18] media: i2c: max9286: Adjust parameters indent Jacopo Mondi
2021-03-15 13:15 ` [PATCH v2 11/18] media: i2c: rdacm21: Fix OV10640 powerdown Jacopo Mondi
2021-03-15 17:20   ` Kieran Bingham
2021-03-15 21:45   ` Laurent Pinchart
2021-03-15 13:15 ` [PATCH v2 12/18] media: i2c: rdacm21: Give more time to OV490 to boot Jacopo Mondi
2021-03-15 17:22   ` Kieran Bingham
2021-03-17 10:04     ` Jacopo Mondi
2021-03-19  0:29       ` Laurent Pinchart
2021-03-19 14:53         ` Jacopo Mondi
2021-03-15 21:52   ` Laurent Pinchart
2021-03-15 13:15 ` [PATCH v2 13/18] media: i2c: max9286: Rename reverse_channel_mv Jacopo Mondi
2021-03-15 13:15 ` [PATCH v2 14/18] media: i2c: max9286: Cache channel amplitude Jacopo Mondi
2021-03-15 13:15 ` [PATCH v2 15/18] media: i2c: max9286: Define high " Jacopo Mondi
2021-03-15 13:15 ` [PATCH v2 16/18] media: v4l2-subdev: De-deprecate init() subdev op Jacopo Mondi
2021-03-15 21:56   ` Laurent Pinchart
2021-03-15 13:15 ` [PATCH v2 17/18] media: gmsl: Reimplement initialization sequence Jacopo Mondi
2021-03-15 13:15 ` [PATCH v2 18/18] media: i2c: max9286: Rework comments in .bound() Jacopo Mondi
2021-03-15 17:28   ` Kieran Bingham
2021-03-15 21:57   ` Laurent Pinchart

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=20210315131512.133720-1-jacopo+renesas@jmondi.org \
    --to=jacopo+renesas@jmondi.org \
    --cc=geert@linux-m68k.org \
    --cc=kieran.bingham+renesas@ideasonboard.com \
    --cc=laurent.pinchart+renesas@ideasonboard.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=mchehab@kernel.org \
    --cc=niklas.soderlund+renesas@ragnatech.se \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.