greybus-dev

greybus-dev@lists.linaro.org

5 participants
790 discussions

[PATCH v4] Fix tiny typo in firmware-management docs

by zntsproj

Signed-off-by: zntsproj <vseokaktusah7(a)gmail.com> --- .../staging/greybus/Documentation/firmware/firmware-management | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/greybus/Documentation/firmware/firmware-management b/drivers/staging/greybus/Documentation/firmware/firmware-management index 7918257e5..393455557 100644 --- a/drivers/staging/greybus/Documentation/firmware/firmware-management +++ b/drivers/staging/greybus/Documentation/firmware/firmware-management @@ -193,7 +193,7 @@ Identifying the Character Device There can be multiple devices present in /dev/ directory with name gb-authenticate-N and user first needs to identify the character device used for -authentication a of particular interface. +authentication of a particular interface. The Authentication core creates a device of class 'gb_authenticate', which shall be used by the user to identify the right character device for it. The class -- 2.51.2

7 months, 1 week

[PATCH v3] Fix tiny typo in firmware-management docs

by zntsproj

7 months, 1 week

[PATCH] Fix tiny typo in firmware-management docs

by zntsproj

7 months, 1 week

[PATCH v2] Fix tiny typo in firmware-management docs

by zntsproj

--- .../staging/greybus/Documentation/firmware/firmware-management | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/greybus/Documentation/firmware/firmware-management b/drivers/staging/greybus/Documentation/firmware/firmware-management index 7918257e5..393455557 100644 --- a/drivers/staging/greybus/Documentation/firmware/firmware-management +++ b/drivers/staging/greybus/Documentation/firmware/firmware-management @@ -193,7 +193,7 @@ Identifying the Character Device There can be multiple devices present in /dev/ directory with name gb-authenticate-N and user first needs to identify the character device used for -authentication a of particular interface. +authentication of a particular interface. The Authentication core creates a device of class 'gb_authenticate', which shall be used by the user to identify the right character device for it. The class -- 2.51.2

7 months, 1 week

[PATCH] Fix tiny typo in firmware-management docs

by zntsproj

7 months, 1 week

[PATCH] staging: greybus: fw-download: Fix find firmware req

by Ayush Singh

According to the Greybus Spec published here [0], the Greybus firmware download find firmware request should have both tag and format as request arguments. However, currently, the linux kernel seems to have defined this request incorrectly since format is missing. The patch adds format to request and am using it as the file extension of the firmware. [0]: https://github.com/projectara/greybus-spec/blob/ac47bc32dce1256489a82ff1f46… Signed-off-by: Ayush Singh <ayush(a)beagleboard.org> --- According to the Greybus Spec published here [0], the Greybus firmware download find firmware request should have both tag and format as request arguments. However, currently, the linux kernel seems to have defined this request incorrectly since format is missing. The patch adds format to request and am using it as the file extension of the firmware. I came across the bug while working on greybus-for-zephyr [1], to get it ready for upstreaming as zephyr module. Open Questions *************** 1. Handle empty format Not sure what to do in case format is just NULL. Should the request fail? There is no reason to not support firmware without extension. So personally, I don't think it should be treated as error. [0]: https://github.com/projectara/greybus-spec/blob/ac47bc32dce1256489a82ff1f46… [1]: https://github.com/Ayush1325/greybus-for-zephyr --- drivers/staging/greybus/fw-download.c | 31 ++++++++++++++++++++++++------- include/linux/greybus/greybus_protocols.h | 2 ++ 2 files changed, 26 insertions(+), 7 deletions(-) diff --git a/drivers/staging/greybus/fw-download.c b/drivers/staging/greybus/fw-download.c index 9a09bd3af79ba0dcf7efa683f4e86246bcd473a5..06f1be8f3121e29551ea8416d5ee2666339b2fe3 100644 --- a/drivers/staging/greybus/fw-download.c +++ b/drivers/staging/greybus/fw-download.c @@ -159,7 +159,7 @@ static int exceeds_release_timeout(struct fw_request *fw_req) /* This returns path of the firmware blob on the disk */ static struct fw_request *find_firmware(struct fw_download *fw_download, - const char *tag) + const char *tag, const char *format) { struct gb_interface *intf = fw_download->connection->bundle->intf; struct fw_request *fw_req; @@ -178,10 +178,17 @@ static struct fw_request *find_firmware(struct fw_download *fw_download, } fw_req->firmware_id = ret; - snprintf(fw_req->name, sizeof(fw_req->name), - FW_NAME_PREFIX "%08x_%08x_%08x_%08x_%s.tftf", - intf->ddbl1_manufacturer_id, intf->ddbl1_product_id, - intf->vendor_id, intf->product_id, tag); + if (strnlen(format, GB_FIRMWARE_FORMAT_MAX_SIZE) == 0) { + snprintf(fw_req->name, sizeof(fw_req->name), + FW_NAME_PREFIX "%08x_%08x_%08x_%08x_%s", + intf->ddbl1_manufacturer_id, intf->ddbl1_product_id, + intf->vendor_id, intf->product_id, tag); + } else { + snprintf(fw_req->name, sizeof(fw_req->name), + FW_NAME_PREFIX "%08x_%08x_%08x_%08x_%s.%s", + intf->ddbl1_manufacturer_id, intf->ddbl1_product_id, + intf->vendor_id, intf->product_id, tag, format); + } dev_info(fw_download->parent, "Requested firmware package '%s'\n", fw_req->name); @@ -225,7 +232,7 @@ static int fw_download_find_firmware(struct gb_operation *op) struct gb_fw_download_find_firmware_request *request; struct gb_fw_download_find_firmware_response *response; struct fw_request *fw_req; - const char *tag; + const char *tag, *format; if (op->request->payload_size != sizeof(*request)) { dev_err(fw_download->parent, @@ -245,7 +252,17 @@ static int fw_download_find_firmware(struct gb_operation *op) return -EINVAL; } - fw_req = find_firmware(fw_download, tag); + format = (const char *)request->format; + + /* firmware_format must be null-terminated */ + if (strnlen(format, GB_FIRMWARE_FORMAT_MAX_SIZE) == + GB_FIRMWARE_FORMAT_MAX_SIZE) { + dev_err(fw_download->parent, + "firmware-format is not null-terminated\n"); + return -EINVAL; + } + + fw_req = find_firmware(fw_download, tag, format); if (IS_ERR(fw_req)) return PTR_ERR(fw_req); diff --git a/include/linux/greybus/greybus_protocols.h b/include/linux/greybus/greybus_protocols.h index 820134b0105c2caf8cea3ff0985c92d48d3a2d4c..48d91154847dbc7d3c01081eadc69f96dbe41a9f 100644 --- a/include/linux/greybus/greybus_protocols.h +++ b/include/linux/greybus/greybus_protocols.h @@ -214,10 +214,12 @@ struct gb_apb_request_cport_flags { #define GB_FW_DOWNLOAD_TYPE_RELEASE_FIRMWARE 0x03 #define GB_FIRMWARE_TAG_MAX_SIZE 10 +#define GB_FIRMWARE_FORMAT_MAX_SIZE 10 /* firmware download find firmware request/response */ struct gb_fw_download_find_firmware_request { __u8 firmware_tag[GB_FIRMWARE_TAG_MAX_SIZE]; + __u8 format[GB_FIRMWARE_FORMAT_MAX_SIZE]; } __packed; struct gb_fw_download_find_firmware_response { --- base-commit: aaa9c3550b60d6259d6ea8b1175ade8d1242444e change-id: 20251022-gb-fw-a5db68692b5d Best regards, -- Ayush Singh <ayush(a)beagleboard.org>

7 months, 2 weeks

[PATCH] staging: greybus: documentation: replace strncpy() with strscpy_pad()

by Vivek BalachandharTN

strncpy() does not guarantee NUL-termination and is deprecated for NUL-terminated strings. Replace it with strscpy_pad(), which guarantees NUL-termination and zero-pads the remaining bytes, matching the fixed-size firmware tag semantics. Signed-off-by: Vivek BalachandharTN <vivek.balachandhar(a)gmail.com> --- .../greybus/Documentation/firmware/firmware.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/greybus/Documentation/firmware/firmware.c b/drivers/staging/greybus/Documentation/firmware/firmware.c index 3b35ef6d4adb..deac8584841b 100644 --- a/drivers/staging/greybus/Documentation/firmware/firmware.c +++ b/drivers/staging/greybus/Documentation/firmware/firmware.c @@ -63,8 +63,8 @@ static int update_intf_firmware(int fd) intf_load.major = 0; intf_load.minor = 0; - strncpy((char *)&intf_load.firmware_tag, firmware_tag, - GB_FIRMWARE_U_TAG_MAX_SIZE); + strscpy_pad((char *)&intf_load.firmware_tag, firmware_tag, + GB_FIRMWARE_U_TAG_MAX_SIZE); ret = ioctl(fd, FW_MGMT_IOC_INTF_LOAD_AND_VALIDATE, &intf_load); if (ret < 0) { @@ -101,8 +101,8 @@ static int update_backend_firmware(int fd) /* Get Backend Firmware Version */ printf("Getting Backend Firmware Version\n"); - strncpy((char *)&backend_fw_info.firmware_tag, firmware_tag, - GB_FIRMWARE_U_TAG_MAX_SIZE); + strscpy_pad((char *)&backend_fw_info.firmware_tag, firmware_tag, + GB_FIRMWARE_U_TAG_MAX_SIZE); retry_fw_version: ret = ioctl(fd, FW_MGMT_IOC_GET_BACKEND_FW, &backend_fw_info); @@ -129,8 +129,8 @@ static int update_backend_firmware(int fd) /* Try Backend Firmware Update over Unipro */ printf("Updating Backend Firmware\n"); - strncpy((char *)&backend_update.firmware_tag, firmware_tag, - GB_FIRMWARE_U_TAG_MAX_SIZE); + strscpy_pad((char *)&backend_update.firmware_tag, firmware_tag, + GB_FIRMWARE_U_TAG_MAX_SIZE); retry_fw_update: backend_update.status = 0; -- 2.34.1

7 months, 4 weeks

[PATCH 0/2] staging: fix W=1 format-truncation warnings

by Masaharu Noguchi

- This series fixes -Wformat-truncation warnings in staging drivers found with W=1 builds: - greybus: prefix device id to widget/control names safely - vc04_services/bcm2835-camera: compose V4L2 bus_info safely Both changes use scnprintf() for the prefix and strscpy() for the remainder. No functional change intended. Masaharu Noguchi (2): staging: greybus: audio_topology: avoid -Wformat-truncation staging: vc04_services: bcm2835-camera: avoid -Wformat-truncation drivers/staging/greybus/audio_topology.c | 11 ++++++----- .../vc04_services/bcm2835-camera/bcm2835-camera.c | 5 +++-- 2 files changed, 9 insertions(+), 7 deletions(-) -- 2.39.5

9 months, 1 week