diff options
author | Viresh Kumar <viresh.kumar@linaro.org> | 2015-12-22 22:04:33 +0530 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@google.com> | 2015-12-22 13:00:52 -0800 |
commit | b32a5c5346bdaac0ad6ef12cb444a9ad69d2b24b (patch) | |
tree | e8a8d94bc6f0e8060043be90f214265bd4604e0f /drivers/staging | |
parent | a5a7723a0d3762387fc765e69f0ab61fd4d47f00 (diff) |
greybus: interface: Prefix DDBL1 attributes with ddbl1_ instead of unipro_
The Device descriptor block Level 1 (DDBL1) attributes are specified by
the MIPI standard and prefixing them with 'unipro_' isn't the best thing
to do. They should be prefixed with DDBL1 instead.
To make it more readable/clear:
- rename macros and variable by prefixing them with ddbl1_.
- write full names for mfg and prod ids as manufacturer and product ids.
- replace mfg (manufacturing) with mfr (manufacturer)
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Reviewed-by: Johan Hovold <johan@hovoldconsulting.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
Diffstat (limited to 'drivers/staging')
-rw-r--r-- | drivers/staging/greybus/firmware.c | 10 | ||||
-rw-r--r-- | drivers/staging/greybus/greybus_protocols.h | 4 | ||||
-rw-r--r-- | drivers/staging/greybus/interface.h | 4 | ||||
-rw-r--r-- | drivers/staging/greybus/svc.c | 4 |
4 files changed, 11 insertions, 11 deletions
diff --git a/drivers/staging/greybus/firmware.c b/drivers/staging/greybus/firmware.c index 9d1739d78cca..5ec3efdd8f01 100644 --- a/drivers/staging/greybus/firmware.c +++ b/drivers/staging/greybus/firmware.c @@ -11,8 +11,8 @@ #include "greybus.h" -#define ES2_UNIPRO_MFG_ID 0x00000126 -#define ES2_UNIPRO_PROD_ID 0x00001000 +#define ES2_DDBL1_MFR_ID 0x00000126 +#define ES2_DDBL1_PROD_ID 0x00001000 struct gb_firmware { struct gb_connection *connection; @@ -49,8 +49,8 @@ static void firmware_es2_fixup_vid_pid(struct gb_firmware *firmware) * - Bridge ASIC chip isn't ES2 * - Received non-zero Vendor/Product ids */ - if (intf->unipro_mfg_id != ES2_UNIPRO_MFG_ID || - intf->unipro_prod_id != ES2_UNIPRO_PROD_ID || + if (intf->ddbl1_manufacturer_id != ES2_DDBL1_MFR_ID || + intf->ddbl1_product_id != ES2_DDBL1_PROD_ID || intf->vendor_id != 0 || intf->product_id != 0) return; @@ -84,7 +84,7 @@ static int download_firmware(struct gb_firmware *firmware, u8 stage) */ snprintf(firmware_name, sizeof(firmware_name), "ara:%08x:%08x:%08x:%08x:%02x.tftf", - intf->unipro_mfg_id, intf->unipro_prod_id, + intf->ddbl1_manufacturer_id, intf->ddbl1_product_id, firmware->vendor_id, firmware->product_id, stage); return request_firmware(&firmware->fw, firmware_name, diff --git a/drivers/staging/greybus/greybus_protocols.h b/drivers/staging/greybus/greybus_protocols.h index a50823c9fae3..76416b5ad848 100644 --- a/drivers/staging/greybus/greybus_protocols.h +++ b/drivers/staging/greybus/greybus_protocols.h @@ -749,8 +749,8 @@ struct gb_svc_intf_device_id_request { struct gb_svc_intf_hotplug_request { __u8 intf_id; struct { - __le32 unipro_mfg_id; - __le32 unipro_prod_id; + __le32 ddbl1_mfr_id; + __le32 ddbl1_prod_id; __le32 ara_vend_id; __le32 ara_prod_id; } data; diff --git a/drivers/staging/greybus/interface.h b/drivers/staging/greybus/interface.h index ca123ce11e76..d192b74ab5e1 100644 --- a/drivers/staging/greybus/interface.h +++ b/drivers/staging/greybus/interface.h @@ -26,8 +26,8 @@ struct gb_interface { char *product_string; /* Information taken from the hotplug event */ - u32 unipro_mfg_id; - u32 unipro_prod_id; + u32 ddbl1_manufacturer_id; + u32 ddbl1_product_id; u32 vendor_id; u32 product_id; diff --git a/drivers/staging/greybus/svc.c b/drivers/staging/greybus/svc.c index 24b6cdf2c440..864051b95376 100644 --- a/drivers/staging/greybus/svc.c +++ b/drivers/staging/greybus/svc.c @@ -417,8 +417,8 @@ static void gb_svc_process_intf_hotplug(struct gb_operation *operation) goto destroy_interface; } - intf->unipro_mfg_id = le32_to_cpu(request->data.unipro_mfg_id); - intf->unipro_prod_id = le32_to_cpu(request->data.unipro_prod_id); + intf->ddbl1_manufacturer_id = le32_to_cpu(request->data.ddbl1_mfr_id); + intf->ddbl1_product_id = le32_to_cpu(request->data.ddbl1_prod_id); intf->vendor_id = le32_to_cpu(request->data.ara_vend_id); intf->product_id = le32_to_cpu(request->data.ara_prod_id); |