RE: [PATCH 06/11] staging: fsl-mc: make fsl_mc_is_root_dprc() global

From: Stuart Yoder
Date: Wed Jun 29 2016 - 10:39:57 EST




> -----Original Message-----
> From: Matthias Brugger [mailto:mbrugger@xxxxxxxx]
> Sent: Wednesday, June 29, 2016 9:17 AM
> To: Stuart Yoder <stuart.yoder@xxxxxxx>; gregkh@xxxxxxxxxxxxxxxxxxx
> Cc: devel@xxxxxxxxxxxxxxxxxxxx; agraf@xxxxxxx; arnd@xxxxxxxx; Jose Rivera <german.rivera@xxxxxxx>;
> linux-kernel@xxxxxxxxxxxxxxx; Yang-Leo Li <leoyang.li@xxxxxxx>
> Subject: Re: [PATCH 06/11] staging: fsl-mc: make fsl_mc_is_root_dprc() global
>
> On 22/06/16 23:40, Stuart Yoder wrote:
> > make fsl_mc_is_root_dprc() global so that the dprc driver
> > can use it
> >
> > Signed-off-by: Stuart Yoder <stuart.yoder@xxxxxxx>
> > ---
> > drivers/staging/fsl-mc/bus/mc-bus.c | 28 +++++++++++++---------------
> > drivers/staging/fsl-mc/include/mc.h | 2 ++
> > 2 files changed, 15 insertions(+), 15 deletions(-)
> >
> > diff --git a/drivers/staging/fsl-mc/bus/mc-bus.c b/drivers/staging/fsl-mc/bus/mc-bus.c
> > index e975adc..a49186e 100644
> > --- a/drivers/staging/fsl-mc/bus/mc-bus.c
> > +++ b/drivers/staging/fsl-mc/bus/mc-bus.c
> > @@ -24,8 +24,6 @@
> >
> > static struct kmem_cache *mc_dev_cache;
> >
> > -static bool fsl_mc_is_root_dprc(struct device *dev);
> > -
> > /**
> > * fsl_mc_bus_match - device to driver matching callback
> > * @dev: the MC object device structure to match against
> > @@ -247,19 +245,6 @@ static void fsl_mc_get_root_dprc(struct device *dev,
> > }
> > }
> >
> > -/**
> > - * fsl_mc_is_root_dprc - function to check if a given device is a root dprc
> > - */
> > -static bool fsl_mc_is_root_dprc(struct device *dev)
> > -{
> > - struct device *root_dprc_dev;
> > -
> > - fsl_mc_get_root_dprc(dev, &root_dprc_dev);
> > - if (!root_dprc_dev)
> > - return false;
> > - return dev == root_dprc_dev;
> > -}
> > -
> > static int get_dprc_attr(struct fsl_mc_io *mc_io,
> > int container_id, struct dprc_attributes *attr)
> > {
> > @@ -424,6 +409,19 @@ error_cleanup_regions:
> > }
> >
> > /**
> > + * fsl_mc_is_root_dprc - function to check if a given device is a root dprc
> > + */
> > +bool fsl_mc_is_root_dprc(struct device *dev)
> > +{
> > + struct device *root_dprc_dev;
> > +
> > + fsl_mc_get_root_dprc(dev, &root_dprc_dev);
> > + if (!root_dprc_dev)
> > + return false;
> > + return dev == root_dprc_dev;
> > +}
> > +
> > +/**
>
> Is there any reason why apart from deleting "static" you move
> fsl_mc_is_root to a different line?

I moved it just to keep internal consistency inside the source file where all the
static functions were grouped together in the first part of the file, and public
functions were in the second part.

Stuart