On Mon, May 04, 2015 at 06:51:32AM -0700, Kevin Cernekee wrote: > On Mon, May 4, 2015 at 4:45 AM, Mark Brown <[email protected]> wrote: > > On Sun, May 03, 2015 at 05:00:18PM -0700, Kevin Cernekee wrote: > >> + if (dev->of_node) { > >> + const struct of_device_id *of_id; > >> + > >> + of_id = of_match_device(tas571x_of_match, dev); > >> + if (of_id) > >> + priv->chip = of_id->data; > >> + }
> > Doesn't of_match_device() do the right thing with devices not registered > > from DT? > Not sure. What kinds of situations are you concerned about? I'm wondering why we need to bother checking for dev->of_node before calling it.
signature.asc
Description: Digital signature
