From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751915AbbIOSWm (ORCPT ); Tue, 15 Sep 2015 14:22:42 -0400 Received: from mail-bn1on0085.outbound.protection.outlook.com ([157.56.110.85]:18841 "EHLO na01-bn1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751532AbbIOSWk (ORCPT ); Tue, 15 Sep 2015 14:22:40 -0400 X-Greylist: delayed 933 seconds by postgrey-1.27 at vger.kernel.org; Tue, 15 Sep 2015 14:22:40 EDT Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=David.Daney@caviumnetworks.com; Message-ID: <55F85E43.5070207@caviumnetworks.com> Date: Tue, 15 Sep 2015 11:06:59 -0700 From: David Daney User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130625 Thunderbird/17.0.7 MIME-Version: 1.0 To: David Daney CC: , , Rob Herring , Frank Rowand , Grant Likely , Bjorn Helgaas , , Will Deacon , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , , David Daney Subject: Re: [PATCH 0/6] PCI: generic: Misc. bug fixes and enhanced support for MSI. References: <1442013719-5001-1-git-send-email-ddaney.cavm@gmail.com> In-Reply-To: <1442013719-5001-1-git-send-email-ddaney.cavm@gmail.com> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [64.2.3.194] X-ClientProxiedBy: SN1PR07CA0041.namprd07.prod.outlook.com (25.162.170.179) To BN3PR0701MB1719.namprd07.prod.outlook.com (25.163.39.18) X-Microsoft-Exchange-Diagnostics: 1;BN3PR0701MB1719;2:PasrME9QmLHzEhrUMg0T0r9p2C4NK4owrO4nkm1mGimY87ok/K36zITvOtJBIdwVTqMLn7Kpadw4hzFq8l5y+JyJMrjxen5EYB0O+6nmb2kPcNnBHMNYcM0TExLKFGg0c/LrVty9g6NRNtLH7qPoyhrOk5o2Hn0FQWzwObcwRz8=;3:20j7QNS8CWXhnJSxU8tS7saCoUGlasgEsP4+E7nPyHco/6hrurazCYtAGahs9MV5CZsh7cJgKn5QXCnrAmtLSGna5W617HiqI6MjkoV/O5nWpekSuj0wRZlGvVkd2ot07ITL9LkKpSOemidk11ocxg==;25:H7Uwp5j+G+/DWHF7GV4FCgsH+FLiugdNvY4UtDBjynr9+89i5bVpmb107rDDPyU5+wnGLmLsuXizxBrzzUiPjp8229NW75iAUW67X/vyyILipVu9ATOKFfeZxv9gEFLkBGRFFVl5jrjhBiYI/UBLejg8R/po8jolp5J/dSrDZYOPexsVB2y8GcUmvfbWMjKFXHTmOir1v8sE6VFwtOTcZFV/VM9NqDNJuz8z9xDfdUce5LHRR2Z1nfpU5Ao8Loa7qGR6N8zCIlngGCnL6BkBkg== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BN3PR0701MB1719; X-Microsoft-Exchange-Diagnostics: 1;BN3PR0701MB1719;20:eeQamHVfOWjfmMSJQudOOovw5VJrqaLUQwHPp5aW7M3wfGcu+b8bH6/AuWO2CGIZIkoO/Nf7k01tju5InWtMpr6lXA3wRhLg1Z8iHungEkqXqKhjQvbxoHQCinnv2gAGJk2StXvGI4shBtB6VQN5jLuH3bWyOD1BepVq0Jf6I209wcHD52ooHKwpmUI4/Ip2Y1vEtxi30j6cydolbqjEjr1aCCLYn6dtTcIODxS3Z3SCyyFYLkZZeaJXu+NGdH/pOpid8lrLnLbKBdnkc+Kr2B14Cf2ce5ZKLWqrjvlZxolHGGyRfWvHo+dGLne8IgSxSvugRwF9ac/pIg8MBOTZgwt5F6doo/b2Y4TO0Ew5+A1+OWv65VoWPceyNxrSAd8zX0YYf+D1nIMGs4mz2Y7WMWUsrd2qSQqymJpOT+RwUHPuWbpNtYLhrH+gVJCZKJfdbBfCXvWFbvFDDvfhr25Wxpgvdg1O8KjcWwsKqRPBUOC0xDLNyH+qIWR7RY9caYyDUKa16fj4+TWONKZvuOY5x0BsT3R90VHSTMg0SHdLLnEJ/V0dNs1kNa5xBbc3F8cgJ43IHD9lgNAy3ZIjkAu3h1LgWoKlPC9u4UwOnEALegY= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(8121501046)(5005006)(520075)(520078)(3002001);SRVR:BN3PR0701MB1719;BCL:0;PCL:0;RULEID:;SRVR:BN3PR0701MB1719; X-Microsoft-Exchange-Diagnostics: 1;BN3PR0701MB1719;4:BtS7OccsiATD1D7bX+mwKXaAt+7YavMD58b/2qknz5ReYkEFTdL0duL2ZogAo79+/y8Q43q4tyb5JZ8EIm2QwXv7L/juehR726Y3PhOiADhUGMY3HqBjkIJG0n78bGXHosCuB6Q84I/tFP/4mN9KXpA3JuJF/z7RR6M/OgoCv8uhqCLsfBJlIL5rP7CjGDIAsO2XrD1QrAnMjZKK3QRRrrA7q7VMvXuB9cA4672QAHriucKcu0cCrF+zvUZ0SZaAWLZqelitjDhBYgg1ch5+SobBDN7wdK+F0itp7nSBr/TNv8C2fc2e1NXdHFsmNvA5M5nBN3/bNicy39fLxrO4sVEKvtWp2ls0Vod378BlY9U= X-Forefront-PRVS: 070092A9D3 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(6009001)(377454003)(479174004)(189002)(24454002)(199003)(46102003)(189998001)(5001860100001)(62966003)(81156007)(77156002)(110136002)(68736005)(5001960100002)(2950100001)(53416004)(122386002)(92566002)(5004730100002)(5001830100001)(105586002)(42186005)(77096005)(5007970100001)(97736004)(4001540100001)(4001350100001)(106356001)(40100003)(76176999)(23756003)(54356999)(33656002)(19580395003)(87266999)(47776003)(101416001)(50466002)(50986999)(80316001)(65816999)(64706001)(36756003)(83506001)(59896002)(65956001)(66066001)(87976001)(69596002)(64126003)(19580405001)(65806001)(41533002);DIR:OUT;SFP:1101;SCL:1;SRVR:BN3PR0701MB1719;H:dl.caveonetworks.com;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?iso-8859-1?Q?1;BN3PR0701MB1719;23:dag7dI0VKcJx5Yi58qfTq5zoxzRwJ68a0zpeY?= =?iso-8859-1?Q?PqXI1BlR2W5kqsES+ObTX8/pHh8kgzFIfLlcV42Vv+/drR2537umcxQyO6?= =?iso-8859-1?Q?Swt3cB3VzVoh+aw/Wk1cNEyR/7By6NwH+U6CJN+D6RezWzEHd0OAQXov+U?= =?iso-8859-1?Q?vuWRTZPkDExbJiOGRab4IwwlCxmnFgrfEuqpFH+79eu/p85EXuAoOXybh9?= =?iso-8859-1?Q?tyBP1HarwWYw8+mO+43BEbJQZ2lDAij+3zrJK6gfLNwbZxe+XAXtPyu0JA?= =?iso-8859-1?Q?Od/8dLd42i5I9o97Oj/muON2a5vqr5OW3Q6mIHn6XdwpfRZfVcfHtAuCL9?= =?iso-8859-1?Q?BGLHaUshkyATmbpOEvkWxKXy/btXCbSZGxGs5ri7Jips1l9zPR5Tl9wUD1?= =?iso-8859-1?Q?2AihR4AhvpluTxK/KBM5/9LXa7WRJRIBlFp/WC/ifArOmQVwTu6RXFSwcW?= =?iso-8859-1?Q?Z3cNOQcAY43jJgqP8cxbzh16AIVqQIl39PkxrGXW8MCbMXT/jK6Q04k+E3?= =?iso-8859-1?Q?n20BWm7laaseIDxUwvUvgZAPLHzuPIpBN3oSJ4TPOLGN66xe6g2D5NaJAK?= =?iso-8859-1?Q?wYp2jtYSENv5UJo7Lqs6O5ZOrJErZC3oiy/rqc/yAUnE7Q+NlFfqIvbX7E?= =?iso-8859-1?Q?c/ZO9dGmQmU+aH5dM89HOYu/XTP7moPbtaCqoit4HXWCXLRyteAE9lHlvM?= =?iso-8859-1?Q?2ZLfhfvMVaB0RuexE+DtITBvDDz3VECQx8TilBh/FPsPJXipiCefl2EXFr?= =?iso-8859-1?Q?yI9aW74hRTHqPhl5hK8P5V3fl2ty1S48MBQJKVip8sAzypEoVGYcluYMA4?= =?iso-8859-1?Q?+4jEzkgYTzvk6eAxZXC20VLQs+6yUlQjUYKpGfHl+3OabWVjM0coUOnJDF?= =?iso-8859-1?Q?EMzpFT9skjbzr+tcAnqAIetlMaCsb/LJmcnEkO/iwxjtK0SJvBPl+knpvp?= =?iso-8859-1?Q?56dBLo02EsW5sxy9DgdNttZvnYalxIi1AxO78V0+liFSaS2H161IBO/faR?= =?iso-8859-1?Q?N7BdeRIk+Cv+gP09Xd01JxmTgXNuoLEW1cPleanivs261LzCagtzAkUP65?= =?iso-8859-1?Q?QEmqgkEGaNgW78mgnP4xi1/A5QDMSqW3AAajHHPPjfgorRskzXIgKgZJQ6?= =?iso-8859-1?Q?AMyYkCCF/6WNL7R7XXWBqEh4Z4OFDgckm4YKlpoq2Du+pLj9KSs3O/c/mn?= =?iso-8859-1?Q?ByjnV85UoxvHsgSLB2nlwhMXHgsa9HUxiTDDQzQF+lFP9c6CzYlXjj6bjp?= =?iso-8859-1?Q?GvCjdnrOfUEBcFCAKQwZVtyzIAxQIfam2UMo2w63FNGWG1oRBcDwxjc++h?= =?iso-8859-1?Q?F3uJpA9qvK1uosPbkKyUtF1UXHrJB7KTMvsQZcG6GA9MlM3JvWiVf+/FcM?= =?iso-8859-1?Q?qalvz39/6HsdIO9AIwJ8dlVdhlVKgm00MU4z4C+Awrtrf+zOcnHt7Ln33D?= =?iso-8859-1?Q?Fb7s8BlRkZM0DPDRj1L03WyZeVaqm2//ytiSbZFdQ6vMgBOkIQO4HCapNj?= =?iso-8859-1?B?Zz09?= X-Microsoft-Exchange-Diagnostics: 1;BN3PR0701MB1719;5:vxB/Bq2ZedX/9TSrbbZuzKzL4TCfRwPEZkpLq3P+JmUTQv4JSOCJF7NiUkhx2LdLFTQ5DCSxJlAJ53Prn7JlQzJaMCiAZghw8nV6CL8CMsiZzlfUBV0SC0X4V1/ZGUKhIL+YzwtZRTrz+p0VdOPatQ==;24:5Z6RcEXiQZkwoqL6UgfNtxwMBuk7r2qodFY8DWT3JM1jPo5rZtx8H1EBdn53pA83kHFlyHa9SKtp0VNchJjj7QD9rkIqI4vCgragcriq/w4=;20:8Jm7S3Lr3Mj53aKAWBj+rNcvqGR/BUwzg3URlTigm0mpx5NwSXAzi7roq5F4lrBM2UbR3sXN4Jter9+ZwUw9Fw== SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Sep 2015 18:07:03.1185 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN3PR0701MB1719 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09/11/2015 04:21 PM, David Daney wrote: > From: David Daney > > While using the pci-host-generic driver to add PCI support for the > Cavium ThunderX processors, several bugs were discovered. We also > need the ability to specify a per-bus MSI controller, so support for > that was added. > > David Daney (6): > PCI: Make global and export pdev_fixup_irq(). > PCI: generic: Only fixup irqs for bus we are creating. > PCI: generic: Quit clobbering our pci_ops. > PCI: generic: Correct, and avoid overflow, in bus_max calculation. > PCI: generic: Pass proper starting bus number to pci_scan_root_bus(). > PCI: generic: Allow bus default MSI controller to be specified. > Based on feedback on this first revision of the patches, I plan on sending an updated patch set in a few days. In the mean time, keep sending feedback... David Daney > .../devicetree/bindings/pci/host-generic-pci.txt | 5 ++ > drivers/pci/host/pci-host-generic.c | 55 +++++++++++++++------- > drivers/pci/setup-irq.c | 7 +-- > include/linux/pci.h | 3 ++ > 4 files changed, 50 insertions(+), 20 deletions(-) >