mbox series

[0/8] interconnect: Prepare making platform remove callbacks return void

Message ID 20220715203652.89912-1-u.kleine-koenig@pengutronix.de
Headers show
Series interconnect: Prepare making platform remove callbacks return void | expand

Message

Uwe Kleine-König July 15, 2022, 8:36 p.m. UTC 
Hello,

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).

My 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 straigt forward
and easy to review.

Best regards
Uwe

Uwe Kleine-König (8):
  interconnect: imx: Ignore return value of icc_provider_del() in
    .remove()
  interconnect: icc-rpm: Ignore return value of icc_provider_del() in
    .remove()
  interconnect: icc-rpmh: Ignore return value of icc_provider_del() in
    .remove()
  interconnect: msm8974: Ignore return value of icc_provider_del() in
    .remove()
  interconnect: osm-l3: Ignore return value of icc_provider_del() in
    .remove()
  interconnect: sm8450: Ignore return value of icc_provider_del() in
    .remove()
  interconnect: Make icc_provider_del() return void
  interconnect: imx: Make imx_icc_unregister() return void

 drivers/interconnect/core.c           | 10 +++-------
 drivers/interconnect/imx/imx.c        |  4 ++--
 drivers/interconnect/imx/imx.h        |  2 +-
 drivers/interconnect/imx/imx8mm.c     |  4 +++-
 drivers/interconnect/imx/imx8mn.c     |  4 +++-
 drivers/interconnect/imx/imx8mq.c     |  4 +++-
 drivers/interconnect/qcom/icc-rpm.c   |  4 +++-
 drivers/interconnect/qcom/icc-rpmh.c  |  4 +++-
 drivers/interconnect/qcom/msm8974.c   |  4 +++-
 drivers/interconnect/qcom/osm-l3.c    |  4 +++-
 drivers/interconnect/qcom/sm8450.c    |  4 +++-
 include/linux/interconnect-provider.h |  2 +-
 12 files changed, 31 insertions(+), 19 deletions(-)


base-commit: f2906aa863381afb0015a9eb7fefad885d4e5a56