Thread (14 messages) 14 messages, 7 authors, 2023-10-14

Re: [PATCH 00/40] soc: Convert to platform remove callback returning void

From: Uwe Kleine-König <hidden>
Date: 2023-09-28 06:16:05
Also in: linux-arm-msm, linux-aspeed, linux-mediatek, linux-pm, linux-riscv, linux-rockchip, linux-samsung-soc, linux-tegra, lkml, loongarch

On Wed, Sep 27, 2023 at 04:01:58PM -0700, Bjorn Andersson wrote:
On Wed, Sep 27, 2023 at 10:43:16AM +0200, Arnd Bergmann wrote:
quoted
On Wed, Sep 27, 2023, at 04:25, Joel Stanley wrote:
quoted
On Mon, 25 Sept 2023 at 09:55, Uwe Kleine-König [off-list ref] wrote:
quoted
this series converts all platform drivers below drivers/soc to use
.remove_new(). The motivation is to get rid of an integer return code
that is (mostly) ignored by the platform driver core and error prone on
the driver side.

See commit 5c5a7680e67b ("platform: Provide a remove callback that
returns no value") for an extended explanation and the eventual goal.

As there is no single maintainer team for drivers/soc, I suggest the
individual maintainers to pick up "their" patches.
I'd be happy if Arnd merged the lot at once. Arnd, what do you think?

If that will be too messy then I understand. I have queued the aspeed
ones locally and will push that out if we decide that's the best way
to go.
The main downside of merging it all at once through the soc tree
is that there may be patches that conflict with other work going on
in individual drivers.

What I'd suggest doing here is:

- have platform maintainers pick up patches for their drivers
  if that is their preference for any reason
I'd prefer this for the qcom drivers at least, please let me know if you
would like me to proceed.
I can send a pull request as Arnd suggested. So iff you want the qcom
drivers not be a part of that PR, just make sure they appear in next
during the next week. :-)

(I'm not sure if "this" in your last quoted sentence is "platform
maintainers pick up" or "merging it all at once through the soc tree". I
think you mean the former. Still if you don't want me to pick up the
qcom patches and won't manage to make them appear in next via your tree,
I ask you to tell me explicitly to skip these in my PR.)

Best regards and thanks,
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help