[PATCH linux dev-4.10 v3 24/31] drivers: fsi: occ: Remove unnecessary platform_set_drvdata call

Eddie James eajames at linux.vnet.ibm.com
Fri Oct 6 11:33:59 AEDT 2017



On 10/05/2017 06:14 PM, Andrew Jeffery wrote:
> On Thu, 2017-10-05 at 14:24 -0500, Eddie James wrote:
>> From: "Edward A. James" <eajames at us.ibm.com>
>>   
>> The purpose was to set that pointer null before probing. However, it is
>> set null by the kernel by default.
>>   
>> Signed-off-by: Edward A. James <eajames at us.ibm.com>
>> ---
>>   drivers/fsi/occ.c | 3 ---
>>   1 file changed, 3 deletions(-)
>>   
>> diff --git a/drivers/fsi/occ.c b/drivers/fsi/occ.c
>> index 916c342..561b029 100644
>> --- a/drivers/fsi/occ.c
>> +++ b/drivers/fsi/occ.c
>> @@ -703,9 +703,6 @@ static int occ_probe(struct platform_device *pdev)
>>   	mutex_init(&occ->occ_lock);
>>   	INIT_WORK(&occ->work, occ_worker);
>>   
>> -	/* ensure NULL before we probe children, so they don't hang FSI */
>> -	platform_set_drvdata(pdev, NULL);
>> -
> Bizarre that it was even there in the first place - do you know the history of
> the comment?

I made the comment. I wasn't sure if there were situations where that 
wouldn't be NULL before probe() somehow. Regarding the hanging FSI... 
that was before some FSI core fixes were in that prevent that situation 
during probe().

>
>>   	if (dev->of_node) {
>>   		rc = of_property_read_u32(dev->of_node, "reg", &reg);
>>   		if (!rc) {



More information about the openbmc mailing list