lkml.org 
[lkml]   [2019]   [Apr]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: Applied "regulator: core: do not report EPROBE_DEFER as error but as debug" to the regulator tree
    From
    Date
    On 4/18/19 12:26, Mark Brown wrote:
    > The patch
    >
    > regulator: core: do not report EPROBE_DEFER as error but as debug
    >
    > has been applied to the regulator tree at
    >
    > https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git for-5.2


    hi Mark, I cant find the commit in the tree. Can you check if it was merged?



    >
    > All being well this means that it will be integrated into the linux-next
    > tree (usually sometime in the next 24 hours) and sent to Linus during
    > the next merge window (or sooner if it is a bug fix), however if
    > problems are discovered then the patch may be dropped or reverted.
    >
    > You may get further e-mails resulting from automated or manual testing
    > and review of the tree, please engage with people reporting problems and
    > send followup patches addressing any issues that are reported if needed.
    >
    > If any updates are required or you are submitting further changes they
    > should be sent as incremental updates against current git, existing
    > patches will not be replaced.
    >
    > Please add any relevant lists and maintainers to the CCs when replying
    > to this mail.
    >
    > Thanks,
    > Mark
    >
    >>From 317f0111220921e87a168e4e6cec275df4e9be01 Mon Sep 17 00:00:00 2001
    > From: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>
    > Date: Wed, 17 Apr 2019 21:24:43 +0200
    > Subject: [PATCH] regulator: core: do not report EPROBE_DEFER as error but as
    > debug
    >
    > Temporary failures to get a regulator (EPROBE_DEFER) should be logged
    > as debug information instead of errors.
    >
    > Signed-off-by: Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>
    > Signed-off-by: Mark Brown <broonie@kernel.org>
    > ---
    > drivers/regulator/core.c | 9 +++++++--
    > 1 file changed, 7 insertions(+), 2 deletions(-)
    >
    > diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c
    > index 5a9ebcf7fe7a..08ccabe07fe3 100644
    > --- a/drivers/regulator/core.c
    > +++ b/drivers/regulator/core.c
    > @@ -4345,8 +4345,6 @@ int regulator_bulk_get(struct device *dev, int num_consumers,
    > consumers[i].supply);
    > if (IS_ERR(consumers[i].consumer)) {
    > ret = PTR_ERR(consumers[i].consumer);
    > - dev_err(dev, "Failed to get supply '%s': %d\n",
    > - consumers[i].supply, ret);
    > consumers[i].consumer = NULL;
    > goto err;
    > }
    > @@ -4355,6 +4353,13 @@ int regulator_bulk_get(struct device *dev, int num_consumers,
    > return 0;
    >
    > err:
    > + if (ret != -EPROBE_DEFER)
    > + dev_err(dev, "Failed to get supply '%s': %d\n",
    > + consumers[i].supply, ret);
    > + else
    > + dev_dbg(dev, "Failed to get supply '%s', deferring\n",
    > + consumers[i].supply);
    > +
    > while (--i >= 0)
    > regulator_put(consumers[i].consumer);
    >
    >

    \
     
     \ /
      Last update: 2019-04-23 11:50    [W:3.210 / U:0.232 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site