On Wed, May 23, 2018 at 1:11 PM, Srinivas Kandagatla <srinivas.kandagatla@xxxxxxxxxx> wrote: > > > On 23/05/18 17:40, Rob Herring wrote: >>> >>> + >>> +- qcom,ngd-id >>> + Usage: required >>> + Value type: <u32> >>> + Definition: ngd instance id in the controller >> >> Why do you need this? >> > Please ignore my comment from previous reply. > > There are more than one instances of ngd in this slim controller. > We need this to make sure we are programming the correct one. Doesn't the parent-child relationship of devices on the bus provide that? If you mean to provide consistent numbering to userspace, then that's not a DT problem (nor one that Linux plans to solve). > We also need this instance ID during powering it up using QMI. Wouldn't that be a QMI ID? Rob -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html