Hello Tejun, The 10/13/2016 19:22, Tejun Heo wrote: > Hello, Adam. > > Sorry about late reply. Was on vacation. NP I was on vacation at the end of the week last week. > > On Thu, Oct 13, 2016 at 04:00:31PM -0700, Adam Manzanares wrote: > > Add a sysfs entry to turn on priority information being passed > > to a ATA device. By default this feature is turned off. > > > > This patch depends on ata: Enabling ATA Command Priorities > > Looks generally good but can we please use a device attribute name > which is more specific - ie. enable_ncq_prio? Will do, I'll also double check the naming scheme of functions and variables also. The functions that check if the device has the ncq prio capability might be too similar to the function that checks if the device attribute is enabled. > > Thanks. > > -- > tejun Take care, Adam -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html