Hi Heikki, On Thu, Aug 15, 2024 at 7:51 AM Heikki Krogerus <heikki.krogerus@xxxxxxxxxxxxxxx> wrote: > > On Thu, Aug 15, 2024 at 11:57:21AM +0300, Heikki Krogerus wrote: > > Hi, > > > > The two first patches prevent potential issues. The rest are just > > cleanups - removing duplicated or otherwise useless code. > > > > James, can you test these? I'm changing the handling of the identity > > discovery in patch 3/5. It would be great if you (or somebody else) > > could confirm that it continues to work as expected. > > > > thanks, > > > > Heikki Krogerus (5): > > usb: typec: ucsi: Don't truncate the reads > > usb: typec: ucsi: Only assign the identity structure if the PPM > > supports it > > usb: typec: ucsi: Common function for the GET_PD_MESSAGE command > > usb: typec: ucsi: Call CANCEL from single location > > usb: typec: ucsi: Remove useless error check from ucsi_read_error() > > > > drivers/usb/typec/ucsi/ucsi.c | 126 ++++++++++------------------------ > > drivers/usb/typec/ucsi/ucsi.h | 14 +--- > > 2 files changed, 39 insertions(+), 101 deletions(-) > > These are in wrong order. They break bisectability. I'll prepare v2. Can you please cc +Abhishek Pandit-Subedi <abhishekpandit@xxxxxxxxxx> and +Bartosz Szpila <bszpila@xxxxxxxxxx> on v2? They're also working on UCSI and can help too. Thanks, Benson -- Benson Leung Staff Software Engineer Chrome OS Kernel Google Inc. bleung@xxxxxxxxxx Chromium OS Project bleung@xxxxxxxxxxxx