diff options
| author |   Georgi Djakov <djakov@kernel.org> | 2022-09-20 15:57:00 +0300 |
|---|---|---|
| committer | Georgi Djakov <djakov@kernel.org> | 2022-09-20 15:57:00 +0300 |
| commit | 7360d55ba1993cb59267507d04b7e62c40bad424 (patch) | |
| tree | 2bf9f750fbff84e79cd383d104306319f9de5424 /include/linux | |
| parent | interconnect: qcom: Kconfig: Make INTERCONNECT_QCOM tristate (diff) | |
| parent | interconnect: imx: Make imx_icc_unregister() return void (diff) | |
| download | linux-dev-7360d55ba1993cb59267507d04b7e62c40bad424.tar.xz linux-dev-7360d55ba1993cb59267507d04b7e62c40bad424.zip | |
Merge branch 'icc-ignore-return-val' into icc-next
Today remove callbacks of platform devices return an int. This is unfortunate
because the device core ignores the return value and so the platform code only
emits a warning (and still removes the device).
The longterm quest is to make these remove callbacks return void instead.
This series is a preparation for that, with the goal to make the remove
callbacks obviously always return 0. This way when the prototype of
these functions is changed to return void, the change is straight
forward and easy to review.
Link: https://lore.kernel.org/r/20220718121409.171773-1-u.kleine-koenig@pengutronix.de
Signed-off-by: Georgi Djakov <djakov@kernel.org>
Diffstat (limited to 'include/linux')
| -rw-r--r-- | include/linux/interconnect-provider.h | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/include/linux/interconnect-provider.h b/include/linux/interconnect-provider.h index 6bd01f7159c6..cd5c5a27557f 100644 --- a/include/linux/interconnect-provider.h +++ b/include/linux/interconnect-provider.h @@ -123,7 +123,7 @@ void icc_node_add(struct icc_node *node, struct icc_provider *provider); void icc_node_del(struct icc_node *node); int icc_nodes_remove(struct icc_provider *provider); int icc_provider_add(struct icc_provider *provider); -int icc_provider_del(struct icc_provider *provider); +void icc_provider_del(struct icc_provider *provider); struct icc_node_data *of_icc_get_from_provider(struct of_phandle_args *spec); void icc_sync_state(struct device *dev); @@ -172,9 +172,8 @@ static inline int icc_provider_add(struct icc_provider *provider) return -ENOTSUPP; } -static inline int icc_provider_del(struct icc_provider *provider) +static inline void icc_provider_del(struct icc_provider *provider) { - return -ENOTSUPP; } static inline struct icc_node_data *of_icc_get_from_provider(struct of_phandle_args *spec) |