On Thu, 2016-02-25 at 15:14 -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> > --- > drivers/target/target_core_device.c | 41 ++++++++++------------------ > drivers/target/target_core_fabric_configfs.c | 32 ++++++++++------------ > drivers/target/target_core_internal.h | 8 +++--- > drivers/target/target_core_spc.c | 3 +- > drivers/target/target_core_tpg.c | 21 +++++++------- > include/target/target_core_base.h | 11 ++------ > 6 files changed, 45 insertions(+), 71 deletions(-) > Applied to target-pending/for-next, with a minor context change fix-up. Thanks Andy. -- 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