Hi Prabhakar, On Sun, Mar 08, 2015 at 11:33:27AM +0000, Lad Prabhakar wrote: ... > +static struct ov2659_platform_data * > +ov2659_get_pdata(struct i2c_client *client) > +{ > + struct ov2659_platform_data *pdata; > + struct device_node *endpoint; > + int ret; > + > + if (!IS_ENABLED(CONFIG_OF) || !client->dev.of_node) { > + dev_err(&client->dev, "ov2659_get_pdata: DT Node found\n"); > + return client->dev.platform_data; > + } > + > + endpoint = of_graph_get_next_endpoint(client->dev.of_node, NULL); > + if (!endpoint) > + return NULL; > + > + pdata = devm_kzalloc(&client->dev, sizeof(*pdata), GFP_KERNEL); > + if (!pdata) > + goto done; > + > + ret = of_property_read_u32(endpoint, "link-frequencies", > + &pdata->link_frequency); This is actually documented as being a 64-bit array. The smiapp wasn't even reading it from the endpoint node. Oh well... -- Regards, Sakari Ailus e-mail: sakari.ailus@xxxxxx XMPP: sailus@xxxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html