On Thu, Feb 25, 2016 at 03:14:32PM -0800, Andy Grover wrote: > se_dev_entry.lun_flags and se_lun.lun_access are only used for keeping > track of read-write vs. read-only state. Since this is an either/or thing > we can represent it as bool, and remove the unneeded enum > transport_lunflags_table, which is left over from when there were more > flags. > > Change code that uses this enum to just use true/false, and make it clear > through variable and param names that true means read-only, false means > read-write. > > Signed-off-by: Andy Grover <agrover@xxxxxxxxxx> Looks fine, Reviewed-by: Christoph Hellwig <hch@xxxxxx> -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html