mbox series

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

Message ID 20240606-tps6598x_fw_update_log-v1-0-2b5b8369a0ba@wolfvision.net
Headers show
Series usb: typec: tipd: minor improvements in the firmware update code | expand

Message

Javier Carrasco June 6, 2024, 9:03 a.m. UTC
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@wolfvision.net>
---
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,