From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932176Ab3COTRe (ORCPT ); Fri, 15 Mar 2013 15:17:34 -0400 Received: from mail-pb0-f47.google.com ([209.85.160.47]:36124 "EHLO mail-pb0-f47.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755447Ab3COTRc (ORCPT ); Fri, 15 Mar 2013 15:17:32 -0400 Date: Fri, 15 Mar 2013 12:17:29 -0700 From: Greg KH To: Alessandro Rubini Cc: linux-kernel@vger.kernel.org, Juan David Gonzalez Cobas , "Emilio G. Cota" , Samuel Iglesias Gonsalvez , Rob Landley , linux-doc@vger.kernel.org Subject: Re: [PATCH 3/8] FMC: add core bus driver Message-ID: <20130315191729.GA11988@kroah.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Feb 21, 2013 at 07:14:01PM +0100, Alessandro Rubini wrote: > -/* Every device must have a release method: provide a default */ > +/* We really have nothing to release in here */ > static void __fmc_release(struct device *dev) { } Then your implementation is wrong, and as per the documentation in the kernel Documentation directory (kobject.txt), I get to make fun of you in public for trying to do this :) Please fix this up, this isn't ok. greg k-h