Re: [PATCH v2] usb: typec: ucsi: Stop a power level change notification storm

[sashiko-bot@kernel.org]

> From: David Vornholt <david@vornholt.online>
> 
> On a Lenovo ThinkPad X1 Carbon Gen 13 (BIOS N4BET78W 1.48, EC N4BHT61W
> 1.41, UCSI 1.0) with an HP E273m monitor connected, the PPM keeps
> signaling a Negotiated Power Level Change although the power contract
> stays the same, and signals it again right after it has been
> acknowledged:
> 
>   ucsi_connector_change: port1 status: change=0040, opmode=3,
>     connected=1, sourcing=0, partner_flags=2, partner_type=4,
>     request_data_obj=1304b12c, BC status=2
> 
> This repeats about seven times a second for as long as the monitor stays
> connected. Since commit a0d4618788f2 ("usb: typec: ucsi: Workaround for
> missed op_mode change"), every Negotiated Power Level Change is handled
> [ ... ]
> The ports, the partner and its 5 V/3 A power supply stay registered.
> 
> Fixes: c1b0bc2dabfa ("usb: typec: Add support for UCSI interface")
> Cc: stable@vger.kernel.org
> Signed-off-by: David Vornholt <david@vornholt.online>

Sashiko has reviewed this patch and found no issues. It looks great!

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261010183917.146821-1-dv@etik.com?part=1