Received-SPF: pass (google.com: domain of linux-kernel+bounces-123309-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 RESEND 0/2] usb: typec: tipd: fix event checking in
 interrupt service routines
Date: Thu, 28 Mar 2024 17:55:50 +0100
Message-Id: <20240328-tps6598x_fix_event_handling-v1-0-502721ff705b@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>, 
 Abdel Alkuor <abdelalkuor@geotab.com>
Cc: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, 
 Javier Carrasco <javier.carrasco@wolfvision.net>, stable@vger.kernel.org
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?SUgyRGRGalhIcjY5VnhKTFIvMU1QT1pPR2xRNGloTmJReVVzMzliTlhFTGZo?=
 =?utf-8?B?dENlYllnVStBYUdqUVRKbEphaituS0hmQlhrWFlWa0lNZ1A1RFFaVkZmdlI1?=
 =?utf-8?B?RzVNYUJMdnhtTXhQMmFSeWdXYjUwaFFiaTdqS2dueGtZUEZwb1RLUXNiVHgx?=
 =?utf-8?B?UnBZMTdORHhnNjNUSXg3VzliOHNRb2RoeGZ1cGQwQm1aQmcwSDZteTNOdks2?=
 =?utf-8?B?RjhNWTlqM3FESXdlNjJRZldUZlBWc0RwaDVBOVVvTzdSbVBEdDJHUkQrdzI3?=
 =?utf-8?B?Ym1SOEFkeFh3TGk3WFZvc2tPRDJiL09HcEY2aHZEWWZFNENTZlg4UGNEYkNW?=
 =?utf-8?B?K3gyUHZGbEtBQWl1U2F1YlFtT24vRkprUUhmZGoyOEo2N1R1WklkRUNycS9B?=
 =?utf-8?B?WFM1TzZMZ0VMTHFTSkMwbGtPaXdNeGhWWllXcmZaK1U4bkdYMkV1QlpLcmJF?=
 =?utf-8?B?bnUzOE9NbW9ITFUxbnJVdTBoUUgxRDNhVHVPUnRhaEU5czNEMXNkWElRb25M?=
 =?utf-8?B?c3ZyWlg0N25XSmx4bnM4d0JFRHd0ZzJMSlhsSFF2S0dYTEp5NFEwUDVSazEr?=
 =?utf-8?B?M3hZL1JMSlJNWUdjQ3J2b01NOFJSa2dudGI5TENEVUdsTDlCeUFhbjVPaW9N?=
 =?utf-8?B?VFJyeG11bVcyZlhoZjYwMEZLTDZGL1UvK283aXpOYVQ1eStjVENCeFR5N1l1?=
 =?utf-8?B?VTVWaEwxOTJ5STRmVW1aMWtDOUxyY2pXdkpsL2plQkdTNGw1eDZPUWFiS2JU?=
 =?utf-8?B?QVVvMjN3M2FQbDAreS84T21Qc1o5blhldWxSNTFtc0w2ai85RWF5TlU5Nitp?=
 =?utf-8?B?MkJubE5sZ1dWRXNVc0xObFc3OW5peE5qMWp3NjdtOWJTeHdUTEZQbVpXQ3lv?=
 =?utf-8?B?aFRoc1VJV3dYNVN5UzVEaUcxVTQ0L3Rqc3FrOGY1RGNnU2dJL2hPL3dYZWNL?=
 =?utf-8?B?TWxwT0VNL1hzY1loVTBBTGRubEtDU1pMeDNvMlBDdzVMV0wwblB4SE8vVXVw?=
 =?utf-8?B?d0dOcjB5UUgxcXhhbHdOZkRwdUx0VlREYy9RN2l2S2ZGdGhVQ2JSQWhJdStQ?=
 =?utf-8?B?USt1M1Q1WHIydTVwZkZoS0VEQ1NGK2lhZ0Jza25IWkRUb1VId0kzazJmYzhX?=
 =?utf-8?B?YUlIRDJBZXgxb3Y2Qk9nN2I3UUhnajNsN29rYVF3bjIvZ3YwR2d2WHY1SWZP?=
 =?utf-8?B?aHNob0FYaStaSXZjaTlkYWhDdityN0FzQzZHaUNKZHlkL0pVclZxbzgwQU4y?=
 =?utf-8?B?Y3l3RFNLOHpQMVREVjN0Q2lwTndSbVA1OGJrNytlV1NOM2ZWV1VLN2pSNzho?=
 =?utf-8?B?OEczSGtaTU1yZDVKa2pVaEFWYTJHNjlQMjduaUdpRkwydFBZZUl1TFlVc0Vm?=
 =?utf-8?B?RGY0UktHN0lCSXBuSTBlWDdnRHZPUTAwZW90TEhaZWJUTUNLV3pHRUJJNFBo?=
 =?utf-8?B?Mm9Fa3FOZVY1UURHcXBXRUR4M1cyWXp3ZlZxQ3VWcHZJY1k2dVd3ajdiOWtn?=
 =?utf-8?B?VjdqZmpBS2xyZXI2bHhONzVZU0xKZ1pYZEhOd3dQS0JmVUJYN2tCWStKZENs?=
 =?utf-8?B?QXFiMkpRdmNpanFUWUdFYUJqQkt3QUZIaG5RS1laU24xekllby9LTGFlRm45?=
 =?utf-8?B?c2hUZHhqSlBEaW5IWGJqenpQQWQxZExldXltU0kzaHlsQUttMU8zd3BnMDhG?=
 =?utf-8?B?S0pIMHlOazkySSt1WTdNSmk2QktKQ0J0S2hIRHBMc0dlQ25YWEszN2diejRv?=
 =?utf-8?B?d0ovSTR2Q0tDRnBWV1VvY3RBSC8ydzZHRGtNb3VsaURTVk5jaW5UanVpbFk2?=
 =?utf-8?B?T2xCd0xaa1k0Q3UxeWpZU2lmWkthUGluRzcreFFTSStNblVJbEc4aHdXTWdG?=
 =?utf-8?B?U3RNL2o4Q21pYW0xQVFKN1JBSSt3RkZzaForM0tSQzFZU2NNS3NKNVhxMjV3?=
 =?utf-8?B?OU1UZFVnb2xrVGdqNjVoT1FoMVE2UTliTkNlMEhEMmVkOUp0NkxqaU5lOVhW?=
 =?utf-8?B?anlqdDRDMTBNSGlTWEUyL3JLdWt0MVI1UHNKeDBqNG8zeWlKRjd5dDY1UGUz?=
 =?utf-8?B?cUZKVC9MME1zLzZjUVRiMVM0UFk4dzJIVnVyZklsdDhVNzBkZngzK1NBT0U3?=
 =?utf-8?B?SkxUMGFKNGNDcndJT2hMRnorZlZ4TjVkNzZKZ1c4UnVXd1paMk52RHBjN0Zi?=
 =?utf-8?Q?Vu1tqY6rMGf6F2jMQQ/WnRs=3D?=
X-OriginatorOrg: wolfvision.net
X-MS-Exchange-CrossTenant-Network-Message-Id: 78943a57-7016-44b6-dfe3-08dc4f47f411
X-MS-Exchange-CrossTenant-AuthSource: VE1PR08MB4974.eurprd08.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Mar 2024 16:56:03.7560
 (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: a430X1iRk3YMKjVGZHQGdpbMLKeI8BMfl/Qj1MBt+uhcrASKffpnTJBJotbMGQwD27WNGQmg90RF6dRDZLg/wcX0GAJ1vUe4aesmqxcp5XI=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR08MB10147

The ISRs of the tps25750 and tps6598x do not handle generated events
properly under all circumstances.

The tps6598x ISR does not read all bits of the INT_EVENTX registers,
leaving events signaled with bits above 64 unattended. Moreover, these
events are not cleared, leaving the interrupt enabled.

The tps25750 reads all bits of the INT_EVENT1 register, but the event
checking is not right because the same event is checked in two different
regions of the same register by means of an OR operation.

This series aims to fix both issues by reading all bits of the
INT_EVENTX registers, and limiting the event checking to the region
where the supported events are defined (currently they are limited to
the first 64 bits of the registers, as the are defined as BIT_ULL()).

If the need for events above the first 64 bits of the INT_EVENTX
registers arises, a different mechanism might be required. But for the
current needs, all definitions can be left as they are.

Note: resend to add the Cc tag for 'stable' (fixes in the series).

Signed-off-by: Javier Carrasco <javier.carrasco@wolfvision.net>
---
Javier Carrasco (2):
      usb: typec: tipd: fix event checking for tps25750
      usb: typec: tipd: fix event checking for tps6598x

 drivers/usb/typec/tipd/core.c | 37 +++++++++++++++++++++----------------
 1 file changed, 21 insertions(+), 16 deletions(-)
---
base-commit: 4cece764965020c22cff7665b18a012006359095
change-id: 20240328-tps6598x_fix_event_handling-3398d3d82f85

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