linux-kernel - Re: [PATCH 06/11] staging: fsl-mc: make fsl_mc_is_root

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Wed, 29 Jun 2016 17:01:52 +0200
From:	Matthias Brugger <mbrugger@...e.com>
To:	Stuart Yoder <stuart.yoder@....com>,
	"gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>
Cc:	"devel@...verdev.osuosl.org" <devel@...verdev.osuosl.org>,
	"agraf@...e.de" <agraf@...e.de>, "arnd@...db.de" <arnd@...db.de>,
	Jose Rivera <german.rivera@....com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Yang-Leo Li <leoyang.li@....com>
Subject: Re: [PATCH 06/11] staging: fsl-mc: make fsl_mc_is_root_dprc() global



On 29/06/16 16:25, Stuart Yoder wrote:
>
>
>> -----Original Message-----
>> From: Matthias Brugger [mailto:mbrugger@...e.com]
>> Sent: Wednesday, June 29, 2016 9:17 AM
>> To: Stuart Yoder <stuart.yoder@....com>; gregkh@...uxfoundation.org
>> Cc: devel@...verdev.osuosl.org; agraf@...e.de; arnd@...db.de; Jose Rivera <german.rivera@....com>;
>> linux-kernel@...r.kernel.org; Yang-Leo Li <leoyang.li@....com>
>> 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@....com>
>>> ---
>>>    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.
>

Ok, thanks for clarification.

Reviewed-by: Matthias Brugger <mbrugger@...e.com>