From patchwork Tue Aug 25 04:22:07 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Badhri Jagan Sridharan X-Patchwork-Id: 11734839 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 529FE138A for ; Tue, 25 Aug 2020 04:23:37 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 39C9A20737 for ; Tue, 25 Aug 2020 04:23:37 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=google.com header.i=@google.com header.b="YqtZ49pJ" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728745AbgHYEX2 (ORCPT ); Tue, 25 Aug 2020 00:23:28 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50922 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728653AbgHYEWy (ORCPT ); Tue, 25 Aug 2020 00:22:54 -0400 Received: from mail-qv1-xf4a.google.com (mail-qv1-xf4a.google.com [IPv6:2607:f8b0:4864:20::f4a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 681F9C061755 for ; Mon, 24 Aug 2020 21:22:54 -0700 (PDT) Received: by mail-qv1-xf4a.google.com with SMTP id h6so7896208qvz.14 for ; Mon, 24 Aug 2020 21:22:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=sender:date:in-reply-to:message-id:mime-version:references:subject :from:to:cc; bh=Nv5tc6KahqHd2NBKcfPXCDAaClr/I8kOXQmUHCJ9qWQ=; b=YqtZ49pJFfw//Tms6srxB9cmG6SIMAsPfgiC2cbBRjzdOKnk3y7KFv8E5qzMu+TQza aMQp2eGCeAlq0EKOoTbiQZoj6jnqDkiErdDtaXvfFO2DAXy2N6MZW2xx0uMV8zgo/+hA KWUzDsx+qQpyUDgyuEQwYOx+rF/BDpRhb+NSSMhCu1r0f/erlqTu2+legGwWHWUsKrzy VQGnzfaBKfu+c717oQoHVwwPSRyOU9Q9cLiT5KkmVKWDcej3+suM5Ky9F784FXv2gRxY X7AIZtPk2y/kzUtNpcaCJ3PAqu3L7+rIsEfA3+N2BzABpSNa/gYL6b6P4Zv55ZyliTI1 TffQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:date:in-reply-to:message-id:mime-version :references:subject:from:to:cc; bh=Nv5tc6KahqHd2NBKcfPXCDAaClr/I8kOXQmUHCJ9qWQ=; b=VbMogXdL/IyZu8jpP3PXeU6TqraAvRaSPq9wBZw4EU0wPAQG/HOxzoLGOIQ9AbAnqO k3Gz4KRAfPDp8KC1UArQnGmENe7L61jw3M6zdLhRxczXEYsYFKw+ofBx5snJ+BkYQMtG /Bwwd9cDXwQ0c/uOYe8ePqEH3/+WZEzB7Lsx7qQbIu/kaSg2CC/ec9K2hoZ0h0pD+CLK hKOk5qi0eGQRwWl35Ez+kQMWj/ZcKyDaZ8fZ3rK6prFJRASrgsXSP3RqmPmp3Nw6bCfh mCUd1bJt6LSayla11HjU0y7SqZVME4n8Iyqnv/1xjs8aROPyG9JgLMB6+qmdroslH3yd 9pUg== X-Gm-Message-State: AOAM5333+i73bak6539f5YvCAgxLlF+A8+7NlIfs7IihgZVdZYbKUaTO ywPVDF3UwMqzwVjplUZUeK/m7JHKL0g= X-Google-Smtp-Source: ABdhPJyxdJTRBEww26jqyEBYkoSZj8teumSAJ0bi3oYNf5zYDACJTuOdVs9cm1hocyvPsrPZUaRz/v9WtsE= X-Received: from badhri.mtv.corp.google.com ([2620:15c:211:1:f292:1cff:fee0:66cf]) (user=badhri job=sendgmr) by 2002:a0c:f607:: with SMTP id r7mr8103797qvm.219.1598329373011; Mon, 24 Aug 2020 21:22:53 -0700 (PDT) Date: Mon, 24 Aug 2020 21:22:07 -0700 In-Reply-To: <20200825042210.300632-1-badhri@google.com> Message-Id: <20200825042210.300632-12-badhri@google.com> Mime-Version: 1.0 References: <20200825042210.300632-1-badhri@google.com> X-Mailer: git-send-email 2.28.0.297.g1956fa8f8d-goog Subject: [PATCH 11/14 v1] usb: typec: tcpci_maxim: Add support for Sink FRS From: Badhri Jagan Sridharan To: Guenter Roeck , Heikki Krogerus , Greg Kroah-Hartman Cc: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, Badhri Jagan Sridharan Sender: linux-usb-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-usb@vger.kernel.org Upon receiving ALERT_EXTENDED.TCPC_SINK_FAST_ROLE_SWAP signal tcpm to start Sink fast role swap signal. Inform when TCPM is sourcing vbus. Signed-off-by: Badhri Jagan Sridharan --- drivers/usb/typec/tcpm/tcpci_maxim.c | 50 +++++++++++++++++++++++++--- 1 file changed, 46 insertions(+), 4 deletions(-) diff --git a/drivers/usb/typec/tcpm/tcpci_maxim.c b/drivers/usb/typec/tcpm/tcpci_maxim.c index b61f290a8f96..6ba808ad901a 100644 --- a/drivers/usb/typec/tcpm/tcpci_maxim.c +++ b/drivers/usb/typec/tcpm/tcpci_maxim.c @@ -106,13 +106,22 @@ static void max_tcpci_init_regs(struct max_tcpci_chip *chip) return; } + ret = max_tcpci_write8(chip, TCPC_ALERT_EXTENDED, 0xff); + if (ret < 0) { + dev_err(chip->dev, "Unable to clear TCPC_ALERT_EXTENDED ret:%d\n", ret); + return; + } + alert_mask = TCPC_ALERT_TX_SUCCESS | TCPC_ALERT_TX_DISCARDED | TCPC_ALERT_TX_FAILED | TCPC_ALERT_RX_HARD_RST | TCPC_ALERT_RX_STATUS | TCPC_ALERT_CC_STATUS | - TCPC_ALERT_VBUS_DISCNCT | TCPC_ALERT_RX_BUF_OVF | TCPC_ALERT_POWER_STATUS; + TCPC_ALERT_VBUS_DISCNCT | TCPC_ALERT_RX_BUF_OVF | TCPC_ALERT_POWER_STATUS | + /* Enable Extended alert for detecting Fast Role Swap Signal */ + TCPC_ALERT_EXTND; ret = max_tcpci_write16(chip, TCPC_ALERT_MASK, alert_mask); if (ret < 0) { - dev_err(chip->dev, "Error writing to TCPC_ALERT_MASK ret:%d\n", ret); + dev_err(chip->dev, + "Error enabling TCPC_ALERT: TCPC_ALERT_MASK write failed ret:%d\n", ret); return; } @@ -122,6 +131,10 @@ static void max_tcpci_init_regs(struct max_tcpci_chip *chip) dev_err(chip->dev, "Error writing to TCPC_POWER_CTRL ret:%d\n", ret); return; } + + ret = max_tcpci_write8(chip, TCPC_ALERT_EXTENDED_MASK, TCPC_SINK_FAST_ROLE_SWAP); + if (ret < 0) + return; } static void process_rx(struct max_tcpci_chip *chip, u16 status) @@ -225,10 +238,23 @@ static void process_power_status(struct max_tcpci_chip *chip) if (ret < 0) return; - if (pwr_status == 0xff) + if (pwr_status == 0xff) { max_tcpci_init_regs(chip); - else + } else if (pwr_status & TCPC_POWER_STATUS_SOURCING_VBUS) { + tcpm_sourcing_vbus(chip->port); + /* + * Alawys re-enable boost here. + * In normal case, when say an headset is attached, TCPM would + * have instructed to TCPC to enable boost, so the call is a + * no-op. + * But for Fast Role Swap case, Boost turns on autonomously without + * AP intervention, but, needs AP to enable source mode explicitly + * for AP to regain control. + */ + max_tcpci_set_vbus(chip->tcpci, &chip->data, true, false); + } else { tcpm_vbus_change(chip->port); + } } static void process_tx(struct max_tcpci_chip *chip, u16 status) @@ -249,6 +275,7 @@ static irqreturn_t _max_tcpci_irq(struct max_tcpci_chip *chip, u16 status) { u16 mask; int ret; + u8 reg_status; /* * Clear alert status for everything except RX_STATUS, which shouldn't @@ -274,6 +301,21 @@ static irqreturn_t _max_tcpci_irq(struct max_tcpci_chip *chip, u16 status) } } + if (status & TCPC_ALERT_EXTND) { + ret = max_tcpci_read8(chip, TCPC_ALERT_EXTENDED, ®_status); + if (ret < 0) + return ret; + + ret = max_tcpci_write8(chip, TCPC_ALERT_EXTENDED, reg_status); + if (ret < 0) + return ret; + + if (reg_status & TCPC_SINK_FAST_ROLE_SWAP) { + dev_info(chip->dev, "FRS Signal"); + tcpm_sink_frs(chip->port); + } + } + if (status & TCPC_ALERT_RX_STATUS) process_rx(chip, status);