Nathan Chancellor 184ff4f721 OPP: Fix -Wunsequenced in _of_add_opp_table_v1() ...

Clang warns (or errors with CONFIG_WERROR=y):

  drivers/opp/of.c:1081:28: error: multiple unsequenced modifications to 'val' [-Werror,-Wunsequenced]
   1081 |                         .freq = be32_to_cpup(val++) * 1000,
        |                                                 ^
   1082 |                         .u_volt = be32_to_cpup(val++),
        |                                                   ~~
  1 error generated.

There is no sequence point in a designated initializer. Move back to
separate variables for the creation of the values, so that there are
sequence points between each evaluation and increment of val.

Fixes: 75bbc92c09d8 ("OPP: Add dev_pm_opp_add_dynamic() to allow more flexibility")
Signed-off-by: Nathan Chancellor <nathan@kernel.org>
Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>

2023-10-06 12:49:29 +05:30

..

core.c

OPP: Remove doc style comments for internal routines

2023-10-06 12:37:34 +05:30

cpu.c

OPP: Fix argument name in doc comment

2023-08-18 10:55:49 +05:30

debugfs.c

OPP: debugfs: Fix warning with W=1 builds

2023-10-06 12:37:34 +05:30

Kconfig

drivers/opp: Remove "select SRCU"

2023-01-05 11:47:21 +05:30

Makefile

treewide: Add SPDX license identifier - Makefile/Kconfig

2019-05-21 10:50:46 +02:00

of.c

OPP: Fix -Wunsequenced in _of_add_opp_table_v1()

2023-10-06 12:49:29 +05:30

opp.h

OPP: Add dev_pm_opp_add_dynamic() to allow more flexibility

2023-10-06 12:37:33 +05:30

ti-opp-supply.c

OPP: ti: Migrate to dev_pm_opp_set_config_regulators()

2022-07-08 11:27:49 +05:30