* [U-Boot] [PATCH] drivers/power/regulator/max77686.c: Don't use switch() on bools
@ 2016-01-17 2:44 Tom Rini
2016-01-18 5:05 ` Bin Meng
` (3 more replies)
0 siblings, 4 replies; 7+ messages in thread
From: Tom Rini @ 2016-01-17 2:44 UTC (permalink / raw)
To: u-boot
With gcc-5.3 we get a warning for using switch() on a bool type.
Rewrite these sections as if/else and update the one section that was
using 1/0 instead of true/false.
Cc: Simon Glass <sjg@chromium.org>
Cc: Przemyslaw Marczak <p.marczak@samsung.com>
Signed-off-by: Tom Rini <trini@konsulko.com>
---
drivers/power/regulator/max77686.c | 28 ++++++++--------------------
1 file changed, 8 insertions(+), 20 deletions(-)
diff --git a/drivers/power/regulator/max77686.c b/drivers/power/regulator/max77686.c
index 71678b6..7479af7 100644
--- a/drivers/power/regulator/max77686.c
+++ b/drivers/power/regulator/max77686.c
@@ -515,25 +515,19 @@ static int max77686_ldo_enable(struct udevice *dev, int op, bool *enable)
switch (on_off) {
case OPMODE_OFF:
- *enable = 0;
+ *enable = false;
break;
case OPMODE_ON:
- *enable = 1;
+ *enable = true;
break;
default:
return -EINVAL;
}
} else if (op == PMIC_OP_SET) {
- switch (*enable) {
- case 0:
- on_off = OPMODE_OFF;
- break;
- case 1:
+ if (*enable)
on_off = OPMODE_ON;
- break;
- default:
- return -EINVAL;
- }
+ else
+ on_off = OPMODE_OFF;
ret = max77686_ldo_mode(dev, op, &on_off);
if (ret)
@@ -651,16 +645,10 @@ static int max77686_buck_enable(struct udevice *dev, int op, bool *enable)
return -EINVAL;
}
} else if (op == PMIC_OP_SET) {
- switch (*enable) {
- case 0:
- on_off = OPMODE_OFF;
- break;
- case 1:
+ if (*enable)
on_off = OPMODE_ON;
- break;
- default:
- return -EINVAL;
- }
+ else
+ on_off = OPMODE_OFF;
ret = max77686_buck_mode(dev, op, &on_off);
if (ret)
--
2.7.0.rc3
^ permalink raw reply related [flat|nested] 7+ messages in thread* [U-Boot] [PATCH] drivers/power/regulator/max77686.c: Don't use switch() on bools
2016-01-17 2:44 [U-Boot] [PATCH] drivers/power/regulator/max77686.c: Don't use switch() on bools Tom Rini
@ 2016-01-18 5:05 ` Bin Meng
2016-01-18 8:16 ` Przemyslaw Marczak
` (2 subsequent siblings)
3 siblings, 0 replies; 7+ messages in thread
From: Bin Meng @ 2016-01-18 5:05 UTC (permalink / raw)
To: u-boot
On Sun, Jan 17, 2016 at 10:44 AM, Tom Rini <trini@konsulko.com> wrote:
> With gcc-5.3 we get a warning for using switch() on a bool type.
> Rewrite these sections as if/else and update the one section that was
> using 1/0 instead of true/false.
>
> Cc: Simon Glass <sjg@chromium.org>
> Cc: Przemyslaw Marczak <p.marczak@samsung.com>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> ---
> drivers/power/regulator/max77686.c | 28 ++++++++--------------------
> 1 file changed, 8 insertions(+), 20 deletions(-)
>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
^ permalink raw reply [flat|nested] 7+ messages in thread* [U-Boot] [PATCH] drivers/power/regulator/max77686.c: Don't use switch() on bools
2016-01-17 2:44 [U-Boot] [PATCH] drivers/power/regulator/max77686.c: Don't use switch() on bools Tom Rini
2016-01-18 5:05 ` Bin Meng
@ 2016-01-18 8:16 ` Przemyslaw Marczak
2016-01-18 8:23 ` Michael Trimarchi
2016-01-19 18:08 ` [U-Boot] " Tom Rini
3 siblings, 0 replies; 7+ messages in thread
From: Przemyslaw Marczak @ 2016-01-18 8:16 UTC (permalink / raw)
To: u-boot
Hello Tom,
On 01/17/2016 03:44 AM, Tom Rini wrote:
> With gcc-5.3 we get a warning for using switch() on a bool type.
> Rewrite these sections as if/else and update the one section that was
> using 1/0 instead of true/false.
>
> Cc: Simon Glass <sjg@chromium.org>
> Cc: Przemyslaw Marczak <p.marczak@samsung.com>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> ---
> drivers/power/regulator/max77686.c | 28 ++++++++--------------------
> 1 file changed, 8 insertions(+), 20 deletions(-)
>
> diff --git a/drivers/power/regulator/max77686.c b/drivers/power/regulator/max77686.c
> index 71678b6..7479af7 100644
> --- a/drivers/power/regulator/max77686.c
> +++ b/drivers/power/regulator/max77686.c
> @@ -515,25 +515,19 @@ static int max77686_ldo_enable(struct udevice *dev, int op, bool *enable)
>
> switch (on_off) {
> case OPMODE_OFF:
> - *enable = 0;
> + *enable = false;
> break;
> case OPMODE_ON:
> - *enable = 1;
> + *enable = true;
> break;
> default:
> return -EINVAL;
> }
> } else if (op == PMIC_OP_SET) {
> - switch (*enable) {
> - case 0:
> - on_off = OPMODE_OFF;
> - break;
> - case 1:
> + if (*enable)
> on_off = OPMODE_ON;
> - break;
> - default:
> - return -EINVAL;
> - }
> + else
> + on_off = OPMODE_OFF;
>
> ret = max77686_ldo_mode(dev, op, &on_off);
> if (ret)
> @@ -651,16 +645,10 @@ static int max77686_buck_enable(struct udevice *dev, int op, bool *enable)
> return -EINVAL;
> }
> } else if (op == PMIC_OP_SET) {
> - switch (*enable) {
> - case 0:
> - on_off = OPMODE_OFF;
> - break;
> - case 1:
> + if (*enable)
> on_off = OPMODE_ON;
> - break;
> - default:
> - return -EINVAL;
> - }
> + else
> + on_off = OPMODE_OFF;
>
> ret = max77686_buck_mode(dev, op, &on_off);
> if (ret)
>
Thank you for pointing that:)
Acked-by: Przemyslaw Marczak <p.marczak@samsung.com>
Best regards,
--
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com
^ permalink raw reply [flat|nested] 7+ messages in thread* [U-Boot] [PATCH] drivers/power/regulator/max77686.c: Don't use switch() on bools
2016-01-17 2:44 [U-Boot] [PATCH] drivers/power/regulator/max77686.c: Don't use switch() on bools Tom Rini
2016-01-18 5:05 ` Bin Meng
2016-01-18 8:16 ` Przemyslaw Marczak
@ 2016-01-18 8:23 ` Michael Trimarchi
2016-01-18 16:40 ` Tom Rini
2016-01-19 18:08 ` [U-Boot] " Tom Rini
3 siblings, 1 reply; 7+ messages in thread
From: Michael Trimarchi @ 2016-01-18 8:23 UTC (permalink / raw)
To: u-boot
Hi
On Sun, Jan 17, 2016 at 3:44 AM, Tom Rini <trini@konsulko.com> wrote:
> With gcc-5.3 we get a warning for using switch() on a bool type.
> Rewrite these sections as if/else and update the one section that was
> using 1/0 instead of true/false.
>
> Cc: Simon Glass <sjg@chromium.org>
> Cc: Przemyslaw Marczak <p.marczak@samsung.com>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> ---
> drivers/power/regulator/max77686.c | 28 ++++++++--------------------
> 1 file changed, 8 insertions(+), 20 deletions(-)
>
I never seen subject that contains the path of a driver.
Michael
> diff --git a/drivers/power/regulator/max77686.c b/drivers/power/regulator/max77686.c
> index 71678b6..7479af7 100644
> --- a/drivers/power/regulator/max77686.c
> +++ b/drivers/power/regulator/max77686.c
> @@ -515,25 +515,19 @@ static int max77686_ldo_enable(struct udevice *dev, int op, bool *enable)
>
> switch (on_off) {
> case OPMODE_OFF:
> - *enable = 0;
> + *enable = false;
> break;
> case OPMODE_ON:
> - *enable = 1;
> + *enable = true;
> break;
> default:
> return -EINVAL;
> }
> } else if (op == PMIC_OP_SET) {
> - switch (*enable) {
> - case 0:
> - on_off = OPMODE_OFF;
> - break;
> - case 1:
> + if (*enable)
> on_off = OPMODE_ON;
> - break;
> - default:
> - return -EINVAL;
> - }
> + else
> + on_off = OPMODE_OFF;
>
> ret = max77686_ldo_mode(dev, op, &on_off);
> if (ret)
> @@ -651,16 +645,10 @@ static int max77686_buck_enable(struct udevice *dev, int op, bool *enable)
> return -EINVAL;
> }
> } else if (op == PMIC_OP_SET) {
> - switch (*enable) {
> - case 0:
> - on_off = OPMODE_OFF;
> - break;
> - case 1:
> + if (*enable)
> on_off = OPMODE_ON;
> - break;
> - default:
> - return -EINVAL;
> - }
> + else
> + on_off = OPMODE_OFF;
>
> ret = max77686_buck_mode(dev, op, &on_off);
> if (ret)
> --
> 2.7.0.rc3
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
--
| Michael Nazzareno Trimarchi Amarula Solutions BV |
| COO - Founder Cruquiuskade 47 |
| +31(0)851119172 Amsterdam 1018 AM NL |
| [`as] http://www.amarulasolutions.com |
^ permalink raw reply [flat|nested] 7+ messages in thread* [U-Boot] [PATCH] drivers/power/regulator/max77686.c: Don't use switch() on bools
2016-01-18 8:23 ` Michael Trimarchi
@ 2016-01-18 16:40 ` Tom Rini
2016-01-18 16:41 ` Michael Trimarchi
0 siblings, 1 reply; 7+ messages in thread
From: Tom Rini @ 2016-01-18 16:40 UTC (permalink / raw)
To: u-boot
On Mon, Jan 18, 2016 at 09:23:54AM +0100, Michael Trimarchi wrote:
> Hi
>
> On Sun, Jan 17, 2016 at 3:44 AM, Tom Rini <trini@konsulko.com> wrote:
> > With gcc-5.3 we get a warning for using switch() on a bool type.
> > Rewrite these sections as if/else and update the one section that was
> > using 1/0 instead of true/false.
> >
> > Cc: Simon Glass <sjg@chromium.org>
> > Cc: Przemyslaw Marczak <p.marczak@samsung.com>
> > Signed-off-by: Tom Rini <trini@konsulko.com>
> > ---
> > drivers/power/regulator/max77686.c | 28 ++++++++--------------------
> > 1 file changed, 8 insertions(+), 20 deletions(-)
> >
>
> I never seen subject that contains the path of a driver.
Uncommon but done sometimes.
>
> Michael
>
> > diff --git a/drivers/power/regulator/max77686.c b/drivers/power/regulator/max77686.c
> > index 71678b6..7479af7 100644
> > --- a/drivers/power/regulator/max77686.c
> > +++ b/drivers/power/regulator/max77686.c
> > @@ -515,25 +515,19 @@ static int max77686_ldo_enable(struct udevice *dev, int op, bool *enable)
> >
> > switch (on_off) {
> > case OPMODE_OFF:
> > - *enable = 0;
> > + *enable = false;
> > break;
> > case OPMODE_ON:
> > - *enable = 1;
> > + *enable = true;
> > break;
> > default:
> > return -EINVAL;
> > }
> > } else if (op == PMIC_OP_SET) {
> > - switch (*enable) {
> > - case 0:
> > - on_off = OPMODE_OFF;
> > - break;
> > - case 1:
> > + if (*enable)
> > on_off = OPMODE_ON;
> > - break;
> > - default:
> > - return -EINVAL;
> > - }
> > + else
> > + on_off = OPMODE_OFF;
> >
> > ret = max77686_ldo_mode(dev, op, &on_off);
> > if (ret)
> > @@ -651,16 +645,10 @@ static int max77686_buck_enable(struct udevice *dev, int op, bool *enable)
> > return -EINVAL;
> > }
> > } else if (op == PMIC_OP_SET) {
> > - switch (*enable) {
> > - case 0:
> > - on_off = OPMODE_OFF;
> > - break;
> > - case 1:
> > + if (*enable)
> > on_off = OPMODE_ON;
> > - break;
> > - default:
> > - return -EINVAL;
> > - }
> > + else
> > + on_off = OPMODE_OFF;
> >
> > ret = max77686_buck_mode(dev, op, &on_off);
> > if (ret)
> > --
> > 2.7.0.rc3
> >
> > _______________________________________________
> > U-Boot mailing list
> > U-Boot at lists.denx.de
> > http://lists.denx.de/mailman/listinfo/u-boot
>
>
>
> --
> | Michael Nazzareno Trimarchi Amarula Solutions BV |
> | COO - Founder Cruquiuskade 47 |
> | +31(0)851119172 Amsterdam 1018 AM NL |
> | [`as] http://www.amarulasolutions.com |
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160118/79bf7311/attachment.sig>
^ permalink raw reply [flat|nested] 7+ messages in thread* [U-Boot] [PATCH] drivers/power/regulator/max77686.c: Don't use switch() on bools
2016-01-18 16:40 ` Tom Rini
@ 2016-01-18 16:41 ` Michael Trimarchi
0 siblings, 0 replies; 7+ messages in thread
From: Michael Trimarchi @ 2016-01-18 16:41 UTC (permalink / raw)
To: u-boot
Hi
On Mon, Jan 18, 2016 at 5:40 PM, Tom Rini <trini@konsulko.com> wrote:
> On Mon, Jan 18, 2016 at 09:23:54AM +0100, Michael Trimarchi wrote:
>> Hi
>>
>> On Sun, Jan 17, 2016 at 3:44 AM, Tom Rini <trini@konsulko.com> wrote:
>> > With gcc-5.3 we get a warning for using switch() on a bool type.
>> > Rewrite these sections as if/else and update the one section that was
>> > using 1/0 instead of true/false.
>> >
>> > Cc: Simon Glass <sjg@chromium.org>
>> > Cc: Przemyslaw Marczak <p.marczak@samsung.com>
>> > Signed-off-by: Tom Rini <trini@konsulko.com>
>> > ---
>> > drivers/power/regulator/max77686.c | 28 ++++++++--------------------
>> > 1 file changed, 8 insertions(+), 20 deletions(-)
>> >
>>
>> I never seen subject that contains the path of a driver.
>
> Uncommon but done sometimes.
>
Can you just fix it up and make more common? I think that patchwork
in general organize better the emails too
Michael
>>
>> Michael
>>
>> > diff --git a/drivers/power/regulator/max77686.c b/drivers/power/regulator/max77686.c
>> > index 71678b6..7479af7 100644
>> > --- a/drivers/power/regulator/max77686.c
>> > +++ b/drivers/power/regulator/max77686.c
>> > @@ -515,25 +515,19 @@ static int max77686_ldo_enable(struct udevice *dev, int op, bool *enable)
>> >
>> > switch (on_off) {
>> > case OPMODE_OFF:
>> > - *enable = 0;
>> > + *enable = false;
>> > break;
>> > case OPMODE_ON:
>> > - *enable = 1;
>> > + *enable = true;
>> > break;
>> > default:
>> > return -EINVAL;
>> > }
>> > } else if (op == PMIC_OP_SET) {
>> > - switch (*enable) {
>> > - case 0:
>> > - on_off = OPMODE_OFF;
>> > - break;
>> > - case 1:
>> > + if (*enable)
>> > on_off = OPMODE_ON;
>> > - break;
>> > - default:
>> > - return -EINVAL;
>> > - }
>> > + else
>> > + on_off = OPMODE_OFF;
>> >
>> > ret = max77686_ldo_mode(dev, op, &on_off);
>> > if (ret)
>> > @@ -651,16 +645,10 @@ static int max77686_buck_enable(struct udevice *dev, int op, bool *enable)
>> > return -EINVAL;
>> > }
>> > } else if (op == PMIC_OP_SET) {
>> > - switch (*enable) {
>> > - case 0:
>> > - on_off = OPMODE_OFF;
>> > - break;
>> > - case 1:
>> > + if (*enable)
>> > on_off = OPMODE_ON;
>> > - break;
>> > - default:
>> > - return -EINVAL;
>> > - }
>> > + else
>> > + on_off = OPMODE_OFF;
>> >
>> > ret = max77686_buck_mode(dev, op, &on_off);
>> > if (ret)
>> > --
>> > 2.7.0.rc3
>> >
>> > _______________________________________________
>> > U-Boot mailing list
>> > U-Boot at lists.denx.de
>> > http://lists.denx.de/mailman/listinfo/u-boot
>>
>>
>>
>> --
>> | Michael Nazzareno Trimarchi Amarula Solutions BV |
>> | COO - Founder Cruquiuskade 47 |
>> | +31(0)851119172 Amsterdam 1018 AM NL |
>> | [`as] http://www.amarulasolutions.com |
>
> --
> Tom
--
| Michael Nazzareno Trimarchi Amarula Solutions BV |
| COO - Founder Cruquiuskade 47 |
| +31(0)851119172 Amsterdam 1018 AM NL |
| [`as] http://www.amarulasolutions.com |
^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] drivers/power/regulator/max77686.c: Don't use switch() on bools
2016-01-17 2:44 [U-Boot] [PATCH] drivers/power/regulator/max77686.c: Don't use switch() on bools Tom Rini
` (2 preceding siblings ...)
2016-01-18 8:23 ` Michael Trimarchi
@ 2016-01-19 18:08 ` Tom Rini
3 siblings, 0 replies; 7+ messages in thread
From: Tom Rini @ 2016-01-19 18:08 UTC (permalink / raw)
To: u-boot
On Sun, Jan 17, 2016 at 02:44:37AM +0000, Tom Rini wrote:
> With gcc-5.3 we get a warning for using switch() on a bool type.
> Rewrite these sections as if/else and update the one section that was
> using 1/0 instead of true/false.
>
> Cc: Simon Glass <sjg@chromium.org>
> Cc: Przemyslaw Marczak <p.marczak@samsung.com>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
> Acked-by: Przemyslaw Marczak <p.marczak@samsung.com>
Reworded the subject and applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160119/da783e1f/attachment.sig>
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-01-19 18:08 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-17 2:44 [U-Boot] [PATCH] drivers/power/regulator/max77686.c: Don't use switch() on bools Tom Rini
2016-01-18 5:05 ` Bin Meng
2016-01-18 8:16 ` Przemyslaw Marczak
2016-01-18 8:23 ` Michael Trimarchi
2016-01-18 16:40 ` Tom Rini
2016-01-18 16:41 ` Michael Trimarchi
2016-01-19 18:08 ` [U-Boot] " Tom Rini
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.