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>