Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932221AbaBEGvv (ORCPT ); Wed, 5 Feb 2014 01:51:51 -0500 Received: from mail-pd0-f178.google.com ([209.85.192.178]:55296 "EHLO mail-pd0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752100AbaBEGvf (ORCPT ); Wed, 5 Feb 2014 01:51:35 -0500 From: Magnus Damm To: linux-pci@vger.kernel.org Cc: linux-sh@vger.kernel.org, linux-kernel@vger.kernel.org, valentine.barshak@cogentembedded.com, horms@verge.net.au, bhelgaas@google.com, Magnus Damm , ben.dooks@codethink.co.uk Date: Wed, 05 Feb 2014 15:52:43 +0900 Message-Id: <20140205065243.29445.76593.sendpatchset@w520> Subject: [PATCH 00/04] PCI: rcar: Driver model and physical address space update Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org PCI: rcar: Driver model and physical address space update [PATCH 01/04] PCI: rcar: Register each instance independently [PATCH 02/04] PCI: rcar: Break out window size handling [PATCH 03/04] PCI: rcar: Add DMABOUNCE support [PATCH 04/04] PCI: rcar: Enable BOUNCE in case of HIGHMEM These patches update the pci-rcar-gen2.c driver in various ways including cleanups for driver model interface (1/4), readability update (2/4) and also bounce buffer support (3/4, 4/4). Basically the first two are just cleanups and the rest are fixes. As it is today without these patches the system memory start address is hard coded at 0x4000000 and the window is fixed at 1GiB. So we have board specific code hidden in the driver which is good to avoid. With these hard coded board specific constants there are some error cases that are not handled, in particular the issue that only maximum 1GiB of physical address space can be used for bus mastering with a single window. The common case of using ARM CONFIG_VMSPLIT_3G results in no visible issues as long as CONFIG_BOUNCE is used, but other CONFIG_VMSPLIT settings will break due to the hard coded 1GiB window not being enough. It has been verified that reducing the window size to 256MB makes the driver behave the same with VMSPLIT_3G as 1GiB window size and other VMSPLIT settings. To handle the maximum 1GiB physical address space limitation two types of bounce buffers are added. The ARM specific DMABOUNCE code is in 3/4 hooked up to a chunk of local memory that is also handed of as coherent memory to the pci devices hanging off the PCI bridge. The driver makes sure to set the window so the local memory is always included. When the PCI devices are operating and in case memory is used outside the window then the DMABOUNCE buffers kicks in. This makes the driver support all kinds of VMSPLIT settings and window sizes. The BOUNCE code in 4/4 is selected to make sure bounce buffers are used for HIGHMEM. With these patches this driver can be used with or without CMA and with or without DMA zone. Basically the system wide memory setup is left to the user. If a DMA zone is used then the PCI window will be setup to cover that. Same thing with CMA. Tested with USB storage using LPAE and various VMSPLIT settings together with renesas git tag renesas-devel-v3.14-rc1-20140204 from kernel.org Signed-off-by: Magnus Damm --- Written against renesas.git tag renesas-devel-v3.14-rc1-20140204 drivers/pci/host/Kconfig | 3 drivers/pci/host/pci-rcar-gen2.c | 367 +++++++++++++++++++++++++++++++------- 2 files changed, 306 insertions(+), 64 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/