* [PATCH] sh-pfc: r8a7778: Remove outdated comments
@ 2013-04-17 10:27 Laurent Pinchart
2013-04-18 0:24 ` Kuninori Morimoto
2013-04-23 11:07 ` Laurent Pinchart
0 siblings, 2 replies; 3+ messages in thread
From: Laurent Pinchart @ 2013-04-17 10:27 UTC (permalink / raw)
To: linux-sh
Function GPIOs have been removed, remove comments that refer to them.
Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
---
drivers/pinctrl/sh-pfc/pfc-r8a7778.c | 26 +++++++++++++-------------
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/drivers/pinctrl/sh-pfc/pfc-r8a7778.c b/drivers/pinctrl/sh-pfc/pfc-r8a7778.c
index ddbd27b..e9cc040 100644
--- a/drivers/pinctrl/sh-pfc/pfc-r8a7778.c
+++ b/drivers/pinctrl/sh-pfc/pfc-r8a7778.c
@@ -376,19 +376,19 @@ enum {
AVS1_MARK,
AVS2_MARK,
- VI0_R0_C_MARK, /* see GPIO_FN_VI0_R0_A */
- VI0_R1_C_MARK, /* see GPIO_FN_VI0_R1_A */
- VI0_R2_C_MARK, /* see GPIO_FN_VI0_R2_A */
- /* VI0_R3_C_MARK, see GPIO_FN_VI0_R3_A */
- VI0_R4_C_MARK, /* see GPIO_FN_VI0_R4_A */
- VI0_R5_C_MARK, /* see GPIO_FN_VI0_R5_A */
-
- VI0_R0_D_MARK, /* see GPIO_FN_VI0_R0_B */
- VI0_R1_D_MARK, /* see GPIO_FN_VI0_R1_B */
- VI0_R2_D_MARK, /* see GPIO_FN_VI0_R2_B */
- VI0_R3_D_MARK, /* see GPIO_FN_VI0_R3_B */
- VI0_R4_D_MARK, /* see GPIO_FN_VI0_R4_B */
- VI0_R5_D_MARK, /* see GPIO_FN_VI0_R5_B */
+ VI0_R0_C_MARK,
+ VI0_R1_C_MARK,
+ VI0_R2_C_MARK,
+ /* VI0_R3_C_MARK, */
+ VI0_R4_C_MARK,
+ VI0_R5_C_MARK,
+
+ VI0_R0_D_MARK,
+ VI0_R1_D_MARK,
+ VI0_R2_D_MARK,
+ VI0_R3_D_MARK,
+ VI0_R4_D_MARK,
+ VI0_R5_D_MARK,
/* IPSR0 */
PRESETOUT_MARK, PWM1_MARK, AUDATA0_MARK,
--
Regards,
Laurent Pinchart
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] sh-pfc: r8a7778: Remove outdated comments
2013-04-17 10:27 [PATCH] sh-pfc: r8a7778: Remove outdated comments Laurent Pinchart
@ 2013-04-18 0:24 ` Kuninori Morimoto
2013-04-23 11:07 ` Laurent Pinchart
1 sibling, 0 replies; 3+ messages in thread
From: Kuninori Morimoto @ 2013-04-18 0:24 UTC (permalink / raw)
To: linux-sh
Hi Laurent
Thank you for your patch
At first, I'm sorry, these comments created confusion.
But, these are indicating caution of strange VI pin settings.
You can find same meaning comments on /* IPSR9 */ category, as /* see sel_vi0 */
So, How about this ?
/* see VI0_Rx_A_MARK */
or
/* see sel_vi0 */
> Function GPIOs have been removed, remove comments that refer to them.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> ---
> drivers/pinctrl/sh-pfc/pfc-r8a7778.c | 26 +++++++++++++-------------
> 1 file changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/pinctrl/sh-pfc/pfc-r8a7778.c b/drivers/pinctrl/sh-pfc/pfc-r8a7778.c
> index ddbd27b..e9cc040 100644
> --- a/drivers/pinctrl/sh-pfc/pfc-r8a7778.c
> +++ b/drivers/pinctrl/sh-pfc/pfc-r8a7778.c
> @@ -376,19 +376,19 @@ enum {
> AVS1_MARK,
> AVS2_MARK,
>
> - VI0_R0_C_MARK, /* see GPIO_FN_VI0_R0_A */
> - VI0_R1_C_MARK, /* see GPIO_FN_VI0_R1_A */
> - VI0_R2_C_MARK, /* see GPIO_FN_VI0_R2_A */
> - /* VI0_R3_C_MARK, see GPIO_FN_VI0_R3_A */
> - VI0_R4_C_MARK, /* see GPIO_FN_VI0_R4_A */
> - VI0_R5_C_MARK, /* see GPIO_FN_VI0_R5_A */
> -
> - VI0_R0_D_MARK, /* see GPIO_FN_VI0_R0_B */
> - VI0_R1_D_MARK, /* see GPIO_FN_VI0_R1_B */
> - VI0_R2_D_MARK, /* see GPIO_FN_VI0_R2_B */
> - VI0_R3_D_MARK, /* see GPIO_FN_VI0_R3_B */
> - VI0_R4_D_MARK, /* see GPIO_FN_VI0_R4_B */
> - VI0_R5_D_MARK, /* see GPIO_FN_VI0_R5_B */
> + VI0_R0_C_MARK,
> + VI0_R1_C_MARK,
> + VI0_R2_C_MARK,
> + /* VI0_R3_C_MARK, */
> + VI0_R4_C_MARK,
> + VI0_R5_C_MARK,
> +
> + VI0_R0_D_MARK,
> + VI0_R1_D_MARK,
> + VI0_R2_D_MARK,
> + VI0_R3_D_MARK,
> + VI0_R4_D_MARK,
> + VI0_R5_D_MARK,
>
> /* IPSR0 */
> PRESETOUT_MARK, PWM1_MARK, AUDATA0_MARK,
> --
> Regards,
>
> Laurent Pinchart
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
Best regards
---
Kuninori Morimoto
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] sh-pfc: r8a7778: Remove outdated comments
2013-04-17 10:27 [PATCH] sh-pfc: r8a7778: Remove outdated comments Laurent Pinchart
2013-04-18 0:24 ` Kuninori Morimoto
@ 2013-04-23 11:07 ` Laurent Pinchart
1 sibling, 0 replies; 3+ messages in thread
From: Laurent Pinchart @ 2013-04-23 11:07 UTC (permalink / raw)
To: linux-sh
Hi Morimoto-san,
On Wednesday 17 April 2013 17:24:49 Kuninori Morimoto wrote:
> Hi Laurent
>
> Thank you for your patch
> At first, I'm sorry, these comments created confusion.
> But, these are indicating caution of strange VI pin settings.
> You can find same meaning comments on /* IPSR9 */ category, as /* see
> sel_vi0 */ So, How about this ?
>
> /* see VI0_Rx_A_MARK */
>
> or
>
> /* see sel_vi0 */
That sounds good to me. I'll resubmit the patch.
> > Function GPIOs have been removed, remove comments that refer to them.
> >
> > Signed-off-by: Laurent Pinchart
> > <laurent.pinchart+renesas@ideasonboard.com>
> > ---
> >
> > drivers/pinctrl/sh-pfc/pfc-r8a7778.c | 26 +++++++++++++-------------
> > 1 file changed, 13 insertions(+), 13 deletions(-)
> >
> > diff --git a/drivers/pinctrl/sh-pfc/pfc-r8a7778.c
> > b/drivers/pinctrl/sh-pfc/pfc-r8a7778.c index ddbd27b..e9cc040 100644
> > --- a/drivers/pinctrl/sh-pfc/pfc-r8a7778.c
> > +++ b/drivers/pinctrl/sh-pfc/pfc-r8a7778.c
> > @@ -376,19 +376,19 @@ enum {
> >
> > AVS1_MARK,
> > AVS2_MARK,
> >
> > - VI0_R0_C_MARK, /* see GPIO_FN_VI0_R0_A */
> > - VI0_R1_C_MARK, /* see GPIO_FN_VI0_R1_A */
> > - VI0_R2_C_MARK, /* see GPIO_FN_VI0_R2_A */
> > - /* VI0_R3_C_MARK, see GPIO_FN_VI0_R3_A */
> > - VI0_R4_C_MARK, /* see GPIO_FN_VI0_R4_A */
> > - VI0_R5_C_MARK, /* see GPIO_FN_VI0_R5_A */
> > -
> > - VI0_R0_D_MARK, /* see GPIO_FN_VI0_R0_B */
> > - VI0_R1_D_MARK, /* see GPIO_FN_VI0_R1_B */
> > - VI0_R2_D_MARK, /* see GPIO_FN_VI0_R2_B */
> > - VI0_R3_D_MARK, /* see GPIO_FN_VI0_R3_B */
> > - VI0_R4_D_MARK, /* see GPIO_FN_VI0_R4_B */
> > - VI0_R5_D_MARK, /* see GPIO_FN_VI0_R5_B */
> > + VI0_R0_C_MARK,
> > + VI0_R1_C_MARK,
> > + VI0_R2_C_MARK,
> > + /* VI0_R3_C_MARK, */
> > + VI0_R4_C_MARK,
> > + VI0_R5_C_MARK,
> > +
> > + VI0_R0_D_MARK,
> > + VI0_R1_D_MARK,
> > + VI0_R2_D_MARK,
> > + VI0_R3_D_MARK,
> > + VI0_R4_D_MARK,
> > + VI0_R5_D_MARK,
> >
> > /* IPSR0 */
> > PRESETOUT_MARK, PWM1_MARK, AUDATA0_MARK,
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-04-23 11:07 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-17 10:27 [PATCH] sh-pfc: r8a7778: Remove outdated comments Laurent Pinchart
2013-04-18 0:24 ` Kuninori Morimoto
2013-04-23 11:07 ` Laurent Pinchart
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).