From: Andrzej Hajda <a.hajda@samsung.com> To: Andy Shevchenko <andy.shevchenko@gmail.com> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>, Marek Szyprowski <m.szyprowski@samsung.com>, "Rafael J. Wysocki" <rafael@kernel.org>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Javier Martinez Canillas <javierm@redhat.com>, linux-arm Mailing List <linux-arm-kernel@lists.infradead.org>, Mark Brown <broonie@kernel.org> Subject: Re: [PATCH 3/3] drivers: use probe_err function in obvious cases Date: Wed, 17 Oct 2018 11:10:57 +0200 [thread overview] Message-ID: <20181017091059eucas1p2919e8b5984d4b8998438251a26987068~eWli2LsTV3025130251eucas1p2r@eucas1p2.samsung.com> (raw) In-Reply-To: <CAHp75VeZEtG7f3WmgvLrirReSwgbOQDp1qiBZ47652poMw8BUQ@mail.gmail.com> On 16.10.2018 15:51, Andy Shevchenko wrote: > On Tue, Oct 16, 2018 at 10:22 AM Andrzej Hajda <a.hajda@samsung.com> wrote: >> The patch replaces obviously matching code with probe_err function. >> There are many more places where probe_err could be used. >> The patch shows how the new function should be used, and how it >> improves the code. >> --- a/drivers/gpio/gpio-pca953x.c >> +++ b/drivers/gpio/gpio-pca953x.c >> @@ -824,9 +824,7 @@ static int pca953x_probe(struct i2c_client *client, >> reg = devm_regulator_get(&client->dev, "vcc"); >> if (IS_ERR(reg)) { >> ret = PTR_ERR(reg); >> - if (ret != -EPROBE_DEFER) >> - dev_err(&client->dev, "reg get err: %d\n", ret); >> - return ret; >> + return probe_err(&client->dev, ret, "reg get err: %d\n", ret); > No need to assign ret above. > As I mentioned, better to print a value for everyone. OK. Thanks for all reviews. I have posted v2 of patches 1 and 2 as replies in this thread. As patch 3 requires splitting, adjusting arguments of probe_err (remove err and newline), and depends on patch 1 I will prepare it and post later, probably next cycle as suggested Javier. Regards Andrzej > >> } > Please, split it at least one patch per subsystem. >
WARNING: multiple messages have this Message-ID (diff)
From: a.hajda@samsung.com (Andrzej Hajda) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 3/3] drivers: use probe_err function in obvious cases Date: Wed, 17 Oct 2018 11:10:57 +0200 [thread overview] Message-ID: <20181017091059eucas1p2919e8b5984d4b8998438251a26987068~eWli2LsTV3025130251eucas1p2r@eucas1p2.samsung.com> (raw) In-Reply-To: <CAHp75VeZEtG7f3WmgvLrirReSwgbOQDp1qiBZ47652poMw8BUQ@mail.gmail.com> On 16.10.2018 15:51, Andy Shevchenko wrote: > On Tue, Oct 16, 2018 at 10:22 AM Andrzej Hajda <a.hajda@samsung.com> wrote: >> The patch replaces obviously matching code with probe_err function. >> There are many more places where probe_err could be used. >> The patch shows how the new function should be used, and how it >> improves the code. >> --- a/drivers/gpio/gpio-pca953x.c >> +++ b/drivers/gpio/gpio-pca953x.c >> @@ -824,9 +824,7 @@ static int pca953x_probe(struct i2c_client *client, >> reg = devm_regulator_get(&client->dev, "vcc"); >> if (IS_ERR(reg)) { >> ret = PTR_ERR(reg); >> - if (ret != -EPROBE_DEFER) >> - dev_err(&client->dev, "reg get err: %d\n", ret); >> - return ret; >> + return probe_err(&client->dev, ret, "reg get err: %d\n", ret); > No need to assign ret above. > As I mentioned, better to print a value for everyone. OK. Thanks for all reviews. I have posted v2 of patches 1 and 2 as replies in this thread. As patch 3 requires splitting, adjusting arguments of probe_err (remove err and newline), and depends on patch 1 I will prepare it and post later, probably next cycle as suggested Javier. Regards Andrzej > >> } > Please, split it at least one patch per subsystem. >
next prev parent reply other threads:[~2018-10-17 9:11 UTC|newest] Thread overview: 58+ messages / expand[flat|nested] mbox.gz Atom feed top [not found] <CGME20181016072248eucas1p18943ce87e084797cd597afd4edb65a65@eucas1p1.samsung.com> 2018-10-16 7:22 ` [PATCH 0/3] driver core: add probe error check helper Andrzej Hajda 2018-10-16 7:22 ` Andrzej Hajda [not found] ` <CGME20181016072249eucas1p2303b8c06a9f7c839a971fe065b0c752e@eucas1p2.samsung.com> 2018-10-16 7:22 ` [PATCH 1/3] driver core: add probe_err log helper Andrzej Hajda 2018-10-16 7:22 ` Andrzej Hajda 2018-10-16 9:32 ` Javier Martinez Canillas 2018-10-16 9:32 ` Javier Martinez Canillas 2018-10-16 10:27 ` Mark Brown 2018-10-16 10:27 ` Mark Brown 2018-10-16 11:09 ` Greg Kroah-Hartman 2018-10-16 11:09 ` Greg Kroah-Hartman 2018-10-16 11:01 ` Andy Shevchenko 2018-10-16 11:01 ` Andy Shevchenko 2018-10-16 11:29 ` Andrzej Hajda 2018-10-16 11:29 ` Andrzej Hajda [not found] ` <605bd00e-ed0d-4259-bdc3-1784b2b3b16a@samsung.com> 2018-10-16 12:55 ` Andrzej Hajda 2018-10-16 12:55 ` Andrzej Hajda 2018-10-16 13:55 ` Andy Shevchenko 2018-10-16 13:55 ` Andy Shevchenko [not found] ` <CGME20181017085832eucas1p23dbb86230149a13634b8a091c3b1301f@eucas1p2.samsung.com> 2018-10-17 8:58 ` [PATCH v2 " Andrzej Hajda 2018-10-17 8:58 ` Andrzej Hajda 2018-10-17 9:56 ` Mark Brown 2018-10-17 9:56 ` Mark Brown 2018-10-17 11:30 ` Andy Shevchenko 2018-10-17 11:30 ` Andy Shevchenko 2018-10-17 11:49 ` Greg Kroah-Hartman 2018-10-17 11:49 ` Greg Kroah-Hartman 2018-10-17 11:29 ` [PATCH " Russell King - ARM Linux 2018-10-17 11:29 ` Russell King - ARM Linux 2018-10-17 11:33 ` Andy Shevchenko 2018-10-17 11:33 ` Andy Shevchenko [not found] ` <CGME20181017132209eucas1p1d23615adc15663caaffb3c117dc61151@eucas1p1.samsung.com> 2018-10-17 13:22 ` [PATCH v3 1/4] " Andrzej Hajda 2018-10-17 13:22 ` Andrzej Hajda 2018-10-18 1:45 ` [PATCH 1/3] " Joe Perches 2018-10-18 1:45 ` Joe Perches [not found] ` <CGME20181016072249eucas1p28855602564eb74a771fe521712962680@eucas1p2.samsung.com> 2018-10-16 7:22 ` [PATCH 2/3] driver core: add deferring probe reason to devices_deferred property Andrzej Hajda 2018-10-16 7:22 ` Andrzej Hajda 2018-10-16 9:25 ` Mark Brown 2018-10-16 9:47 ` Javier Martinez Canillas 2018-10-16 9:47 ` Javier Martinez Canillas 2018-10-16 13:42 ` Andy Shevchenko 2018-10-16 13:42 ` Andy Shevchenko [not found] ` <CGME20181017085952eucas1p2ff93b96d300aab2dc70b97bf266978ab@eucas1p2.samsung.com> 2018-10-17 8:59 ` [PATCH v2 " Andrzej Hajda 2018-10-17 8:59 ` Andrzej Hajda 2018-10-17 11:35 ` Andy Shevchenko 2018-10-17 11:35 ` Andy Shevchenko [not found] ` <CGME20181017132404eucas1p2a413f6853af8f11a874650b6289d56cf@eucas1p2.samsung.com> 2018-10-17 13:24 ` [PATCH v3 2/4] " Andrzej Hajda 2018-10-17 13:24 ` Andrzej Hajda 2018-10-17 14:17 ` Andy Shevchenko 2018-10-17 14:17 ` Andy Shevchenko [not found] ` <CGME20181018064916eucas1p22f03a19eb09cecdb0061dc5a4bf99ce6@eucas1p2.samsung.com> 2018-10-18 6:49 ` [PATCH v4 2/3] " Andrzej Hajda 2018-10-18 6:49 ` Andrzej Hajda [not found] ` <CGME20181016072250eucas1p1a763670c8509d20a6e6847eadb246817@eucas1p1.samsung.com> 2018-10-16 7:22 ` [PATCH 3/3] drivers: use probe_err function in obvious cases Andrzej Hajda 2018-10-16 9:52 ` Javier Martinez Canillas 2018-10-16 9:52 ` Javier Martinez Canillas 2018-10-16 13:51 ` Andy Shevchenko 2018-10-16 13:51 ` Andy Shevchenko 2018-10-17 9:10 ` Andrzej Hajda [this message] 2018-10-17 9:10 ` Andrzej Hajda
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='20181017091059eucas1p2919e8b5984d4b8998438251a26987068~eWli2LsTV3025130251eucas1p2r@eucas1p2.samsung.com' \ --to=a.hajda@samsung.com \ --cc=andy.shevchenko@gmail.com \ --cc=b.zolnierkie@samsung.com \ --cc=broonie@kernel.org \ --cc=gregkh@linuxfoundation.org \ --cc=javierm@redhat.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=m.szyprowski@samsung.com \ --cc=rafael@kernel.org \ /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: linkBe 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.