On Thu, Dec 11, 2014 at 11:14:39AM +0800, Bo Shen wrote: > +static const struct of_device_id wm8904_of_match[] = { > + { .compatible = "wlf,wm8904", }, > + { .compatible = "wlf,wm8912", }, > + { .compatible = "wlf,wm8918", }, > + { } > +}; > +MODULE_DEVICE_TABLE(of, wm8904_of_match); > + > static struct i2c_driver wm8904_i2c_driver = { > .driver = { > .name = "wm8904", > .owner = THIS_MODULE, > + .of_match_table = wm8904_of_match, I'm not seeing anything here that ensures that devtype is set correctly when probing using the OF IDs - currently we rely on i2c->driver_data.
Attachment:
signature.asc
Description: Digital signature
- Follow-Ups:
- Re: [PATCH 1/3] ASoC: codecs: wm8904: add dt ids table
- From: Bo Shen
- Re: [PATCH 1/3] ASoC: codecs: wm8904: add dt ids table
- References:
- [PATCH 1/3] ASoC: codecs: wm8904: add dt ids table
- From: Bo Shen
- [PATCH 1/3] ASoC: codecs: wm8904: add dt ids table
- Prev by Date: [PATCH 2/3] ARM: at91: sama5d3xek/dt: use dt ids for wm8904
- Next by Date: Re: [PATCH 1/3] ASoC: codecs: wm8904: add dt ids table
- Previous by thread: [PATCH 2/3] ARM: at91: sama5d3xek/dt: use dt ids for wm8904
- Next by thread: Re: [PATCH 1/3] ASoC: codecs: wm8904: add dt ids table
- Index(es):