From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752567AbcAXUhm (ORCPT ); Sun, 24 Jan 2016 15:37:42 -0500 Received: from cassarossa.samfundet.no ([193.35.52.29]:46648 "EHLO cassarossa.samfundet.no" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751590AbcAXUhh (ORCPT ); Sun, 24 Jan 2016 15:37:37 -0500 Date: Sun, 24 Jan 2016 21:37:21 +0100 From: Hans-Christian Noren Egtvedt To: =?iso-8859-1?Q?M=E5ns_Rullg=E5rd?= Cc: Viresh Kumar , Andy Shevchenko , Vinod Koul , linux-kernel@vger.kernel.org, dmaengine@vger.kernel.org, Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Haavard Skinnemoen , Tejun Heo , Dan Williams , Daniel Mack , Haojian Zhuang , Robert Jarzmik , Mark Brown , Greg Kroah-Hartman , Jiri Slaby , devicetree@vger.kernel.org, linux-ide@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-spi@vger.kernel.org, linux-serial@vger.kernel.org Subject: Re: [PATCH 03/15] dmaengine: dw: rename masters to reflect actual topology Message-ID: <20160124203720.GA29010@samfundet.no> References: <1453663322-14474-1-git-send-email-mans@mansr.com> <1453663322-14474-4-git-send-email-mans@mansr.com> <20160124200959.GA15633@samfundet.no> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Around Sun 24 Jan 2016 20:19:46 +0000 or thereabout, Måns Rullgård wrote: > Hans-Christian Noren Egtvedt writes: >> Around Sun 24 Jan 2016 19:21:50 +0000 or thereabout, Mans Rullgard wrote: >>> From: Andy Shevchenko >>> >>> The source and destination masters are reflecting buses or their layers to >>> where the different devices can be connected. The patch changes the master >>> names to reflect which one is related to which independently on the transfer >>> direction. >>> >>> The outcome of the change is that the memory data width is now always limited >>> by a data width of the master which is dedicated to communicate to memory. >>> >>> The patch will not break anything since all current users have the same data >>> width for all masters. Though it would be nice to revisit avr32 plaforms to >>> check what is the actual hardware topology is used there. It seems that it has >>> one bus and two masters on it as stated by Table 8-2, that's why everything >>> works independently on the master in use. The purpose of the sequential patch >>> is to fix the driver for configuration of more that one bus. >> >> Not entirely sure what you want to have confirmed here. There are multiple >> masters and slaves on the HMATRIX internal bus on AVR32, and the DMA >> controller supports up to three simultaneous configurations. >> >> Sounds good to support configuration of more than one bus. I thought we >> always did support that? Perhaps it was a non-standard avr32 implementation. > > The DW DMA controller on the AT32AP7000 serves the MCI, AC97, and ABDAC > peripherals. It appears to work regardless of the values put in the > various master select fields. Perhaps the topology is hardwired in the > DMA controller and those fields are ignored. The AVR32 works both > before and after this patch series, the main purpose of which (at least > my patches) is to fix the SATA driver on 460EX. DEST_PER and SRC_PER in the DMA controller selects this, numbers placed here should match the table you most likely found, 9-3. Wiring the handshake connections is done with the struct dw_dma_slave src_id or dst_id member, depending on data direction. Configured in the at32ap700x.c machine code. It is not hard wired on avr32, as there are not one-to-one configurations and masters. >>> The change is done in the assumption that src_master and dst_master are >>> reflecting a connection to the memory and peripheral correspondently on all >>> platforms except 460ex. OK, I have no knowledge about the 460ex. >>> Signed-off-by: Andy Shevchenko >>> Signed-off-by: Mans Rullgard >> >> For the avr32 related stuff: >> >> Acked-by: Hans-Christian Egtvedt >> >>> --- >>> Documentation/devicetree/bindings/dma/snps-dma.txt | 4 ++-- >>> arch/avr32/mach-at32ap/at32ap700x.c | 16 ++++++++-------- >>> drivers/ata/sata_dwc_460ex.c | 4 ++-- >>> drivers/dma/dw/core.c | 15 +++++++-------- >>> drivers/dma/dw/platform.c | 12 ++++++------ >>> drivers/dma/dw/regs.h | 4 ++-- >>> drivers/spi/spi-pxa2xx-pci.c | 8 ++++---- >>> drivers/tty/serial/8250/8250_pci.c | 8 ++++---- >>> include/linux/platform_data/dma-dw.h | 8 ++++---- >>> 9 files changed, 39 insertions(+), 40 deletions(-) >> >> -- Best regards, Hans-Christian Egtvedt