On 04/09/15 09:42, Ming Lin wrote:
diff --git a/include/target/target_core_fabric.h b/include/target/target_core_fabric.h
index e0a8191..abe3fcb 100644
--- a/include/target/target_core_fabric.h
+++ b/include/target/target_core_fabric.h
@@ -4,8 +4,8 @@
struct target_core_fabric_ops {
struct configfs_subsystem *tf_subsys;
size_t node_acl_size;
+ u8 fabric_proto_ident;
char *(*get_fabric_name)(void);
- u8 (*get_fabric_proto_ident)(struct se_portal_group *);
char *(*tpg_get_wwn)(struct se_portal_group *);
u16 (*tpg_get_tag)(struct se_portal_group *);
u32 (*tpg_get_default_depth)(struct se_portal_group *);
Hello Ming,
Please do not use the "fabric_" prefix in the data structure member
name. Since this structure member exists inside the
target_core_fabric_ops data structure it is already clear that it refers
to target code and it is not needed to use the "fabric_" prefix. How
about using the name "scsi_protocol" instead ?
Bart.
--
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