Received-SPF: pass (google.com: domain of linux-kernel+bounces-203949-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
From: Javier Carrasco <javier.carrasco@wolfvision.net>
Subject: [PATCH 0/3] usb: typec: tipd: minor improvements in the firmware
 update code
Date: Thu, 06 Jun 2024 11:03:54 +0200
Message-Id: <20240606-tps6598x_fw_update_log-v1-0-2b5b8369a0ba@wolfvision.net>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
To: Heikki Krogerus <heikki.krogerus@linux.intel.com>, 
 Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, 
 Javier Carrasco <javier.carrasco@wolfvision.net>, 
 Julia Lawall <julia.lawall@inria.fr>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?NSs0cldzUWxrQ2QwQmQrd2tQVDlTbWNSa2ptaCtyMFhoVU5FZ1prYmhoWWRs?=
 =?utf-8?B?Y3pxSHJwVXNDY3o2U05VSGIyNnQzb0t4c3N1S1ZYbVZEcEpMY1dFTFl0Y0t3?=
 =?utf-8?B?VGVFaEtjOGMzTzVmYi9EdnNMU2ZzTFRVRDE4SThnZEwzaE81a0ZOMERqYWFJ?=
 =?utf-8?B?c2h5ZU5KVDJrUWJlak1BMXMvZ3ZYbUVjVVBKNWU0K0MvcTd3aDROQUYwUXFj?=
 =?utf-8?B?ZkU2UHpXWTRMRGpJdWNSK29JbVEvdjdUanZtdEhXN05hcGdUNkw2cjNlZlNC?=
 =?utf-8?B?cW9aTW9yeElDcjRLaER5WDk3YU4raFJZMFc4dU1xelEvcE5SdHFNRWMrZUlv?=
 =?utf-8?B?elFMQkEvS1Z1ZkZYd3JyMHJaZEZaL1NGUVNwL0NYVlA4ZmZQTUMwN0dtbElz?=
 =?utf-8?B?ZndxNHZxZGxMcE5MRG9FeUpQcUgzRjZjamFZRXFVUHVVczh5WmRqVEtSa3Q4?=
 =?utf-8?B?VkxSSmlwa28wWk1qY2U4a3lYdUxrYTZJNk4vajc4V3FTUnNiazJTcDVINWk1?=
 =?utf-8?B?ejhBUUFEUkFLZGk0dmk5VzhtOElEMElmcHdnUkpncUJzU21JdHVOdFhaSGsv?=
 =?utf-8?B?d3F5aWJLcTBVamlSTG1CQ0ZmcE10UzdWWlNZVzdicEFCR1ZXZVRXUEpqUTR6?=
 =?utf-8?B?b0MvY042Q1YxRnFrMVVBRFBOM0xibTl6QnhzMEx3VFluSEg3OU5EcUlYbm5x?=
 =?utf-8?B?cHJHVzgvWWs1dk1SVW1WMzNxRFFjTlY3dnhNSFpuZEhNM09Ca09seEdWUHRj?=
 =?utf-8?B?bVhUYVMzTjAzRzArY0t5NnRVKzdCelovQW81WjhMUDRXamgxNUMrOW82NXZV?=
 =?utf-8?B?QXMrOFpGK2dkWGJoZVFJc0ZZQnFTTThXRFVDV1VRbWk3S0JZWGVpUENTS2xU?=
 =?utf-8?B?anE2YUQ4WW44NzRUL0JuQ0V0NnNTU20xcFhza3FXUEJQMXRsYzZiemtxTHpt?=
 =?utf-8?B?R281UGZqV01HdWJ6L0IzeitYblhRc3dSai9odEpoK3hEQ01HUDJtTEo3Ym9J?=
 =?utf-8?B?SkJIT0lnaWZMZGRQQkxKMnhxN1BoNVJiT1JtRmxoOEw3TTBBZWRYaWF1dEtB?=
 =?utf-8?B?UXNwZ0hjMy9MckFucTQ3TjlNRGFjMGd0eXRVNzBuaTl0VXM5djVHS0s0NVRI?=
 =?utf-8?B?NVBBb0x5bVpmclJSU1FQNTRJMGIrRVh6Zmh1ZnZrS2d1TUVJa0RiTUNSSVQ3?=
 =?utf-8?B?U0RFcUUySm9wVit0WGpybEwvV3Qxa3dyb2U5dGFsdnNjek03bll0bE9hcnlO?=
 =?utf-8?B?TTdnZTVMK3hLczNPOGlrdmFqMmFyYjZhMmw4RVdaTDd6YnZBY29yU1cvK1Vw?=
 =?utf-8?B?bDk2SWR6VW1wSHBHanBEZjFNaTJTOFlIMGRJRTgzdGNKR3VSQURFMkorL0lq?=
 =?utf-8?B?cFpKOGR0VHlRNGtpOHMyK2x0c3I4cDlEN0Z3c25JMkRyeDZiMXFSVjNyOGhS?=
 =?utf-8?B?L3NjM1I5THdPTGRiTWFuRVp5cDF6aGZUMURPbE5uT1E0Q1NIRmNuamFRK2la?=
 =?utf-8?B?NTFkL0ltWWxFM0tkd2c4N2RkbDA3ZzJjRENjaDk3MkVTS2ZZNGVSamNwSFBK?=
 =?utf-8?B?R2VIUldOMDUzNnB2WGpSQ2xrZGl0enhHWi9FdDhiRjBuZWEzeGVVYjRCM0FV?=
 =?utf-8?B?WlJvV1ZacEtUQjhlUm03cUZqUkpyMVdGeTkzNi95ek1pMSt4OVBCTkZERi9C?=
 =?utf-8?B?M1NCeDVnb05KWFo2WFROcWVGM2VjTzhZa2U2a0lHd3o2akc0V1pzdEJ3Q2NJ?=
 =?utf-8?B?Q1hDQnNjUURhdDl5aWJzcEQ5aHNWRHdFUFBDVVorUHNqSzJQcGRqNzZkdXB1?=
 =?utf-8?B?a0YwL1lVTXFEU1VFcENEUTZCazhNSk85Sys3SldTVHBxZHlpNW9HWmdNb2pP?=
 =?utf-8?B?QnpGZnhrUmtFKzRQUWpiZlp3NVM4dmwyUi9OSVJaYzlSdjc2eVFiQ2dBZ3ZE?=
 =?utf-8?B?dm9kQ2xwd2pmWjcrbGk3V0dDaVBBNkY3c1ZmNjlUb3l0NkZPaVMxYXBRak1q?=
 =?utf-8?B?ZVNBL0Q1TFpJSStDa1BpUm1rNkUzdEt3YjN2VTNIcXFVM3NBU0hQaW1xR29Q?=
 =?utf-8?B?NnA1QXRaSGh2YkZTbnlmWm5xTWZ0Vm5XUWM5NXkwbXFSQlNWOTQvWm1Yb0RJ?=
 =?utf-8?B?U2lZd0ludm9JYXhwSXBUR09HMDM2NmU2aDBjTWZxNEowZWt0MjR6a3dIcEZi?=
 =?utf-8?B?SHc9PQ==?=
X-OriginatorOrg: wolfvision.net
X-MS-Exchange-CrossTenant-Network-Message-Id: 856275c7-dbbe-4023-b90c-08dc86079cc0
X-MS-Exchange-CrossTenant-AuthSource: VE1PR08MB4974.eurprd08.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Jun 2024 09:04:03.3301
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: e94ec9da-9183-471e-83b3-51baa8eb804f
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: IkCvwTHuMuwBHZVRZowskuI+VYeSP0o543DVUKeg5FlP2Rd/jAiAzy5p87r6aJ6sPVT8QSm4Y5DYCEeb/paOOEB8o4WXQ1cDgPD4LZbq37Q=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR08MB9513

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,
-- 
Javier Carrasco <javier.carrasco@wolfvision.net>