Received-SPF: pass (google.com: domain of linux-kernel+bounces-67708-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Precedence: bulk
MIME-Version: 1.0
References: <20240209223824.622869-1-abhishekpandit@chromium.org>
In-Reply-To: <20240209223824.622869-1-abhishekpandit@chromium.org>
From: Abhishek Pandit-Subedi <abhishekpandit@chromium.org>
Date: Thu, 15 Feb 2024 13:40:26 -0800
Message-ID: <CANFp7mWoUFZ_J-yYqCj9AE_EymQBR_3czFHxLKvZ7ZmBJe-u7Q@mail.gmail.com>
Subject: Re: [PATCH v5 0/3] usb: typec: ucsi: Adding support for UCSI 3.0
To: Heikki Krogerus <heikki.krogerus@linux.intel.com>, linux-usb@vger.kernel.org, 
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: pmalani@chromium.org, jthies@google.com, 
	"Christian A. Ehrhardt" <lk@c--e.de>, Dmitry Baryshkov <dmitry.baryshkov@linaro.org>, 
	Fabrice Gasnier <fabrice.gasnier@foss.st.com>, Hans de Goede <hdegoede@redhat.com>, 
	Neil Armstrong <neil.armstrong@linaro.org>, Rajaram Regupathy <rajaram.regupathy@intel.com>, 
	Saranya Gopal <saranya.gopal@intel.com>, linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Fri, Feb 9, 2024 at 2:38=E2=80=AFPM Abhishek Pandit-Subedi
<abhishekpandit@chromium.org> wrote:
>
>
> Hi Heikki,
>
> This series starts the work adding UCSI 3.0 support to the UCSI driver.
>
> There's a couple of pieces to start here:
> * Add version checks and limit read size on 1.2.
> * Update Connector Status and Connector Capability structures.
> * Expose Partner PD revision from Capability data.
>
> These were tested against on a 6.6 kernel running a usermode PPM against
> a Realtek Evaluation board.
>
> One additional note: there are a lot more unaligned fields in UCSI now
> and the struct definitions are getting a bit out of hand. We can discuss
> alternate mechanisms for defining these structs in the patch that
> changes these structures.
>
> Thanks,
> Abhishek
>
> Changes in v5:
>   - Change min_t to clamp
>
> Changes in v4:
>   - Added missing Tested-By tags from v1 and reviewed-by tags.
>   - Fix BCD translation of PD Major Rev
>   - Replace IS_MIN_VERSION macro and just compare version directly.
>
> Changes in v3:
>   - Change include to asm/unaligned.h and reorder include.
>
> Changes in v2:
>   - Changed log message to DEBUG
>   - Formatting changes and update macro to use brackets.
>   - Fix incorrect guard condition when checking connector capability.
>
> Abhishek Pandit-Subedi (3):
>   usb: typec: ucsi: Limit read size on v1.2
>   usb: typec: ucsi: Update connector cap and status
>   usb: typec: ucsi: Get PD revision for partner
>
>  drivers/usb/typec/ucsi/ucsi.c | 49 +++++++++++++++++++++++--
>  drivers/usb/typec/ucsi/ucsi.h | 67 ++++++++++++++++++++++++++++++++---
>  2 files changed, 110 insertions(+), 6 deletions(-)
>
> --
> 2.43.0.687.g38aa6559b0-goog
>

Hi Greg,

This series should be ready to merge. Please take a look.

Thanks,
Abhishek