From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Subject: Re: [PATCH 1/2] net: sh_eth: use correct "struct device" when calling DMA mapping functions Date: Tue, 5 Dec 2017 10:39:57 +0100 Message-ID: References: <20171204133327.3505-1-thomas.petazzoni@free-electrons.com> <20171204133327.3505-2-thomas.petazzoni@free-electrons.com> <607df840-613e-b86c-2224-43815b50627c@cogentembedded.com> <20171205103835.6fb9192f@windsurf.lan> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Cc: Sergei Shtylyov , "David S. Miller" , =?UTF-8?Q?Niklas_S=C3=B6derlund?= , Geert Uytterhoeven , Simon Horman , "netdev@vger.kernel.org" , Linux-Renesas To: Thomas Petazzoni Return-path: Received: from mail-lf0-f68.google.com ([209.85.215.68]:44748 "EHLO mail-lf0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753171AbdLEJkA (ORCPT ); Tue, 5 Dec 2017 04:40:00 -0500 In-Reply-To: <20171205103835.6fb9192f@windsurf.lan> Sender: netdev-owner@vger.kernel.org List-ID: Hi Thomas, On Tue, Dec 5, 2017 at 10:38 AM, Thomas Petazzoni wrote: > On Tue, 5 Dec 2017 09:39:35 +0100, Geert Uytterhoeven wrote: >> > Using 'ndev->dev.parent' (as in ravb) also should work... not sure which >> > is better >> >> That was going to be my comment, too. I also haven't checked which >> generates the smallest code. > > Using ndev->dev.parent actually generates bigger code (44 bytes larger) : Thanks a lot for checking! Gr{oetje,eeting}s, Geert