From patchwork Sun Jan 7 00:16:59 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Christian A. Ehrhardt" X-Patchwork-Id: 185701 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7300:37c1:b0:101:2151:f287 with SMTP id y1csp335375dyq; Sat, 6 Jan 2024 16:26:24 -0800 (PST) X-Google-Smtp-Source: AGHT+IEpXmECTZ28+ekGkTk4Wwz4qQHKOEIrKBgOBXoSCjGMHwsdu/e7uqXZo7vpIZoiQjlvc+VD X-Received: by 2002:a05:6402:1ada:b0:557:1aa5:3b89 with SMTP id ba26-20020a0564021ada00b005571aa53b89mr900811edb.48.1704587184432; Sat, 06 Jan 2024 16:26:24 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1704587184; cv=none; d=google.com; s=arc-20160816; b=rYthJwEPuHXjIdYnWsCQq7NYEszbsFQHXgt12Bj18RggHq1MlKe/NkyQyCg53GVpGO 8kzTx5WRpORs2JPAlZz86eHkTWCkJSS2TLVL4E4a6skKOTtzGkjpXzkGSxzu3TyEQayv lBUI+a7USWD/IPVDqJCQCZYN534EPbZOiB+e46pG6Sy3buk9YE6UEiHURBDteY03SuAD 4yrrRatmPSWI24Pcfyb5IXJqt1nGvChSMukJ0U+H5CL+CavLIk3CEXOFbNSbaBr5BIdC UB1M0zyCeC13eZxC1ghdu3Qfevj3JhSEzmh5aoYDuCAiq6iwi0+EhKBiJVyhYmsR/4sj 7xGw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:in-reply-to:message-id :date:subject:cc:to:from; bh=Z3UoC9RP2K1SjNROulm2tbB2VjJNU4LKSyuRFz3/Ny0=; fh=PBT+T+L/Wk7fQyAuF/ivTwxWqTQzcP6BniPDJctp9Xs=; b=aMKN0TQF9dAkoZ7Yi6ieRKroGcpZpWlM3NYX/VzfxC74dkVsOuAtEYBWKfumzURyMJ YcPvtqxWx9jVI3RAD8QMftSeurmFV01F1UiNt9/e64EbK6Yf/kokYuU27BEEq3qSTQiB zktcTXLN1EfbOME16Y0+p4JQj62ZdjM4nIF9jdrD/Pceh5gq/Y0bRBWt1VjyePLtsdEY CgK1ghMJyHB/9ZDf4dCmaRFlWQYoQ4OEVyz9VdRc490gssthYijnGvvhFl1SDJvSVxfz lMiFhgE4xXGfityAFnerjU/UsxCZg0S48tjhzZCBIn6lA913blYqWNAwRWcQP35HOUNU 9u7w== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel+bounces-18744-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-18744-ouuuleilei=gmail.com@vger.kernel.org" Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [147.75.80.249]) by mx.google.com with ESMTPS id g17-20020a0564021ed100b0055703da55aasi1857574edg.225.2024.01.06.16.26.24 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 06 Jan 2024 16:26:24 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-18744-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel+bounces-18744-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-18744-ouuuleilei=gmail.com@vger.kernel.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 am.mirrors.kernel.org (Postfix) with ESMTPS id 054D81F22FE7 for ; Sun, 7 Jan 2024 00:18:47 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 4B8F07EC; Sun, 7 Jan 2024 00:17:55 +0000 (UTC) X-Original-To: linux-kernel@vger.kernel.org Received: from cae.in-ulm.de (cae.in-ulm.de [217.10.14.231]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 6875423B4; Sun, 7 Jan 2024 00:17:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=c--e.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=c--e.de Received: by cae.in-ulm.de (Postfix, from userid 1000) id 2388F140346; Sun, 7 Jan 2024 01:17:50 +0100 (CET) From: "Christian A. Ehrhardt" To: Heikki Krogerus , linux-usb@vger.kernel.org Cc: "Christian A. Ehrhardt" , Dell.Client.Kernel@dell.com, Greg Kroah-Hartman , Neil Armstrong , Hans de Goede , Jack Pham , Fabrice Gasnier , =?utf-8?q?Samuel_=C4=8Cavoj?= , linux-kernel@vger.kernel.org Subject: [PATCH 2/4] usb: ucsi: Add quirk infrastructure Date: Sun, 7 Jan 2024 01:16:59 +0100 Message-Id: <20240107001701.130535-3-lk@c--e.de> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20240107001701.130535-1-lk@c--e.de> References: <20240107001701.130535-1-lk@c--e.de> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1787389211532368431 X-GMAIL-MSGID: 1787389211532368431 Allow bus drivers to specify quirks for the UCSI core on attach. Allow the user to override the quirks on the command line. Signed-off-by: Christian A. Ehrhardt --- Documentation/admin-guide/kernel-parameters.txt | 5 +++++ drivers/usb/typec/ucsi/ucsi.c | 12 +++++++++++- drivers/usb/typec/ucsi/ucsi.h | 6 +++++- drivers/usb/typec/ucsi/ucsi_acpi.c | 2 +- drivers/usb/typec/ucsi/ucsi_ccg.c | 2 +- drivers/usb/typec/ucsi/ucsi_glink.c | 2 +- drivers/usb/typec/ucsi/ucsi_stm32g0.c | 2 +- 7 files changed, 25 insertions(+), 6 deletions(-) diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt index 0a6a4b7f7a3b..fd8152dd4450 100644 --- a/Documentation/admin-guide/kernel-parameters.txt +++ b/Documentation/admin-guide/kernel-parameters.txt @@ -6783,6 +6783,11 @@ ,,,,,,, See also Documentation/input/devices/joystick-parport.rst + typec_ucsi.quirks= [USB] + A hex value specifying the quirks to enable for + the USB Type-C connector system software interface + driver. This overrides auto detected quirks. + udbg-immortal [PPC] When debugging early kernel crashes that happen after console_init() and before a proper console driver takes over, this boot options might diff --git a/drivers/usb/typec/ucsi/ucsi.c b/drivers/usb/typec/ucsi/ucsi.c index 8f9dff993b3d..00b23292f46f 100644 --- a/drivers/usb/typec/ucsi/ucsi.c +++ b/drivers/usb/typec/ucsi/ucsi.c @@ -36,6 +36,11 @@ */ #define UCSI_SWAP_TIMEOUT_MS 5000 +/* Override for auto detected quirks. */ +static unsigned int quirk_override = ~0U; +module_param_named(quirks, quirk_override, uint, S_IRUGO); +MODULE_PARM_DESC(quirks, "Override quirks for UCSI"); + static int ucsi_acknowledge_command(struct ucsi *ucsi) { u64 ctrl; @@ -1507,7 +1512,8 @@ EXPORT_SYMBOL_GPL(ucsi_set_drvdata); * @dev: Device interface to the PPM (Platform Policy Manager) * @ops: I/O routines */ -struct ucsi *ucsi_create(struct device *dev, const struct ucsi_operations *ops) +struct ucsi *ucsi_create(struct device *dev, const struct ucsi_operations *ops, + unsigned int quirks) { struct ucsi *ucsi; @@ -1523,6 +1529,10 @@ struct ucsi *ucsi_create(struct device *dev, const struct ucsi_operations *ops) mutex_init(&ucsi->ppm_lock); ucsi->dev = dev; ucsi->ops = ops; + if (quirk_override != ~0U) + ucsi->quirks = quirk_override; + else + ucsi->quirks = quirks; return ucsi; } diff --git a/drivers/usb/typec/ucsi/ucsi.h b/drivers/usb/typec/ucsi/ucsi.h index 474315a72c77..5e6d2225c9c8 100644 --- a/drivers/usb/typec/ucsi/ucsi.h +++ b/drivers/usb/typec/ucsi/ucsi.h @@ -57,7 +57,8 @@ struct ucsi_operations { struct ucsi_altmode *updated); }; -struct ucsi *ucsi_create(struct device *dev, const struct ucsi_operations *ops); +struct ucsi *ucsi_create(struct device *dev, const struct ucsi_operations *ops, + unsigned int quirks); void ucsi_destroy(struct ucsi *ucsi); int ucsi_register(struct ucsi *ucsi); void ucsi_unregister(struct ucsi *ucsi); @@ -317,6 +318,9 @@ struct ucsi { #define EVENT_PENDING 0 #define COMMAND_PENDING 1 #define ACK_PENDING 2 + + /* Enabled quirks. */ + unsigned int quirks; }; #define UCSI_MAX_SVID 5 diff --git a/drivers/usb/typec/ucsi/ucsi_acpi.c b/drivers/usb/typec/ucsi/ucsi_acpi.c index 8062d0a4b523..78a0d13584ad 100644 --- a/drivers/usb/typec/ucsi/ucsi_acpi.c +++ b/drivers/usb/typec/ucsi/ucsi_acpi.c @@ -185,7 +185,7 @@ static int ucsi_acpi_probe(struct platform_device *pdev) if (dmi_check_system(zenbook_dmi_id)) ops = &ucsi_zenbook_ops; - ua->ucsi = ucsi_create(&pdev->dev, ops); + ua->ucsi = ucsi_create(&pdev->dev, ops, 0); if (IS_ERR(ua->ucsi)) return PTR_ERR(ua->ucsi); diff --git a/drivers/usb/typec/ucsi/ucsi_ccg.c b/drivers/usb/typec/ucsi/ucsi_ccg.c index 449c125f6f87..d491b6547fc3 100644 --- a/drivers/usb/typec/ucsi/ucsi_ccg.c +++ b/drivers/usb/typec/ucsi/ucsi_ccg.c @@ -1387,7 +1387,7 @@ static int ucsi_ccg_probe(struct i2c_client *client) if (uc->info.mode & CCG_DEVINFO_PDPORTS_MASK) uc->port_num++; - uc->ucsi = ucsi_create(dev, &ucsi_ccg_ops); + uc->ucsi = ucsi_create(dev, &ucsi_ccg_ops, 0); if (IS_ERR(uc->ucsi)) return PTR_ERR(uc->ucsi); diff --git a/drivers/usb/typec/ucsi/ucsi_glink.c b/drivers/usb/typec/ucsi/ucsi_glink.c index db6e248f8208..49587960b6a3 100644 --- a/drivers/usb/typec/ucsi/ucsi_glink.c +++ b/drivers/usb/typec/ucsi/ucsi_glink.c @@ -318,7 +318,7 @@ static int pmic_glink_ucsi_probe(struct auxiliary_device *adev, init_completion(&ucsi->sync_ack); mutex_init(&ucsi->lock); - ucsi->ucsi = ucsi_create(dev, &pmic_glink_ucsi_ops); + ucsi->ucsi = ucsi_create(dev, &pmic_glink_ucsi_ops, 0); if (IS_ERR(ucsi->ucsi)) return PTR_ERR(ucsi->ucsi); diff --git a/drivers/usb/typec/ucsi/ucsi_stm32g0.c b/drivers/usb/typec/ucsi/ucsi_stm32g0.c index 93d7806681cf..269caa3b4e84 100644 --- a/drivers/usb/typec/ucsi/ucsi_stm32g0.c +++ b/drivers/usb/typec/ucsi/ucsi_stm32g0.c @@ -641,7 +641,7 @@ static int ucsi_stm32g0_probe(struct i2c_client *client) init_completion(&g0->complete); i2c_set_clientdata(client, g0); - g0->ucsi = ucsi_create(dev, &ucsi_stm32g0_ops); + g0->ucsi = ucsi_create(dev, &ucsi_stm32g0_ops, 0); if (IS_ERR(g0->ucsi)) return PTR_ERR(g0->ucsi);