All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anatolij Gustschin <agust@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4 04/13] regmap: Improve error handling
Date: Mon, 6 Aug 2018 15:09:40 +0200	[thread overview]
Message-ID: <20180806150940.70f80054@crub> (raw)
In-Reply-To: <20180803080118.27748-4-mario.six@gdsys.cc>

On Fri,  3 Aug 2018 10:01:09 +0200
Mario Six mario.six at gdsys.cc wrote:

> ofnode_read_simple_addr_cells may fail and return a negative error code.
> Check for this when initializing regmaps.
> 
> Also check if both_len is zero, since this is perfectly possible, and
> would lead to a division-by-zero further down the line.
> 
> Reviewed-by: Anatolij Gustschin <agust@denx.de>
> Reviewed-by: Simon Glass <sjg@chromium.org>
> Signed-off-by: Mario Six <mario.six@gdsys.cc>
> 
> ---
> 
> v3 -> v4:
> No changes
> 
> v2 -> v3:
> New in v3
> 
> ---
>  drivers/core/regmap.c | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)

Applied to u-boot-staging/agust at denx.de, thanks!

--
Anatolij

  reply	other threads:[~2018-08-06 13:09 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-08-03  8:01 [U-Boot] [PATCH v4 01/13] test: regmap: Increase size of syscon0 memory Mario Six
2018-08-03  8:01 ` [U-Boot] [PATCH v4 02/13] regmap: Fix documentation Mario Six
2018-08-06 13:01   ` Anatolij Gustschin
2018-08-03  8:01 ` [U-Boot] [PATCH v4 03/13] regmap: Add documentation Mario Six
2018-08-06 13:02   ` Anatolij Gustschin
2018-08-03  8:01 ` [U-Boot] [PATCH v4 04/13] regmap: Improve error handling Mario Six
2018-08-06 13:09   ` Anatolij Gustschin [this message]
2018-08-03  8:01 ` [U-Boot] [PATCH v4 05/13] regmap: Introduce init_range Mario Six
2018-08-06 13:10   ` Anatolij Gustschin
2018-08-03  8:01 ` [U-Boot] [PATCH v4 06/13] regmap: Add error output Mario Six
2018-08-06 13:10   ` Anatolij Gustschin
2018-08-03  8:01 ` [U-Boot] [PATCH v4 07/13] regmap: Add raw read/write functions Mario Six
2018-08-06 13:11   ` Anatolij Gustschin
2018-08-06 19:11   ` Anatolij Gustschin
2018-08-07  7:36     ` Mario Six
2018-08-07  7:45       ` Anatolij Gustschin
2018-08-07 14:31       ` Daniel Schwierzeck
2018-08-08  6:19         ` Mario Six
2018-08-03  8:01 ` [U-Boot] [PATCH v4 08/13] regmap: Support reading from specific range Mario Six
2018-08-06 13:16   ` Anatolij Gustschin
2018-08-06 13:16   ` Anatolij Gustschin
2018-08-03  8:01 ` [U-Boot] [PATCH v4 09/13] regmap: Define regmap_{get,set} Mario Six
2018-08-03  8:01 ` [U-Boot] [PATCH v4 10/13] test: regmap: Add test for regmap_{set, get} Mario Six
2018-08-06 13:26   ` Anatolij Gustschin
2018-08-03  8:01 ` [U-Boot] [PATCH v4 11/13] misc: Sort Makefile entries Mario Six
2018-08-06 14:18   ` [U-Boot] [PATCH v5 " Anatolij Gustschin
2018-08-06 14:42     ` Anatolij Gustschin
2018-08-06 14:20   ` [U-Boot] [PATCH v4 " Anatolij Gustschin
2018-08-03  8:01 ` [U-Boot] [PATCH v4 12/13] misc: Add gdsys_soc driver Mario Six
2018-08-06 16:13   ` Anatolij Gustschin
2018-08-03  8:01 ` [U-Boot] [PATCH v4 13/13] misc: Add IHS FPGA driver Mario Six
2018-08-06 16:16   ` Anatolij Gustschin
2018-08-06 13:01 ` [U-Boot] [PATCH v4 01/13] test: regmap: Increase size of syscon0 memory Anatolij Gustschin
2018-08-06 20:56 ` Anatolij Gustschin
2018-08-07  7:37   ` Mario Six

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=20180806150940.70f80054@crub \
    --to=agust@denx.de \
    --cc=u-boot@lists.denx.de \
    /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.