Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757819AbbBFSh4 (ORCPT ); Fri, 6 Feb 2015 13:37:56 -0500 Received: from arroyo.ext.ti.com ([192.94.94.40]:51413 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754732AbbBFShy (ORCPT ); Fri, 6 Feb 2015 13:37:54 -0500 Message-ID: <54D509C9.9010204@ti.com> Date: Fri, 6 Feb 2015 13:36:57 -0500 From: Murali Karicheri User-Agent: Mozilla/5.0 (X11; Linux i686; rv:12.0) Gecko/20120430 Thunderbird/12.0.1 MIME-Version: 1.0 To: Bjorn Helgaas CC: Catalin Marinas , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "iommu@lists.linux-foundation.org" , "devicetree@vger.kernel.org" , "linux-pci@vger.kernel.org" , Russell King , Arnd Bergmann , Joerg Roedel , Will Deacon , Rob Herring , "suravee.suthikulpanit@amd.com" , "grant.likely@linaro.org" Subject: Re: [PATCH v6 0/7] PCI: get DMA configuration from parent device References: <1423173179-10227-1-git-send-email-m-karicheri2@ti.com> <20150206151542.GD23190@e104818-lin.cambridge.arm.com> <54D4DD95.7000303@ti.com> In-Reply-To: Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2987 Lines: 76 On 02/06/2015 12:53 PM, Bjorn Helgaas wrote: > On Fri, Feb 6, 2015 at 9:28 AM, Murali Karicheri wrote: >> On 02/06/2015 10:15 AM, Catalin Marinas wrote: >>> >>> On Thu, Feb 05, 2015 at 09:52:52PM +0000, Murali Karicheri wrote: >>>> >>>> This patch add an important capability to PCI driver on Keystone. I hope >>>> to >>>> have this merged to the upstream branch so that it is available for >>>> v3.20. >>> >>> It's very late for 3.20 and the code hasn't been in linux-next at all >>> (but it's not me who's merging this code), unless you can squeeze it in >>> as a bug-fix. >> >> This is in fact a bug fix as PCI on Keystone is broken without this. > > Oh, sorry, I didn't realize that this was so urgent. I guess I read > "this adds an important capability" in the cover letter and concluded > that it was new functionality. Bjorn, Thanks for responding. Let me give you some context on this without which my explanation won't be complete. For using PCI driver on Keystone, I had submitted patches related to machine and DTS to the arm mailing list to enable the driver on Keystone. Subsequenty one of the patch from my series was Nack-ed and I was asked to implememented in a different way and started this series. You can refer to the discussion of this at http://www.gossamer-threads.com/lists/linux/kernel/2024591 The PCI driver enablement on Keystone is still a working in progress and I am trying to get it fully functional on the upstream. Another missing piece is the SerDes phy driver patch. We have started working on the other part (SerDes phy driver) already as the initial one was not accepted. So it is fine if we are too late for the v3.20 merge window to merge this series and this can be applied to the next branch for v3.21. > > Anyway, if it's broken, presumably PCI on Keystone *did* work at one > point. Can you identify the commit that broke it and requires these > fixes, so we can figure out how far the fixes need to be backported? > I am trying to get this driver enabled on Keystone by adding the missing pieces as described above. So I guess we don't have to back port anything here. > If I merge it, I would like to get into my for-linus branch and get a > little time in -next before asking Linus to pull it. The merge window > is a little wrinkle there -- I don't like to add new things to the mix > during the window. But if it's an important fix we can still get it > in before the final v3.20. Please apply this to next branch for v3.21. It currently apply cleanly to v3.19-rc7. If you want me rebase to another branch, let me know I can apply and send you an updated patch. Thanks Murali > > Bjorn -- Murali Karicheri Linux Kernel, Texas Instruments -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/