6 Jun
2013
6 Jun
'13
5:21 a.m.
On Wed, Jun 05, 2013 at 01:25:11PM +0100, Mark Brown wrote:
On Wed, Jun 05, 2013 at 08:12:55PM +0800, Nicolin Chen wrote:
struct wm8962_priv {
- struct wm8962_pdata *pdata;
More idiomatic style for this is to just embed a copy of the platform data struct in the private data then copy any driver model platform data on top of it. This simplifies usage as now the driver can assume that there is platform data available at all times.
Hmm..sorry I don't fully get it. Does that mean I should do something like: struct wm8962_priv { struct wm8962_pdata pdata;
instead of pointer? so no devm_kzalloc() for it any more?