From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id F0533C3064D for ; Wed, 26 Jun 2024 14:24:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=7blsmaiSEE93ERiXdkPkH75OKRurhykG5nIryfRiw5o=; b=Ym5zqXrtbqAUrPHUg20VlRL/6G CGCjBEZWdcgYEtuqz4M3Z2ZVEAGqB+Yrq8PrH/LBdHpEc11CoYWx5tTvB4vv9Aq9tYvIuEq3lBNAW uTXm9Q/bS0uqXcG8mZfOF8Xn0IzegFKqGAd/F4CP1AZW7XSA6MPSLLGUjnMEyrYq+OPJykZFprx8f QZ/VR6SJxs1wjas3L6GVP68oCRM1Y0XX6aAT+qXx/HdAoXs7uZUvEy3og38yuT420O5I1v2GOX98i Cc9tWIkMVaX4ptV9bwTUXxOpQUXc+SNCpLa/NaOWwzxeny6BUY+qiCR/+R7JKnpqSbY1XugW9Iofz r6KZ3Wsw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sMTZb-000000079Cb-0t4C; Wed, 26 Jun 2024 14:24:35 +0000 Received: from mgamail.intel.com ([192.198.163.15]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sMTZT-000000079C2-2GZL for linux-arm-kernel@lists.infradead.org; Wed, 26 Jun 2024 14:24:29 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1719411867; x=1750947867; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=CuHPqyAPRoQfZVkmRu3Q18KTjaKXZmJG02/XUes9hgQ=; b=HB4IFK1fabZAkL39ZItTBgQcTZEJkseRy5shpnmV0WotwSs0HuuyoO/o rTdDwdVkC8g47Ny8QC9VpmeWrDs8T8Up/47Vkblq2o0RlLk9mZB+5P0IN qlqEx9Q/v6D+mharkAlQCNHUMV8oyc64cTI2CY6Ni7KWrSBERzJfOFvX9 0YuAguYUaDdwfKHpd2mrmK3nVsBTHGIkEytTCy05xzCgS9VMnZ3OVaVhT gljY7kB3RhSrn5cJPjswAax9TYgH08/uX/GcxBuoQaPFdKR6BO+IJOlrL x0iwMUpHP3clm7aQMtYOgyjUD0Mx5tGSxmYcd4m07YIXtAz+kZCXysDYw w==; X-CSE-ConnectionGUID: L3uV3R38SXynySHXg3lm2A== X-CSE-MsgGUID: 4snMTJTJQIOXKX73UkoIpg== X-IronPort-AV: E=McAfee;i="6700,10204,11115"; a="16634422" X-IronPort-AV: E=Sophos;i="6.08,267,1712646000"; d="scan'208";a="16634422" Received: from orviesa003.jf.intel.com ([10.64.159.143]) by fmvoesa109.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Jun 2024 07:24:26 -0700 X-CSE-ConnectionGUID: YVxzdlZZTNCOOsIetqYlkQ== X-CSE-MsgGUID: wXROosdSRtqH8MtrKN2U2g== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.08,267,1712646000"; d="scan'208";a="48943302" Received: from kuha.fi.intel.com ([10.237.72.185]) by orviesa003.jf.intel.com with SMTP; 26 Jun 2024 07:24:23 -0700 Received: by kuha.fi.intel.com (sSMTP sendmail emulation); Wed, 26 Jun 2024 17:24:22 +0300 Date: Wed, 26 Jun 2024 17:24:22 +0300 From: Heikki Krogerus To: Dmitry Baryshkov Cc: Greg Kroah-Hartman , Maxime Coquelin , Alexandre Torgue , Nikita Travkin , Neil Armstrong , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v3 0/7] usb: typec: ucsi: rework glue driver interface Message-ID: References: <20240625-ucsi-rework-interface-v3-0-7a6c8e17be3a@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240625-ucsi-rework-interface-v3-0-7a6c8e17be3a@linaro.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240626_072427_624177_7DF899E7 X-CRM114-Status: GOOD ( 17.95 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Jun 25, 2024 at 05:54:25PM +0300, Dmitry Baryshkov wrote: > The interface between UCSI and the glue driver is very low-level. It > allows reading the UCSI data from any offset (but in reality the UCSI > driver reads only VERSION, CCI an MESSAGE_IN data). All event handling > is to be done by the glue driver (which already resulted in several > similar-but-slightly different implementations). It leaves no place to > optimize the write-read-read sequence for the command execution (which > might be beneficial for some of the drivers), etc. > > The patchseries attempts to restructure the UCSI glue driver interface > in order to provide sensible operations instead of a low-level read / > write calls. > > If this approach is found to be acceptable, I plan to further rework the > command interface, moving reading CCI and MESSAGE_IN to the common > control code, which should simplify driver's implementation and remove > necessity to split quirks between sync_control and read_message_in e.g. > as implemented in the ucsi_ccg.c. > > Note, the series was tested only on the ucsi_glink platforms. Further > testing is appreciated. I tested these on couple of systems that use the acpi mailbox, and didn't see any problems. I'll be away for most of July, so if there's nothing else, for the series: Tested-by: Heikki Krogerus Reviewed-by: Heikki Krogerus thanks, -- heikki