From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from bear.ext.ti.com ([192.94.94.41]:39017 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750951Ab3GLFYH (ORCPT ); Fri, 12 Jul 2013 01:24:07 -0400 Message-ID: <51DF92E7.2070109@ti.com> Date: Fri, 12 Jul 2013 10:53:51 +0530 From: Kishon Vijay Abraham I MIME-Version: 1.0 To: Jingoo Han CC: "'Bjorn Helgaas'" , , , "'Kukjin Kim'" , "'Pratyush Anand'" , "'Mohit KUMAR'" , "'Arnd Bergmann'" , "'Sean Cross'" , "'SRIKANTH TUMKUR SHIVANAND'" , , "'Thierry Reding'" Subject: Re: [PATCH V2] pci: exynos: split into two parts such as Synopsys part and Exynos part References: <001201ce7dfa$716b3370$54419a50$@samsung.com> <51DE72CC.9050001@ti.com> <001301ce7e92$d19a04e0$74ce0ea0$@samsung.com> In-Reply-To: <001301ce7e92$d19a04e0$74ce0ea0$@samsung.com> Content-Type: text/plain; charset="ISO-8859-1" Sender: linux-pci-owner@vger.kernel.org List-ID: On Friday 12 July 2013 05:30 AM, Jingoo Han wrote: > On Thursday, July 11, 2013 5:55 PM, Kishon Vijay Abraham I wrote: >> On Thursday 11 July 2013 11:19 AM, Jingoo Han wrote: >>> Exynos PCIe IP consists of Synopsys specific part and Exynos >>> specific part. Only core block is a Synopsys designware part; >>> other parts are Exynos specific. >>> Also, the Synopsys designware part can be shared with other >>> platforms; thus, it can be split two parts such as Synopsys >>> designware part and Exynos specific part. >>> >>> Signed-off-by: Jingoo Han >>> Cc: Pratyush Anand >>> Cc: Mohit KUMAR >>> --- >> . >> . >> >> . >> . >>> + >>> +/* Exynos PCIe driver does not allow module unload */ >> >> Just curious, why is this restriction? > > CC'ed Thierry Reding, > > Hi Kishon, > > That's a good question. > > Now, we don't have the solution to "be able to load and unload > the PCI host driver in a loop definitely without crashing or exposing > any races or leaks", as Arnd Bergmann said. > Please refer to the following thread in mailing-list. > (http://archive.arm.linux.org.uk/lurker/message/20130614.123849.4ff363c5.pl.html) That explains. Thanks Kishon