Received: by 2002:ab2:b82:0:b0:1f3:401:3cfb with SMTP id 2csp521809lqh; Thu, 28 Mar 2024 08:32:37 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCVi8YZSlzfoxwYX/EiH+Kkvz82fxtrGXE2V63rVOaxFy+02acqfqcd+msUQt9mrclhyUQ2r71w0JcMSVBevsHSooLca2L7WQLPAe8jVdw== X-Google-Smtp-Source: AGHT+IGcFjdrVQAT3NmuS4V8rN7+NRQ0/9Xo8R6nF3BK4ZPbzRLUn4QvkzEHbbyhIS8p2PoZfdLi X-Received: by 2002:a05:620a:4588:b0:788:7160:931f with SMTP id bp8-20020a05620a458800b007887160931fmr3922214qkb.30.1711639957155; Thu, 28 Mar 2024 08:32:37 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1711639957; cv=pass; d=google.com; s=arc-20160816; b=oY5w9jItZ/MRZFcIGihJTirSuk0sbKv+cY1iDyBkeF5fC/hC8TMfzymOOs+q67XJxm L6kbP4w6d/bSgmUuQWkf3tRx3vypFQi+9XahT9ixfKhbW8t1iJS6E6eUTNgLIJ5prVTo StM/YCiuopyCiHj9vkSA6Tc36mnuOPdqGBHSgl0uOnuzlV3BVV+jZ+K5J7wKrHES6g+h NMnC0397d/PM+jMFyJN+zmWNAKL8SczdMBvPPk3PyNIzMNcSG/1j9XS3zMEIMUEPuNiP p4dAvHxW+DZX7ga9Fgm1FXp6b0wXW6r+6QdHJDvuOcvC1OqAV/tzdSzcX6K4+dX+Vhoo 3NzQ== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:date:message-id:dkim-signature; bh=lXM/B0D5ELcLLV0Ges/OZ3bWdZ/dJWeP2NzZMjp2T8A=; fh=Cs0IK7V3Od6RIuxMFNbSOOSvPpoelyw+cuGxc8kAzNY=; b=jicIuuEKlRBmcB5oUZh/d8KYLWRjje83aEApxZWqx0d6LpR8Alq5yA3z1Cr1fobqeZ Dvjp+GRITRG/wfgGgkb0W8psZO4kBC8SHdOIWib9ecv1eHRTbMAta2RLNUDh2dBtP9ny w6SxCTeIBF7QHI9RpRw9+pMLFxhnioM+EIi503Z8U+XcXpsEZBQPPpDp34ouyUFswzHf zTvk7q9pn16uQlpActDkbj/CBAh75rvhvdoUJ+yY7Vks/pXQL/1XXhbguMtotEr0xxbP eWnBOQgnESdYkjvRZ1ZDgsvdkMnjKssVIfUM/UuBuDWoZYiOx8AID1mK+fkdZdaz3fjO D30g==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=D76hPW6n; arc=pass (i=1 spf=pass spfdomain=linaro.org dkim=pass dkdomain=linaro.org dmarc=pass fromdomain=linaro.org); spf=pass (google.com: domain of linux-kernel+bounces-123154-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-123154-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [147.75.199.223]) by mx.google.com with ESMTPS id a11-20020a05620a066b00b0078a3a38a52csi1572975qkh.218.2024.03.28.08.32.36 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 28 Mar 2024 08:32:37 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-123154-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=D76hPW6n; arc=pass (i=1 spf=pass spfdomain=linaro.org dkim=pass dkdomain=linaro.org dmarc=pass fromdomain=linaro.org); spf=pass (google.com: domain of linux-kernel+bounces-123154-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-123154-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id C80751C2543F for ; Thu, 28 Mar 2024 15:32:36 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 57F8212F5B2; Thu, 28 Mar 2024 15:32:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="D76hPW6n" Received: from mail-lf1-f49.google.com (mail-lf1-f49.google.com [209.85.167.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id C0E6B12F384 for ; Thu, 28 Mar 2024 15:32:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1711639949; cv=none; b=GaXJ/fp5Nn210UBUquF/3YQM4rW/f7ET+ST5CMp5Am3+ijyN6IsRKsbImHtyxWVP3I6mj9VbNvd19d0N/ClOdMfckLJFMRDzXC2QDl42WU3pauRO0sHrjFywdKSxQebfjwZTXZj3cFOkhO5qk5KDjn2K4F6guUx2F72AP9sqjtU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1711639949; c=relaxed/simple; bh=MxJfELHb/ZTHQmCUFrKhSwgtmJ0UUkjaK/hhdG86Tbg=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=JeUKLkSlwB3GRFVABvYrNG/nzSbojsQ+pjcMa8awfPFn7Edrnl7EK04V3U6m1IZ/Man9dG/yzOpIdUNdQpYlT74Z7P0qAYfFjLtAMZoGfFiw8dLSvZjA5TAz0Ct+f+3qguD3rL/NXkg6d1crZrDL6d5IKIL2LghjYUvIe+xe9Zk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=D76hPW6n; arc=none smtp.client-ip=209.85.167.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Received: by mail-lf1-f49.google.com with SMTP id 2adb3069b0e04-513d599dbabso1395139e87.1 for ; Thu, 28 Mar 2024 08:32:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1711639946; x=1712244746; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=lXM/B0D5ELcLLV0Ges/OZ3bWdZ/dJWeP2NzZMjp2T8A=; b=D76hPW6nFroPOzm9ZqZZ2GI1tTIYRhUqFtFUxOjK/lXkgu9E7qw9G53VFEVNlFQM2u G8ccgsfV+c1eBXQc8BFT25jH5rdLoAnCGeUKYUiuXIFQap/BtVCcTOSCjlIrXtcm39Fm F9MtwxUZcYVB9Vaz1/NOy8V88+VoFPmoxz2BHgcjEU1+XtUP64w/OWO0ltzF6YyLCr1D +4GXqjR3PbfLX0Qqo7vuThOFaLrF7st8ZXXXpAIRFU70X4vmBSlPJjQ8GjKmUMrEg/1T U2vl03NUjq0BCQWPptP1Vz8kgPAB9Vd57YikDpYGrRcVs1IZ873Uu+08A9xW1Z2zXtdH yZvA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1711639946; x=1712244746; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=lXM/B0D5ELcLLV0Ges/OZ3bWdZ/dJWeP2NzZMjp2T8A=; b=WqevUATubVIa2TUCEKh9gn5DMegE2ffHD5oz11AVN09CFmCGITmzgazEQz6a0TBjyh v1FeGXPmIFfm9/bOu3fFqz7t7FcUl60RgfJZECtbyvVjhG9p8Ye5RoLpqj9YvaUN0jwg Cq99B0zRV8+ijAr//VPmPckRwlTHzBeQOgCzq7Dy1z5NSaOhWhuxX7/8WiSBG1pvbQRL YABn4SEsVA5GQHjr/FgqodZTJRxGj8afTB7BJ3gTxzcoHc4Hs0BpdFPi/FpC/eopUMx5 tWLMc77Utl6VFOQWlvUyYLYuidckkaYbvAt9LE482h0E65lpk0fDhUkz/hnvFrAAyXn2 4tFw== X-Gm-Message-State: AOJu0YzbzaHh8wbqcnS1Goziu/mRI4fHhmJEwI0H/EuL0Ds+tQOQubmd EdBeC0JUhmyS1UDDWJYXPwxBI1IBY91iJyMN2qCDVFUPk0Gtlow1VVo/XIGj/xE= X-Received: by 2002:a05:6512:534:b0:515:a876:68a0 with SMTP id o20-20020a056512053400b00515a87668a0mr2226473lfc.14.1711639945799; Thu, 28 Mar 2024 08:32:25 -0700 (PDT) Received: from ?IPV6:2001:14ba:a00e:a300::227? (dzyjmhyyyyyyyyyyyykxt-3.rev.dnainternet.fi. [2001:14ba:a00e:a300::227]) by smtp.gmail.com with ESMTPSA id j11-20020ac2550b000000b00515a6e4bdbdsm238086lfk.250.2024.03.28.08.32.25 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 28 Mar 2024 08:32:25 -0700 (PDT) Message-ID: <1b040be3-acd7-40dc-bc9b-24ea6d4b8377@linaro.org> Date: Thu, 28 Mar 2024 17:32:24 +0200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 3/3] platform/chrome: cros_ec_ucsi: Implement UCSI PDC driver To: Pavan Holla , Greg Kroah-Hartman , Heikki Krogerus , Benson Leung , Tzung-Bi Shih , Guenter Roeck Cc: linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org, Abhishek Pandit-Subedi , chrome-platform@lists.linux.dev References: <20240325-public-ucsi-h-v2-0-a6d716968bb1@chromium.org> <20240325-public-ucsi-h-v2-3-a6d716968bb1@chromium.org> Content-Language: en-GB From: Dmitry Baryshkov In-Reply-To: <20240325-public-ucsi-h-v2-3-a6d716968bb1@chromium.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 26/03/2024 01:42, Pavan Holla wrote: > Implementation of transport driver for UCSI. This driver will be used > if the ChromeOS EC implements a PPM. > > Signed-off-by: Pavan Holla > --- > drivers/platform/chrome/Kconfig | 14 ++ > drivers/platform/chrome/Makefile | 1 + > drivers/platform/chrome/cros_ec_ucsi.c | 247 +++++++++++++++++++++++++ > include/linux/platform_data/cros_ec_commands.h | 19 ++ While it's fine to use platform/chrome for platform drivers, please place drivers which have a subsystem into the subsystem dir. I think we don't want to hunt UCSI implementations all over the codebase. Please use drivers/usb/typec/ucsi/ location for your driver. This also removes a need for a global header. -- With best wishes Dmitry