From patchwork Fri Mar 17 07:16:29 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Manish Jaggi X-Patchwork-Id: 740183 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3vkxV93DZSz9ryT for ; Fri, 17 Mar 2017 18:16:49 +1100 (AEDT) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=CAVIUMNETWORKS.onmicrosoft.com header.i=@CAVIUMNETWORKS.onmicrosoft.com header.b="dgV8hgmd"; dkim-atps=neutral Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751021AbdCQHQn (ORCPT ); Fri, 17 Mar 2017 03:16:43 -0400 Received: from mail-bn3nam01on0049.outbound.protection.outlook.com ([104.47.33.49]:6512 "EHLO NAM01-BN3-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751011AbdCQHQm (ORCPT ); Fri, 17 Mar 2017 03:16:42 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=2HK+sF1xQZsRLj7+vSs3s7MMyPFDiZKATEvaCYDObaE=; b=dgV8hgmd50K5vCjvXbEdLaLNMaauAYvvNwZ6/aeexEsjMz54LPFJ8GLAfBm7cnwBWI/07EvFruI4yazu6d/MlUwuInN6d/EYP6wREno4SKTKVtIHEkaFHbZNLtHCACggjylNz6Ly15eMQNoj6eTlieYIixsBL77bMI3GaPZqWus= Authentication-Results: google.com; dkim=none (message not signed) header.d=none;google.com; dmarc=none action=none header.from=caviumnetworks.com; Received: from [10.160.0.34] (111.93.218.67) by BLUPR0701MB1586.namprd07.prod.outlook.com (10.163.84.148) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.947.12; Fri, 17 Mar 2017 07:16:36 +0000 To: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org From: Manish Jaggi Subject: [PATCH] pci: quirk cavium: Add device ID range check Cc: Alex Williamson , "Daney, David" , bhelgaas@google.com Message-ID: <43ff1f3f-b098-f25d-b9dc-9e8ef2278110@caviumnetworks.com> Date: Fri, 17 Mar 2017 12:46:29 +0530 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 X-Originating-IP: [111.93.218.67] X-ClientProxiedBy: MWHPR15CA0051.namprd15.prod.outlook.com (10.174.254.13) To BLUPR0701MB1586.namprd07.prod.outlook.com (10.163.84.148) X-MS-Office365-Filtering-Correlation-Id: 89cfd968-5b9f-4a90-1d16-08d46d058e03 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001); SRVR:BLUPR0701MB1586; X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1586; 3:JemZy7u5ih4IdZ/wG1cRas4WCahdPVuZ9BKnqAtwgp2AYAFyPTZAKKUfgoWDuNgsGRduCkrBTpLjfW2oI8EyVIka1MZiJLtsYUCvBbSJbNQlz0/XkC8F/ojG7CHv/sLlh+u14WCkJXlOAn+BObzR3xHFQAExWCPg1jy/IG+PozS2GGv4wWPoaZ4iTIXMLYpu89/B7SCF+VIrk4krOBVufmUNfeO9NyKsfdGV4m6SkgEJTQPk5iQ4i5ID7JzZ7F0t36XuPT7yk3KnhXUNu0llnw==; 25:kNiLo/PAGTTMK/ZGJYVWLljzWt5CcptRD9Ca2JKPtmHccClkxW5s9iL0PBu1AOeduHqFpLxpf2zJJidSINwjN6zwiZEhsvTYPxdQH/QHt8KsKuSb9rhCmRFmY7R1isYp7nzm6fm5hpBXMkc+RF8l+cjLii02eGNHDSB2dSUngJWLdHxpScxdJ7+Ux9/SMywPNO+aMyVbijMEBoJHZaN8LjumB/I0BtnnXhppx/zUOpA1y0+KE1a9Ay81vEPQZaQ81ra8OE6NcrNVmqNn1oaBLiQSM+LQS2A+VO4S/lbACtpCaQRTkr09WQpKJpklD/2Lk0AO5/fJxYJv6B6GDMo5f/e0+SeiaIaQOkLG1QZJwwRXcZL1WJSvEzwKHfRqse/GrtmfIYb4ZfxoHlDTymm3bIIdDeNqRFDhYqrYxkLwROaJRxBQqw3EcHFKAbK48nNi00CApRb0apsGoq9/yTn8fA== X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1586; 31:tEPWTo3v0CgZeK6goS8y9nUMdjXR9qpLWSPk7YtrhNrqI0oFgnmM3EX1vK9uBN56T04baQRFCtwisHtUf3MlkzUkPm2zOKH6+Y7vWfVF7OOEGkp6D03U5/vg89Uii4LaUJaAxx2Hjs2iQ/mPKti57EWaOOseLIy6Pt5or3Hb1xCfdb0U1gvJAh6VFO/XAGej4PNXCiLKOwwTo7eOoAQMePGLPc2YJy9o8vejWzSLY/z4P8JI+dJgvnNdGGHwK4MF; 20:0Qfr8ipuZCghigeI0Ja+Qn3xMd0JOSsLmecLMkfOaKx781PULnzkhTPwkdEQfCKNhreOY052vT8g6N53j4+ISVSqlHXnjR6FV5sbYaF+6A7MX2z0JMwSuyvOVEpAzriLWs2lkYHDXD+P4xX/KhQLc/SIiEJZjdtLrEUoPR6MN09YG9xwbf9WHZyngV7eynw9P/xUG6xwOTd+/ptXd+C/1aQwhpzXwsZyzgElbMcPc8NBjWvEtkQhGHbydtYcYIH+x9p8FxNwsg+WYQltCBEhSL911w67CEQdKSTj3FcLmxI6A5wb5zPzb+deyPvyTGaSssVpQchnJiuOwQVX7k8izUPWcP3vo8p0mDW3SJ5AgoCDv5j2JSJJGDVZaUrqe6BmwN+/LURjXSzDzWLhklMBrsMFnqV92dOm6EHa+2uqjwB8GjvTJev7ZsileAsfpZYepN0o8046sfYqkYrqhpQLNgv45541kitWknU3BDz+xaLPci0pitKVxnkpGA7DKi1gVBLBRV5s4SK0Tck317EKB9aeVQyd89L/49pdXQ07wA27/KBq3RmswRE/0myl9y1faUaAAIFxVBNkTp/zP0fHRvkl7W/4xUAChilHT2Ts6CM= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(6040375)(601004)(2401047)(5005006)(8121501046)(3002001)(10201501046)(6041248)(20161123560025)(20161123562025)(20161123558025)(20161123555025)(20161123564025)(6072148); SRVR:BLUPR0701MB1586; BCL:0; PCL:0; RULEID:; SRVR:BLUPR0701MB1586; X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1586; 4:v9XIGTwL2MrhR/zC+SUdkQ9r1UyU0oLcr7KrNKzcrC282O6WrJCHYJgj+L8tABw0SaFX8hGgTPevl81HhQHCjKg/HIFphdfS/JSn06XIZWivVbS/f2tFPPJI6gCTEx3okeloEu+d44wxdiBmNFQnjMp7zEZuwkUHrk4PrL1mHzsZzX/f3+x3mPIAoIuEsLumZ1qPKdkqs0oLQT0/496Ta4lnDJIKtwarTLvZ6HdVIxR+D9Oy5LAuUJDErpEOppd27BN5JLpRT55C87nJ8UFYswMBFCIs8nYn6GqmGcrbx7XWwQe2/eYzZKQOcLeABkfeBy148JQd/rMcerk7/rBF7lszo3CB7XxZDu/aZrmHsrPzbJ0oZ38GNRJKb+taxaV2/3Ebtw7tKWmQH19Ooy07QCXue2kmWaHyP8pOpUbMSCzPeT0dXnR8Ut2RU6c+ptD3DDBkVr5PTtJjZ65tS1XhWDoVOZDUcjc1rjgNURbSwwBNK1IhwCWp0GWRz/s7UNApBxGMT2XoebbXz16LbnJmnuh7ysg4SssWnvMtRwPH0kRhI282W7QBKObdjBqn7j6Yh8PDYecplGB1mnLIAt8b1fw4ad3e3Sb0Q6ZBN/e1mmk= X-Forefront-PRVS: 0249EFCB0B X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(4630300001)(6009001)(6049001)(39410400002)(39830400002)(39450400003)(189998001)(90366009)(65826007)(53936002)(77096006)(6486002)(5660300001)(575784001)(110136004)(25786008)(83506001)(305945005)(38730400002)(6666003)(42882006)(7736002)(54906002)(4001350100001)(42186005)(2906002)(54356999)(50986999)(4326008)(5009440100003)(33646002)(81166006)(8676002)(230700001)(36756003)(50466002)(47776003)(3846002)(23676002)(31696002)(6116002)(31686004)(66066001)(19627235001); DIR:OUT; SFP:1101; SCL:1; SRVR:BLUPR0701MB1586; H:[10.160.0.34]; FPR:; SPF:None; MLV:sfv; LANG:en; X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtCTFVQUjA3MDFNQjE1ODY7MjM6YUczcFlOZVJ6RHI0Wm1CZUlPZVAvYitr?= =?utf-8?B?eTEzdHBveHBwU2kwTmVzVngvZkw1U1liM3VpOUd2ZTd3c0UxRTgyUUtJRnBU?= =?utf-8?B?eUZPalIzL3plSEpLMXNoWFdNWUVZOE1GWXM4OHpiWTRwaTNic0dQRDJPZHZp?= =?utf-8?B?VHNMSjJTSGEyUG1nWjBMNitLbE1LcnN6T3ZsUlU0OEVwdVFGd2RuYU9FQVBQ?= =?utf-8?B?anZobTJOQWZBZmRpL1c3b1VjelhsVlJwcGlEcndGNUFMZWh3NVFScGZmbTRT?= =?utf-8?B?S3c0dHk5SzNVR3ArVlRZVDhCU2pOVnZaOHVWUmJLZjh2WkI0ZWVDQmwvNVEy?= =?utf-8?B?aldGMG1Ma254MndDTEQrODVJUkNNSktDcU1tNUhQVG95alRoSS9yeENGUFZr?= =?utf-8?B?ZlBiTktkY1c2UXc1Mk1RNnQ0Z3h6VGduKzlEQU1qWkFzK2VabmJFQ2RIZWIw?= =?utf-8?B?N2ExUWtUdkEzTmVzaklRcUN4aHAzQWVQVmQwUTVlL0dwdTZoUHRhMmpwMTJB?= =?utf-8?B?QS9xeWdtOEsraVo5RDVFQXVVM1h0QVBjOUlFMEpSdVkxYzVBdnB1Y2U0N3o1?= =?utf-8?B?OHkxTHd5cWRuZ1hIMEJjazVXOUtDdHlJQWJxbHlIZkpsQ2NFMDI4NUUyamd1?= =?utf-8?B?T0x3MEZ1clloUEp4TmZVRUhwSW5aYTlvUVhlNHV6bVBGS0NRVDlBVDRVV1Jp?= =?utf-8?B?Y2ZNcFJhQWFyUExlZmxGRDIyNVdURFpQUGNucDVyUWhISEFDQ3NZMEZDZ1l1?= =?utf-8?B?c3hMelNOS1ZzcnJRWG5nd0xiOXpwTVVOSU5FdzRGY0w0K0R1RnhlYnRUNXk1?= =?utf-8?B?WXZmVjVQNG93d3d0M0QyaW1zTGtaanczQ1VMMUJGa3BIOVZKL1RQdm5ZNVdE?= =?utf-8?B?bkxnMUwyajhXSW9JSk1LUkdPWEMxRWFMekZWVnZXZVc1czdacVlZTms2TDRx?= =?utf-8?B?L1BpMmZXK1pEYm1henArOWlDaDJnRHk2dDBQY1pobjFXOGhWS2p1VVh0aXZJ?= =?utf-8?B?ZXVFNUR3SU1NQnFYTTkyaldxdmdsZVIrbXdqcnNaL2lMbnM2QTRBWkFVTE1Z?= =?utf-8?B?MTNTb1ZHekVqdHNKbXk4WXJHdHZHNkpHd1N4RmtwVE0yMXBmK2JrOEtTbmVq?= =?utf-8?B?NlZIRmZ3eXJic1c5UkN6MVk3cGtUb0tJdXNRYUhzd05ib2JGM05qcVlsc3Jn?= =?utf-8?B?Q2c4TU1Dc1l4ZTN5dEU3Wm1yY3ZJZlhNaSt0Y3hRdE1reWhRdmRHa2FhMzBT?= =?utf-8?B?TXlpRW9GWDVjeFVsbnc1ZFBOSW1TZ0lUSC9RTytsKzh5ZXM5ZmNQQjJ3enJV?= =?utf-8?B?dmY1Nk5penlURUZzUzV0cjlHNXBoUFo0ZFlTODRCNDkxVVlaR05zZUxISXN1?= =?utf-8?B?RFJJeWl3V09nMWZqdmF6QmdEa3lCcW1qL3JncHQyZGhIR01YNEM1MnpuUWVV?= =?utf-8?B?NldPUlhTK3ZXTnM4M0NvemdhZjNKSnkxZEwzU0lJSEZzb05uVEFsVGFtdnhn?= =?utf-8?Q?C5CrlkMV5KLE0mfjiYZAAlVSQ1k=3D?= X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1586; 6:PIZSMcrnPtJE9JgcqhyE9+RPs2E55oPD88bG/lTWSW2i0T3gV9HkJOLqc8/BsmwaIswaIpU3ThyxGdBNIyFZYr8tL1ErExJk1PpidHo6RtZAhkSW9vYQ2Edow2+dOZ0o0wfrj5iKRp/TRDx/82Z1e7RJzI+eIMlLpWkfVh0auTLSSB+eQg9phdJ57c/4VLMv9n00W8TuRUMRx2/QoTKTHBAjv2Tz5I258jGvVRKv467y53u0XoJA0oDVtBj+986TWd5PTfpbiZ3V5RUoaB4qYUi4LJYKAWrUOklqH8UFbqLW8VwO7dLQmSl0zW4uwZ4lYeqC67pJtAkd/k16+7gmLDCHVW8iKL+nWd/5EHb0qyo+EkcYxD/2/xHgbs7CA7bsVu/AK1rY0JrmmzgHfgtuGw==; 5:YGo2R6sU3KpGhaKs0LLt3U1DVfWOv8uHpaV7bybenB6N6Sjj/fxGkhWDgQOI4AD001OCTU2ywKt62hlFE2QNP4oaS2dBbHpkZ07jFNkcbEI+gUKVmmpNReGe/zrTOVL14Zdp7KXyPz9kvxCwdlP5xw==; 24:MzIHTHSZlFXkRcHUK8R8H1uqddQ/nAA6CIDinWIdHFcPZyLii+4J6Iu54ICPiWS2rGhfPJRRcUJo1QRToAOVqDgvs6xQ+E9Qqg6VvSLM/Ik= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1586; 7:RtxBQ1cDnawGwBV0QukMndCtq6viulF14Ok54Me70HJ/OwaUbYz05ao24aL/0US9WtzFmKK+cKwx8sDLcCx4vqvbTEqeuP2fgPehuWBnPID3ZXMr4bMRkCL3id5gGzjfpMp262qwPUMnkJ3TR7I1HnGshHaX/0kldQaB6D041IQCLGUzf4OBq3ho73cjQORxj6iO1EtD9uK+N8sIVP+LGGYqDFqXSNDfTDNCgMF1wX35jGL2MID0yssQPhMLd7tyerjHNnVp7kmLHGFPy0MyNhy30f+04xFK5mg4ML0t5P6Ir6Zey41og5mJ63hIC8K9jEz9l4txJ6N8zHeq1rQSTA== X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Mar 2017 07:16:36.6070 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BLUPR0701MB1586 Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org ACS quirk applies to devices with ID in the range a000-a0XX. The patch adds a check.This matches on-chip pci devices for CN81xx/CN83xx/CN88xx This patch adds check to b404bcfbf0 : PCI: Add ACS quirk for all Cavium devices Signed-off-by: Manish Jaggi Acked-by: David Daney Reported-by: Alex Williamson Acked-by: Alex Williamson --- drivers/pci/quirks.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index f754453..178242d 100644 --- a/drivers/pci/quirks.c +++ b/drivers/pci/quirks.c @@ -4094,6 +4094,9 @@ static int pci_quirk_cavium_acs(struct pci_dev *dev, u16 acs_flags) acs_flags &= ~(PCI_ACS_SV | PCI_ACS_TB | PCI_ACS_RR | PCI_ACS_CR | PCI_ACS_UF | PCI_ACS_DT); + if (!((dev->device >= 0xa000) && (dev->device <= 0xa0ff))) + return -ENOTTY; + return acs_flags ? 0 : 1; }