From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [v6,2/2] PCI: Rockchip: Add Rockchip PCIe controller support Date: Fri, 08 Jul 2016 22:10:14 +0200 Message-ID: <2013221.obfd8Z8cde@wuerfel> References: <1467789398-13501-2-git-send-email-shawn.lin@rock-chips.com> <20160708163511.GA23348@roeck-us.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20160708163511.GA23348-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+glpar-linux-rockchip=m.gmane.org-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org To: Guenter Roeck Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Brian Norris , Heiko Stuebner , Marc Zyngier , linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Shawn Lin , Wenrui Li , linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Doug Anderson , linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Rob Herring , Bjorn Helgaas List-Id: devicetree@vger.kernel.org On Friday, July 8, 2016 9:35:11 AM CEST Guenter Roeck wrote: > On Wed, Jul 06, 2016 at 03:16:38PM +0800, Shawn Lin wrote: > > This patch adds Rockchip PCIe controller support found > > on RK3399 Soc platform. > > > > Signed-off-by: Shawn Lin > > --- > > [ ... ] > > > + /* Get the I/O and memory ranges from DT */ > > + resource_list_for_each_entry(win, &res) { > > + switch (resource_type(win->res)) { > > + case IORESOURCE_IO: > > + io = win->res; > > + io->name = "I/O"; > > + io_size = resource_size(io); > > + io_bus_addr = io->start - win->offset; > > + err = pci_remap_iospace(io, io_base); > > This function is not exported. Either it will have to be exported > (which may be difficult since it is declared as __weak), or the > driver must only be built into the kernel, or the function can > not be used. > I think we can simply drop the __weak annotation, there is no other implementation of this, and I doubt there will ever be. Even if it was needed, we could do it in a different way. Arnd