From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To: "Rafael J. Wysocki" <rafael@kernel.org>
Cc: Heikki Krogerus <heikki.krogerus@linux.intel.com>,
"Rafael J. Wysocki" <rjw@rjwysocki.net>,
Felipe Balbi <balbi@kernel.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
"open list:ULTRA-WIDEBAND (UWB) SUBSYSTEM:"
<linux-usb@vger.kernel.org>,
ACPI Devel Maling List <linux-acpi@vger.kernel.org>
Subject: Re: [PATCH 0/2] Remove one more platform_device_add_properties() call
Date: Mon, 23 Nov 2020 19:36:03 +0200 [thread overview]
Message-ID: <20201123173603.GJ4077@smile.fi.intel.com> (raw)
In-Reply-To: <CAJZ5v0jAaz2zELkJoKjHtxyfuKEi=ORuCCad-F0yp6KephieGg@mail.gmail.com>
On Mon, Nov 23, 2020 at 06:06:31PM +0100, Rafael J. Wysocki wrote:
> On Mon, Nov 23, 2020 at 4:32 PM Heikki Krogerus
> <heikki.krogerus@linux.intel.com> wrote:
> >
> > Hi,
> >
> > I originally introduced these as part of my series where I was
> > proposing PM ops for software nodes [1], but since that still needs
> > work, I'm sending these two separately.
> >
> > So basically I'm only modifying dwc3-pci.c so it registers a software
> > node directly at this point. That will remove one more user of
> > platform_device_add_properties().
> >
> > [1] https://lore.kernel.org/lkml/20201029105941.63410-1-heikki.krogerus@linux.intel.com/
> >
> > thanks,
> >
> > Heikki Krogerus (2):
> > software node: Introduce device_add_software_node()
> > usb: dwc3: pci: Register a software node for the dwc3 platform device
> >
> > drivers/base/swnode.c | 69 ++++++++++++++++++++++++++++++++-----
> > drivers/usb/dwc3/dwc3-pci.c | 61 +++++++++++++++++++-------------
> > include/linux/property.h | 3 ++
> > 3 files changed, 100 insertions(+), 33 deletions(-)
> >
> > --
>
> These look good to me.
>
> If you want me to take them, though, I need an ACK from the dwc3 side.
Btw, I have tested this on one of the platform with DWC3 and found no
regression, so feel free to add
Tested-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
--
With Best Regards,
Andy Shevchenko
next prev parent reply other threads:[~2020-11-23 17:35 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-23 15:31 [PATCH 0/2] Remove one more platform_device_add_properties() call Heikki Krogerus
2020-11-23 15:31 ` [PATCH 1/2] software node: Introduce device_add_software_node() Heikki Krogerus
2020-11-23 15:31 ` [PATCH 2/2] usb: dwc3: pci: Register a software node for the dwc3 platform device Heikki Krogerus
2020-11-23 17:06 ` [PATCH 0/2] Remove one more platform_device_add_properties() call Rafael J. Wysocki
2020-11-23 17:36 ` Andy Shevchenko [this message]
2020-12-04 11:23 ` Heikki Krogerus
2021-01-11 12:56 ` Heikki Krogerus
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=20201123173603.GJ4077@smile.fi.intel.com \
--to=andriy.shevchenko@linux.intel.com \
--cc=balbi@kernel.org \
--cc=heikki.krogerus@linux.intel.com \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=rafael@kernel.org \
--cc=rjw@rjwysocki.net \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.