Greg Kroah-Hartman | eb50fd3 | 2017-11-07 14:58:41 +0100 | [diff] [blame] | 1 | // SPDX-License-Identifier: GPL-2.0 |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 2 | /* |
Viresh Kumar | a93db2d | 2015-04-01 20:32:02 +0530 | [diff] [blame] | 3 | * Greybus manifest parsing |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 4 | * |
Alex Elder | d8187aa | 2015-03-27 15:06:24 -0500 | [diff] [blame] | 5 | * Copyright 2014-2015 Google Inc. |
| 6 | * Copyright 2014-2015 Linaro Ltd. |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 7 | */ |
| 8 | |
Greg Kroah-Hartman | ec0ad86 | 2019-08-25 07:54:27 +0200 | [diff] [blame] | 9 | #include <linux/greybus.h> |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 10 | |
Viresh Kumar | 19b3b2c | 2015-03-24 17:08:13 +0530 | [diff] [blame] | 11 | static const char *get_descriptor_type_string(u8 type) |
| 12 | { |
Quentin Lambert | 3dd2226 | 2016-09-27 11:42:08 +0200 | [diff] [blame] | 13 | switch (type) { |
Viresh Kumar | 19b3b2c | 2015-03-24 17:08:13 +0530 | [diff] [blame] | 14 | case GREYBUS_TYPE_INVALID: |
| 15 | return "invalid"; |
Viresh Kumar | 19b3b2c | 2015-03-24 17:08:13 +0530 | [diff] [blame] | 16 | case GREYBUS_TYPE_STRING: |
| 17 | return "string"; |
| 18 | case GREYBUS_TYPE_INTERFACE: |
| 19 | return "interface"; |
| 20 | case GREYBUS_TYPE_CPORT: |
| 21 | return "cport"; |
Viresh Kumar | 83a0cb5 | 2015-04-01 20:31:59 +0530 | [diff] [blame] | 22 | case GREYBUS_TYPE_BUNDLE: |
| 23 | return "bundle"; |
Viresh Kumar | 19b3b2c | 2015-03-24 17:08:13 +0530 | [diff] [blame] | 24 | default: |
| 25 | WARN_ON(1); |
| 26 | return "unknown"; |
| 27 | } |
| 28 | } |
| 29 | |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 30 | /* |
| 31 | * We scan the manifest once to identify where all the descriptors |
| 32 | * are. The result is a list of these manifest_desc structures. We |
| 33 | * then pick through them for what we're looking for (starting with |
Viresh Kumar | a93db2d | 2015-04-01 20:32:02 +0530 | [diff] [blame] | 34 | * the interface descriptor). As each is processed we remove it from |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 35 | * the list. When we're done the list should (probably) be empty. |
| 36 | */ |
| 37 | struct manifest_desc { |
| 38 | struct list_head links; |
| 39 | |
| 40 | size_t size; |
| 41 | void *data; |
| 42 | enum greybus_descriptor_type type; |
| 43 | }; |
| 44 | |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 45 | static void release_manifest_descriptor(struct manifest_desc *descriptor) |
| 46 | { |
| 47 | list_del(&descriptor->links); |
| 48 | kfree(descriptor); |
| 49 | } |
| 50 | |
Greg Kroah-Hartman | 86cad66 | 2014-12-23 15:16:50 -0800 | [diff] [blame] | 51 | static void release_manifest_descriptors(struct gb_interface *intf) |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 52 | { |
| 53 | struct manifest_desc *descriptor; |
| 54 | struct manifest_desc *next; |
| 55 | |
Greg Kroah-Hartman | 86cad66 | 2014-12-23 15:16:50 -0800 | [diff] [blame] | 56 | list_for_each_entry_safe(descriptor, next, &intf->manifest_descs, links) |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 57 | release_manifest_descriptor(descriptor); |
| 58 | } |
| 59 | |
Johan Hovold | f2152eb | 2015-11-25 15:59:25 +0100 | [diff] [blame] | 60 | static void release_cport_descriptors(struct list_head *head, u8 bundle_id) |
| 61 | { |
| 62 | struct manifest_desc *desc, *tmp; |
| 63 | struct greybus_descriptor_cport *desc_cport; |
| 64 | |
| 65 | list_for_each_entry_safe(desc, tmp, head, links) { |
| 66 | desc_cport = desc->data; |
| 67 | |
| 68 | if (desc->type != GREYBUS_TYPE_CPORT) |
| 69 | continue; |
| 70 | |
| 71 | if (desc_cport->bundle == bundle_id) |
| 72 | release_manifest_descriptor(desc); |
| 73 | } |
| 74 | } |
| 75 | |
Rui Miguel Silva | 5c864e7 | 2015-11-16 19:23:25 +0000 | [diff] [blame] | 76 | static struct manifest_desc *get_next_bundle_desc(struct gb_interface *intf) |
| 77 | { |
| 78 | struct manifest_desc *descriptor; |
| 79 | struct manifest_desc *next; |
| 80 | |
| 81 | list_for_each_entry_safe(descriptor, next, &intf->manifest_descs, links) |
| 82 | if (descriptor->type == GREYBUS_TYPE_BUNDLE) |
| 83 | return descriptor; |
| 84 | |
| 85 | return NULL; |
| 86 | } |
| 87 | |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 88 | /* |
| 89 | * Validate the given descriptor. Its reported size must fit within |
Viresh Kumar | 696e0cc | 2014-11-21 11:26:30 +0530 | [diff] [blame] | 90 | * the number of bytes remaining, and it must have a recognized |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 91 | * type. Check that the reported size is at least as big as what |
| 92 | * we expect to see. (It could be bigger, perhaps for a new version |
| 93 | * of the format.) |
| 94 | * |
Alex Elder | d393c98 | 2015-06-09 17:42:53 -0500 | [diff] [blame] | 95 | * Returns the (non-zero) number of bytes consumed by the descriptor, |
| 96 | * or a negative errno. |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 97 | */ |
Greg Kroah-Hartman | 86cad66 | 2014-12-23 15:16:50 -0800 | [diff] [blame] | 98 | static int identify_descriptor(struct gb_interface *intf, |
| 99 | struct greybus_descriptor *desc, size_t size) |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 100 | { |
| 101 | struct greybus_descriptor_header *desc_header = &desc->header; |
| 102 | struct manifest_desc *descriptor; |
Alex Elder | d8187aa | 2015-03-27 15:06:24 -0500 | [diff] [blame] | 103 | size_t desc_size; |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 104 | size_t expected_size; |
| 105 | |
| 106 | if (size < sizeof(*desc_header)) { |
Greg Kroah-Hartman | ee989b0 | 2019-08-25 07:54:24 +0200 | [diff] [blame] | 107 | dev_err(&intf->dev, "manifest too small (%zu < %zu)\n", size, |
| 108 | sizeof(*desc_header)); |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 109 | return -EINVAL; /* Must at least have header */ |
| 110 | } |
| 111 | |
Alex Elder | d8187aa | 2015-03-27 15:06:24 -0500 | [diff] [blame] | 112 | desc_size = le16_to_cpu(desc_header->size); |
| 113 | if (desc_size > size) { |
Johan Hovold | a0b5542 | 2016-02-11 13:52:46 +0100 | [diff] [blame] | 114 | dev_err(&intf->dev, "descriptor too big (%zu > %zu)\n", |
Greg Kroah-Hartman | ee989b0 | 2019-08-25 07:54:24 +0200 | [diff] [blame] | 115 | desc_size, size); |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 116 | return -EINVAL; |
| 117 | } |
| 118 | |
Viresh Kumar | 19b3b2c | 2015-03-24 17:08:13 +0530 | [diff] [blame] | 119 | /* Descriptor needs to at least have a header */ |
| 120 | expected_size = sizeof(*desc_header); |
| 121 | |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 122 | switch (desc_header->type) { |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 123 | case GREYBUS_TYPE_STRING: |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 124 | expected_size += sizeof(struct greybus_descriptor_string); |
Viresh Kumar | 19b3b2c | 2015-03-24 17:08:13 +0530 | [diff] [blame] | 125 | expected_size += desc->string.length; |
Viresh Kumar | fa2fbf1 | 2015-04-28 19:51:35 +0530 | [diff] [blame] | 126 | |
| 127 | /* String descriptors are padded to 4 byte boundaries */ |
| 128 | expected_size = ALIGN(expected_size, 4); |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 129 | break; |
Alex Elder | 63cc932 | 2014-10-02 12:30:02 -0500 | [diff] [blame] | 130 | case GREYBUS_TYPE_INTERFACE: |
Viresh Kumar | a93db2d | 2015-04-01 20:32:02 +0530 | [diff] [blame] | 131 | expected_size += sizeof(struct greybus_descriptor_interface); |
Alex Elder | 63cc932 | 2014-10-02 12:30:02 -0500 | [diff] [blame] | 132 | break; |
Viresh Kumar | 7c183f7 | 2015-04-01 20:32:00 +0530 | [diff] [blame] | 133 | case GREYBUS_TYPE_BUNDLE: |
| 134 | expected_size += sizeof(struct greybus_descriptor_bundle); |
| 135 | break; |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 136 | case GREYBUS_TYPE_CPORT: |
Viresh Kumar | 19b3b2c | 2015-03-24 17:08:13 +0530 | [diff] [blame] | 137 | expected_size += sizeof(struct greybus_descriptor_cport); |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 138 | break; |
| 139 | case GREYBUS_TYPE_INVALID: |
| 140 | default: |
Johan Hovold | a0b5542 | 2016-02-11 13:52:46 +0100 | [diff] [blame] | 141 | dev_err(&intf->dev, "invalid descriptor type (%u)\n", |
Greg Kroah-Hartman | ee989b0 | 2019-08-25 07:54:24 +0200 | [diff] [blame] | 142 | desc_header->type); |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 143 | return -EINVAL; |
| 144 | } |
| 145 | |
Viresh Kumar | 19b3b2c | 2015-03-24 17:08:13 +0530 | [diff] [blame] | 146 | if (desc_size < expected_size) { |
Johan Hovold | a0b5542 | 2016-02-11 13:52:46 +0100 | [diff] [blame] | 147 | dev_err(&intf->dev, "%s descriptor too small (%zu < %zu)\n", |
Greg Kroah-Hartman | ee989b0 | 2019-08-25 07:54:24 +0200 | [diff] [blame] | 148 | get_descriptor_type_string(desc_header->type), |
| 149 | desc_size, expected_size); |
Viresh Kumar | 19b3b2c | 2015-03-24 17:08:13 +0530 | [diff] [blame] | 150 | return -EINVAL; |
| 151 | } |
| 152 | |
Viresh Kumar | 55b930c | 2015-04-29 11:02:08 +0530 | [diff] [blame] | 153 | /* Descriptor bigger than what we expect */ |
| 154 | if (desc_size > expected_size) { |
Johan Hovold | a0b5542 | 2016-02-11 13:52:46 +0100 | [diff] [blame] | 155 | dev_warn(&intf->dev, "%s descriptor size mismatch (want %zu got %zu)\n", |
Greg Kroah-Hartman | ee989b0 | 2019-08-25 07:54:24 +0200 | [diff] [blame] | 156 | get_descriptor_type_string(desc_header->type), |
| 157 | expected_size, desc_size); |
Viresh Kumar | 55b930c | 2015-04-29 11:02:08 +0530 | [diff] [blame] | 158 | } |
| 159 | |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 160 | descriptor = kzalloc(sizeof(*descriptor), GFP_KERNEL); |
| 161 | if (!descriptor) |
| 162 | return -ENOMEM; |
| 163 | |
| 164 | descriptor->size = desc_size; |
Alex Elder | d393c98 | 2015-06-09 17:42:53 -0500 | [diff] [blame] | 165 | descriptor->data = (char *)desc + sizeof(*desc_header); |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 166 | descriptor->type = desc_header->type; |
Greg Kroah-Hartman | 86cad66 | 2014-12-23 15:16:50 -0800 | [diff] [blame] | 167 | list_add_tail(&descriptor->links, &intf->manifest_descs); |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 168 | |
Alex Elder | d393c98 | 2015-06-09 17:42:53 -0500 | [diff] [blame] | 169 | /* desc_size is positive and is known to fit in a signed int */ |
Alex Elder | d8187aa | 2015-03-27 15:06:24 -0500 | [diff] [blame] | 170 | |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 171 | return desc_size; |
| 172 | } |
| 173 | |
| 174 | /* |
| 175 | * Find the string descriptor having the given id, validate it, and |
| 176 | * allocate a duplicate copy of it. The duplicate has an extra byte |
| 177 | * which guarantees the returned string is NUL-terminated. |
| 178 | * |
| 179 | * String index 0 is valid (it represents "no string"), and for |
| 180 | * that a null pointer is returned. |
| 181 | * |
| 182 | * Otherwise returns a pointer to a newly-allocated copy of the |
| 183 | * descriptor string, or an error-coded pointer on failure. |
| 184 | */ |
Greg Kroah-Hartman | 86cad66 | 2014-12-23 15:16:50 -0800 | [diff] [blame] | 185 | static char *gb_string_get(struct gb_interface *intf, u8 string_id) |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 186 | { |
| 187 | struct greybus_descriptor_string *desc_string; |
| 188 | struct manifest_desc *descriptor; |
| 189 | bool found = false; |
| 190 | char *string; |
| 191 | |
| 192 | /* A zero string id means no string (but no error) */ |
| 193 | if (!string_id) |
| 194 | return NULL; |
| 195 | |
Greg Kroah-Hartman | 86cad66 | 2014-12-23 15:16:50 -0800 | [diff] [blame] | 196 | list_for_each_entry(descriptor, &intf->manifest_descs, links) { |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 197 | if (descriptor->type != GREYBUS_TYPE_STRING) |
| 198 | continue; |
| 199 | |
Matt Porter | 7a13e2f | 2014-10-06 09:58:44 -0400 | [diff] [blame] | 200 | desc_string = descriptor->data; |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 201 | if (desc_string->id == string_id) { |
| 202 | found = true; |
| 203 | break; |
| 204 | } |
| 205 | } |
| 206 | if (!found) |
| 207 | return ERR_PTR(-ENOENT); |
| 208 | |
| 209 | /* Allocate an extra byte so we can guarantee it's NUL-terminated */ |
Alex Elder | d393c98 | 2015-06-09 17:42:53 -0500 | [diff] [blame] | 210 | string = kmemdup(&desc_string->string, desc_string->length + 1, |
Greg Kroah-Hartman | ee989b0 | 2019-08-25 07:54:24 +0200 | [diff] [blame] | 211 | GFP_KERNEL); |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 212 | if (!string) |
| 213 | return ERR_PTR(-ENOMEM); |
| 214 | string[desc_string->length] = '\0'; |
| 215 | |
| 216 | /* Ok we've used this string, so we're done with it */ |
| 217 | release_manifest_descriptor(descriptor); |
| 218 | |
| 219 | return string; |
| 220 | } |
| 221 | |
Alex Elder | d88bfb5 | 2014-10-01 21:54:17 -0500 | [diff] [blame] | 222 | /* |
Alex Elder | d393c98 | 2015-06-09 17:42:53 -0500 | [diff] [blame] | 223 | * Find cport descriptors in the manifest associated with the given |
| 224 | * bundle, and set up data structures for the functions that use |
| 225 | * them. Returns the number of cports set up for the bundle, or 0 |
| 226 | * if there is an error. |
Alex Elder | c095bbc | 2014-10-01 21:54:18 -0500 | [diff] [blame] | 227 | */ |
Alex Elder | c46839d | 2015-06-09 17:42:54 -0500 | [diff] [blame] | 228 | static u32 gb_manifest_parse_cports(struct gb_bundle *bundle) |
Alex Elder | c095bbc | 2014-10-01 21:54:18 -0500 | [diff] [blame] | 229 | { |
Alex Elder | c46839d | 2015-06-09 17:42:54 -0500 | [diff] [blame] | 230 | struct gb_interface *intf = bundle->intf; |
Johan Hovold | 98fdf5a | 2016-01-21 17:34:09 +0100 | [diff] [blame] | 231 | struct greybus_descriptor_cport *desc_cport; |
Johan Hovold | d6fba3d | 2016-01-21 17:34:10 +0100 | [diff] [blame] | 232 | struct manifest_desc *desc, *next, *tmp; |
Johan Hovold | 98fdf5a | 2016-01-21 17:34:09 +0100 | [diff] [blame] | 233 | LIST_HEAD(list); |
Alex Elder | a6b13eb | 2015-06-09 17:42:55 -0500 | [diff] [blame] | 234 | u8 bundle_id = bundle->id; |
Bryan O'Donoghue | b38fe34 | 2015-07-21 09:10:27 +0100 | [diff] [blame] | 235 | u16 cport_id; |
Alex Elder | c095bbc | 2014-10-01 21:54:18 -0500 | [diff] [blame] | 236 | u32 count = 0; |
Johan Hovold | 98fdf5a | 2016-01-21 17:34:09 +0100 | [diff] [blame] | 237 | int i; |
Alex Elder | c095bbc | 2014-10-01 21:54:18 -0500 | [diff] [blame] | 238 | |
Alex Elder | a6b13eb | 2015-06-09 17:42:55 -0500 | [diff] [blame] | 239 | /* Set up all cport descriptors associated with this bundle */ |
| 240 | list_for_each_entry_safe(desc, next, &intf->manifest_descs, links) { |
Alex Elder | a6b13eb | 2015-06-09 17:42:55 -0500 | [diff] [blame] | 241 | if (desc->type != GREYBUS_TYPE_CPORT) |
| 242 | continue; |
| 243 | |
| 244 | desc_cport = desc->data; |
| 245 | if (desc_cport->bundle != bundle_id) |
| 246 | continue; |
Alex Elder | c095bbc | 2014-10-01 21:54:18 -0500 | [diff] [blame] | 247 | |
Alex Elder | fb690ca | 2015-06-13 11:02:09 -0500 | [diff] [blame] | 248 | cport_id = le16_to_cpu(desc_cport->id); |
| 249 | if (cport_id > CPORT_ID_MAX) |
Bryan O'Donoghue | b38fe34 | 2015-07-21 09:10:27 +0100 | [diff] [blame] | 250 | goto exit; |
Alex Elder | fb690ca | 2015-06-13 11:02:09 -0500 | [diff] [blame] | 251 | |
Viresh Kumar | 42830f7 | 2016-06-11 08:01:01 +0530 | [diff] [blame] | 252 | /* Nothing else should have its cport_id as control cport id */ |
| 253 | if (cport_id == GB_CONTROL_CPORT_ID) { |
| 254 | dev_err(&bundle->dev, "invalid cport id found (%02u)\n", |
| 255 | cport_id); |
| 256 | goto exit; |
| 257 | } |
| 258 | |
Johan Hovold | d6fba3d | 2016-01-21 17:34:10 +0100 | [diff] [blame] | 259 | /* |
| 260 | * Found one, move it to our temporary list after checking for |
| 261 | * duplicates. |
| 262 | */ |
| 263 | list_for_each_entry(tmp, &list, links) { |
| 264 | desc_cport = tmp->data; |
Greg Kroah-Hartman | d1a9c05 | 2016-02-02 21:31:19 -0800 | [diff] [blame] | 265 | if (cport_id == le16_to_cpu(desc_cport->id)) { |
Johan Hovold | d6fba3d | 2016-01-21 17:34:10 +0100 | [diff] [blame] | 266 | dev_err(&bundle->dev, |
Greg Kroah-Hartman | ee989b0 | 2019-08-25 07:54:24 +0200 | [diff] [blame] | 267 | "duplicate CPort %u found\n", cport_id); |
Johan Hovold | d6fba3d | 2016-01-21 17:34:10 +0100 | [diff] [blame] | 268 | goto exit; |
| 269 | } |
| 270 | } |
Viresh Kumar | 4a7908c | 2016-02-12 21:48:03 +0530 | [diff] [blame] | 271 | list_move_tail(&desc->links, &list); |
Alex Elder | c095bbc | 2014-10-01 21:54:18 -0500 | [diff] [blame] | 272 | count++; |
Johan Hovold | 98fdf5a | 2016-01-21 17:34:09 +0100 | [diff] [blame] | 273 | } |
| 274 | |
| 275 | if (!count) |
| 276 | return 0; |
| 277 | |
| 278 | bundle->cport_desc = kcalloc(count, sizeof(*bundle->cport_desc), |
Greg Kroah-Hartman | ee989b0 | 2019-08-25 07:54:24 +0200 | [diff] [blame] | 279 | GFP_KERNEL); |
Johan Hovold | 98fdf5a | 2016-01-21 17:34:09 +0100 | [diff] [blame] | 280 | if (!bundle->cport_desc) |
| 281 | goto exit; |
| 282 | |
| 283 | bundle->num_cports = count; |
| 284 | |
| 285 | i = 0; |
| 286 | list_for_each_entry_safe(desc, next, &list, links) { |
| 287 | desc_cport = desc->data; |
| 288 | memcpy(&bundle->cport_desc[i++], desc_cport, |
Greg Kroah-Hartman | ee989b0 | 2019-08-25 07:54:24 +0200 | [diff] [blame] | 289 | sizeof(*desc_cport)); |
Alex Elder | a6b13eb | 2015-06-09 17:42:55 -0500 | [diff] [blame] | 290 | |
Alex Elder | c095bbc | 2014-10-01 21:54:18 -0500 | [diff] [blame] | 291 | /* Release the cport descriptor */ |
Alex Elder | a6b13eb | 2015-06-09 17:42:55 -0500 | [diff] [blame] | 292 | release_manifest_descriptor(desc); |
Alex Elder | c095bbc | 2014-10-01 21:54:18 -0500 | [diff] [blame] | 293 | } |
| 294 | |
| 295 | return count; |
Bryan O'Donoghue | b38fe34 | 2015-07-21 09:10:27 +0100 | [diff] [blame] | 296 | exit: |
Johan Hovold | 98fdf5a | 2016-01-21 17:34:09 +0100 | [diff] [blame] | 297 | release_cport_descriptors(&list, bundle_id); |
Viresh Kumar | 4317f87 | 2015-09-07 16:01:20 +0530 | [diff] [blame] | 298 | /* |
| 299 | * Free all cports for this bundle to avoid 'excess descriptors' |
| 300 | * warnings. |
| 301 | */ |
Johan Hovold | f2152eb | 2015-11-25 15:59:25 +0100 | [diff] [blame] | 302 | release_cport_descriptors(&intf->manifest_descs, bundle_id); |
Viresh Kumar | 4317f87 | 2015-09-07 16:01:20 +0530 | [diff] [blame] | 303 | |
Alex Elder | 52e8ce3 | 2015-06-12 10:21:09 -0500 | [diff] [blame] | 304 | return 0; /* Error; count should also be 0 */ |
Alex Elder | c095bbc | 2014-10-01 21:54:18 -0500 | [diff] [blame] | 305 | } |
| 306 | |
| 307 | /* |
Greg Kroah-Hartman | 1db0a5f | 2014-12-12 17:10:17 -0500 | [diff] [blame] | 308 | * Find bundle descriptors in the manifest and set up their data |
| 309 | * structures. Returns the number of bundles set up for the |
Viresh Kumar | 7c183f7 | 2015-04-01 20:32:00 +0530 | [diff] [blame] | 310 | * given interface. |
Alex Elder | d88bfb5 | 2014-10-01 21:54:17 -0500 | [diff] [blame] | 311 | */ |
Greg Kroah-Hartman | 4ab9b3c | 2014-12-19 14:56:31 -0800 | [diff] [blame] | 312 | static u32 gb_manifest_parse_bundles(struct gb_interface *intf) |
Alex Elder | d88bfb5 | 2014-10-01 21:54:17 -0500 | [diff] [blame] | 313 | { |
Alex Elder | c27a253 | 2015-06-09 17:42:56 -0500 | [diff] [blame] | 314 | struct manifest_desc *desc; |
Alex Elder | 2a64fb0 | 2015-06-12 10:21:12 -0500 | [diff] [blame] | 315 | struct gb_bundle *bundle; |
| 316 | struct gb_bundle *bundle_next; |
Alex Elder | d88bfb5 | 2014-10-01 21:54:17 -0500 | [diff] [blame] | 317 | u32 count = 0; |
Viresh Kumar | 98d7fbc | 2015-09-07 16:01:19 +0530 | [diff] [blame] | 318 | u8 bundle_id; |
Johan Hovold | 47091af | 2015-11-25 15:59:26 +0100 | [diff] [blame] | 319 | u8 class; |
Alex Elder | d88bfb5 | 2014-10-01 21:54:17 -0500 | [diff] [blame] | 320 | |
Rui Miguel Silva | 5c864e7 | 2015-11-16 19:23:25 +0000 | [diff] [blame] | 321 | while ((desc = get_next_bundle_desc(intf))) { |
Viresh Kumar | 7c183f7 | 2015-04-01 20:32:00 +0530 | [diff] [blame] | 322 | struct greybus_descriptor_bundle *desc_bundle; |
Alex Elder | d88bfb5 | 2014-10-01 21:54:17 -0500 | [diff] [blame] | 323 | |
Greg Kroah-Hartman | 1db0a5f | 2014-12-12 17:10:17 -0500 | [diff] [blame] | 324 | /* Found one. Set up its bundle structure*/ |
Alex Elder | c27a253 | 2015-06-09 17:42:56 -0500 | [diff] [blame] | 325 | desc_bundle = desc->data; |
Viresh Kumar | 98d7fbc | 2015-09-07 16:01:19 +0530 | [diff] [blame] | 326 | bundle_id = desc_bundle->id; |
Johan Hovold | 47091af | 2015-11-25 15:59:26 +0100 | [diff] [blame] | 327 | class = desc_bundle->class; |
Viresh Kumar | 6c68da2 | 2015-06-22 16:42:27 +0530 | [diff] [blame] | 328 | |
Johan Hovold | 47091af | 2015-11-25 15:59:26 +0100 | [diff] [blame] | 329 | /* Done with this bundle descriptor */ |
| 330 | release_manifest_descriptor(desc); |
| 331 | |
| 332 | /* Ignore any legacy control bundles */ |
Viresh Kumar | 98d7fbc | 2015-09-07 16:01:19 +0530 | [diff] [blame] | 333 | if (bundle_id == GB_CONTROL_BUNDLE_ID) { |
Johan Hovold | 47091af | 2015-11-25 15:59:26 +0100 | [diff] [blame] | 334 | dev_dbg(&intf->dev, "%s - ignoring control bundle\n", |
Greg Kroah-Hartman | ee989b0 | 2019-08-25 07:54:24 +0200 | [diff] [blame] | 335 | __func__); |
Johan Hovold | 47091af | 2015-11-25 15:59:26 +0100 | [diff] [blame] | 336 | release_cport_descriptors(&intf->manifest_descs, |
Greg Kroah-Hartman | ee989b0 | 2019-08-25 07:54:24 +0200 | [diff] [blame] | 337 | bundle_id); |
Johan Hovold | 47091af | 2015-11-25 15:59:26 +0100 | [diff] [blame] | 338 | continue; |
Viresh Kumar | 6c68da2 | 2015-06-22 16:42:27 +0530 | [diff] [blame] | 339 | } |
| 340 | |
Viresh Kumar | 730a2f6 | 2015-06-22 16:42:30 +0530 | [diff] [blame] | 341 | /* Nothing else should have its class set to control class */ |
Johan Hovold | 47091af | 2015-11-25 15:59:26 +0100 | [diff] [blame] | 342 | if (class == GREYBUS_CLASS_CONTROL) { |
Bryan O'Donoghue | b38fe34 | 2015-07-21 09:10:27 +0100 | [diff] [blame] | 343 | dev_err(&intf->dev, |
Johan Hovold | 100e900 | 2015-12-07 15:05:38 +0100 | [diff] [blame] | 344 | "bundle %u cannot use control class\n", |
Viresh Kumar | 98d7fbc | 2015-09-07 16:01:19 +0530 | [diff] [blame] | 345 | bundle_id); |
Bryan O'Donoghue | b38fe34 | 2015-07-21 09:10:27 +0100 | [diff] [blame] | 346 | goto cleanup; |
| 347 | } |
Viresh Kumar | 730a2f6 | 2015-06-22 16:42:30 +0530 | [diff] [blame] | 348 | |
Johan Hovold | 47091af | 2015-11-25 15:59:26 +0100 | [diff] [blame] | 349 | bundle = gb_bundle_create(intf, bundle_id, class); |
Greg Kroah-Hartman | 1db0a5f | 2014-12-12 17:10:17 -0500 | [diff] [blame] | 350 | if (!bundle) |
Alex Elder | 2a64fb0 | 2015-06-12 10:21:12 -0500 | [diff] [blame] | 351 | goto cleanup; |
Alex Elder | c095bbc | 2014-10-01 21:54:18 -0500 | [diff] [blame] | 352 | |
Viresh Kumar | 98d7fbc | 2015-09-07 16:01:19 +0530 | [diff] [blame] | 353 | /* |
| 354 | * Now go set up this bundle's functions and cports. |
| 355 | * |
| 356 | * A 'bundle' represents a device in greybus. It may require |
| 357 | * multiple cports for its functioning. If we fail to setup any |
| 358 | * cport of a bundle, we better reject the complete bundle as |
| 359 | * the device may not be able to function properly then. |
| 360 | * |
| 361 | * But, failing to setup a cport of bundle X doesn't mean that |
| 362 | * the device corresponding to bundle Y will not work properly. |
| 363 | * Bundles should be treated as separate independent devices. |
| 364 | * |
| 365 | * While parsing manifest for an interface, treat bundles as |
| 366 | * separate entities and don't reject entire interface and its |
| 367 | * bundles on failing to initialize a cport. But make sure the |
| 368 | * bundle which needs the cport, gets destroyed properly. |
Viresh Kumar | 98d7fbc | 2015-09-07 16:01:19 +0530 | [diff] [blame] | 369 | */ |
| 370 | if (!gb_manifest_parse_cports(bundle)) { |
Viresh Kumar | 98d7fbc | 2015-09-07 16:01:19 +0530 | [diff] [blame] | 371 | gb_bundle_destroy(bundle); |
| 372 | continue; |
| 373 | } |
| 374 | |
Bryan O'Donoghue | b38fe34 | 2015-07-21 09:10:27 +0100 | [diff] [blame] | 375 | count++; |
Alex Elder | d88bfb5 | 2014-10-01 21:54:17 -0500 | [diff] [blame] | 376 | } |
| 377 | |
| 378 | return count; |
Alex Elder | 2a64fb0 | 2015-06-12 10:21:12 -0500 | [diff] [blame] | 379 | cleanup: |
| 380 | /* An error occurred; undo any changes we've made */ |
| 381 | list_for_each_entry_safe(bundle, bundle_next, &intf->bundles, links) { |
| 382 | gb_bundle_destroy(bundle); |
| 383 | count--; |
| 384 | } |
| 385 | return 0; /* Error; count should also be 0 */ |
Alex Elder | d88bfb5 | 2014-10-01 21:54:17 -0500 | [diff] [blame] | 386 | } |
| 387 | |
Viresh Kumar | a93db2d | 2015-04-01 20:32:02 +0530 | [diff] [blame] | 388 | static bool gb_manifest_parse_interface(struct gb_interface *intf, |
| 389 | struct manifest_desc *interface_desc) |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 390 | { |
Viresh Kumar | a93db2d | 2015-04-01 20:32:02 +0530 | [diff] [blame] | 391 | struct greybus_descriptor_interface *desc_intf = interface_desc->data; |
Johan Hovold | 7c8eb12 | 2016-04-13 19:19:04 +0200 | [diff] [blame] | 392 | struct gb_control *control = intf->control; |
Johan Hovold | 7d963cb | 2016-04-13 19:18:55 +0200 | [diff] [blame] | 393 | char *str; |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 394 | |
| 395 | /* Handle the strings first--they can fail */ |
Johan Hovold | 7d963cb | 2016-04-13 19:18:55 +0200 | [diff] [blame] | 396 | str = gb_string_get(intf, desc_intf->vendor_stringid); |
| 397 | if (IS_ERR(str)) |
Alex Elder | 937d0da | 2014-10-03 14:14:25 -0500 | [diff] [blame] | 398 | return false; |
Johan Hovold | 7c8eb12 | 2016-04-13 19:19:04 +0200 | [diff] [blame] | 399 | control->vendor_string = str; |
Alex Elder | 937d0da | 2014-10-03 14:14:25 -0500 | [diff] [blame] | 400 | |
Johan Hovold | 7d963cb | 2016-04-13 19:18:55 +0200 | [diff] [blame] | 401 | str = gb_string_get(intf, desc_intf->product_stringid); |
| 402 | if (IS_ERR(str)) |
Viresh Kumar | 50fc08f | 2014-11-13 18:14:32 +0530 | [diff] [blame] | 403 | goto out_free_vendor_string; |
Johan Hovold | 7c8eb12 | 2016-04-13 19:19:04 +0200 | [diff] [blame] | 404 | control->product_string = str; |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 405 | |
Bryan O'Donoghue | 8c81d46 | 2016-05-15 19:37:49 +0100 | [diff] [blame] | 406 | /* Assign feature flags communicated via manifest */ |
| 407 | intf->features = desc_intf->features; |
| 408 | |
Viresh Kumar | a93db2d | 2015-04-01 20:32:02 +0530 | [diff] [blame] | 409 | /* Release the interface descriptor, now that we're done with it */ |
| 410 | release_manifest_descriptor(interface_desc); |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 411 | |
Greg Kroah-Hartman | 1db0a5f | 2014-12-12 17:10:17 -0500 | [diff] [blame] | 412 | /* An interface must have at least one bundle descriptor */ |
Greg Kroah-Hartman | 4ab9b3c | 2014-12-19 14:56:31 -0800 | [diff] [blame] | 413 | if (!gb_manifest_parse_bundles(intf)) { |
Bryan O'Donoghue | 09fb10f | 2015-07-21 09:10:28 +0100 | [diff] [blame] | 414 | dev_err(&intf->dev, "manifest bundle descriptors not valid\n"); |
Alex Elder | 937d0da | 2014-10-03 14:14:25 -0500 | [diff] [blame] | 415 | goto out_err; |
Alex Elder | d88bfb5 | 2014-10-01 21:54:17 -0500 | [diff] [blame] | 416 | } |
| 417 | |
Alex Elder | 937d0da | 2014-10-03 14:14:25 -0500 | [diff] [blame] | 418 | return true; |
| 419 | out_err: |
Johan Hovold | 7c8eb12 | 2016-04-13 19:19:04 +0200 | [diff] [blame] | 420 | kfree(control->product_string); |
| 421 | control->product_string = NULL; |
Viresh Kumar | 50fc08f | 2014-11-13 18:14:32 +0530 | [diff] [blame] | 422 | out_free_vendor_string: |
Johan Hovold | 7c8eb12 | 2016-04-13 19:19:04 +0200 | [diff] [blame] | 423 | kfree(control->vendor_string); |
| 424 | control->vendor_string = NULL; |
Alex Elder | 937d0da | 2014-10-03 14:14:25 -0500 | [diff] [blame] | 425 | |
| 426 | return false; |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 427 | } |
| 428 | |
| 429 | /* |
Alex Elder | d393c98 | 2015-06-09 17:42:53 -0500 | [diff] [blame] | 430 | * Parse a buffer containing an interface manifest. |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 431 | * |
| 432 | * If we find anything wrong with the content/format of the buffer |
| 433 | * we reject it. |
| 434 | * |
| 435 | * The first requirement is that the manifest's version is |
| 436 | * one we can parse. |
| 437 | * |
| 438 | * We make an initial pass through the buffer and identify all of |
| 439 | * the descriptors it contains, keeping track for each its type |
| 440 | * and the location size of its data in the buffer. |
| 441 | * |
Alex Elder | d393c98 | 2015-06-09 17:42:53 -0500 | [diff] [blame] | 442 | * Next we scan the descriptors, looking for an interface descriptor; |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 443 | * there must be exactly one of those. When found, we record the |
| 444 | * information it contains, and then remove that descriptor (and any |
| 445 | * string descriptors it refers to) from further consideration. |
| 446 | * |
Greg Kroah-Hartman | 4ab9b3c | 2014-12-19 14:56:31 -0800 | [diff] [blame] | 447 | * After that we look for the interface's bundles--there must be at |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 448 | * least one of those. |
| 449 | * |
Alex Elder | 937d0da | 2014-10-03 14:14:25 -0500 | [diff] [blame] | 450 | * Returns true if parsing was successful, false otherwise. |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 451 | */ |
Greg Kroah-Hartman | 4ab9b3c | 2014-12-19 14:56:31 -0800 | [diff] [blame] | 452 | bool gb_manifest_parse(struct gb_interface *intf, void *data, size_t size) |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 453 | { |
| 454 | struct greybus_manifest *manifest; |
| 455 | struct greybus_manifest_header *header; |
| 456 | struct greybus_descriptor *desc; |
| 457 | struct manifest_desc *descriptor; |
Viresh Kumar | a93db2d | 2015-04-01 20:32:02 +0530 | [diff] [blame] | 458 | struct manifest_desc *interface_desc = NULL; |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 459 | u16 manifest_size; |
| 460 | u32 found = 0; |
Viresh Kumar | 43d9431 | 2014-11-13 18:14:30 +0530 | [diff] [blame] | 461 | bool result; |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 462 | |
Viresh Kumar | 1dd90df | 2014-11-14 17:25:03 +0530 | [diff] [blame] | 463 | /* Manifest descriptor list should be empty here */ |
Greg Kroah-Hartman | 86cad66 | 2014-12-23 15:16:50 -0800 | [diff] [blame] | 464 | if (WARN_ON(!list_empty(&intf->manifest_descs))) |
Viresh Kumar | 1dd90df | 2014-11-14 17:25:03 +0530 | [diff] [blame] | 465 | return false; |
| 466 | |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 467 | /* we have to have at _least_ the manifest header */ |
Alex Elder | d393c98 | 2015-06-09 17:42:53 -0500 | [diff] [blame] | 468 | if (size < sizeof(*header)) { |
Johan Hovold | a0b5542 | 2016-02-11 13:52:46 +0100 | [diff] [blame] | 469 | dev_err(&intf->dev, "short manifest (%zu < %zu)\n", |
Greg Kroah-Hartman | ee989b0 | 2019-08-25 07:54:24 +0200 | [diff] [blame] | 470 | size, sizeof(*header)); |
Alex Elder | 937d0da | 2014-10-03 14:14:25 -0500 | [diff] [blame] | 471 | return false; |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 472 | } |
| 473 | |
| 474 | /* Make sure the size is right */ |
| 475 | manifest = data; |
| 476 | header = &manifest->header; |
| 477 | manifest_size = le16_to_cpu(header->size); |
| 478 | if (manifest_size != size) { |
Johan Hovold | a0b5542 | 2016-02-11 13:52:46 +0100 | [diff] [blame] | 479 | dev_err(&intf->dev, "manifest size mismatch (%zu != %u)\n", |
Greg Kroah-Hartman | ee989b0 | 2019-08-25 07:54:24 +0200 | [diff] [blame] | 480 | size, manifest_size); |
Alex Elder | 937d0da | 2014-10-03 14:14:25 -0500 | [diff] [blame] | 481 | return false; |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 482 | } |
| 483 | |
| 484 | /* Validate major/minor number */ |
| 485 | if (header->version_major > GREYBUS_VERSION_MAJOR) { |
Johan Hovold | a0b5542 | 2016-02-11 13:52:46 +0100 | [diff] [blame] | 486 | dev_err(&intf->dev, "manifest version too new (%u.%u > %u.%u)\n", |
Greg Kroah-Hartman | ee989b0 | 2019-08-25 07:54:24 +0200 | [diff] [blame] | 487 | header->version_major, header->version_minor, |
| 488 | GREYBUS_VERSION_MAJOR, GREYBUS_VERSION_MINOR); |
Alex Elder | 937d0da | 2014-10-03 14:14:25 -0500 | [diff] [blame] | 489 | return false; |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 490 | } |
| 491 | |
| 492 | /* OK, find all the descriptors */ |
Sachin Pandhare | fc25d90 | 2015-11-24 07:59:10 +0530 | [diff] [blame] | 493 | desc = manifest->descriptors; |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 494 | size -= sizeof(*header); |
| 495 | while (size) { |
| 496 | int desc_size; |
| 497 | |
Greg Kroah-Hartman | 86cad66 | 2014-12-23 15:16:50 -0800 | [diff] [blame] | 498 | desc_size = identify_descriptor(intf, desc, size); |
Viresh Kumar | 13fe6a9 | 2015-03-24 17:08:14 +0530 | [diff] [blame] | 499 | if (desc_size < 0) { |
Matt Porter | ff8aed5 | 2014-10-06 13:46:36 -0400 | [diff] [blame] | 500 | result = false; |
Alex Elder | 937d0da | 2014-10-03 14:14:25 -0500 | [diff] [blame] | 501 | goto out; |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 502 | } |
| 503 | desc = (struct greybus_descriptor *)((char *)desc + desc_size); |
| 504 | size -= desc_size; |
Greg Kroah-Hartman | 86bf33a | 2014-11-14 14:37:56 -0800 | [diff] [blame] | 505 | } |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 506 | |
Viresh Kumar | a93db2d | 2015-04-01 20:32:02 +0530 | [diff] [blame] | 507 | /* There must be a single interface descriptor */ |
Greg Kroah-Hartman | 86cad66 | 2014-12-23 15:16:50 -0800 | [diff] [blame] | 508 | list_for_each_entry(descriptor, &intf->manifest_descs, links) { |
Viresh Kumar | a93db2d | 2015-04-01 20:32:02 +0530 | [diff] [blame] | 509 | if (descriptor->type == GREYBUS_TYPE_INTERFACE) |
Greg Kroah-Hartman | 86bf33a | 2014-11-14 14:37:56 -0800 | [diff] [blame] | 510 | if (!found++) |
Viresh Kumar | a93db2d | 2015-04-01 20:32:02 +0530 | [diff] [blame] | 511 | interface_desc = descriptor; |
Greg Kroah-Hartman | 86bf33a | 2014-11-14 14:37:56 -0800 | [diff] [blame] | 512 | } |
| 513 | if (found != 1) { |
Johan Hovold | a0b5542 | 2016-02-11 13:52:46 +0100 | [diff] [blame] | 514 | dev_err(&intf->dev, "manifest must have 1 interface descriptor (%u found)\n", |
Greg Kroah-Hartman | ee989b0 | 2019-08-25 07:54:24 +0200 | [diff] [blame] | 515 | found); |
Greg Kroah-Hartman | 86bf33a | 2014-11-14 14:37:56 -0800 | [diff] [blame] | 516 | result = false; |
| 517 | goto out; |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 518 | } |
| 519 | |
Viresh Kumar | a93db2d | 2015-04-01 20:32:02 +0530 | [diff] [blame] | 520 | /* Parse the manifest, starting with the interface descriptor */ |
| 521 | result = gb_manifest_parse_interface(intf, interface_desc); |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 522 | |
| 523 | /* |
| 524 | * We really should have no remaining descriptors, but we |
| 525 | * don't know what newer format manifests might leave. |
| 526 | */ |
Greg Kroah-Hartman | 86cad66 | 2014-12-23 15:16:50 -0800 | [diff] [blame] | 527 | if (result && !list_empty(&intf->manifest_descs)) |
Johan Hovold | a0b5542 | 2016-02-11 13:52:46 +0100 | [diff] [blame] | 528 | dev_info(&intf->dev, "excess descriptors in interface manifest\n"); |
Alex Elder | 937d0da | 2014-10-03 14:14:25 -0500 | [diff] [blame] | 529 | out: |
Greg Kroah-Hartman | 86cad66 | 2014-12-23 15:16:50 -0800 | [diff] [blame] | 530 | release_manifest_descriptors(intf); |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 531 | |
Matt Porter | ff8aed5 | 2014-10-06 13:46:36 -0400 | [diff] [blame] | 532 | return result; |
Alex Elder | b09c94a | 2014-10-01 21:54:16 -0500 | [diff] [blame] | 533 | } |