From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Bresticker Subject: Re: [PATCH] pinctrl: pistachio: fix mfio84-89 function description and pinmux. Date: Fri, 4 Mar 2016 10:48:33 -0800 Message-ID: References: <1457105302-15070-1-git-send-email-Govindraj.Raja@imgtec.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <1457105302-15070-1-git-send-email-Govindraj.Raja@imgtec.com> Sender: stable-owner@vger.kernel.org To: Govindraj Raja Cc: "linux-gpio@vger.kernel.org" , Linus Walleij , "devicetree@vger.kernel.org" , Rob Herring , Linux-MIPS , James Hartley , "Stable kernel (v4.1)" List-Id: devicetree@vger.kernel.org Govindraj, On Fri, Mar 4, 2016 at 7:28 AM, Govindraj Raja wrote: > mfio 84 to 89 are described wrongly, fix it to describe > the right pin and add them to right pin-mux group. > > The correct order is: > pll1_lock => mips_pll -- MFIO_83 > pll2_lock => audio_pll -- MFIO_84 > pll3_lock => rpu_v_pll -- MFIO_85 > pll4_lock => rpu_l_pll -- MFIO_86 > pll5_lock => sys_pll -- MFIO_87 > pll6_lock => wifi_pll -- MFIO_88 > pll7_lock => bt_pll -- MFIO_89 > > Fixes: cefc03e5995e("pinctrl: Add Pistachio SoC pin control driver") > Signed-off-by: Govindraj Raja > Cc: linux-gpio@vger.kernel.org > Cc: devicetree@vger.kernel.org > Cc: Rob Herring > Cc: Linus Walleij > Cc: Andrew Bresticker > Cc: linux-mips@linux-mips.org > Cc: James Hartley > Cc: # v4.2+ Acked-by: Andrew Bresticker > Do I need to split this patch into dt & pinctrl? > Or can it be picked up through pinctrl subsystem with dt maintainers Ack? I would think that since this is a correction to the existing binding/driver that shouldn't be necessary, but that's up to Linus. > > .../bindings/pinctrl/img,pistachio-pinctrl.txt | 12 +++++------ > drivers/pinctrl/pinctrl-pistachio.c | 24 +++++++++++----------- > 2 files changed, 18 insertions(+), 18 deletions(-) > > diff --git a/Documentation/devicetree/bindings/pinctrl/img,pistachio-pinctrl.txt b/Documentation/devicetree/bindings/pinctrl/img,pistachio-pinctrl.txt > index 08a4a32..0326154 100644 > --- a/Documentation/devicetree/bindings/pinctrl/img,pistachio-pinctrl.txt > +++ b/Documentation/devicetree/bindings/pinctrl/img,pistachio-pinctrl.txt > @@ -134,12 +134,12 @@ mfio80 ddr_debug, mips_trace_data, mips_debug > mfio81 dreq0, mips_trace_data, eth_debug > mfio82 dreq1, mips_trace_data, eth_debug > mfio83 mips_pll_lock, mips_trace_data, usb_debug > -mfio84 sys_pll_lock, mips_trace_data, usb_debug > -mfio85 wifi_pll_lock, mips_trace_data, sdhost_debug > -mfio86 bt_pll_lock, mips_trace_data, sdhost_debug > -mfio87 rpu_v_pll_lock, dreq2, socif_debug > -mfio88 rpu_l_pll_lock, dreq3, socif_debug > -mfio89 audio_pll_lock, dreq4, dreq5 > +mfio84 audio_pll_lock, mips_trace_data, usb_debug > +mfio85 rpu_v_pll_lock, mips_trace_data, sdhost_debug > +mfio86 rpu_l_pll_lock, mips_trace_data, sdhost_debug > +mfio87 sys_pll_lock, dreq2, socif_debug > +mfio88 wifi_pll_lock, dreq3, socif_debug > +mfio89 bt_pll_lock, dreq4, dreq5 > tck > trstn > tdi > diff --git a/drivers/pinctrl/pinctrl-pistachio.c b/drivers/pinctrl/pinctrl-pistachio.c > index 856f736..2673cd9 100644 > --- a/drivers/pinctrl/pinctrl-pistachio.c > +++ b/drivers/pinctrl/pinctrl-pistachio.c > @@ -469,27 +469,27 @@ static const char * const pistachio_mips_pll_lock_groups[] = { > "mfio83", > }; > > -static const char * const pistachio_sys_pll_lock_groups[] = { > +static const char * const pistachio_audio_pll_lock_groups[] = { > "mfio84", > }; > > -static const char * const pistachio_wifi_pll_lock_groups[] = { > +static const char * const pistachio_rpu_v_pll_lock_groups[] = { > "mfio85", > }; > > -static const char * const pistachio_bt_pll_lock_groups[] = { > +static const char * const pistachio_rpu_l_pll_lock_groups[] = { > "mfio86", > }; > > -static const char * const pistachio_rpu_v_pll_lock_groups[] = { > +static const char * const pistachio_sys_pll_lock_groups[] = { > "mfio87", > }; > > -static const char * const pistachio_rpu_l_pll_lock_groups[] = { > +static const char * const pistachio_wifi_pll_lock_groups[] = { > "mfio88", > }; > > -static const char * const pistachio_audio_pll_lock_groups[] = { > +static const char * const pistachio_bt_pll_lock_groups[] = { > "mfio89", > }; > > @@ -559,12 +559,12 @@ enum pistachio_mux_option { > PISTACHIO_FUNCTION_DREQ4, > PISTACHIO_FUNCTION_DREQ5, > PISTACHIO_FUNCTION_MIPS_PLL_LOCK, > + PISTACHIO_FUNCTION_AUDIO_PLL_LOCK, > + PISTACHIO_FUNCTION_RPU_V_PLL_LOCK, > + PISTACHIO_FUNCTION_RPU_L_PLL_LOCK, > PISTACHIO_FUNCTION_SYS_PLL_LOCK, > PISTACHIO_FUNCTION_WIFI_PLL_LOCK, > PISTACHIO_FUNCTION_BT_PLL_LOCK, > - PISTACHIO_FUNCTION_RPU_V_PLL_LOCK, > - PISTACHIO_FUNCTION_RPU_L_PLL_LOCK, > - PISTACHIO_FUNCTION_AUDIO_PLL_LOCK, > PISTACHIO_FUNCTION_DEBUG_RAW_CCA_IND, > PISTACHIO_FUNCTION_DEBUG_ED_SEC20_CCA_IND, > PISTACHIO_FUNCTION_DEBUG_ED_SEC40_CCA_IND, > @@ -620,12 +620,12 @@ static const struct pistachio_function pistachio_functions[] = { > FUNCTION(dreq4), > FUNCTION(dreq5), > FUNCTION(mips_pll_lock), > + FUNCTION(audio_pll_lock), > + FUNCTION(rpu_v_pll_lock), > + FUNCTION(rpu_l_pll_lock), > FUNCTION(sys_pll_lock), > FUNCTION(wifi_pll_lock), > FUNCTION(bt_pll_lock), > - FUNCTION(rpu_v_pll_lock), > - FUNCTION(rpu_l_pll_lock), > - FUNCTION(audio_pll_lock), > FUNCTION(debug_raw_cca_ind), > FUNCTION(debug_ed_sec20_cca_ind), > FUNCTION(debug_ed_sec40_cca_ind), > -- > 2.5.0 >