MIME-Version: 1.0
References: <20211115121001.77041-1-heikki.krogerus@linux.intel.com>
In-Reply-To: <20211115121001.77041-1-heikki.krogerus@linux.intel.com>
From:   "Rafael J. Wysocki" <rafael@kernel.org>
Date:   Wed, 24 Nov 2021 14:59:01 +0100
Message-ID: <CAJZ5v0jsWVw4OyVbkdn2374tLAXAShZ_B3CKDmnQOE_QEXXPiQ@mail.gmail.com>
Subject: Re: [PATCH v4 0/3] device property: Remove device_add_properties()
To:     Heikki Krogerus <heikki.krogerus@linux.intel.com>
Cc:     "Rafael J. Wysocki" <rafael@kernel.org>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Bjorn Helgaas <bhelgaas@google.com>,
        Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
        Zhangfei Gao <zhangfei.gao@linaro.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Linux PCI <linux-pci@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Mon, Nov 15, 2021 at 1:10 PM Heikki Krogerus
<heikki.krogerus@linux.intel.com> wrote:
>
> Hi,
>
> One more version. Hopefully the commit messages are now OK. No other
> changes since v3:
>
> https://lore.kernel.org/lkml/20211006112643.77684-1-heikki.krogerus@linux.intel.com/
>
>
> v3 cover letter:
>
> In this third version of this series, the second patch is now split in
> two. The device_remove_properties() call is first removed from
> device_del() in its own patch, and the
> device_add/remove_properties() API is removed separately in the last
> patch. I hope the commit messages are clear enough this time.
>
>
> v2 cover letter:
>
> This is the second version where I only modified the commit message of
> the first patch according to comments from Bjorn.
>
>
> Original cover letter:
>
> There is one user left for the API, so converting that to use software
> node API instead, and removing the function.
>
>
> thanks,
>
> Heikki Krogerus (3):
>   PCI: Convert to device_create_managed_software_node()
>   driver core: Don't call device_remove_properties() from device_del()
>   device property: Remove device_add_properties() API
>
>  drivers/base/core.c      |  1 -
>  drivers/base/property.c  | 48 ----------------------------------------
>  drivers/pci/quirks.c     |  2 +-
>  include/linux/property.h |  4 ----
>  4 files changed, 1 insertion(+), 54 deletions(-)

Has this been picked up already or am I expected to pick it up?