Message-ID: <473EF47E.8090402@freescale.com>
Date: Sat, 17 Nov 2007 08:02:38 -0600
From: Timur Tabi <timur@freescale.com>
Organization: Freescale
User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.1.8) Gecko/20071008 SeaMonkey/1.1.5
MIME-Version: 1.0
To: Mikael Pettersson <mikpe@it.uu.se>
CC: gregkh@suse.de, linux-kernel@vger.kernel.org
Subject: Re: Add a private_data pointer to struct device_attribute
References: <200711170115.lAH1F5kZ009153@harpo.it.uu.se>
In-Reply-To: <200711170115.lAH1F5kZ009153@harpo.it.uu.se>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 793
Lines: 21

Mikael Pettersson wrote:

> A common trick is to embed a generic struct inside a specific one
> containing add-on data fields, and then to map from the generic
> one to the specific one using container_of() in your ops (function
> pointers). This is both faster and less wasteful of memory than
> adding void *private all over the place.
> 
> Any reason that won't work here?

Yes, that will work.  Sorry, I should have thought of that myself, since I've 
used that trick a number of times before.

-- 
Timur Tabi
Linux Kernel Developer @ Freescale
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/