Re: [PATCH v3 03/10] drivers/peci: Add support for PECI bus driver core

From: Greg KH
Date: Mon Apr 23 2018 - 06:52:42 EST


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