Messages in this thread | | | From | Li Yang <> | Date | Fri, 16 Aug 2019 14:54:07 -0500 | Subject | Re: [PATCH 1/3] bus: fsl-mc: remove explicit device_link_del |
| |
On Fri, Aug 16, 2019 at 1:04 AM Laurentiu Tudor <laurentiu.tudor@nxp.com> wrote: > > Hi Leo, > > > -----Original Message----- > > From: Li Yang <leoyang.li@nxp.com> > > Sent: Friday, August 16, 2019 2:13 AM > > To: Ioana Ciornei <ioana.ciornei@nxp.com> > > Cc: Laurentiu Tudor <laurentiu.tudor@nxp.com>; Roy Pledge > > <roy.pledge@nxp.com>; lkml <linux-kernel@vger.kernel.org> > > Subject: Re: [PATCH 1/3] bus: fsl-mc: remove explicit device_link_del > > Importance: High > > > > On Wed, Jul 3, 2019 at 9:58 AM Ioana Ciornei <ioana.ciornei@nxp.com> wrote: > > > > > > Starting with commit 72175d4ea4c4 ("driver core: Make driver core own > > > stateful device links") stateful device links are owned by the driver > > > core and should not be explicitly removed on device unbind. Delete all > > > device_link_del appearances from the fsl-mc bus. > > > > > > Signed-off-by: Ioana Ciornei <ioana.ciornei@nxp.com> > > > > Hi Laurentiu, > > > > What do you think of this patches? I can take it through fsl/soc if > > you can ACK it. > > Looks good to me, so for the whole series: > > Acked-By: Laurentiu Tudor <laurentiu.tudor@nxp.com>
Thanks. Applied for next.
> > > Regards, > > Leo > > > > > --- > > > drivers/bus/fsl-mc/fsl-mc-allocator.c | 1 - > > > drivers/bus/fsl-mc/mc-io.c | 1 - > > > 2 files changed, 2 deletions(-) > > > > > > diff --git a/drivers/bus/fsl-mc/fsl-mc-allocator.c b/drivers/bus/fsl- > > mc/fsl-mc-allocator.c > > > index 8ad77246f322..cc7bb900f524 100644 > > > --- a/drivers/bus/fsl-mc/fsl-mc-allocator.c > > > +++ b/drivers/bus/fsl-mc/fsl-mc-allocator.c > > > @@ -330,7 +330,6 @@ void fsl_mc_object_free(struct fsl_mc_device > > *mc_adev) > > > > > > fsl_mc_resource_free(resource); > > > > > > - device_link_del(mc_adev->consumer_link); > > > mc_adev->consumer_link = NULL; > > > } > > > EXPORT_SYMBOL_GPL(fsl_mc_object_free); > > > diff --git a/drivers/bus/fsl-mc/mc-io.c b/drivers/bus/fsl-mc/mc-io.c > > > index 3ae574a58cce..d9629fc13a15 100644 > > > --- a/drivers/bus/fsl-mc/mc-io.c > > > +++ b/drivers/bus/fsl-mc/mc-io.c > > > @@ -255,7 +255,6 @@ void fsl_mc_portal_free(struct fsl_mc_io *mc_io) > > > fsl_destroy_mc_io(mc_io); > > > fsl_mc_resource_free(resource); > > > > > > - device_link_del(dpmcp_dev->consumer_link); > > > dpmcp_dev->consumer_link = NULL; > > > } > > > EXPORT_SYMBOL_GPL(fsl_mc_portal_free); > > > -- > > > 1.9.1 > > >
| |