[PATCH v3 03/10] drivers/peci: Add support for PECI bus driver core
Jae Hyun Yoo
jae.hyun.yoo at linux.intel.com
Tue Apr 24 03:40:53 AEST 2018
On 4/23/2018 3:52 AM, Greg KH wrote:
> On Tue, Apr 10, 2018 at 11:32:05AM -0700, Jae Hyun Yoo wrote:
>> +static void peci_adapter_dev_release(struct device *dev)
>> + /* do nothing */
> As per the in-kernel documentation, I am now allowed to make fun of you.
> You are trying to "out smart" the kernel by getting rid of a warning
> message that was explicitly put there for you to do something. To think
> that by just providing an "empty" function you are somehow fulfilling
> the API requirement is quite bold, don't you think?
> This has to be fixed. I didn't put that warning in there for no good
> reason. Please go read the documentation again...
> greg k-h
Thanks a lot for your review.
I think, it should contain actual device resource release code which is
being done by peci_del_adapter(), or a coupling logic should be added
between peci_adapter_dev_release() and peci_del_adapter().
As you suggested, I'll check it again after reading documentation and
understanding core.c code more deeply.
More information about the openbmc