From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751797AbcI1Hyu (ORCPT ); Wed, 28 Sep 2016 03:54:50 -0400 Received: from arroyo.ext.ti.com ([198.47.19.12]:49058 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751521AbcI1Hym (ORCPT ); Wed, 28 Sep 2016 03:54:42 -0400 Subject: Re: [PATCH 1/1] ARM: dma: fix dma_max_pfn() To: Santosh Shilimkar , Russell King - ARM Linux References: <1471435517-7840-1-git-send-email-rogerq@ti.com> <1471435517-7840-2-git-send-email-rogerq@ti.com> <20160818142459.GX1041@n2100.armlinux.org.uk> <8003b6ed-6071-9384-1b88-0b84604bdce8@oracle.com> <20160818230754.GE1041@n2100.armlinux.org.uk> <9c38663b-1867-213d-21ba-f1dec7edfb6a@ti.com> <81dedbcc-14f6-f02f-2ce9-114d57c8d9d4@oracle.com> CC: , , , , , , Greg Kroah-Hartman , Arnd Bergmann , Olof Johansson , Catalin Marinas , Linus Walleij From: Roger Quadros Message-ID: <135ccf6f-ef3b-98a5-9d3d-4cd7f0967ceb@ti.com> Date: Wed, 28 Sep 2016 10:53:49 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On 12/09/16 14:38, Roger Quadros wrote: > Hi Santosh & Russell, > > On 19/08/16 19:38, Santosh Shilimkar wrote: >> >> On 8/19/2016 12:30 AM, Roger Quadros wrote: >>> Hi Santosh, >>> >> >>>>> So I'm 99.9% convinced that the proposed change is correct. >>>>> >>>> I will got with that then :-) and take my objection back. Just >>>> saying that if there other breakages which I can't recollect now, >>>> those drivers needs to be patched as well. >>>> >>> I was able to boot the Keystone2 Edison EVM over NFS with the $subject patch. >>> Boot log is below. Do you see anything suspicious? >>> >> Logs looks ok to me. Probably do some tests where DMA and bounce buffers etc gets tested. Running it through your internal regression >> suit will be good idea as well if thats possible. >> > > This has been running in our internal test suite for a week on various TI > platforms. There haven't been any surprises. > > Is it a good idea to at least put this in -next for a wider test audience? Gentle reminder. regards, -roger From mboxrd@z Thu Jan 1 00:00:00 1970 From: rogerq@ti.com (Roger Quadros) Date: Wed, 28 Sep 2016 10:53:49 +0300 Subject: [PATCH 1/1] ARM: dma: fix dma_max_pfn() In-Reply-To: References: <1471435517-7840-1-git-send-email-rogerq@ti.com> <1471435517-7840-2-git-send-email-rogerq@ti.com> <20160818142459.GX1041@n2100.armlinux.org.uk> <8003b6ed-6071-9384-1b88-0b84604bdce8@oracle.com> <20160818230754.GE1041@n2100.armlinux.org.uk> <9c38663b-1867-213d-21ba-f1dec7edfb6a@ti.com> <81dedbcc-14f6-f02f-2ce9-114d57c8d9d4@oracle.com> Message-ID: <135ccf6f-ef3b-98a5-9d3d-4cd7f0967ceb@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi, On 12/09/16 14:38, Roger Quadros wrote: > Hi Santosh & Russell, > > On 19/08/16 19:38, Santosh Shilimkar wrote: >> >> On 8/19/2016 12:30 AM, Roger Quadros wrote: >>> Hi Santosh, >>> >> >>>>> So I'm 99.9% convinced that the proposed change is correct. >>>>> >>>> I will got with that then :-) and take my objection back. Just >>>> saying that if there other breakages which I can't recollect now, >>>> those drivers needs to be patched as well. >>>> >>> I was able to boot the Keystone2 Edison EVM over NFS with the $subject patch. >>> Boot log is below. Do you see anything suspicious? >>> >> Logs looks ok to me. Probably do some tests where DMA and bounce buffers etc gets tested. Running it through your internal regression >> suit will be good idea as well if thats possible. >> > > This has been running in our internal test suite for a week on various TI > platforms. There haven't been any surprises. > > Is it a good idea to at least put this in -next for a wider test audience? Gentle reminder. regards, -roger