From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko =?ISO-8859-1?Q?St=FCbner?= Subject: Re: [PATCH 1/2 v3] ARM: s3c24xx: get rid of custom Date: Tue, 07 Jan 2014 20:36:22 +0100 Message-ID: <3906866.u3Hx8EuvTo@phil> References: <1386939220-12776-1-git-send-email-linus.walleij@linaro.org> <201401071215.06064.arnd@arndb.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: Received: from gloria.sntech.de ([95.129.55.99]:38339 "EHLO gloria.sntech.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750851AbaAGTgo (ORCPT ); Tue, 7 Jan 2014 14:36:44 -0500 In-Reply-To: Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Linus Walleij Cc: Arnd Bergmann , "linux-arm-kernel@lists.infradead.org" , linux-samsung-soc , Kukjin Kim , Ben Dooks , Sylwester Nawrocki , Tomasz Figa Am Dienstag, 7. Januar 2014, 19:27:49 schrieb Linus Walleij: > On Tue, Jan 7, 2014 at 12:15 PM, Arnd Bergmann wrote: > > On Friday 13 December 2013, Linus Walleij wrote: > > * The file includes , which is not a bug yet, but > > > > will be once we move s3c24xx to multiplatform, which would make > > it impossible to include this file from outside of arch/arm. > > Yes that needs to be a step for multiplatform enablement. > My series only tries to make the problem smaller and remove > the dependence on . All the and > stuff needs to go away eventually ... > > > Note that on Exynos, the solution for the gpio driver dependencies > > was to scrap the driver and use pinctrl-exynos instead. > > I think the S3C driver is a different piece of hardware unfortunately. both the s3c24xx as well as the exynos pinctrl driver use the common pinctrl- samsung infrastructure. The biggest part of the pinctrl-{exynos,s3c24xx} drivers is the handling of the gpio-interrupts and defining the available banks. In general my multiplatform-plan is to provide the basics (pinctrl, common- clock, etc) and then take the easy way out, described in some arm-summit-notes by simply removing the boards and let people with the hardware that care about it readd the specifics :-) . For example it seems nobody started a S3C2442 machine for quite some time, as it was/is missing the clkdev entries for stuff like the serial ports. Heiko From mboxrd@z Thu Jan 1 00:00:00 1970 From: heiko@sntech.de (Heiko =?ISO-8859-1?Q?St=FCbner?=) Date: Tue, 07 Jan 2014 20:36:22 +0100 Subject: [PATCH 1/2 v3] ARM: s3c24xx: get rid of custom In-Reply-To: References: <1386939220-12776-1-git-send-email-linus.walleij@linaro.org> <201401071215.06064.arnd@arndb.de> Message-ID: <3906866.u3Hx8EuvTo@phil> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Am Dienstag, 7. Januar 2014, 19:27:49 schrieb Linus Walleij: > On Tue, Jan 7, 2014 at 12:15 PM, Arnd Bergmann wrote: > > On Friday 13 December 2013, Linus Walleij wrote: > > * The file includes , which is not a bug yet, but > > > > will be once we move s3c24xx to multiplatform, which would make > > it impossible to include this file from outside of arch/arm. > > Yes that needs to be a step for multiplatform enablement. > My series only tries to make the problem smaller and remove > the dependence on . All the and > stuff needs to go away eventually ... > > > Note that on Exynos, the solution for the gpio driver dependencies > > was to scrap the driver and use pinctrl-exynos instead. > > I think the S3C driver is a different piece of hardware unfortunately. both the s3c24xx as well as the exynos pinctrl driver use the common pinctrl- samsung infrastructure. The biggest part of the pinctrl-{exynos,s3c24xx} drivers is the handling of the gpio-interrupts and defining the available banks. In general my multiplatform-plan is to provide the basics (pinctrl, common- clock, etc) and then take the easy way out, described in some arm-summit-notes by simply removing the boards and let people with the hardware that care about it readd the specifics :-) . For example it seems nobody started a S3C2442 machine for quite some time, as it was/is missing the clkdev entries for stuff like the serial ports. Heiko