From mboxrd@z Thu Jan  1 00:00:00 1970
From: Heiko Stuebner <heiko@sntech.de>
Date: Sun, 06 Jan 2019 21:49:25 +0100
Subject: [U-Boot] [PATCH v2 6/9] pinctrl: rockchip: Add common rockchip
	pinctrl driver
In-Reply-To: <7CA45F3C-F6E4-4E49-BF41-C2E9DCC66519@theobroma-systems.com>
References: <20190102125105.25734-1-david.wu@rock-chips.com>
 <9566905.BjVVr1PO9Y@phil>
 <7CA45F3C-F6E4-4E49-BF41-C2E9DCC66519@theobroma-systems.com>
Message-ID: <2424928.rjXneoo3dz@phil>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
To: u-boot@lists.denx.de

Hi Philipp,

Am Sonntag, 6. Januar 2019, 19:57:42 CET schrieb Philipp Tomsich:
> > On 05.01.2019, at 18:27, Heiko Stuebner <heiko@sntech.de> wrote:
> > Am Mittwoch, 2. Januar 2019, 14:00:55 CET schrieb David Wu:
> >> Use this driver to fit all Rockchip SOCs and to support
> >> the desired pinctrl configuration via DTS.
> >> 
> >> Signed-off-by: David Wu <david.wu@rock-chips.com>
> >> ---
> >> 
> >> Changes in v2:
> >> - Remove px30, rk2928, rk3066*.
> >> - Split it to multiple files for the relevant per-SoC data structures.
> > 
> > Moving so nicely close to the Linux pinctrl driver is way cool. And while
> > I haven't looked in that much detail, this looks nice, so
> > 
> > Acked-by: Heiko Stuebner <heiko@sntech.de>
> > 
> > Out of curiosity, why did you remove px30, rk2928 and rk3066 support?
> > I'm still hoping to support those at some point (someone was working
> > on rk3066 uboot support for example last year or so), so having the
> > pinctrl support already present might encourage things?
> > Or was that a review-request from v1?
> 
> I requested to not have any drivers includeed for chips that are not
> supported in U-Boot.  Each additional pinctrl-driver should then come
> in once the respective SoC is added (i.e. I’d like to have the pinctrl
> driver part of the respective enablement series)…

fair enough, thanks for the heads up :-)


Heiko