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 S933362AbdC3KFx (ORCPT <rfc822;w@1wt.eu>);
        Thu, 30 Mar 2017 06:05:53 -0400
Received: from mail.free-electrons.com ([62.4.15.54]:33869 "EHLO
        mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S933329AbdC3KFg (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Thu, 30 Mar 2017 06:05:36 -0400
Date: Thu, 30 Mar 2017 12:05:22 +0200
From: Boris Brezillon <boris.brezillon@free-electrons.com>
To: Max Filippov <jcmvbkbc@gmail.com>
Cc: Christoph Hellwig <hch@infradead.org>, Chris Zankel <chris@zankel.net>,
        "linux-xtensa@linux-xtensa.org" <linux-xtensa@linux-xtensa.org>,
        Maxime Ripard <maxime.ripard@free-electrons.com>,
        Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
        LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] xtensa: Fix mmap() support
Message-ID: <20170330120522.4af7bcbc@bbrezillon>
In-Reply-To: <CAMo8Bf+D1kr8UtOQ+e86MtN9=mMO9KsJvY62qOXCyEBF_Jj-yA@mail.gmail.com>
References: <1490689242-5097-1-git-send-email-boris.brezillon@free-electrons.com>
        <CAMo8BfJZDR5A5MRO0SOL7r5U+_7ETuVx_LH95J7K-ZRZe9EM=A@mail.gmail.com>
        <20170330103032.35558cf3@bbrezillon>
        <CAMo8Bf+D1kr8UtOQ+e86MtN9=mMO9KsJvY62qOXCyEBF_Jj-yA@mail.gmail.com>
X-Mailer: Claws Mail 3.13.2 (GTK+ 2.24.30; x86_64-pc-linux-gnu)
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
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

On Thu, 30 Mar 2017 02:05:28 -0700
Max Filippov <jcmvbkbc@gmail.com> wrote:

> On Thu, Mar 30, 2017 at 1:30 AM, Boris Brezillon
> <boris.brezillon@free-electrons.com> wrote:
> >> Could you please instead check if the dma_common_mmap works for you
> >> with the attached patch?  
> >
> > I will. BTW, shouldn't it be
> >
> >         if (off >= XCHAL_KSEG_SIZE)
> >
> > instead of
> >
> >         if (off > XCHAL_KSEG_SIZE)
> > ?  
> 
> Oops. Yes, you're right.
> 

It works, you can add my

Tested-by: Boris Brezillon <boris.brezillon@free-electrons.com>
Reviewed-by: Boris Brezillon <boris.brezillon@free-electrons.com>

when submitting the patch.

Thanks,

Boris