Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934189AbcCIVHy (ORCPT ); Wed, 9 Mar 2016 16:07:54 -0500 Received: from 5520-maca-inet1-outside.broadcom.com ([216.31.211.11]:22361 "EHLO mail-irv-18.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933627AbcCIVHf (ORCPT ); Wed, 9 Mar 2016 16:07:35 -0500 Subject: Re: [PATCH 2/3] pinctrl: ns2: add pinmux driver support for Broadcom NS2 SoC To: Linus Walleij , Yendapally Reddy Dhananjaya Reddy , Ray Jui , Stefan Wahren , Stephen Warren , Florian Fainelli , Pramod KUMAR , Jonathan Bell References: <1457005303-26829-1-git-send-email-yendapally.reddy@broadcom.com> <1457005303-26829-3-git-send-email-yendapally.reddy@broadcom.com> Cc: Scott Branden , "linux-kernel@vger.kernel.org" , bcm-kernel-feedback-list , "linux-arm-kernel@lists.infradead.org" , "linux-gpio@vger.kernel.org" From: Ray Jui Message-ID: Date: Wed, 9 Mar 2016 13:07:31 -0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1490 Lines: 48 Hi Linus, On 3/9/2016 3:04 AM, Linus Walleij wrote: > On Thu, Mar 3, 2016 at 6:41 PM, Yendapally Reddy Dhananjaya Reddy > wrote: > >> This adds the initial support of the Broadcom NS2 pinmux driver >> >> Signed-off-by: Yendapally Reddy Dhananjaya Reddy > > Now yet another Broadcom driver! > > Can I get the help of the maintainers of the other > Broadcom drivers to say: > > - If this looks sane > I've reviewed this driver internally before it was sent out to public. Here's my reviewed-by: Reviewed-by: Ray Jui Like I mentioned before, we have been working closely with our ASIC team to have one pad per register based register layout in our next-gen SoC. That way we are able to use the generic pinctrl-single driver. It's definitely our best interest to re-use a generic driver when possible, to minimize the ongoing maintenance cost. Note we only have control on chips that we work on; there are other Broadcom SoCs that we do not have a say, unfortunately. > - If this hardware looks suspiciously similar to one of the > other drivers and should rather me a subdriver of one of > the existing ones The IOMUX block in NS2 seems different from that of Cygnus and NSP. It only supports group based mux configuration and does not have any mechanism to allow GPIO pin function override. It also supports per pad based pinconf functions. > > Yours, > Linus Walleij > Thanks, Ray