From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752237AbbJMI7Y (ORCPT ); Tue, 13 Oct 2015 04:59:24 -0400 Received: from arroyo.ext.ti.com ([192.94.94.40]:50133 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751451AbbJMI7V (ORCPT ); Tue, 13 Oct 2015 04:59:21 -0400 Subject: Re: [PATCH v4 11/25] ARM/dmaengine: edma: Merge the two drivers under drivers/dma/ To: Vinod Koul References: <1443088932-21731-1-git-send-email-peter.ujfalusi@ti.com> <1443088932-21731-12-git-send-email-peter.ujfalusi@ti.com> <20151012160024.GF27370@localhost> CC: , , , , , , , From: Peter Ujfalusi Message-ID: <561CC7B9.8000201@ti.com> Date: Tue, 13 Oct 2015 11:58:33 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <20151012160024.GF27370@localhost> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/12/2015 07:00 PM, Vinod Koul wrote: > On Thu, Sep 24, 2015 at 01:01:58PM +0300, Peter Ujfalusi wrote: >> Move the code out from arch/arm/common and merge it inside of the dmaengine >> driver. >> This change is done with as minimal change to the code as possible to avoid >> any possibilities to introducing regression. > > Is this a pure move patch or code has been modified, if latter am > disappointed that existing code style issue have not been fixed Yes, it is mostly code move, I have done minimal changes only needed to get the moved code working in the new location. Patch 13 in this series will go through the file and will fix up most (I hope all) of the outstanding coding style issues. At this point I wanted to have as small change as possible. >> +static inline void edma_write(struct edma_cc *ecc, int offset, int val) >> +{ >> + __raw_writel(val, ecc->base + offset); >> +} >> +static inline void edma_modify(struct edma_cc *ecc, int offset, unsigned and, >> + unsigned or) > > This looks bad on my 80 char screen, and few more places below > >> +{ >> + unsigned val = edma_read(ecc, offset); > > checkpatch should have asked you to add empty line here, many places below > too > >> + val &= and; >> + val |= or; >> + edma_write(ecc, offset, val); >> +} > > empty line here and few more places > > More later :) > -- Péter