From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751744AbbLUTu7 (ORCPT ); Mon, 21 Dec 2015 14:50:59 -0500 Received: from unicorn.mansr.com ([81.2.72.234]:54651 "EHLO unicorn.mansr.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751166AbbLUTu5 convert rfc822-to-8bit (ORCPT ); Mon, 21 Dec 2015 14:50:57 -0500 From: =?iso-8859-1?Q?M=E5ns_Rullg=E5rd?= To: Andy Shevchenko Cc: Andy Shevchenko , Viresh Kumar , Julian Margetson , Tejun Heo , linux-ide@vger.kernel.org, "linux-kernel\@vger.kernel.org" Subject: Re: [PATCH 1/3] ata: sata_dwc_460ex: use "dmas" DT property to find dma channel References: <1450221935-6034-1-git-send-email-mans@mansr.com> <56758F33.20804@candw.ms> <5675A84F.2070208@candw.ms> <5675BB2F.6060107@candw.ms> <5675C452.2080206@candw.ms> <5676E906.1060603@candw.ms> <1450725813.30729.256.camel@linux.intel.com> Date: Mon, 21 Dec 2015 19:50:55 +0000 In-Reply-To: <1450725813.30729.256.camel@linux.intel.com> (Andy Shevchenko's message of "Mon, 21 Dec 2015 21:23:33 +0200") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Andy Shevchenko writes: > On Mon, 2015-12-21 at 18:16 +0000, Måns Rullgård wrote: >> Andy Shevchenko writes: >> >> > On Mon, Dec 21, 2015 at 2:15 PM, Måns Rullgård >> > wrote: >> > > Andy Shevchenko writes: >> > >  >> > I used to have a patch to implement this in dw_dmac driver. >> > However, I >> > dropped it at some point. Seems we need it back and now I possible >> > have a good explanation why. >> >> Are you still able to find that patch?  Shouldn't be too hard to do >> from scratch if not. > > Yes, I found a version of it, let me mock up tomorrow something > working. > >> >> > > If those values didn't matter, why would the fields exist in the >> > > first place? >> > >> > Because someone can have more than one AHB bus on the system and >> > connect DMA to all of them (up to 4). >> >> Which apparently these guys did.  Well, not a full-blown AHB bus, but >> they seem to be using two master interfaces. > > To different buses? Intel HW uses two masters and they are quite equal > (at least from OS point of view, it might be HW adjusts it). Judging by the block diagram in the 460EX datasheet [1], and by the fact that the old SATA driver works despite using an invalid address, the DMA FIFO of the controller isn't connected to the AHB bus at all but directly to master 0 on the DW DMA controller. Master 1 of the DMA controller is connected to the AHB bus, which is bridged to the main system bus. I haven't managed to find a full manual for the 460EX. [1] http://datasheet.octopart.com/PPC460EX-NUB800T-AMCC-datasheet-11553412.pdf -- Måns Rullgård