linux-um.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: Richard Weinberger <richard@nod.at>
To: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
Cc: anton ivanov <anton.ivanov@cambridgegreys.com>,
	 Johannes Berg <johannes@sipsolutions.net>,
	 linux-um <linux-um@lists.infradead.org>,
	 kernel <kernel@pengutronix.de>
Subject: Re: [PATCH 0/2] um: Convert to platform remove callback returning void
Date: Tue, 30 Apr 2024 14:18:13 +0200 (CEST)	[thread overview]
Message-ID: <1268129847.49378.1714479493850.JavaMail.zimbra@nod.at> (raw)
In-Reply-To: <4bxrtz7jnavmzplkiwzpq62omswpfbogd2pep7ju4jlbum6ccr@xbegqcpfmnf2>

Uwe,

----- Ursprüngliche Mail -----
> Von: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
> An: "richard" <richard@nod.at>, "anton ivanov" <anton.ivanov@cambridgegreys.com>, "Johannes Berg"
> <johannes@sipsolutions.net>
> CC: "linux-um" <linux-um@lists.infradead.org>, "kernel" <kernel@pengutronix.de>
> Gesendet: Montag, 29. April 2024 09:35:49
> Betreff: Re: [PATCH 0/2] um: Convert to platform remove callback returning void

> Hello Richard,
> 
> On Sun, Mar 10, 2024 at 08:52:26AM +0100, Uwe Kleine-König wrote:
>> this series converts all platform drivers below arch/um to stop using
>> struct platform_driver::remove(). See commit 5c5a7680e67b ("platform:
>> Provide a remove callback that returns no value") for an extended
>> explanation and the eventual goal.
>> 
>> Both conversations are trivial, because the driver's .remove() callbacks
>> returned zero unconditionally.
>> 
>> There are no interdependencies between these patches, still I'd expect
>> them to be picked up together.
> 
> I intend to send the change to struct platform_driver::remove() to Greg
> soon after the upcoming merge window closed. It would be great if these
> patches made it in before. If not, I'll ask Greg to apply these patches
> together with the change to platform_driver::remove(). If that's your
> preferred resolution, feel free to send an Ack for that.

I've applied them and will appear soon in -next.

Thanks,
//richard


      reply	other threads:[~2024-04-30 12:18 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-10  7:52 [PATCH 0/2] um: Convert to platform remove callback returning void Uwe Kleine-König
2024-03-10  7:52 ` [PATCH 1/2] um: rtc: " Uwe Kleine-König
2024-03-11  8:22   ` Geert Uytterhoeven
2024-03-10  7:52 ` [PATCH 2/2] um: virtio_uml: " Uwe Kleine-König
2024-03-11  8:22   ` Geert Uytterhoeven
2024-04-29  7:35 ` [PATCH 0/2] um: " Uwe Kleine-König
2024-04-30 12:18   ` Richard Weinberger [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1268129847.49378.1714479493850.JavaMail.zimbra@nod.at \
    --to=richard@nod.at \
    --cc=anton.ivanov@cambridgegreys.com \
    --cc=johannes@sipsolutions.net \
    --cc=kernel@pengutronix.de \
    --cc=linux-um@lists.infradead.org \
    --cc=u.kleine-koenig@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).