All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: spi: rewrites the name of the function.
@ 2016-11-07  1:00 ` Cao Minh Hiep
  0 siblings, 0 replies; 6+ messages in thread
From: Cao Minh Hiep @ 2016-11-07  1:00 UTC (permalink / raw)
  To: broonie, geert+renesas, linux-spi
  Cc: kuninori.morimoto.gx, yoshihiro.shimoda.uh, ryusuke.sakato.bx,
	linux-renesas-soc, nv-dung, h-inayoshi, cm-hiep

From: Hiep Cao Minh <cm-hiep@jinso.co.jp>

This patch rewrites the name of rspi_pio_transfer_in_or_out
function.

Signed-off-by: Hiep Cao Minh <cm-hiep@jinso.co.jp>
---
 drivers/spi/spi-rspi.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/spi/spi-rspi.c b/drivers/spi/spi-rspi.c
index fb12bc5..7d3a825 100644
--- a/drivers/spi/spi-rspi.c
+++ b/drivers/spi/spi-rspi.c
@@ -515,7 +515,7 @@ static int rspi_pio_transfer(struct rspi_data *rspi, const u8 *tx, u8 *rx,
 	return 0;
 }
 
-static int rspi_pio_transfer_in_or_our(struct rspi_data *rspi, const u8 *tx, 
+static int rspi_pio_transfer_in_or_out(struct rspi_data *rspi, const u8 *tx, 
 				       u8 *rx, unsigned int n)
 {
 	unsigned int i, len;
@@ -839,7 +839,7 @@ static int qspi_transfer_out(struct rspi_data *rspi, struct spi_transfer *xfer)
 			return ret;
 	}
 
-	ret = rspi_pio_transfer_in_or_our(rspi, xfer->tx_buf, NULL, xfer->len);
+	ret = rspi_pio_transfer_in_or_out(rspi, xfer->tx_buf, NULL, xfer->len);
 	if (ret < 0)
 		return ret;
 
@@ -857,7 +857,7 @@ static int qspi_transfer_in(struct rspi_data *rspi, struct spi_transfer *xfer)
 			return ret;
 	}
 
-	return rspi_pio_transfer_in_or_our(rspi, NULL, xfer->rx_buf, xfer->len);
+	return rspi_pio_transfer_in_or_out(rspi, NULL, xfer->rx_buf, xfer->len);
 }
 
 static int qspi_transfer_one(struct spi_master *master, struct spi_device *spi,
-- 
1.9.1

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

* [PATCH] ARM: spi: rewrites the name of the function.
@ 2016-11-07  1:00 ` Cao Minh Hiep
  0 siblings, 0 replies; 6+ messages in thread
From: Cao Minh Hiep @ 2016-11-07  1:00 UTC (permalink / raw)
  To: broonie-DgEjT+Ai2ygdnm+yROfE0A,
	geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ,
	linux-spi-u79uwXL29TY76Z2rM5mHXA
  Cc: kuninori.morimoto.gx-zM6kxYcvzFBBDgjK7y7TUQ,
	yoshihiro.shimoda.uh-zM6kxYcvzFBBDgjK7y7TUQ,
	ryusuke.sakato.bx-zM6kxYcvzFBBDgjK7y7TUQ,
	linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA,
	nv-dung-HEF513clHfp3+QwDJ9on6Q,
	h-inayoshi-HEF513clHfp3+QwDJ9on6Q,
	cm-hiep-HEF513clHfp3+QwDJ9on6Q

From: Hiep Cao Minh <cm-hiep-HEF513clHfp3+QwDJ9on6Q@public.gmane.org>

This patch rewrites the name of rspi_pio_transfer_in_or_out
function.

Signed-off-by: Hiep Cao Minh <cm-hiep-HEF513clHfp3+QwDJ9on6Q@public.gmane.org>
---
 drivers/spi/spi-rspi.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/spi/spi-rspi.c b/drivers/spi/spi-rspi.c
index fb12bc5..7d3a825 100644
--- a/drivers/spi/spi-rspi.c
+++ b/drivers/spi/spi-rspi.c
@@ -515,7 +515,7 @@ static int rspi_pio_transfer(struct rspi_data *rspi, const u8 *tx, u8 *rx,
 	return 0;
 }
 
-static int rspi_pio_transfer_in_or_our(struct rspi_data *rspi, const u8 *tx, 
+static int rspi_pio_transfer_in_or_out(struct rspi_data *rspi, const u8 *tx, 
 				       u8 *rx, unsigned int n)
 {
 	unsigned int i, len;
@@ -839,7 +839,7 @@ static int qspi_transfer_out(struct rspi_data *rspi, struct spi_transfer *xfer)
 			return ret;
 	}
 
-	ret = rspi_pio_transfer_in_or_our(rspi, xfer->tx_buf, NULL, xfer->len);
+	ret = rspi_pio_transfer_in_or_out(rspi, xfer->tx_buf, NULL, xfer->len);
 	if (ret < 0)
 		return ret;
 
@@ -857,7 +857,7 @@ static int qspi_transfer_in(struct rspi_data *rspi, struct spi_transfer *xfer)
 			return ret;
 	}
 
-	return rspi_pio_transfer_in_or_our(rspi, NULL, xfer->rx_buf, xfer->len);
+	return rspi_pio_transfer_in_or_out(rspi, NULL, xfer->rx_buf, xfer->len);
 }
 
 static int qspi_transfer_one(struct spi_master *master, struct spi_device *spi,
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* Re: [PATCH] ARM: spi: rewrites the name of the function.
@ 2016-11-07 10:25   ` Geert Uytterhoeven
  0 siblings, 0 replies; 6+ messages in thread
From: Geert Uytterhoeven @ 2016-11-07 10:25 UTC (permalink / raw)
  To: Cao Minh Hiep
  Cc: Mark Brown, Geert Uytterhoeven, linux-spi, Kuninori Morimoto,
	Yoshihiro Shimoda, Ryusuke Sakato, Linux-Renesas,
	Dung:人ソ, 稲吉

Hi Hiep,

On Mon, Nov 7, 2016 at 2:00 AM, Cao Minh Hiep <cm-hiep@jinso.co.jp> wrote:
> From: Hiep Cao Minh <cm-hiep@jinso.co.jp>
>
> This patch rewrites the name of rspi_pio_transfer_in_or_out
> function.
>
> Signed-off-by: Hiep Cao Minh <cm-hiep@jinso.co.jp>

Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>

But you want to use proper prefixes matching the style of the subsystem,
to avoid Mark's canned message reply system.

Always use e.g. "git log --oneline drivers/spi/spi-rspi.c" to find out...

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

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

* Re: [PATCH] ARM: spi: rewrites the name of the function.
@ 2016-11-07 10:25   ` Geert Uytterhoeven
  0 siblings, 0 replies; 6+ messages in thread
From: Geert Uytterhoeven @ 2016-11-07 10:25 UTC (permalink / raw)
  To: Cao Minh Hiep
  Cc: Mark Brown, Geert Uytterhoeven, linux-spi, Kuninori Morimoto,
	Yoshihiro Shimoda, Ryusuke Sakato, Linux-Renesas,
	Dung:人ソ, 稲吉

Hi Hiep,

On Mon, Nov 7, 2016 at 2:00 AM, Cao Minh Hiep <cm-hiep-HEF513clHfp3+QwDJ9on6Q@public.gmane.org> wrote:
> From: Hiep Cao Minh <cm-hiep-HEF513clHfp3+QwDJ9on6Q@public.gmane.org>
>
> This patch rewrites the name of rspi_pio_transfer_in_or_out
> function.
>
> Signed-off-by: Hiep Cao Minh <cm-hiep-HEF513clHfp3+QwDJ9on6Q@public.gmane.org>

Acked-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>

But you want to use proper prefixes matching the style of the subsystem,
to avoid Mark's canned message reply system.

Always use e.g. "git log --oneline drivers/spi/spi-rspi.c" to find out...

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* Re: [PATCH] ARM: spi: rewrites the name of the function.
@ 2016-11-08  0:17     ` Hiep Cao Minh
  0 siblings, 0 replies; 6+ messages in thread
From: Hiep Cao Minh @ 2016-11-08  0:17 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Mark Brown, Geert Uytterhoeven, linux-spi, Kuninori Morimoto,
	Yoshihiro Shimoda, Ryusuke Sakato, Linux-Renesas,
	Dung:人ソ, 稲吉

Hi Geert,

Thanks for your comment.

On 11/07/2016 07:25 PM, Geert Uytterhoeven wrote:
> Hi Hiep,
>
> On Mon, Nov 7, 2016 at 2:00 AM, Cao Minh Hiep <cm-hiep@jinso.co.jp> wrote:
>> From: Hiep Cao Minh <cm-hiep@jinso.co.jp>
>>
>> This patch rewrites the name of rspi_pio_transfer_in_or_out
>> function.
>>
>> Signed-off-by: Hiep Cao Minh <cm-hiep@jinso.co.jp>
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
>
> But you want to use proper prefixes matching the style of the subsystem,
> to avoid Mark's canned message reply system.
>
> Always use e.g. "git log --oneline drivers/spi/spi-rspi.c" to find out...
Thanks, I will make matching the style of the subsystem.

Jinzai Solution.
Hiep.

> Gr{oetje,eeting}s,
>
>                          Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                  -- Linus Torvalds
>

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

* Re: [PATCH] ARM: spi: rewrites the name of the function.
@ 2016-11-08  0:17     ` Hiep Cao Minh
  0 siblings, 0 replies; 6+ messages in thread
From: Hiep Cao Minh @ 2016-11-08  0:17 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Mark Brown, Geert Uytterhoeven, linux-spi, Kuninori Morimoto,
	Yoshihiro Shimoda, Ryusuke Sakato, Linux-Renesas,
	Dung:人ソ, 稲吉

Hi Geert,

Thanks for your comment.

On 11/07/2016 07:25 PM, Geert Uytterhoeven wrote:
> Hi Hiep,
>
> On Mon, Nov 7, 2016 at 2:00 AM, Cao Minh Hiep <cm-hiep-HEF513clHfp3+QwDJ9on6Q@public.gmane.org> wrote:
>> From: Hiep Cao Minh <cm-hiep-HEF513clHfp3+QwDJ9on6Q@public.gmane.org>
>>
>> This patch rewrites the name of rspi_pio_transfer_in_or_out
>> function.
>>
>> Signed-off-by: Hiep Cao Minh <cm-hiep-HEF513clHfp3+QwDJ9on6Q@public.gmane.org>
> Acked-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
>
> But you want to use proper prefixes matching the style of the subsystem,
> to avoid Mark's canned message reply system.
>
> Always use e.g. "git log --oneline drivers/spi/spi-rspi.c" to find out...
Thanks, I will make matching the style of the subsystem.

Jinzai Solution.
Hiep.

> Gr{oetje,eeting}s,
>
>                          Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                  -- Linus Torvalds
>

--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

end of thread, other threads:[~2016-11-08  2:06 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-07  1:00 [PATCH] ARM: spi: rewrites the name of the function Cao Minh Hiep
2016-11-07  1:00 ` Cao Minh Hiep
2016-11-07 10:25 ` Geert Uytterhoeven
2016-11-07 10:25   ` Geert Uytterhoeven
2016-11-08  0:17   ` Hiep Cao Minh
2016-11-08  0:17     ` Hiep Cao Minh

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.