From patchwork Thu Dec 12 21:21:25 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Doug Anderson X-Patchwork-Id: 11289429 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 A628C14DB for ; Thu, 12 Dec 2019 21:21:53 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 8430E214AF for ; Thu, 12 Dec 2019 21:21:53 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b="XOSrWBeB" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731097AbfLLVVx (ORCPT ); Thu, 12 Dec 2019 16:21:53 -0500 Received: from mail-pf1-f195.google.com ([209.85.210.195]:43497 "EHLO mail-pf1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1731094AbfLLVVx (ORCPT ); Thu, 12 Dec 2019 16:21:53 -0500 Received: by mail-pf1-f195.google.com with SMTP id h14so18271pfe.10 for ; Thu, 12 Dec 2019 13:21:52 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=A3MllguTlm4RWEMVtAXxk8WCZHqQzGllKbJkD1LpkZo=; b=XOSrWBeBMymaJn4z1PR8VDb2JrXgYLfUBwv5WuGOHadZF8aLKPGK7s3i80NML9ei5o Wz+pX+KdMmyRfIOuG5pMCh3ms1vGqADyMXJweI9dlCYaufAZ4i3qR6Iy3JnnJ8bgvEGd TueSwhM5i2JoLKcIA8OmAjTC7bOIrDgKDzJB4= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=A3MllguTlm4RWEMVtAXxk8WCZHqQzGllKbJkD1LpkZo=; b=PsAe+os/+PESfYNbeTfqWNsND+UcOliXnHX1BaYHS2Ehq9qhDC0NhPtL9arkv6j7wb 1CfNhfEseT1U/afnBjvqNtGFpQcrjUOKjmmxTEwrryvfn/y7LrCVIBiF6GQ056zDafpn 8PYwSL00z9PgeAvm0QlIU3p4XmciGdfAI1D6iQT1GoMNIoeRJWk6kmQBsTHyhGMTib/k SDZlHRo5qilgwMPKaTCLI1C0+K2D/TlMKJRFyOiFRZk+UtLBwndtuSG6eshkwMtu7iUh 8B0dJcOmG8tRdOUTz2E75P3Fvd3ckmsCFlfHleRm0N3PLshkGRjwge3l4tXEjUwUl/Jh p7Yg== X-Gm-Message-State: APjAAAU1yduRyRcVF7sDjUyCYqyOsh2iqCH3RJGJr7HoU6Beek0As6vD cq/YJEYlXIzIcmEWYPE2NK+dbw== X-Google-Smtp-Source: APXvYqxCXvvMmFnm700MmbQmtWurHwBNW8hqqwHFReXZcDmV8WyGaUc29LSksY5z4mtikc0J/wRD1g== X-Received: by 2002:a63:504f:: with SMTP id q15mr12761862pgl.8.1576185712148; Thu, 12 Dec 2019 13:21:52 -0800 (PST) Received: from tictac2.mtv.corp.google.com ([2620:15c:202:1:24fa:e766:52c9:e3b2]) by smtp.gmail.com with ESMTPSA id y62sm8969595pfg.45.2019.12.12.13.21.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 12 Dec 2019 13:21:51 -0800 (PST) From: Douglas Anderson To: Felipe Balbi Cc: Bjorn Andersson , Manu Gautam , Andy Gross , Sandeep Maheswaram , Jeffrey Hugo , Douglas Anderson , linux-arm-msm@vger.kernel.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, Greg Kroah-Hartman Subject: [PATCH] usb: dwc3: qcom: Remove useless compatible strings from the match table Date: Thu, 12 Dec 2019 13:21:25 -0800 Message-Id: <20191212132122.1.I85a23bdcff04dbce48cc46ddb8f1ffe7a51015eb@changeid> X-Mailer: git-send-email 2.24.1.735.g03f4e72817-goog MIME-Version: 1.0 Sender: linux-arm-msm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org The of match table in dwc3-qcom.c had an entry per Qualcomm SoC known to have dwc3. That's not needed. Here's why: 1. The bindings specify that the compatible string in the device tree should contain the SoC-specific compatible string followed by the string "qcom,dwc3". 2. All known Qualcomm SoC dts files (confirmed via git grep in mainline Linux) using dwc3 follow the rules and do, in fact, contain the SoC-specific compatible string followed by the string "qcom,dwc3". 3. The source code does nothing special with the per-SoC strings--they are only used to match the node. Let's remove the extra strings from the table. Doing so will avoid the need to land future useless patches [1] that do nothing more than add yet more strings to the table. NOTE: if later we _do_ find some SoC-specific quirk we need to handle in the code we can add back a subset of these strings. At the time we will probably also add some data in the match table to make it easier to generalize this hypothetical quirk across all the SoCs it affects. [1] https://lore.kernel.org/r/1574940787-1004-2-git-send-email-sanm@codeaurora.org Signed-off-by: Douglas Anderson Reviewed-by: Bjorn Andersson Reviewed-by: Manu Gautam --- drivers/usb/dwc3/dwc3-qcom.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/usb/dwc3/dwc3-qcom.c b/drivers/usb/dwc3/dwc3-qcom.c index 261af9e38ddd..72e867e02a1c 100644 --- a/drivers/usb/dwc3/dwc3-qcom.c +++ b/drivers/usb/dwc3/dwc3-qcom.c @@ -751,9 +751,6 @@ static const struct dev_pm_ops dwc3_qcom_dev_pm_ops = { static const struct of_device_id dwc3_qcom_of_match[] = { { .compatible = "qcom,dwc3" }, - { .compatible = "qcom,msm8996-dwc3" }, - { .compatible = "qcom,msm8998-dwc3" }, - { .compatible = "qcom,sdm845-dwc3" }, { } }; MODULE_DEVICE_TABLE(of, dwc3_qcom_of_match);