From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752285AbdJTHo7 (ORCPT ); Fri, 20 Oct 2017 03:44:59 -0400 Received: from lb2-smtp-cloud9.xs4all.net ([194.109.24.26]:47829 "EHLO lb2-smtp-cloud9.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751961AbdJTHo5 (ORCPT ); Fri, 20 Oct 2017 03:44:57 -0400 Subject: Re: [PATCH] pinctrl: rockchip: Add iomux-route switching support for rk3288 To: =?UTF-8?Q?Heiko_St=c3=bcbner?= References: <20171013225337.5196-1-phh@phh.me> <13020229.tRmotBUImn@phil> <52d1a9ee-a6c6-5c7f-330f-9b672be9c2c6@xs4all.nl> <2040825.vRYCsv903Y@diego> Cc: Pierre-Hugues Husson , linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org From: Hans Verkuil Message-ID: <68b8765d-3d55-3fdd-e23e-776e978c38bd@xs4all.nl> Date: Fri, 20 Oct 2017 09:44:55 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 In-Reply-To: <2040825.vRYCsv903Y@diego> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit X-CMAE-Envelope: MS4wfANhltoWTk9EgMUZI9gdKFz6lKP0B8G7f/LRu3vReIphzKE614RNm7C/krh0dXZdpEJ4VdPAjL5F/M6iUiAueR+a3Ppcu4uX0spcwEtzlDUgZnfvzQO4 1VGZ3sWP4pr5yJZr5mB9r+6fvum5UMOc3yfLM3//rVVsvCRNlzrDojN5+lXI19V8/nJtm2q27/JyL27IjjtAmHnPRPAAv9kzFYwTTbCvtDq9/Sipxp8lC4FF d827IwGSzkzlifWwidqhVWxbudEqK/MG1j/xRI+vwBf8mnlwOortTNIG/OqQlz2w Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 20/10/17 09:38, Heiko Stübner wrote: > Hi Hans, > > Am Freitag, 20. Oktober 2017, 09:28:58 CEST schrieb Hans Verkuil: >> On 14/10/17 17:39, Heiko Stuebner wrote: >>> So far only the hdmi cec supports using one of two different pins >>> as source, so add the route switching for it. >>> >>> Signed-off-by: Heiko Stuebner >> >> Just tested this on my firefly reload and it works great! >> >> Tested-by: Hans Verkuil > > oh cool. I really only wrote this based on the soc manual, > so it actually surprises me, that it works on the first try :-) One note though: I've only tested it on my Firefly Reload. I don't have a regular Firefly, so I can't be certain it works there. Just covering my ass here :-) > >> I'll post some dts patches later today to fully bring up the first HDMI >> output on the Firefly Reload. >> >> Will you process this patch further to get it mainlined? > > Yep, I'll do that. Thanks! Regards, Hans