On Wed, Apr 18, 2018 at 7:06 PM, Qi, Fuli <[email protected]> wrote: >> > Thank you very much, I made a v2 patch by referring to your sample patch. >> > >> > Going back to the OPT_STRING_LIST, I think it is necessary for ndctl. >> > Because the other options in monitor like --dimm-event, --bus-event >> > also need to support multiple space-separated arguments, these options >> > should be made as a string_list. >> >> Ok, yes, lets bring in string_list for that case. >> > > Could you merge the OPT_STRING_LIST first?
Oh, I was assuming when you said string_list that you would redo the patch to bring in the struct string_list implementation from git, and not use the strtok() approach. I'd like to not diverge from that implementation. _______________________________________________ Linux-nvdimm mailing list [email protected] https://lists.01.org/mailman/listinfo/linux-nvdimm
