On 5/15/19 8:09 PM, Subash Abhinov Kasiviswanathan wrote: . . . > Hi Alex > > Could we instead have the rmnet header definition in > include/linux/if_rmnet.h
I have no objection to that, but I don't actually know what the criteria are for putting a file in that directory. Glancing at other "if_*" files there it seems sensible, but because I don't know, I'd like to have a little better justification. Can you provide a good explanation about why these definitions belong in "include/linux/if_rmnet.h" instead of "include/soc/qcom/rmnet.h"? Thanks. -Alex