linux-mips.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Add Ingenic X1000 serial support.
@ 2019-01-28  9:19 Zhou Yanjie
  2019-01-28  9:19 ` [PATCH 1/2] Serial: Ingenic: Add support for the X1000 Zhou Yanjie
                   ` (3 more replies)
  0 siblings, 4 replies; 17+ messages in thread
From: Zhou Yanjie @ 2019-01-28  9:19 UTC (permalink / raw)
  To: linux-mips
  Cc: linux-kernel, linux-serial, devicetree, robh+dt, paul.burton,
	gregkh, jslaby, mark.rutland, syq, jiaxun.yang, 772753199

Add Ingenic X1000 serial support.



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

* [PATCH 1/2] Serial: Ingenic: Add support for the X1000.
  2019-01-28  9:19 Add Ingenic X1000 serial support Zhou Yanjie
@ 2019-01-28  9:19 ` Zhou Yanjie
  2019-01-28  9:30   ` Greg KH
  2019-01-28 10:53   ` Geert Uytterhoeven
  2019-01-28  9:19 ` [PATCH 2/2] " Zhou Yanjie
                   ` (2 subsequent siblings)
  3 siblings, 2 replies; 17+ messages in thread
From: Zhou Yanjie @ 2019-01-28  9:19 UTC (permalink / raw)
  To: linux-mips
  Cc: linux-kernel, linux-serial, devicetree, robh+dt, paul.burton,
	gregkh, jslaby, mark.rutland, syq, jiaxun.yang, 772753199

Add support for probing the 8250_ingenic driver on the
X1000 Soc from Ingenic.

Signed-off-by: Zhou Yanjie <zhouyanjie@zoho.com>
---
 drivers/tty/serial/8250/8250_ingenic.c | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/drivers/tty/serial/8250/8250_ingenic.c b/drivers/tty/serial/8250/8250_ingenic.c
index 15a8c8d..1999e3b 100644
--- a/drivers/tty/serial/8250/8250_ingenic.c
+++ b/drivers/tty/serial/8250/8250_ingenic.c
@@ -145,6 +145,10 @@ EARLYCON_DECLARE(jz4780_uart, ingenic_early_console_setup);
 OF_EARLYCON_DECLARE(jz4780_uart, "ingenic,jz4780-uart",
 		    ingenic_early_console_setup);
 
+EARLYCON_DECLARE(x1000_uart, ingenic_early_console_setup);
+OF_EARLYCON_DECLARE(x1000_uart, "ingenic,x1000-uart",
+		    ingenic_early_console_setup);
+
 static void ingenic_uart_serial_out(struct uart_port *p, int offset, int value)
 {
 	int ier;
@@ -328,12 +332,18 @@ static const struct ingenic_uart_config jz4780_uart_config = {
 	.fifosize = 64,
 };
 
+static const struct ingenic_uart_config x1000_uart_config = {
+	.tx_loadsz = 32,
+	.fifosize = 64,
+};
+
 static const struct of_device_id of_match[] = {
 	{ .compatible = "ingenic,jz4740-uart", .data = &jz4740_uart_config },
 	{ .compatible = "ingenic,jz4760-uart", .data = &jz4760_uart_config },
 	{ .compatible = "ingenic,jz4770-uart", .data = &jz4760_uart_config },
 	{ .compatible = "ingenic,jz4775-uart", .data = &jz4760_uart_config },
 	{ .compatible = "ingenic,jz4780-uart", .data = &jz4780_uart_config },
+	{ .compatible = "ingenic,x1000-uart", .data = &x1000_uart_config },
 	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, of_match);
-- 
2.7.4



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

* [PATCH 2/2] Serial: Ingenic: Add support for the X1000.
  2019-01-28  9:19 Add Ingenic X1000 serial support Zhou Yanjie
  2019-01-28  9:19 ` [PATCH 1/2] Serial: Ingenic: Add support for the X1000 Zhou Yanjie
@ 2019-01-28  9:19 ` Zhou Yanjie
  2019-01-28 13:56 ` Serial: Ingenic: Add X1000 suppor for the UART driver Zhou Yanjie
  2019-01-28 17:03 ` Add Ingenic X1000 serial support Zhou Yanjie
  3 siblings, 0 replies; 17+ messages in thread
From: Zhou Yanjie @ 2019-01-28  9:19 UTC (permalink / raw)
  To: linux-mips
  Cc: linux-kernel, linux-serial, devicetree, robh+dt, paul.burton,
	gregkh, jslaby, mark.rutland, syq, jiaxun.yang, 772753199

Add support for probing the 8250_ingenic driver on the
X1000 Soc from Ingenic.

Signed-off-by: Zhou Yanjie <zhouyanjie@zoho.com>
---
 Documentation/devicetree/bindings/serial/ingenic,uart.txt | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/serial/ingenic,uart.txt b/Documentation/devicetree/bindings/serial/ingenic,uart.txt
index c3c6406..24ed876 100644
--- a/Documentation/devicetree/bindings/serial/ingenic,uart.txt
+++ b/Documentation/devicetree/bindings/serial/ingenic,uart.txt
@@ -6,7 +6,8 @@ Required properties:
   - "ingenic,jz4760-uart",
   - "ingenic,jz4770-uart",
   - "ingenic,jz4775-uart",
-  - "ingenic,jz4780-uart".
+  - "ingenic,jz4780-uart",
+  - "ingenic,x1000-uart".
 - reg : offset and length of the register set for the device.
 - interrupts : should contain uart interrupt.
 - clocks : phandles to the module & baud clocks.
-- 
2.7.4



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

* Re: [PATCH 1/2] Serial: Ingenic: Add support for the X1000.
  2019-01-28  9:19 ` [PATCH 1/2] Serial: Ingenic: Add support for the X1000 Zhou Yanjie
@ 2019-01-28  9:30   ` Greg KH
  2019-01-28 12:44     ` Zhou Yanjie
  2019-01-28 10:53   ` Geert Uytterhoeven
  1 sibling, 1 reply; 17+ messages in thread
From: Greg KH @ 2019-01-28  9:30 UTC (permalink / raw)
  To: Zhou Yanjie
  Cc: linux-mips, linux-kernel, linux-serial, devicetree, robh+dt,
	paul.burton, jslaby, mark.rutland, syq, jiaxun.yang, 772753199

On Mon, Jan 28, 2019 at 05:19:35PM +0800, Zhou Yanjie wrote:
> Add support for probing the 8250_ingenic driver on the
> X1000 Soc from Ingenic.
> 
> Signed-off-by: Zhou Yanjie <zhouyanjie@zoho.com>
> ---
>  drivers/tty/serial/8250/8250_ingenic.c | 10 ++++++++++
>  1 file changed, 10 insertions(+)

You sent two different patches with the same subject: line, yet they did
totally different things.

Please fix this up and resend with a better set of subject lines.

thanks,

greg k-h

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

* Re: [PATCH 1/2] Serial: Ingenic: Add support for the X1000.
  2019-01-28  9:19 ` [PATCH 1/2] Serial: Ingenic: Add support for the X1000 Zhou Yanjie
  2019-01-28  9:30   ` Greg KH
@ 2019-01-28 10:53   ` Geert Uytterhoeven
  1 sibling, 0 replies; 17+ messages in thread
From: Geert Uytterhoeven @ 2019-01-28 10:53 UTC (permalink / raw)
  To: Zhou Yanjie
  Cc: linux-mips, Linux Kernel Mailing List, open list:SERIAL DRIVERS,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	Rob Herring, Paul Burton, Greg KH, Jiri Slaby, Mark Rutland, syq,
	jiaxun.yang, 772753199

Hi Zhou,

On Mon, Jan 28, 2019 at 10:23 AM Zhou Yanjie <zhouyanjie@zoho.com> wrote:
> Add support for probing the 8250_ingenic driver on the
> X1000 Soc from Ingenic.
>
> Signed-off-by: Zhou Yanjie <zhouyanjie@zoho.com>

Thanks for your patch!

> --- a/drivers/tty/serial/8250/8250_ingenic.c
> +++ b/drivers/tty/serial/8250/8250_ingenic.c
> @@ -145,6 +145,10 @@ EARLYCON_DECLARE(jz4780_uart, ingenic_early_console_setup);
>  OF_EARLYCON_DECLARE(jz4780_uart, "ingenic,jz4780-uart",
>                     ingenic_early_console_setup);
>
> +EARLYCON_DECLARE(x1000_uart, ingenic_early_console_setup);
> +OF_EARLYCON_DECLARE(x1000_uart, "ingenic,x1000-uart",
> +                   ingenic_early_console_setup);

As of commit 2eaa790989e03900 ("earlycon: Use common framework for
earlycon declarations") it is no longer needer to specify both
EARLYCON_DECLARE() and OF_EARLYCON_DECLARE().

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] 17+ messages in thread

* Re: [PATCH 1/2] Serial: Ingenic: Add support for the X1000.
  2019-01-28  9:30   ` Greg KH
@ 2019-01-28 12:44     ` Zhou Yanjie
  0 siblings, 0 replies; 17+ messages in thread
From: Zhou Yanjie @ 2019-01-28 12:44 UTC (permalink / raw)
  To: Greg KH
  Cc: linux-mips, linux-kernel, linux-serial, devicetree, robh+dt,
	paul.burton, jslaby, mark.rutland, syq, jiaxun.yang, 772753199

My fault, I will fix these in v2.

On 2019年01月28日 17:30, Greg KH wrote:
> On Mon, Jan 28, 2019 at 05:19:35PM +0800, Zhou Yanjie wrote:
>> Add support for probing the 8250_ingenic driver on the
>> X1000 Soc from Ingenic.
>>
>> Signed-off-by: Zhou Yanjie <zhouyanjie@zoho.com>
>> ---
>>   drivers/tty/serial/8250/8250_ingenic.c | 10 ++++++++++
>>   1 file changed, 10 insertions(+)
> You sent two different patches with the same subject: line, yet they did
> totally different things.
>
> Please fix this up and resend with a better set of subject lines.
>
> thanks,
>
> greg k-h




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

* Serial: Ingenic: Add X1000 suppor for the UART driver.
  2019-01-28  9:19 Add Ingenic X1000 serial support Zhou Yanjie
  2019-01-28  9:19 ` [PATCH 1/2] Serial: Ingenic: Add support for the X1000 Zhou Yanjie
  2019-01-28  9:19 ` [PATCH 2/2] " Zhou Yanjie
@ 2019-01-28 13:56 ` Zhou Yanjie
  2019-01-28 13:57   ` [PATCH v2 1/2] " Zhou Yanjie
  2019-01-28 13:57   ` [PATCH v2 2/2] " Zhou Yanjie
  2019-01-28 17:03 ` Add Ingenic X1000 serial support Zhou Yanjie
  3 siblings, 2 replies; 17+ messages in thread
From: Zhou Yanjie @ 2019-01-28 13:56 UTC (permalink / raw)
  To: linux-mips
  Cc: linux-kernel, linux-serial, devicetree, robh+dt, paul.burton,
	gregkh, jslaby, mark.rutland, syq, jiaxun.yang, 772753199

v1->v2: Remove unnecessary "EARLYCON_DECLARE".



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

* [PATCH v2 1/2] Serial: Ingenic: Add X1000 suppor for the UART driver.
  2019-01-28 13:56 ` Serial: Ingenic: Add X1000 suppor for the UART driver Zhou Yanjie
@ 2019-01-28 13:57   ` Zhou Yanjie
  2019-01-28 16:22     ` Greg KH
  2019-01-28 13:57   ` [PATCH v2 2/2] " Zhou Yanjie
  1 sibling, 1 reply; 17+ messages in thread
From: Zhou Yanjie @ 2019-01-28 13:57 UTC (permalink / raw)
  To: linux-mips
  Cc: linux-kernel, linux-serial, devicetree, robh+dt, paul.burton,
	gregkh, jslaby, mark.rutland, syq, jiaxun.yang, 772753199

Add support for probing the 8250_ingenic driver on the
X1000 Soc from Ingenic.

Signed-off-by: Zhou Yanjie <zhouyanjie@zoho.com>
---
 drivers/tty/serial/8250/8250_ingenic.c | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/drivers/tty/serial/8250/8250_ingenic.c b/drivers/tty/serial/8250/8250_ingenic.c
index 15a8c8d..424c07c 100644
--- a/drivers/tty/serial/8250/8250_ingenic.c
+++ b/drivers/tty/serial/8250/8250_ingenic.c
@@ -129,22 +129,21 @@ static int __init ingenic_early_console_setup(struct earlycon_device *dev,
 	return 0;
 }
 
-EARLYCON_DECLARE(jz4740_uart, ingenic_early_console_setup);
 OF_EARLYCON_DECLARE(jz4740_uart, "ingenic,jz4740-uart",
 		    ingenic_early_console_setup);
 
-EARLYCON_DECLARE(jz4770_uart, ingenic_early_console_setup);
 OF_EARLYCON_DECLARE(jz4770_uart, "ingenic,jz4770-uart",
 		    ingenic_early_console_setup);
 
-EARLYCON_DECLARE(jz4775_uart, ingenic_early_console_setup);
 OF_EARLYCON_DECLARE(jz4775_uart, "ingenic,jz4775-uart",
 		    ingenic_early_console_setup);
 
-EARLYCON_DECLARE(jz4780_uart, ingenic_early_console_setup);
 OF_EARLYCON_DECLARE(jz4780_uart, "ingenic,jz4780-uart",
 		    ingenic_early_console_setup);
 
+OF_EARLYCON_DECLARE(x1000_uart, "ingenic,x1000-uart",
+		    ingenic_early_console_setup);
+
 static void ingenic_uart_serial_out(struct uart_port *p, int offset, int value)
 {
 	int ier;
@@ -328,12 +327,18 @@ static const struct ingenic_uart_config jz4780_uart_config = {
 	.fifosize = 64,
 };
 
+static const struct ingenic_uart_config x1000_uart_config = {
+	.tx_loadsz = 32,
+	.fifosize = 64,
+};
+
 static const struct of_device_id of_match[] = {
 	{ .compatible = "ingenic,jz4740-uart", .data = &jz4740_uart_config },
 	{ .compatible = "ingenic,jz4760-uart", .data = &jz4760_uart_config },
 	{ .compatible = "ingenic,jz4770-uart", .data = &jz4760_uart_config },
 	{ .compatible = "ingenic,jz4775-uart", .data = &jz4760_uart_config },
 	{ .compatible = "ingenic,jz4780-uart", .data = &jz4780_uart_config },
+	{ .compatible = "ingenic,x1000-uart", .data = &x1000_uart_config },
 	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, of_match);
-- 
2.7.4



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

* [PATCH v2 2/2] Serial: Ingenic: Add X1000 suppor for the UART driver.
  2019-01-28 13:56 ` Serial: Ingenic: Add X1000 suppor for the UART driver Zhou Yanjie
  2019-01-28 13:57   ` [PATCH v2 1/2] " Zhou Yanjie
@ 2019-01-28 13:57   ` Zhou Yanjie
  1 sibling, 0 replies; 17+ messages in thread
From: Zhou Yanjie @ 2019-01-28 13:57 UTC (permalink / raw)
  To: linux-mips
  Cc: linux-kernel, linux-serial, devicetree, robh+dt, paul.burton,
	gregkh, jslaby, mark.rutland, syq, jiaxun.yang, 772753199

Add support for probing the 8250_ingenic driver on the
X1000 Soc from Ingenic.

Signed-off-by: Zhou Yanjie <zhouyanjie@zoho.com>
---
 Documentation/devicetree/bindings/serial/ingenic,uart.txt | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/serial/ingenic,uart.txt b/Documentation/devicetree/bindings/serial/ingenic,uart.txt
index c3c6406..24ed876 100644
--- a/Documentation/devicetree/bindings/serial/ingenic,uart.txt
+++ b/Documentation/devicetree/bindings/serial/ingenic,uart.txt
@@ -6,7 +6,8 @@ Required properties:
   - "ingenic,jz4760-uart",
   - "ingenic,jz4770-uart",
   - "ingenic,jz4775-uart",
-  - "ingenic,jz4780-uart".
+  - "ingenic,jz4780-uart",
+  - "ingenic,x1000-uart".
 - reg : offset and length of the register set for the device.
 - interrupts : should contain uart interrupt.
 - clocks : phandles to the module & baud clocks.
-- 
2.7.4



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

* Re: [PATCH v2 1/2] Serial: Ingenic: Add X1000 suppor for the UART driver.
  2019-01-28 13:57   ` [PATCH v2 1/2] " Zhou Yanjie
@ 2019-01-28 16:22     ` Greg KH
  2019-01-28 16:45       ` Zhou Yanjie
  0 siblings, 1 reply; 17+ messages in thread
From: Greg KH @ 2019-01-28 16:22 UTC (permalink / raw)
  To: Zhou Yanjie
  Cc: linux-mips, linux-kernel, linux-serial, devicetree, robh+dt,
	paul.burton, jslaby, mark.rutland, syq, jiaxun.yang, 772753199

On Mon, Jan 28, 2019 at 09:57:00PM +0800, Zhou Yanjie wrote:
> Add support for probing the 8250_ingenic driver on the
> X1000 Soc from Ingenic.
> 
> Signed-off-by: Zhou Yanjie <zhouyanjie@zoho.com>
> ---
>  drivers/tty/serial/8250/8250_ingenic.c | 13 +++++++++----
>  1 file changed, 9 insertions(+), 4 deletions(-)

Your subject lines are still identical for both patches :(

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

* Re: [PATCH v2 1/2] Serial: Ingenic: Add X1000 suppor for the UART driver.
  2019-01-28 16:22     ` Greg KH
@ 2019-01-28 16:45       ` Zhou Yanjie
  0 siblings, 0 replies; 17+ messages in thread
From: Zhou Yanjie @ 2019-01-28 16:45 UTC (permalink / raw)
  To: Greg KH
  Cc: linux-mips, linux-kernel, linux-serial, devicetree, robh+dt,
	paul.burton, jslaby, mark.rutland, syq, jiaxun.yang, 772753199

I am very sorry, I understood wrong before.

On 2019年01月29日 00:22, Greg KH wrote:
> On Mon, Jan 28, 2019 at 09:57:00PM +0800, Zhou Yanjie wrote:
>> Add support for probing the 8250_ingenic driver on the
>> X1000 Soc from Ingenic.
>>
>> Signed-off-by: Zhou Yanjie <zhouyanjie@zoho.com>
>> ---
>>   drivers/tty/serial/8250/8250_ingenic.c | 13 +++++++++----
>>   1 file changed, 9 insertions(+), 4 deletions(-)
> Your subject lines are still identical for both patches :(




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

* Add Ingenic X1000 serial support
  2019-01-28  9:19 Add Ingenic X1000 serial support Zhou Yanjie
                   ` (2 preceding siblings ...)
  2019-01-28 13:56 ` Serial: Ingenic: Add X1000 suppor for the UART driver Zhou Yanjie
@ 2019-01-28 17:03 ` Zhou Yanjie
  2019-01-28 17:03   ` [PATCH v3 1/2] Serial: Ingenic: Add support for the X1000 Zhou Yanjie
  2019-01-28 17:03   ` [PATCH v3 2/2] Dt-bindings: Serial: Add X1000 serial bindings Zhou Yanjie
  3 siblings, 2 replies; 17+ messages in thread
From: Zhou Yanjie @ 2019-01-28 17:03 UTC (permalink / raw)
  To: linux-mips
  Cc: linux-kernel, linux-serial, devicetree, robh+dt, paul.burton,
	gregkh, jslaby, mark.rutland, syq, jiaxun.yang, 772753199

v1->v2: Remove unnecessary "EARLYCON_DECLARE".
v2->v3: Use different subject line for each patch.



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

* [PATCH v3 1/2] Serial: Ingenic: Add support for the X1000.
  2019-01-28 17:03 ` Add Ingenic X1000 serial support Zhou Yanjie
@ 2019-01-28 17:03   ` Zhou Yanjie
  2019-10-12  7:35     ` Greg KH
  2019-01-28 17:03   ` [PATCH v3 2/2] Dt-bindings: Serial: Add X1000 serial bindings Zhou Yanjie
  1 sibling, 1 reply; 17+ messages in thread
From: Zhou Yanjie @ 2019-01-28 17:03 UTC (permalink / raw)
  To: linux-mips
  Cc: linux-kernel, linux-serial, devicetree, robh+dt, paul.burton,
	gregkh, jslaby, mark.rutland, syq, jiaxun.yang, 772753199

Add support for probing the 8250_ingenic driver on the
X1000 Soc from Ingenic.

Signed-off-by: Zhou Yanjie <zhouyanjie@zoho.com>
---
 drivers/tty/serial/8250/8250_ingenic.c | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/drivers/tty/serial/8250/8250_ingenic.c b/drivers/tty/serial/8250/8250_ingenic.c
index 15a8c8d..424c07c 100644
--- a/drivers/tty/serial/8250/8250_ingenic.c
+++ b/drivers/tty/serial/8250/8250_ingenic.c
@@ -129,22 +129,21 @@ static int __init ingenic_early_console_setup(struct earlycon_device *dev,
 	return 0;
 }
 
-EARLYCON_DECLARE(jz4740_uart, ingenic_early_console_setup);
 OF_EARLYCON_DECLARE(jz4740_uart, "ingenic,jz4740-uart",
 		    ingenic_early_console_setup);
 
-EARLYCON_DECLARE(jz4770_uart, ingenic_early_console_setup);
 OF_EARLYCON_DECLARE(jz4770_uart, "ingenic,jz4770-uart",
 		    ingenic_early_console_setup);
 
-EARLYCON_DECLARE(jz4775_uart, ingenic_early_console_setup);
 OF_EARLYCON_DECLARE(jz4775_uart, "ingenic,jz4775-uart",
 		    ingenic_early_console_setup);
 
-EARLYCON_DECLARE(jz4780_uart, ingenic_early_console_setup);
 OF_EARLYCON_DECLARE(jz4780_uart, "ingenic,jz4780-uart",
 		    ingenic_early_console_setup);
 
+OF_EARLYCON_DECLARE(x1000_uart, "ingenic,x1000-uart",
+		    ingenic_early_console_setup);
+
 static void ingenic_uart_serial_out(struct uart_port *p, int offset, int value)
 {
 	int ier;
@@ -328,12 +327,18 @@ static const struct ingenic_uart_config jz4780_uart_config = {
 	.fifosize = 64,
 };
 
+static const struct ingenic_uart_config x1000_uart_config = {
+	.tx_loadsz = 32,
+	.fifosize = 64,
+};
+
 static const struct of_device_id of_match[] = {
 	{ .compatible = "ingenic,jz4740-uart", .data = &jz4740_uart_config },
 	{ .compatible = "ingenic,jz4760-uart", .data = &jz4760_uart_config },
 	{ .compatible = "ingenic,jz4770-uart", .data = &jz4760_uart_config },
 	{ .compatible = "ingenic,jz4775-uart", .data = &jz4760_uart_config },
 	{ .compatible = "ingenic,jz4780-uart", .data = &jz4780_uart_config },
+	{ .compatible = "ingenic,x1000-uart", .data = &x1000_uart_config },
 	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, of_match);
-- 
2.7.4



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

* [PATCH v3 2/2] Dt-bindings: Serial: Add X1000 serial bindings.
  2019-01-28 17:03 ` Add Ingenic X1000 serial support Zhou Yanjie
  2019-01-28 17:03   ` [PATCH v3 1/2] Serial: Ingenic: Add support for the X1000 Zhou Yanjie
@ 2019-01-28 17:03   ` Zhou Yanjie
  2019-01-30 18:46     ` Rob Herring
  1 sibling, 1 reply; 17+ messages in thread
From: Zhou Yanjie @ 2019-01-28 17:03 UTC (permalink / raw)
  To: linux-mips
  Cc: linux-kernel, linux-serial, devicetree, robh+dt, paul.burton,
	gregkh, jslaby, mark.rutland, syq, jiaxun.yang, 772753199

Add the serial bindings for the X1000 Soc from Ingenic.

Signed-off-by: Zhou Yanjie <zhouyanjie@zoho.com>
---
 Documentation/devicetree/bindings/serial/ingenic,uart.txt | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/serial/ingenic,uart.txt b/Documentation/devicetree/bindings/serial/ingenic,uart.txt
index c3c6406..24ed876 100644
--- a/Documentation/devicetree/bindings/serial/ingenic,uart.txt
+++ b/Documentation/devicetree/bindings/serial/ingenic,uart.txt
@@ -6,7 +6,8 @@ Required properties:
   - "ingenic,jz4760-uart",
   - "ingenic,jz4770-uart",
   - "ingenic,jz4775-uart",
-  - "ingenic,jz4780-uart".
+  - "ingenic,jz4780-uart",
+  - "ingenic,x1000-uart".
 - reg : offset and length of the register set for the device.
 - interrupts : should contain uart interrupt.
 - clocks : phandles to the module & baud clocks.
-- 
2.7.4



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

* Re: [PATCH v3 2/2] Dt-bindings: Serial: Add X1000 serial bindings.
  2019-01-28 17:03   ` [PATCH v3 2/2] Dt-bindings: Serial: Add X1000 serial bindings Zhou Yanjie
@ 2019-01-30 18:46     ` Rob Herring
  0 siblings, 0 replies; 17+ messages in thread
From: Rob Herring @ 2019-01-30 18:46 UTC (permalink / raw)
  To: Zhou Yanjie
  Cc: linux-mips, linux-kernel, linux-serial, devicetree, robh+dt,
	paul.burton, gregkh, jslaby, mark.rutland, syq, jiaxun.yang,
	772753199

On Tue, 29 Jan 2019 01:03:49 +0800, Zhou Yanjie wrote:
> Add the serial bindings for the X1000 Soc from Ingenic.
> 
> Signed-off-by: Zhou Yanjie <zhouyanjie@zoho.com>
> ---
>  Documentation/devicetree/bindings/serial/ingenic,uart.txt | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

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

* Re: [PATCH v3 1/2] Serial: Ingenic: Add support for the X1000.
  2019-01-28 17:03   ` [PATCH v3 1/2] Serial: Ingenic: Add support for the X1000 Zhou Yanjie
@ 2019-10-12  7:35     ` Greg KH
  2019-10-14  3:28       ` Zhou Yanjie
  0 siblings, 1 reply; 17+ messages in thread
From: Greg KH @ 2019-10-12  7:35 UTC (permalink / raw)
  To: Zhou Yanjie
  Cc: linux-mips, linux-kernel, linux-mmc, devicetree, robh+dt,
	paul.burton, mark.rutland, syq, ulf.hansson, linus.walleij,
	armijn, tglx, yuehaibing, malat, ezequiel, paul, linux-serial,
	jslaby, jiaxun.yang, 772753199

On Sat, Oct 12, 2019 at 01:13:23PM +0800, Zhou Yanjie wrote:
> Add support for probing the 8250_ingenic driver on the
> X1000 Soc from Ingenic.
> 
> Signed-off-by: Zhou Yanjie <zhouyanjie@zoho.com>
> ---
>  drivers/tty/serial/8250/8250_ingenic.c | 13 +++++++++----
>  1 file changed, 9 insertions(+), 4 deletions(-)

<snip>

you seem to have two patches merged into one here, please fix up and
resend.

thanks,

greg k-h

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

* Re: [PATCH v3 1/2] Serial: Ingenic: Add support for the X1000.
  2019-10-12  7:35     ` Greg KH
@ 2019-10-14  3:28       ` Zhou Yanjie
  0 siblings, 0 replies; 17+ messages in thread
From: Zhou Yanjie @ 2019-10-14  3:28 UTC (permalink / raw)
  To: Greg KH
  Cc: linux-mips, linux-kernel, linux-mmc, devicetree, robh+dt,
	paul.burton, mark.rutland, syq, ulf.hansson, linus.walleij,
	armijn, tglx, yuehaibing, malat, ezequiel, paul, linux-serial,
	jslaby, jiaxun.yang, 772753199

Hi Greg,

I'm sorry, maybe it was a problem when I git send-email,
causing the wrong patch to be sent to you. Just ignore
the email about the serial patch please.

Best regards!

On 2019年10月12日 15:35, Greg KH wrote:
> On Sat, Oct 12, 2019 at 01:13:23PM +0800, Zhou Yanjie wrote:
>> Add support for probing the 8250_ingenic driver on the
>> X1000 Soc from Ingenic.
>>
>> Signed-off-by: Zhou Yanjie <zhouyanjie@zoho.com>
>> ---
>>   drivers/tty/serial/8250/8250_ingenic.c | 13 +++++++++----
>>   1 file changed, 9 insertions(+), 4 deletions(-)
> <snip>
>
> you seem to have two patches merged into one here, please fix up and
> resend.
>
> thanks,
>
> greg k-h




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

end of thread, other threads:[~2019-10-14  3:29 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-28  9:19 Add Ingenic X1000 serial support Zhou Yanjie
2019-01-28  9:19 ` [PATCH 1/2] Serial: Ingenic: Add support for the X1000 Zhou Yanjie
2019-01-28  9:30   ` Greg KH
2019-01-28 12:44     ` Zhou Yanjie
2019-01-28 10:53   ` Geert Uytterhoeven
2019-01-28  9:19 ` [PATCH 2/2] " Zhou Yanjie
2019-01-28 13:56 ` Serial: Ingenic: Add X1000 suppor for the UART driver Zhou Yanjie
2019-01-28 13:57   ` [PATCH v2 1/2] " Zhou Yanjie
2019-01-28 16:22     ` Greg KH
2019-01-28 16:45       ` Zhou Yanjie
2019-01-28 13:57   ` [PATCH v2 2/2] " Zhou Yanjie
2019-01-28 17:03 ` Add Ingenic X1000 serial support Zhou Yanjie
2019-01-28 17:03   ` [PATCH v3 1/2] Serial: Ingenic: Add support for the X1000 Zhou Yanjie
2019-10-12  7:35     ` Greg KH
2019-10-14  3:28       ` Zhou Yanjie
2019-01-28 17:03   ` [PATCH v3 2/2] Dt-bindings: Serial: Add X1000 serial bindings Zhou Yanjie
2019-01-30 18:46     ` Rob Herring

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