Received-SPF: pass (google.com: domain of linux-kernel+bounces-128440-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <82e0f16a-8f25-4e69-97bb-ee3c78f2b183@wolfvision.net>
Date: Tue, 2 Apr 2024 19:39:56 +0200
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH RESEND 0/2] usb: typec: tipd: fix event checking in
 interrupt service routines
To: Heikki Krogerus <heikki.krogerus@linux.intel.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
 Abdel Alkuor <abdelalkuor@geotab.com>, linux-usb@vger.kernel.org,
 linux-kernel@vger.kernel.org, stable@vger.kernel.org
References: <20240328-tps6598x_fix_event_handling-v1-0-502721ff705b@wolfvision.net>
 <ZgvcJPTJS7d4aYgr@kuha.fi.intel.com>
Content-Language: en-US, de-AT
From: Javier Carrasco <javier.carrasco@wolfvision.net>
In-Reply-To: <ZgvcJPTJS7d4aYgr@kuha.fi.intel.com>
Content-Type: text/plain; charset=UTF-8
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?VTIxcWd2TXVPd2d5eHJ3eWFFbmVXeTVpWFhEMmxQRTRvZDYyUEFzUGVTdFdK?=
 =?utf-8?B?dXdRUXVZVUZSK2JyTmNBamhWU2twQTlxbEw0Nkt4RFVRYTUrcnQ5L0xhdjlW?=
 =?utf-8?B?WmdIVmQ1UFhJelNNZy9pRzZTb1BUaTlmRkxFZWIzMUtLbXpzQldqUDFrN1d2?=
 =?utf-8?B?WkxSeWxvOTlKN09BRHd1TlRpeWtUUXlzNVZlaHZWd3RFMlpnRnd1aitzZ01t?=
 =?utf-8?B?MnNtZTA3TFRsSU1LYkwvVkZ0Wnk3UUJkK1hGaDg4bG05N3pDT3hoR2hpWUh5?=
 =?utf-8?B?RTJ6V2VDZTNjRDR6UmQ3SHNEbWtoWXJteEhjLzd4NHJndHpKYWZuQktCcnBu?=
 =?utf-8?B?REQ4SnRXdmNSRHN3QjU4dFVrRWkzbTlmWkFsTFJNRExLRUQ4QjBKZnVCb3FZ?=
 =?utf-8?B?M0FlVURGOTFkNHR0dnV0WTBiaENWa3ZTcnhESFhHZ2ZLbTlPR0pjb2xHTFNa?=
 =?utf-8?B?K0lTSjlkT0E4cVc2dVJLQVEydjRRTmdIeDZLZ0NXSll4QWRGbHR5UmwvN2Vu?=
 =?utf-8?B?Uis5V3F1SlFOdTdCZjRndmN3Y280Zjkzazh3cURnTFAyWmI5U0xQazdIc0pH?=
 =?utf-8?B?QU5EdHBWL2FyME15bG44ZFhqSWZNc1V0SjZEZlFrWUxHQThDSTE1TFc0TW1P?=
 =?utf-8?B?bnBkMytOZGRsOVIvbC8rWkRZZy9ndnNFcUl1clN1T01EQXNPOUgrVlJNbGZQ?=
 =?utf-8?B?MTZTM2E4UDN4Z3R0WXBWTVdRUXVsSDBxQUZhOUN1YUlucVF3MzNuQldtNWE4?=
 =?utf-8?B?SGQ5T2Y3Q1FqaVZNZW5zQzlJODhNY251SFZVNEFML0pzZ1dFbC9ocFZNUHV1?=
 =?utf-8?B?RFZ6ZHFmTVBJQWRuRWtCME9KY1pKbUhtZk05Y1VZK3J3S3YxSGZaTkFGQWIv?=
 =?utf-8?B?OFNNekprWVgyY2sweTNEYVl6ZzZCY2c0VTJweTRtN1lHVGk5eFdiRE0yN1VN?=
 =?utf-8?B?SGxsa2tVajl4Q0s3RjgxQ1lrM1RvbmFTSDlXZCtZUE5DdEdYZGZrOU1TRW1B?=
 =?utf-8?B?TlZiMzNDQ0NOSldhUEF2NnR3MitVWXZkQTkyNjRqZzU1WDI5WDlhWVpza0ln?=
 =?utf-8?B?eHlwRnhYeDRGYmo5aTBCeitLdkoxNllhVDhZVzNBV3JWemdvOFBoaUNqNUVY?=
 =?utf-8?B?ZDBOSWtvSkFZN2RHUktJSEpyK0NPN0txOEtieVpVRjY5T09ySFU2OFJQZWJQ?=
 =?utf-8?B?emZ4ekp0b3NXeTVLa0dnbytTcXVyRGM4Vko0M0lHcUg4YjcwcVJRTm5RYWwy?=
 =?utf-8?B?QkFaUm1EZXlNa0pZVTYxM3YwSWkxREhURjlnNXB1NTI5OTh2Z0pCVkd6MUlm?=
 =?utf-8?B?a1ZmRTdnMCs0QUFDUXJFL3FZdy9LOEVKZytWTWpEVFRNczFTcERIcDJ1c3Zx?=
 =?utf-8?B?dGY2RXF1dndZVHFIT1ZmZ1p0NENVQjZJZ1BoUHg2emoxSDZPejgyd0VyOSsz?=
 =?utf-8?B?NHBRTlhIeUMxeTVQMThWbTl2NDlLdFlkcUpXaWR2c1gyUmd3U0dmMGtQTmEz?=
 =?utf-8?B?MEpBdTk1T2IvQ29RL21NZ0IrUFk0MGMvdkhCYkJMWlpZTnZ2VGNJQ0pHdVVB?=
 =?utf-8?B?aVF1Q2xpbGt6aGMvQWdlbVd0SWpLT2JST2wyVmpEcXlXWC9aTGhzcnVyZ29O?=
 =?utf-8?B?VTV4aGNKS1pHbDEwT3JJLzlWK3JnYnZvMmtXN2Faa3UzRWtKVjhFMVRiK3dh?=
 =?utf-8?B?cEhQMTZDbWh3QnFMY1RydDdrUjZES0lycHJFcjZsY1FKbjRtUnJOeWNTS05l?=
 =?utf-8?B?Y0VhZFdmVkZqb0JMRXd0WEZsN3VwaUpYMGhjY05wbGdSZzZHSDJLc3hkRi83?=
 =?utf-8?B?OUphOG91MDFrVDcyNVBZR3JCYk0xYXUvNmVkMW1aM1F3UlhGOW0xSUZUY29q?=
 =?utf-8?B?THNzK3hVM1VndDhuU09zYy9jN0dGTVhOb21Ddm5qMHRSMkxJQUxyRlFKVW1B?=
 =?utf-8?B?cFFkL1VCK3owRXl6czRiK2U2Z1QwTmRVQStNdXJpeHlXK0JpK1ZWM0IvMVRC?=
 =?utf-8?B?QlJrYUY2enlJK2Y2YVBzcjE3UFFXK3JFbVVjWlh0VHdiV0Z6bmlic2VMNlJ6?=
 =?utf-8?B?UjFHem4zaXBXcHJCZytTZGRJd0x5VHBCWkdWaDRXMWYxVUM2eGxVUW13UHVI?=
 =?utf-8?B?VExVaHFrYU1wZE5UNUI2ckVySkhBZTVFOGVMWEJ2TW5GeGZ0Qk5TcFpaUHlk?=
 =?utf-8?B?MXc9PQ==?=
X-OriginatorOrg: wolfvision.net
X-MS-Exchange-CrossTenant-Network-Message-Id: 530d2136-080d-4735-aa1f-08dc533beb01
X-MS-Exchange-CrossTenant-AuthSource: VE1PR08MB4974.eurprd08.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Apr 2024 17:39:59.2426
 (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: C6JZKNhpev/TmRTZnkGYihb5gUrI1KdkvfJUxClwVQIL7k0jZqxR1U5cI7vIb6Mosq9ITGG5hMZgQAMeu8Yzpw6rqUhAqpBMmPxMVKERtrc=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PA6PR08MB10568

On 4/2/24 12:21, Heikki Krogerus wrote:
> On Thu, Mar 28, 2024 at 05:55:50PM +0100, Javier Carrasco wrote:
>> 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).
> 
> So this should be v3 (or v2?). Next time please follow the guide
> when submitting patches:
> https://www.kernel.org/doc/html/latest/process/submitting-patches.html#the-canonical-patch-format
> 
> thanks,
> 
The first resend only added 'stable' to Cc (which was wrong) with no
modifications at all, and the second resend (this one) actually modified
the commit description to include 'stable', which should have turned it
into a v2.

I will tag the next version as v3 to account for this, thanks for the
feedback.

Best regards,
Javier Carrasco