On Wed, Apr 19, 2017 at 08:53:05PM +0800, Biaoyong Hu wrote:

> kfree(pin); pin will be freed. So, "pin->request_count = 0;" should be
> removed?

Yes, it's redundant.

Please don't top post, reply in line with needed context.  This allows
readers to readily follow the flow of conversation and understand what
you are talking about and also helps ensure that everything in the
discussion is being addressed.