From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1758111AbcILLjP (ORCPT <rfc822;w@1wt.eu>);
        Mon, 12 Sep 2016 07:39:15 -0400
Received: from arroyo.ext.ti.com ([198.47.19.12]:55660 "EHLO arroyo.ext.ti.com"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1754771AbcILLjM (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
        Mon, 12 Sep 2016 07:39:12 -0400
Subject: Re: [PATCH 1/1] ARM: dma: fix dma_max_pfn()
To: Santosh Shilimkar <santosh.shilimkar@oracle.com>,
        Russell King - ARM Linux <linux@armlinux.org.uk>
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>
 <f8a05c02-df36-a0cf-6433-c67eb4a0f5b6@oracle.com>
 <9c38663b-1867-213d-21ba-f1dec7edfb6a@ti.com>
 <81dedbcc-14f6-f02f-2ce9-114d57c8d9d4@oracle.com>
CC: <ssantosh@kernel.org>, <grygorii.strashko@ti.com>,
        <linux-arm-kernel@lists.infradead.org>, <linux-usb@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>, <stable@vger.kernel.org>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Arnd Bergmann <arnd@arndb.de>, Olof Johansson <olof@lixom.net>,
        Catalin Marinas <catalin.marinas@arm.com>,
        Linus Walleij <linus.walleij@linaro.org>
From: Roger Quadros <rogerq@ti.com>
Message-ID: <c29290ee-9d74-73f9-7405-112357359f55@ti.com>
Date: Mon, 12 Sep 2016 14:38:24 +0300
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101
 Thunderbird/45.2.0
MIME-Version: 1.0
In-Reply-To: <81dedbcc-14f6-f02f-2ce9-114d57c8d9d4@oracle.com>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

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?

cheers,
-roger