linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] pinctrl: at91: correct a few typos
@ 2013-10-16 14:12 Alexandre Belloni
  2013-10-16 14:14 ` Nicolas Ferre
  2013-10-23  9:08 ` Linus Walleij
  0 siblings, 2 replies; 3+ messages in thread
From: Alexandre Belloni @ 2013-10-16 14:12 UTC (permalink / raw)
  To: Jean-Christophe Plagniol-Villard
  Cc: Nicolas Ferre, linux-kernel, linux-arm-kernel, Linus Walleij,
	Alexandre Belloni

Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
 drivers/pinctrl/pinctrl-at91.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/pinctrl/pinctrl-at91.c b/drivers/pinctrl/pinctrl-at91.c
index f350fd2..ad48179 100644
--- a/drivers/pinctrl/pinctrl-at91.c
+++ b/drivers/pinctrl/pinctrl-at91.c
@@ -243,7 +243,7 @@ static int at91_dt_node_to_map(struct pinctrl_dev *pctldev,
 	int i;
 
 	/*
-	 * first find the group of this node and check if we need create
+	 * first find the group of this node and check if we need to create
 	 * config maps for pins
 	 */
 	grp = at91_pinctrl_find_group_by_name(info, np->name);
@@ -564,7 +564,7 @@ static int at91_pmx_enable(struct pinctrl_dev *pctldev, unsigned selector,
 		info->functions[selector].name, info->groups[group].name);
 
 	/* first check that all the pins of the group are valid with a valid
-	 * paramter */
+	 * parameter */
 	for (i = 0; i < npins; i++) {
 		pin = &pins_conf[i];
 		ret = pin_check_config(info, info->groups[group].name, i, pin);
@@ -958,7 +958,7 @@ static int at91_pinctrl_probe_dt(struct platform_device *pdev,
 	at91_pinctrl_child_count(info, np);
 
 	if (info->nbanks < 1) {
-		dev_err(&pdev->dev, "you need to specify atleast one gpio-controller\n");
+		dev_err(&pdev->dev, "you need to specify at least one gpio-controller\n");
 		return -EINVAL;
 	}
 
@@ -1495,7 +1495,7 @@ static int at91_gpio_of_irq_setup(struct device_node *node,
 	if (at91_gpio->pioc_idx)
 		prev = gpio_chips[at91_gpio->pioc_idx - 1];
 
-	/* The toplevel handler handles one bank of GPIOs, except
+	/* The top level handler handles one bank of GPIOs, except
 	 * on some SoC it can handles up to three...
 	 * We only set up the handler for the first of the list.
 	 */
-- 
1.8.1.2


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] pinctrl: at91: correct a few typos
  2013-10-16 14:12 [PATCH] pinctrl: at91: correct a few typos Alexandre Belloni
@ 2013-10-16 14:14 ` Nicolas Ferre
  2013-10-23  9:08 ` Linus Walleij
  1 sibling, 0 replies; 3+ messages in thread
From: Nicolas Ferre @ 2013-10-16 14:14 UTC (permalink / raw)
  To: Alexandre Belloni, Jean-Christophe Plagniol-Villard, Linus Walleij
  Cc: linux-kernel, linux-arm-kernel

On 16/10/2013 16:12, Alexandre Belloni :
> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>

Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>

Thanks.

> ---
>   drivers/pinctrl/pinctrl-at91.c | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/pinctrl/pinctrl-at91.c b/drivers/pinctrl/pinctrl-at91.c
> index f350fd2..ad48179 100644
> --- a/drivers/pinctrl/pinctrl-at91.c
> +++ b/drivers/pinctrl/pinctrl-at91.c
> @@ -243,7 +243,7 @@ static int at91_dt_node_to_map(struct pinctrl_dev *pctldev,
>   	int i;
>
>   	/*
> -	 * first find the group of this node and check if we need create
> +	 * first find the group of this node and check if we need to create
>   	 * config maps for pins
>   	 */
>   	grp = at91_pinctrl_find_group_by_name(info, np->name);
> @@ -564,7 +564,7 @@ static int at91_pmx_enable(struct pinctrl_dev *pctldev, unsigned selector,
>   		info->functions[selector].name, info->groups[group].name);
>
>   	/* first check that all the pins of the group are valid with a valid
> -	 * paramter */
> +	 * parameter */
>   	for (i = 0; i < npins; i++) {
>   		pin = &pins_conf[i];
>   		ret = pin_check_config(info, info->groups[group].name, i, pin);
> @@ -958,7 +958,7 @@ static int at91_pinctrl_probe_dt(struct platform_device *pdev,
>   	at91_pinctrl_child_count(info, np);
>
>   	if (info->nbanks < 1) {
> -		dev_err(&pdev->dev, "you need to specify atleast one gpio-controller\n");
> +		dev_err(&pdev->dev, "you need to specify at least one gpio-controller\n");
>   		return -EINVAL;
>   	}
>
> @@ -1495,7 +1495,7 @@ static int at91_gpio_of_irq_setup(struct device_node *node,
>   	if (at91_gpio->pioc_idx)
>   		prev = gpio_chips[at91_gpio->pioc_idx - 1];
>
> -	/* The toplevel handler handles one bank of GPIOs, except
> +	/* The top level handler handles one bank of GPIOs, except
>   	 * on some SoC it can handles up to three...
>   	 * We only set up the handler for the first of the list.
>   	 */
>


-- 
Nicolas Ferre

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] pinctrl: at91: correct a few typos
  2013-10-16 14:12 [PATCH] pinctrl: at91: correct a few typos Alexandre Belloni
  2013-10-16 14:14 ` Nicolas Ferre
@ 2013-10-23  9:08 ` Linus Walleij
  1 sibling, 0 replies; 3+ messages in thread
From: Linus Walleij @ 2013-10-23  9:08 UTC (permalink / raw)
  To: Alexandre Belloni
  Cc: Jean-Christophe Plagniol-Villard, Nicolas Ferre, linux-kernel,
	linux-arm-kernel

On Wed, Oct 16, 2013 at 4:12 PM, Alexandre Belloni
<alexandre.belloni@free-electrons.com> wrote:

> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>

Patch applied with Nicolas' ACK.

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-10-23  9:08 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-10-16 14:12 [PATCH] pinctrl: at91: correct a few typos Alexandre Belloni
2013-10-16 14:14 ` Nicolas Ferre
2013-10-23  9:08 ` Linus Walleij

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).