From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from hqemgate14.nvidia.com ([216.228.121.143]:3772 "EHLO hqemgate14.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755869Ab3HFMLv (ORCPT ); Tue, 6 Aug 2013 08:11:51 -0400 Message-ID: <5200EB2C.9040103@nvidia.com> Date: Tue, 6 Aug 2013 17:55:16 +0530 From: Laxman Dewangan MIME-Version: 1.0 Subject: Re: [PATCH V2 1/3] pinctrl: add utility functions for add map/configs References: <1375688014-6117-1-git-send-email-ldewangan@nvidia.com> <1375688014-6117-2-git-send-email-ldewangan@nvidia.com> <52001B32.7070107@wwwdotorg.org> In-Reply-To: <52001B32.7070107@wwwdotorg.org> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Sender: devicetree-owner@vger.kernel.org To: Stephen Warren Cc: "linus.walleij@linaro.org" , "ian.campbell@citrix.com" , "rob.herring@calxeda.com" , "pawel.moll@arm.com" , "mark.rutland@arm.com" , "rob@landley.net" , "sameo@linux.intel.com" , "lee.jones@linaro.org" , "grant.likely@linaro.org" , "broonie@kernel.org" , "devicetree@vger.kernel.org" , "linux-doc@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "gg@slimlogic.co.uk" , "kishon@ti.com" List-ID: On Tuesday 06 August 2013 03:07 AM, Stephen Warren wrote: > On 08/05/2013 01:33 AM, Laxman Dewangan wrote: >> Some of pincontrol driver needs the utility function to create map >> list. The utility function needed for adding mux, configs etc. >> >> In place of duplicating this in each driver, add the common utility >> function in common file and use from device specific driver. This will >> reduce the duplicating of code across drivers. > This patch, > Reviewed-by: Stephen Warren Thanks for review. > > Some more documentation mnight be useful though, now these are common > utility functions rather than just some internal functions in the Tegra > pinctrl driver. > > Hopefully there will be a patch to convert at least the Tegra pinctrl > driver to use these new functions. Yes, I will have the patches for Tegra also to use these common utils functions.