* [PATCH v2 0/4] serial: uartps: Driver updates
@ 2018-12-18 12:18 ` Michal Simek
0 siblings, 0 replies; 18+ messages in thread
From: Michal Simek @ 2018-12-18 12:18 UTC (permalink / raw)
To: linux-kernel, monstr, michal.simek, Greg Kroah-Hartman,
Shubhrajyoti Datta
Cc: linux-arm-kernel, linux-serial, Jiri Slaby
Hi,
I am sending 4 patches in series to fix some issues we found.
Patches were sent separately but I have been asked to send them in
serial that's why I am also adding cover letter to explain this v2
version.
Thanks,
Michal
Changes in v2:
- Change description based on request from Greg
- Fix Nava's email address to align with patch email address
Michal Simek (1):
serial: uartps: Fix error path when alloc failed
Nava kishore Manne (1):
serial: uartps: Fix interrupt mask issue to handle the RX interrupts
properly
Shubhrajyoti Datta (2):
serial: uartps: Add the device_init_wakeup
serial: uartps: Check if the device is a console
drivers/tty/serial/xilinx_uartps.c | 16 ++++++++++------
1 file changed, 10 insertions(+), 6 deletions(-)
--
1.9.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread* [PATCH v2 1/4] serial: uartps: Add the device_init_wakeup
2018-12-18 12:18 ` Michal Simek
@ 2018-12-18 12:18 ` Michal Simek
-1 siblings, 0 replies; 18+ messages in thread
From: Michal Simek @ 2018-12-18 12:18 UTC (permalink / raw)
To: linux-kernel, monstr, michal.simek, Greg Kroah-Hartman,
Shubhrajyoti Datta
Cc: Jiri Slaby, linux-serial, linux-arm-kernel
From: Shubhrajyoti Datta <shubhrajyoti.datta@xilinx.com>
Initialise the device wakeup.
The device_init_wakeup is needed for the wakeup to work by default.
Uart can be configured as the primary wakeup source so it is good to
enable wakeup by default.
The same functionality is enabled also by 8250_omap, atmel_serial,
omap-serial and stm32-usart.
Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
Changes in v2:
- Change description based on request from Greg
drivers/tty/serial/xilinx_uartps.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/tty/serial/xilinx_uartps.c b/drivers/tty/serial/xilinx_uartps.c
index 379242b96790..0140644391df 100644
--- a/drivers/tty/serial/xilinx_uartps.c
+++ b/drivers/tty/serial/xilinx_uartps.c
@@ -1624,6 +1624,7 @@ static int cdns_uart_probe(struct platform_device *pdev)
pm_runtime_set_autosuspend_delay(&pdev->dev, UART_AUTOSUSPEND_TIMEOUT);
pm_runtime_set_active(&pdev->dev);
pm_runtime_enable(&pdev->dev);
+ device_init_wakeup(port->dev, true);
#ifdef CONFIG_SERIAL_XILINX_PS_UART_CONSOLE
/*
@@ -1702,6 +1703,7 @@ static int cdns_uart_remove(struct platform_device *pdev)
pm_runtime_disable(&pdev->dev);
pm_runtime_set_suspended(&pdev->dev);
pm_runtime_dont_use_autosuspend(&pdev->dev);
+ device_init_wakeup(&pdev->dev, false);
#ifdef CONFIG_SERIAL_XILINX_PS_UART_CONSOLE
if (console_port == port)
--
1.9.1
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v2 1/4] serial: uartps: Add the device_init_wakeup
@ 2018-12-18 12:18 ` Michal Simek
0 siblings, 0 replies; 18+ messages in thread
From: Michal Simek @ 2018-12-18 12:18 UTC (permalink / raw)
To: linux-kernel, monstr, michal.simek, Greg Kroah-Hartman,
Shubhrajyoti Datta
Cc: linux-arm-kernel, linux-serial, Jiri Slaby
From: Shubhrajyoti Datta <shubhrajyoti.datta@xilinx.com>
Initialise the device wakeup.
The device_init_wakeup is needed for the wakeup to work by default.
Uart can be configured as the primary wakeup source so it is good to
enable wakeup by default.
The same functionality is enabled also by 8250_omap, atmel_serial,
omap-serial and stm32-usart.
Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
Changes in v2:
- Change description based on request from Greg
drivers/tty/serial/xilinx_uartps.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/tty/serial/xilinx_uartps.c b/drivers/tty/serial/xilinx_uartps.c
index 379242b96790..0140644391df 100644
--- a/drivers/tty/serial/xilinx_uartps.c
+++ b/drivers/tty/serial/xilinx_uartps.c
@@ -1624,6 +1624,7 @@ static int cdns_uart_probe(struct platform_device *pdev)
pm_runtime_set_autosuspend_delay(&pdev->dev, UART_AUTOSUSPEND_TIMEOUT);
pm_runtime_set_active(&pdev->dev);
pm_runtime_enable(&pdev->dev);
+ device_init_wakeup(port->dev, true);
#ifdef CONFIG_SERIAL_XILINX_PS_UART_CONSOLE
/*
@@ -1702,6 +1703,7 @@ static int cdns_uart_remove(struct platform_device *pdev)
pm_runtime_disable(&pdev->dev);
pm_runtime_set_suspended(&pdev->dev);
pm_runtime_dont_use_autosuspend(&pdev->dev);
+ device_init_wakeup(&pdev->dev, false);
#ifdef CONFIG_SERIAL_XILINX_PS_UART_CONSOLE
if (console_port == port)
--
1.9.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v2 2/4] serial: uartps: Check if the device is a console
2018-12-18 12:18 ` Michal Simek
@ 2018-12-18 12:18 ` Michal Simek
-1 siblings, 0 replies; 18+ messages in thread
From: Michal Simek @ 2018-12-18 12:18 UTC (permalink / raw)
To: linux-kernel, monstr, michal.simek, Greg Kroah-Hartman,
Shubhrajyoti Datta
Cc: Jiri Slaby, linux-serial, linux-arm-kernel
From: Shubhrajyoti Datta <shubhrajyoti.datta@xilinx.com>
While checking for console_suspend_enabled also check if the
device is a console.
Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
Changes in v2: None
drivers/tty/serial/xilinx_uartps.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/tty/serial/xilinx_uartps.c b/drivers/tty/serial/xilinx_uartps.c
index 0140644391df..9cdc36be5b13 100644
--- a/drivers/tty/serial/xilinx_uartps.c
+++ b/drivers/tty/serial/xilinx_uartps.c
@@ -1255,7 +1255,7 @@ static int cdns_uart_suspend(struct device *device)
may_wake = device_may_wakeup(device);
- if (console_suspend_enabled && may_wake) {
+ if (console_suspend_enabled && uart_console(port) && may_wake) {
unsigned long flags = 0;
spin_lock_irqsave(&port->lock, flags);
@@ -1293,7 +1293,7 @@ static int cdns_uart_resume(struct device *device)
may_wake = device_may_wakeup(device);
- if (console_suspend_enabled && !may_wake) {
+ if (console_suspend_enabled && uart_console(port) && !may_wake) {
clk_enable(cdns_uart->pclk);
clk_enable(cdns_uart->uartclk);
--
1.9.1
^ permalink raw reply related [flat|nested] 18+ messages in thread* [PATCH v2 2/4] serial: uartps: Check if the device is a console
@ 2018-12-18 12:18 ` Michal Simek
0 siblings, 0 replies; 18+ messages in thread
From: Michal Simek @ 2018-12-18 12:18 UTC (permalink / raw)
To: linux-kernel, monstr, michal.simek, Greg Kroah-Hartman,
Shubhrajyoti Datta
Cc: linux-arm-kernel, linux-serial, Jiri Slaby
From: Shubhrajyoti Datta <shubhrajyoti.datta@xilinx.com>
While checking for console_suspend_enabled also check if the
device is a console.
Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
Changes in v2: None
drivers/tty/serial/xilinx_uartps.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/tty/serial/xilinx_uartps.c b/drivers/tty/serial/xilinx_uartps.c
index 0140644391df..9cdc36be5b13 100644
--- a/drivers/tty/serial/xilinx_uartps.c
+++ b/drivers/tty/serial/xilinx_uartps.c
@@ -1255,7 +1255,7 @@ static int cdns_uart_suspend(struct device *device)
may_wake = device_may_wakeup(device);
- if (console_suspend_enabled && may_wake) {
+ if (console_suspend_enabled && uart_console(port) && may_wake) {
unsigned long flags = 0;
spin_lock_irqsave(&port->lock, flags);
@@ -1293,7 +1293,7 @@ static int cdns_uart_resume(struct device *device)
may_wake = device_may_wakeup(device);
- if (console_suspend_enabled && !may_wake) {
+ if (console_suspend_enabled && uart_console(port) && !may_wake) {
clk_enable(cdns_uart->pclk);
clk_enable(cdns_uart->uartclk);
--
1.9.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v2 3/4] serial: uartps: Fix error path when alloc failed
2018-12-18 12:18 ` Michal Simek
@ 2018-12-18 12:18 ` Michal Simek
-1 siblings, 0 replies; 18+ messages in thread
From: Michal Simek @ 2018-12-18 12:18 UTC (permalink / raw)
To: linux-kernel, monstr, michal.simek, Greg Kroah-Hartman,
Shubhrajyoti Datta
Cc: Jiri Slaby, linux-serial, linux-arm-kernel
When cdns_uart_console allocation failed there is a need to also clear
ID from ID list.
Fixes: ae1cca3fa347 ("serial: uartps: Change uart ID port allocation")
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
Changes in v2: None
drivers/tty/serial/xilinx_uartps.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/tty/serial/xilinx_uartps.c b/drivers/tty/serial/xilinx_uartps.c
index 9cdc36be5b13..c6d38617d622 100644
--- a/drivers/tty/serial/xilinx_uartps.c
+++ b/drivers/tty/serial/xilinx_uartps.c
@@ -1508,8 +1508,10 @@ static int cdns_uart_probe(struct platform_device *pdev)
#ifdef CONFIG_SERIAL_XILINX_PS_UART_CONSOLE
cdns_uart_console = devm_kzalloc(&pdev->dev, sizeof(*cdns_uart_console),
GFP_KERNEL);
- if (!cdns_uart_console)
- return -ENOMEM;
+ if (!cdns_uart_console) {
+ rc = -ENOMEM;
+ goto err_out_id;
+ }
strncpy(cdns_uart_console->name, CDNS_UART_TTY_NAME,
sizeof(cdns_uart_console->name));
--
1.9.1
^ permalink raw reply related [flat|nested] 18+ messages in thread* [PATCH v2 3/4] serial: uartps: Fix error path when alloc failed
@ 2018-12-18 12:18 ` Michal Simek
0 siblings, 0 replies; 18+ messages in thread
From: Michal Simek @ 2018-12-18 12:18 UTC (permalink / raw)
To: linux-kernel, monstr, michal.simek, Greg Kroah-Hartman,
Shubhrajyoti Datta
Cc: linux-arm-kernel, linux-serial, Jiri Slaby
When cdns_uart_console allocation failed there is a need to also clear
ID from ID list.
Fixes: ae1cca3fa347 ("serial: uartps: Change uart ID port allocation")
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
Changes in v2: None
drivers/tty/serial/xilinx_uartps.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/tty/serial/xilinx_uartps.c b/drivers/tty/serial/xilinx_uartps.c
index 9cdc36be5b13..c6d38617d622 100644
--- a/drivers/tty/serial/xilinx_uartps.c
+++ b/drivers/tty/serial/xilinx_uartps.c
@@ -1508,8 +1508,10 @@ static int cdns_uart_probe(struct platform_device *pdev)
#ifdef CONFIG_SERIAL_XILINX_PS_UART_CONSOLE
cdns_uart_console = devm_kzalloc(&pdev->dev, sizeof(*cdns_uart_console),
GFP_KERNEL);
- if (!cdns_uart_console)
- return -ENOMEM;
+ if (!cdns_uart_console) {
+ rc = -ENOMEM;
+ goto err_out_id;
+ }
strncpy(cdns_uart_console->name, CDNS_UART_TTY_NAME,
sizeof(cdns_uart_console->name));
--
1.9.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH v2 4/4] serial: uartps: Fix interrupt mask issue to handle the RX interrupts properly
2018-12-18 12:18 ` Michal Simek
@ 2018-12-18 12:18 ` Michal Simek
-1 siblings, 0 replies; 18+ messages in thread
From: Michal Simek @ 2018-12-18 12:18 UTC (permalink / raw)
To: linux-kernel, monstr, michal.simek, Greg Kroah-Hartman,
Shubhrajyoti Datta
Cc: Nava kishore Manne, Jiri Slaby, linux-serial, linux-arm-kernel
From: Nava kishore Manne <nava.manne@xilinx.com>
This patch Correct the RX interrupt mask value to handle the
RX interrupts properly.
Fixes: c8dbdc842d30 ("serial: xuartps: Rewrite the interrupt handling logic")
Signed-off-by: Nava kishore Manne <nava.manne@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
Changes in v2:
- Fix Nava's email address to align with patch email address
drivers/tty/serial/xilinx_uartps.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/tty/serial/xilinx_uartps.c b/drivers/tty/serial/xilinx_uartps.c
index c6d38617d622..094f2958cb2b 100644
--- a/drivers/tty/serial/xilinx_uartps.c
+++ b/drivers/tty/serial/xilinx_uartps.c
@@ -123,7 +123,7 @@
#define CDNS_UART_IXR_RXTRIG 0x00000001 /* RX FIFO trigger interrupt */
#define CDNS_UART_IXR_RXFULL 0x00000004 /* RX FIFO full interrupt. */
#define CDNS_UART_IXR_RXEMPTY 0x00000002 /* RX FIFO empty interrupt. */
-#define CDNS_UART_IXR_MASK 0x00001FFF /* Valid bit mask */
+#define CDNS_UART_IXR_RXMASK 0x000021e7 /* Valid RX bit mask */
/*
* Do not enable parity error interrupt for the following
@@ -364,7 +364,7 @@ static irqreturn_t cdns_uart_isr(int irq, void *dev_id)
cdns_uart_handle_tx(dev_id);
isrstatus &= ~CDNS_UART_IXR_TXEMPTY;
}
- if (isrstatus & CDNS_UART_IXR_MASK)
+ if (isrstatus & CDNS_UART_IXR_RXMASK)
cdns_uart_handle_rx(dev_id, isrstatus);
spin_unlock(&port->lock);
--
1.9.1
^ permalink raw reply related [flat|nested] 18+ messages in thread* [PATCH v2 4/4] serial: uartps: Fix interrupt mask issue to handle the RX interrupts properly
@ 2018-12-18 12:18 ` Michal Simek
0 siblings, 0 replies; 18+ messages in thread
From: Michal Simek @ 2018-12-18 12:18 UTC (permalink / raw)
To: linux-kernel, monstr, michal.simek, Greg Kroah-Hartman,
Shubhrajyoti Datta
Cc: Nava kishore Manne, linux-arm-kernel, linux-serial, Jiri Slaby
From: Nava kishore Manne <nava.manne@xilinx.com>
This patch Correct the RX interrupt mask value to handle the
RX interrupts properly.
Fixes: c8dbdc842d30 ("serial: xuartps: Rewrite the interrupt handling logic")
Signed-off-by: Nava kishore Manne <nava.manne@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
Changes in v2:
- Fix Nava's email address to align with patch email address
drivers/tty/serial/xilinx_uartps.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/tty/serial/xilinx_uartps.c b/drivers/tty/serial/xilinx_uartps.c
index c6d38617d622..094f2958cb2b 100644
--- a/drivers/tty/serial/xilinx_uartps.c
+++ b/drivers/tty/serial/xilinx_uartps.c
@@ -123,7 +123,7 @@
#define CDNS_UART_IXR_RXTRIG 0x00000001 /* RX FIFO trigger interrupt */
#define CDNS_UART_IXR_RXFULL 0x00000004 /* RX FIFO full interrupt. */
#define CDNS_UART_IXR_RXEMPTY 0x00000002 /* RX FIFO empty interrupt. */
-#define CDNS_UART_IXR_MASK 0x00001FFF /* Valid bit mask */
+#define CDNS_UART_IXR_RXMASK 0x000021e7 /* Valid RX bit mask */
/*
* Do not enable parity error interrupt for the following
@@ -364,7 +364,7 @@ static irqreturn_t cdns_uart_isr(int irq, void *dev_id)
cdns_uart_handle_tx(dev_id);
isrstatus &= ~CDNS_UART_IXR_TXEMPTY;
}
- if (isrstatus & CDNS_UART_IXR_MASK)
+ if (isrstatus & CDNS_UART_IXR_RXMASK)
cdns_uart_handle_rx(dev_id, isrstatus);
spin_unlock(&port->lock);
--
1.9.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH v2 0/4] serial: uartps: Driver updates
2018-12-18 12:18 ` Michal Simek
(?)
@ 2018-12-19 18:40 ` Maarten Brock
-1 siblings, 0 replies; 18+ messages in thread
From: Maarten Brock @ 2018-12-19 18:40 UTC (permalink / raw)
To: Michal Simek
Cc: linux-kernel, monstr, Greg Kroah-Hartman, Shubhrajyoti Datta,
Jiri Slaby, linux-serial, linux-arm-kernel, linux-serial-owner
Hello Michal,
On 2018-12-18 13:18, Michal Simek wrote:
> Hi,
>
> I am sending 4 patches in series to fix some issues we found.
> Patches were sent separately but I have been asked to send them in
> serial that's why I am also adding cover letter to explain this v2
> version.
>
> Thanks,
> Michal
I'm wondering why, when reading the linux-serial mailing list, I can
only see this cover letter.
Are you perhaps using a different To/Cc for the actual patches? And if
so, is that on purpose?
Kind regards,
Maarten
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2 0/4] serial: uartps: Driver updates
@ 2018-12-19 18:40 ` Maarten Brock
0 siblings, 0 replies; 18+ messages in thread
From: Maarten Brock @ 2018-12-19 18:40 UTC (permalink / raw)
To: Michal Simek
Cc: monstr, linux-serial-owner, Greg Kroah-Hartman,
Shubhrajyoti Datta, linux-kernel, linux-serial, Jiri Slaby,
linux-arm-kernel
Hello Michal,
On 2018-12-18 13:18, Michal Simek wrote:
> Hi,
>
> I am sending 4 patches in series to fix some issues we found.
> Patches were sent separately but I have been asked to send them in
> serial that's why I am also adding cover letter to explain this v2
> version.
>
> Thanks,
> Michal
I'm wondering why, when reading the linux-serial mailing list, I can
only see this cover letter.
Are you perhaps using a different To/Cc for the actual patches? And if
so, is that on purpose?
Kind regards,
Maarten
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2 0/4] serial: uartps: Driver updates
@ 2018-12-19 18:40 ` Maarten Brock
0 siblings, 0 replies; 18+ messages in thread
From: Maarten Brock @ 2018-12-19 18:40 UTC (permalink / raw)
To: Michal Simek
Cc: monstr, linux-serial-owner, Greg Kroah-Hartman,
Shubhrajyoti Datta, linux-kernel, linux-serial, Jiri Slaby,
linux-arm-kernel
Hello Michal,
On 2018-12-18 13:18, Michal Simek wrote:
> Hi,
>
> I am sending 4 patches in series to fix some issues we found.
> Patches were sent separately but I have been asked to send them in
> serial that's why I am also adding cover letter to explain this v2
> version.
>
> Thanks,
> Michal
I'm wondering why, when reading the linux-serial mailing list, I can
only see this cover letter.
Are you perhaps using a different To/Cc for the actual patches? And if
so, is that on purpose?
Kind regards,
Maarten
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2 0/4] serial: uartps: Driver updates
2018-12-19 18:40 ` Maarten Brock
(?)
@ 2018-12-20 6:52 ` Michal Simek
-1 siblings, 0 replies; 18+ messages in thread
From: Michal Simek @ 2018-12-20 6:52 UTC (permalink / raw)
To: Maarten Brock, Michal Simek
Cc: linux-kernel, monstr, Greg Kroah-Hartman, Shubhrajyoti Datta,
Jiri Slaby, linux-serial, linux-arm-kernel, linux-serial-owner
Hi,
On 19. 12. 18 19:40, Maarten Brock wrote:
> Hello Michal,
>
> On 2018-12-18 13:18, Michal Simek wrote:
>> Hi,
>>
>> I am sending 4 patches in series to fix some issues we found.
>> Patches were sent separately but I have been asked to send them in
>> serial that's why I am also adding cover letter to explain this v2
>> version.
>>
>> Thanks,
>> Michal
>
> I'm wondering why, when reading the linux-serial mailing list, I can
> only see this cover letter.
> Are you perhaps using a different To/Cc for the actual patches? And if
> so, is that on purpose?
>
I have checked linux-serial mailing list and I see all of them there.
https://www.spinics.net/lists/linux-serial/
Individual here.
https://www.spinics.net/lists/linux-serial/msg32919.html
https://www.spinics.net/lists/linux-serial/msg32916.html
https://www.spinics.net/lists/linux-serial/msg32917.html
https://www.spinics.net/lists/linux-serial/msg32918.html
Also I see linux-serial@ in CC in all emails I have sent.
Thanks,
Michal
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2 0/4] serial: uartps: Driver updates
@ 2018-12-20 6:52 ` Michal Simek
0 siblings, 0 replies; 18+ messages in thread
From: Michal Simek @ 2018-12-20 6:52 UTC (permalink / raw)
To: Maarten Brock, Michal Simek
Cc: monstr, linux-serial-owner, Greg Kroah-Hartman,
Shubhrajyoti Datta, linux-kernel, linux-serial, Jiri Slaby,
linux-arm-kernel
Hi,
On 19. 12. 18 19:40, Maarten Brock wrote:
> Hello Michal,
>
> On 2018-12-18 13:18, Michal Simek wrote:
>> Hi,
>>
>> I am sending 4 patches in series to fix some issues we found.
>> Patches were sent separately but I have been asked to send them in
>> serial that's why I am also adding cover letter to explain this v2
>> version.
>>
>> Thanks,
>> Michal
>
> I'm wondering why, when reading the linux-serial mailing list, I can
> only see this cover letter.
> Are you perhaps using a different To/Cc for the actual patches? And if
> so, is that on purpose?
>
I have checked linux-serial mailing list and I see all of them there.
https://www.spinics.net/lists/linux-serial/
Individual here.
https://www.spinics.net/lists/linux-serial/msg32919.html
https://www.spinics.net/lists/linux-serial/msg32916.html
https://www.spinics.net/lists/linux-serial/msg32917.html
https://www.spinics.net/lists/linux-serial/msg32918.html
Also I see linux-serial@ in CC in all emails I have sent.
Thanks,
Michal
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2 0/4] serial: uartps: Driver updates
@ 2018-12-20 6:52 ` Michal Simek
0 siblings, 0 replies; 18+ messages in thread
From: Michal Simek @ 2018-12-20 6:52 UTC (permalink / raw)
To: Maarten Brock, Michal Simek
Cc: linux-kernel, monstr, Greg Kroah-Hartman, Shubhrajyoti Datta,
Jiri Slaby, linux-serial, linux-arm-kernel, linux-serial-owner
Hi,
On 19. 12. 18 19:40, Maarten Brock wrote:
> Hello Michal,
>
> On 2018-12-18 13:18, Michal Simek wrote:
>> Hi,
>>
>> I am sending 4 patches in series to fix some issues we found.
>> Patches were sent separately but I have been asked to send them in
>> serial that's why I am also adding cover letter to explain this v2
>> version.
>>
>> Thanks,
>> Michal
>
> I'm wondering why, when reading the linux-serial mailing list, I can
> only see this cover letter.
> Are you perhaps using a different To/Cc for the actual patches? And if
> so, is that on purpose?
>
I have checked linux-serial mailing list and I see all of them there.
https://www.spinics.net/lists/linux-serial/
Individual here.
https://www.spinics.net/lists/linux-serial/msg32919.html
https://www.spinics.net/lists/linux-serial/msg32916.html
https://www.spinics.net/lists/linux-serial/msg32917.html
https://www.spinics.net/lists/linux-serial/msg32918.html
Also I see linux-serial@ in CC in all emails I have sent.
Thanks,
Michal
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2 0/4] serial: uartps: Driver updates
2018-12-20 6:52 ` Michal Simek
@ 2018-12-20 11:04 ` Maarten Brock
-1 siblings, 0 replies; 18+ messages in thread
From: Maarten Brock @ 2018-12-20 11:04 UTC (permalink / raw)
To: Michal Simek
Cc: linux-kernel, monstr, Greg Kroah-Hartman, Shubhrajyoti Datta,
Jiri Slaby, linux-serial, linux-arm-kernel, linux-serial-owner
On 2018-12-20 07:52, Michal Simek wrote:
> Hi,
>
> On 19. 12. 18 19:40, Maarten Brock wrote:
>> Hello Michal,
>>
>> On 2018-12-18 13:18, Michal Simek wrote:
>>> Hi,
>>>
>>> I am sending 4 patches in series to fix some issues we found.
>>> Patches were sent separately but I have been asked to send them in
>>> serial that's why I am also adding cover letter to explain this v2
>>> version.
>>>
>>> Thanks,
>>> Michal
>>
>> I'm wondering why, when reading the linux-serial mailing list, I can
>> only see this cover letter.
>> Are you perhaps using a different To/Cc for the actual patches? And if
>> so, is that on purpose?
>>
>
> I have checked linux-serial mailing list and I see all of them there.
> https://www.spinics.net/lists/linux-serial/
>
> Individual here.
> https://www.spinics.net/lists/linux-serial/msg32919.html
> https://www.spinics.net/lists/linux-serial/msg32916.html
> https://www.spinics.net/lists/linux-serial/msg32917.html
> https://www.spinics.net/lists/linux-serial/msg32918.html
>
> Also I see linux-serial@ in CC in all emails I have sent.
>
> Thanks,
> Michal
Thanks, I see them there too. But the odd thing is that I did not
receive
them in my mailbox. And I had noticed this before that I was missing
your
posts following a cover letter.
I normally do receive other peoples patches after a cover letter.
Maarten
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH v2 0/4] serial: uartps: Driver updates
@ 2018-12-20 11:04 ` Maarten Brock
0 siblings, 0 replies; 18+ messages in thread
From: Maarten Brock @ 2018-12-20 11:04 UTC (permalink / raw)
To: Michal Simek
Cc: monstr, linux-serial-owner, Greg Kroah-Hartman,
Shubhrajyoti Datta, linux-kernel, linux-serial, Jiri Slaby,
linux-arm-kernel
On 2018-12-20 07:52, Michal Simek wrote:
> Hi,
>
> On 19. 12. 18 19:40, Maarten Brock wrote:
>> Hello Michal,
>>
>> On 2018-12-18 13:18, Michal Simek wrote:
>>> Hi,
>>>
>>> I am sending 4 patches in series to fix some issues we found.
>>> Patches were sent separately but I have been asked to send them in
>>> serial that's why I am also adding cover letter to explain this v2
>>> version.
>>>
>>> Thanks,
>>> Michal
>>
>> I'm wondering why, when reading the linux-serial mailing list, I can
>> only see this cover letter.
>> Are you perhaps using a different To/Cc for the actual patches? And if
>> so, is that on purpose?
>>
>
> I have checked linux-serial mailing list and I see all of them there.
> https://www.spinics.net/lists/linux-serial/
>
> Individual here.
> https://www.spinics.net/lists/linux-serial/msg32919.html
> https://www.spinics.net/lists/linux-serial/msg32916.html
> https://www.spinics.net/lists/linux-serial/msg32917.html
> https://www.spinics.net/lists/linux-serial/msg32918.html
>
> Also I see linux-serial@ in CC in all emails I have sent.
>
> Thanks,
> Michal
Thanks, I see them there too. But the odd thing is that I did not
receive
them in my mailbox. And I had noticed this before that I was missing
your
posts following a cover letter.
I normally do receive other peoples patches after a cover letter.
Maarten
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread