[PATCH 0/3] usb: typec: tipd: minor improvements in the firmware update code

From: Javier Carrasco
Date: Thu Jun 06 2024 - 05:13:19 EST


This series makes some minor modifications to the firmware update
mechanism, mainly (but not only) for the tps6598x variant.

The modifications are the following:

1. Use a single call to read the "firmware-name" property. The current
implementations (tps25750 and tps6598x) of the firmware update
mechanism read that property, even though tps_request_firmware()
already does it.
2. Provide the same information via error logs for the both
implementations (firmware name and size missing in tps6598x).
3. Minor code cleanup found by Coccinelle (min() instead of if...else).

The series has been tested with a TPS65987DDHRSHR.

Signed-off-by: Javier Carrasco <javier.carrasco@xxxxxxxxxxxxxx>
---
Javier Carrasco (3):
usb: typec: tipd: drop second firmware name read
usb: typec: tipd: add error log to provide firmware name and size
usb: typec: tipd: use min() to set tps6598x firmware packet size

drivers/usb/typec/tipd/core.c | 33 ++++++++++++---------------------
1 file changed, 12 insertions(+), 21 deletions(-)
---
base-commit: 234cb065ad82915ff8d06ce01e01c3e640b674d2
change-id: 20240605-tps6598x_fw_update_log-1d181faab2ba

Best regards,
--
Javier Carrasco <javier.carrasco@xxxxxxxxxxxxxx>