From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail1.g1.pair.com ([66.39.3.162]:57997 "EHLO mail1.g1.pair.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750821Ab3I0Dyq (ORCPT ); Thu, 26 Sep 2013 23:54:46 -0400 Message-ID: <52450180.2070807@kosagi.com> Date: Fri, 27 Sep 2013 11:54:40 +0800 From: Sean Cross MIME-Version: 1.0 To: Jingoo Han CC: 'Shawn Guo' , 'Zhu Richard-R65037' , 'Tim Harvey' , devicetree@vger.kernel.org, linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, 'Sascha Hauer' , bhelgaas@google.com Subject: Re: [PATCH v6 3/3] PCI: imx6: Add support for i.MX6 PCIe controller References: <1379319655-20210-1-git-send-email-xobs@kosagi.com> <1379319655-20210-4-git-send-email-xobs@kosagi.com> <0E83723C55F66F43A6041464FE31119D3F7E6A@039-SN2MPN1-012.039d.mgd.msft.net> <20130927021925.GF25146@S2101-09.ap.freescale.net> <017e01cebb33$44fa65b0$ceef3110$%han@samsung.com> In-Reply-To: <017e01cebb33$44fa65b0$ceef3110$%han@samsung.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Sender: linux-pci-owner@vger.kernel.org List-ID: On 27/9/13 11:40 AM, Jingoo Han wrote: > On Friday, September 27, 2013 11:19 AM, Shawn Guo wrote: >> On Fri, Sep 27, 2013 at 01:38:39AM +0000, Zhu Richard-R65037 wrote: >>> Hi Tim: >>> Thanks for your tests. >>> Yes, it is. There are some problems when the PCIe switch is used >>> on the 3.1x kernel at my side too, >> although the PCIe switch >>> is ok on FSL imx_3.0.35 kernels. >>> I'm debugging this issue now, and would summit the patches asap. >>> >> From what I read, it seems Sean's series does not support PCIe switch. >> At least, he hasn't got any device to test it. And there is some >> problem with PCIe switch running the latest kernel, which might not be >> related to the controller driver. >> >> I suggest we ask Bjorn to merge the v7 I sent yesterday, so that people >> can have a same base to work on, and add PCIe switch support with >> incremental patches. Does that sound good to you, Tim? >> > Hi Shawn Guo, > > I agree with your suggestion. :-) > The PCIe switch support patches can be added later, after the v7 patch for > i.MX6 PCIe controller driver is merged. > > Also, if the PCIe switch support patches modify the synopsys designware PCIe > driver (pcie-designware.c), it should be confirmed by other related people. > Correct. My series doesn't support PCIe switches mostly because I don't have one to test with. It would be interesting to see if PCIe switches work with other Designware drivers. --- Sean From mboxrd@z Thu Jan 1 00:00:00 1970 From: xobs@kosagi.com (Sean Cross) Date: Fri, 27 Sep 2013 11:54:40 +0800 Subject: [PATCH v6 3/3] PCI: imx6: Add support for i.MX6 PCIe controller In-Reply-To: <017e01cebb33$44fa65b0$ceef3110$%han@samsung.com> References: <1379319655-20210-1-git-send-email-xobs@kosagi.com> <1379319655-20210-4-git-send-email-xobs@kosagi.com> <0E83723C55F66F43A6041464FE31119D3F7E6A@039-SN2MPN1-012.039d.mgd.msft.net> <20130927021925.GF25146@S2101-09.ap.freescale.net> <017e01cebb33$44fa65b0$ceef3110$%han@samsung.com> Message-ID: <52450180.2070807@kosagi.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 27/9/13 11:40 AM, Jingoo Han wrote: > On Friday, September 27, 2013 11:19 AM, Shawn Guo wrote: >> On Fri, Sep 27, 2013 at 01:38:39AM +0000, Zhu Richard-R65037 wrote: >>> Hi Tim: >>> Thanks for your tests. >>> Yes, it is. There are some problems when the PCIe switch is used >>> on the 3.1x kernel at my side too, >> although the PCIe switch >>> is ok on FSL imx_3.0.35 kernels. >>> I'm debugging this issue now, and would summit the patches asap. >>> >> From what I read, it seems Sean's series does not support PCIe switch. >> At least, he hasn't got any device to test it. And there is some >> problem with PCIe switch running the latest kernel, which might not be >> related to the controller driver. >> >> I suggest we ask Bjorn to merge the v7 I sent yesterday, so that people >> can have a same base to work on, and add PCIe switch support with >> incremental patches. Does that sound good to you, Tim? >> > Hi Shawn Guo, > > I agree with your suggestion. :-) > The PCIe switch support patches can be added later, after the v7 patch for > i.MX6 PCIe controller driver is merged. > > Also, if the PCIe switch support patches modify the synopsys designware PCIe > driver (pcie-designware.c), it should be confirmed by other related people. > Correct. My series doesn't support PCIe switches mostly because I don't have one to test with. It would be interesting to see if PCIe switches work with other Designware drivers. --- Sean From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sean Cross Subject: Re: [PATCH v6 3/3] PCI: imx6: Add support for i.MX6 PCIe controller Date: Fri, 27 Sep 2013 11:54:40 +0800 Message-ID: <52450180.2070807@kosagi.com> References: <1379319655-20210-1-git-send-email-xobs@kosagi.com> <1379319655-20210-4-git-send-email-xobs@kosagi.com> <0E83723C55F66F43A6041464FE31119D3F7E6A@039-SN2MPN1-012.039d.mgd.msft.net> <20130927021925.GF25146@S2101-09.ap.freescale.net> <017e01cebb33$44fa65b0$ceef3110$%han@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <017e01cebb33$44fa65b0$ceef3110$%han-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jingoo Han Cc: 'Shawn Guo' , 'Zhu Richard-R65037' , 'Tim Harvey' , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, 'Sascha Hauer' , bhelgaas-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org List-Id: devicetree@vger.kernel.org On 27/9/13 11:40 AM, Jingoo Han wrote: > On Friday, September 27, 2013 11:19 AM, Shawn Guo wrote: >> On Fri, Sep 27, 2013 at 01:38:39AM +0000, Zhu Richard-R65037 wrote: >>> Hi Tim: >>> Thanks for your tests. >>> Yes, it is. There are some problems when the PCIe switch is used >>> on the 3.1x kernel at my side too, >> although the PCIe switch >>> is ok on FSL imx_3.0.35 kernels. >>> I'm debugging this issue now, and would summit the patches asap. >>> >> From what I read, it seems Sean's series does not support PCIe switch. >> At least, he hasn't got any device to test it. And there is some >> problem with PCIe switch running the latest kernel, which might not be >> related to the controller driver. >> >> I suggest we ask Bjorn to merge the v7 I sent yesterday, so that people >> can have a same base to work on, and add PCIe switch support with >> incremental patches. Does that sound good to you, Tim? >> > Hi Shawn Guo, > > I agree with your suggestion. :-) > The PCIe switch support patches can be added later, after the v7 patch for > i.MX6 PCIe controller driver is merged. > > Also, if the PCIe switch support patches modify the synopsys designware PCIe > driver (pcie-designware.c), it should be confirmed by other related people. > Correct. My series doesn't support PCIe switches mostly because I don't have one to test with. It would be interesting to see if PCIe switches work with other Designware drivers. --- Sean -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html