2016-03-22 3:47 GMT+08:00 Timur Tabi <timur@xxxxxxxxxxxxxx>: > Jun Nie wrote: > >> - snprintf(uap->type, sizeof(uap->type), "SBSA"); >> + snprintf(uap->type, sizeof(uap->type), "PL011 plat"); > > > I'm not crazy about this. I think you need to add a "const char *name" > field to 'struct vendor_data'. Will address this suggestion in next version patch. How about "PL011 SBSA" "PL011 ZTE" "PL011 ARM rev1" "PL011 ST rev1"? Jun > > -- > Qualcomm Innovation Center, Inc. > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora > Forum, a Linux Foundation collaborative project. -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html