From mboxrd@z Thu Jan 1 00:00:00 1970 From: Timur Tabi Subject: Re: [PATCH 1/2] [v4] net: emac: emac gigabit ethernet controller driver Date: Fri, 15 Apr 2016 12:00:35 -0500 Message-ID: <57111E33.606@codeaurora.org> References: <1460570393-19838-1-git-send-email-timur@codeaurora.org> <570EC541.6080603@gmail.com> <570FFB6B.5060305@codeaurora.org> <57100962.40404@gmail.com> <571012E6.6050303@codeaurora.org> <57102920.7000104@codeaurora.org> <20160415164433.GV391@tuxbot> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20160415164433.GV391@tuxbot> Sender: linux-arm-msm-owner@vger.kernel.org To: Bjorn Andersson Cc: Vikram Sethi , Florian Fainelli , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org, sdharia@codeaurora.org, Shanker Donthineni , Greg Kroah-Hartman , cov@codeaurora.org, gavidov@codeaurora.org, Rob Herring , andrew@lunn.ch, Mark Langsdorf , Jon Masters , Andy Gross , "David S. Miller" List-Id: devicetree@vger.kernel.org Bjorn Andersson wrote: > For platform devices being populated via from DT you will pass: > of_platform_bus_create() > of_platform_device_create_pdata() > of_dma_configure() > > Which calls of_dma_get_range() to acquire this information from the > dma-ranges property and set up the dma ops and properties. This seems excessive. I have to create a platform bus just to configure the DMA mask? Most drivers just call dma_set_mask and give it a number, and that's not device-tree specific. I also need to come up with a way to get this to work on ACPI. I just seems like a lot of work only because I need to determine at runtime what my DMA mask is. I also don't see any drivers that call of_dma_configure(). -- Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation collaborative project.