On Fri, 2018-01-19 at 15:44 +0100, Xose Vazquez Perez wrote: > On 01/19/2018 12:52 PM, Martin Wilck wrote: > > There was an extra parenthesis in the patch I submitted > > previously. > > .vendor = "NVME", > > - .product = "(NetApp |)ONTAP Controller)", > > + .product = "(NetApp |)ONTAP Controller", > > Out of curiosity, what are the complete product and vendor id > strings? > All NVMe devices use "NVME" as vendor string. The device in question uses "NetApp ONTAP Controller" for the product (-> "model" in NVMe terms). IIUC there's older FW around that uses "ONTAP Controller" only. Regards, Martin -- Dr. Martin Wilck <mwilck@xxxxxxxx>, Tel. +49 (0)911 74053 2107 SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel