From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomasz Figa Subject: Re: [PATCH 2/2] pinctrl: samsung: Fix link error with CONFIG_PINCTRL_EXYNOS disabled Date: Tue, 23 Apr 2013 14:09:46 +0200 Message-ID: <6940961.67Lk2axxPL@amdc1227> References: <1366060483-20342-1-git-send-email-tomasz.figa@gmail.com> <1398545.9FqHoC0WZW@flatron> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7Bit Return-path: In-reply-to: Sender: linux-samsung-soc-owner@vger.kernel.org To: Linus Walleij Cc: Tomasz Figa , Mark Brown , linux-samsung-soc , "linux-arm-kernel@lists.infradead.org" , "devicetree-discuss@lists.ozlabs.org" , Kukjin Kim , kwangwoo.lee@gmail.com, Peter Korsgaard , augulis.darius@gmail.com, mcuelenaere@gmail.com, Russell King - ARM Linux , Rob Herring , Mark Rutland , Thomas Abraham List-Id: devicetree@vger.kernel.org Hi Linus, On Tuesday 23 of April 2013 14:03:12 Linus Walleij wrote: > On Wed, Apr 17, 2013 at 12:24 AM, Tomasz Figa wrote: > > Commit f67faf487 added driver data for Exynos5250, which is available > > only if CONFIG_PINCTRL_EXYNOS is enabled. However probably due to merge > > error, the reference to this driver data structure was located outside > > of relevant ifdef and compiled unconditionally, leading to link error > > with CONFIG_PINCTRL_EXYNOS. > > > > This patch fixes mentioned link error by moving driver data entry under > > ifdef CONFIG_PINCTRL_EXYNOS. > > > > Signed-off-by: Tomasz Figa > > This does not apply to my "devel" branch in the pinctrl tree. > > Could you rebase it on that branch? > > Or is the error not even in my tree? This one got sent incorrectly due to my mistake. Please try the correct version: [PATCH RESEND 2/2] pinctrl: samsung: Fix link error with CONFIG_PINCTRL_EXYNOS disabled Best regards, -- Tomasz Figa Samsung Poland R&D Center SW Solution Development, Kernel and System Framework