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:

quoted

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