Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752821AbbK3RFp (ORCPT ); Mon, 30 Nov 2015 12:05:45 -0500 Received: from mail-gw3-out.broadcom.com ([216.31.210.64]:29027 "EHLO mail-gw3-out.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751484AbbK3RFn (ORCPT ); Mon, 30 Nov 2015 12:05:43 -0500 X-IronPort-AV: E=Sophos;i="5.20,365,1444719600"; d="scan'208";a="81735206" Subject: Re: [PATCH 2/3] pinctrl: nsp: add pinmux driver support for Broadcom NSP SoC To: Linus Walleij , "Yendapally Reddy Dhananjaya Reddy" , Stephen Warren References: <1448042245-25702-1-git-send-email-yrdreddy@broadcom.com> CC: Scott Branden , Jon Mason , "linux-kernel@vger.kernel.org" , "linux-gpio@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , bcm-kernel-feedback-list From: Ray Jui Message-ID: <565C81E4.90507@broadcom.com> Date: Mon, 30 Nov 2015 09:05:40 -0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.4.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: 1169 Lines: 37 Hi Linus, On 11/30/2015 4:49 AM, Linus Walleij wrote: > On Fri, Nov 20, 2015 at 6:57 PM, Yendapally Reddy Dhananjaya Reddy > wrote: > >> This adds the initial support of the Broadcom NSP pinmux driver. >> >> Signed-off-by: Yendapally Reddy Dhananjaya Reddy > > So what does Ray and Stephen say about this? > > Is the hardware unique enough to warrant its own driver? > > Or should it utilize/extend one of the existing Broadcom > drivers? Currently, the IOMUX configurations in Cygnus, NSP, and NS2 are completely different and require a separate driver to each of these SoCs. We now realize the shortcomings and have been communicating with our ASIC team. We'd like to fix this for the next-gen of our SoC. We are driving towards using a register layout that will allow us to use just "pinctrl-single" and "pinconf-single." > > Yours, > Linus Walleij > Thanks, Ray -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/