On Mar 10, 2014, at 4:19 PM, Santosh Shilimkar <santosh.shilimkar@xxxxxx> wrote: >>> + -- reg-names : Names for the above register regions. The name to be >>> + used is as follows: >>> + - "config" : Queue configuration region. >>> + - "status" : Queue status RAM. >>> + - "region" : Descriptor memory setup region. >>> + - "push" : Queue Management/Queue Proxy region. >>> + - "pop" : Queue Management/Queue Proxy region. >>> + - "peek" : Queue Peek region. >> >> reg-names should be optional. Also you have the order different from >> reg. Be consistent as to what is the correct order. >> > We thought of using reg-names to remove the ordering to index > the regs. So if we make the reg-names optional, then indexing > by order will be used. We can do that. We keep having this conversation, why does reg-names need to be optional for new bindings? - k -- Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html