![](https://secure.gravatar.com/avatar/d930951cb00393ecf9c3db3a56d78fa9.jpg?s=120&d=mm&r=g)
2 Nov
2017
2 Nov
'17
12:30 p.m.
On Thu, Nov 02, 2017 at 07:21:16PM +0800, Oder Chiou wrote:
- else if (i2c->dev.of_node)
rt5514_parse_dt(rt5514, &i2c->dev);
- else
rt5514_parse_dp(rt5514, &i2c->dev);
Should this be changed to i2c->dev.of_node || ACPI_HANDLE(&i2c->dev) or will the function cope fine if there are no properties at all?