2013-03-18 10:56:59

by Sourav Poddar

[permalink] [raw]
Subject: [PATCH 1/2] driver: serial-omap: move max uart count into generic header file.

OMAP_MAX_HSUART_PORTS is moved to omap_serial header file.

Cc: Santosh Shilimkar <[email protected]>
Cc: Felipe Balbi <[email protected]>
Cc: Rajendra nayak <[email protected]>
Signed-off-by: Sourav Poddar <[email protected]>
---
drivers/tty/serial/omap-serial.c | 2 --
include/linux/platform_data/serial-omap.h | 1 +
2 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/drivers/tty/serial/omap-serial.c b/drivers/tty/serial/omap-serial.c
index 5722eaf..4c2ba9f 100644
--- a/drivers/tty/serial/omap-serial.c
+++ b/drivers/tty/serial/omap-serial.c
@@ -43,8 +43,6 @@
#include <linux/pinctrl/consumer.h>
#include <linux/platform_data/serial-omap.h>

-#define OMAP_MAX_HSUART_PORTS 6
-
#define UART_BUILD_REVISION(x, y) (((x) << 8) | (y))

#define OMAP_UART_REV_42 0x0402
diff --git a/include/linux/platform_data/serial-omap.h b/include/linux/platform_data/serial-omap.h
index ff9b0aa..92c48c2 100644
--- a/include/linux/platform_data/serial-omap.h
+++ b/include/linux/platform_data/serial-omap.h
@@ -29,6 +29,7 @@
* is used as console uart.
*/
#define OMAP_SERIAL_NAME "ttyO"
+#define OMAP_MAX_HSUART_PORTS 6

struct omap_uart_port_info {
bool dma_enabled; /* To specify DMA Mode */
--
1.7.1


2013-03-18 18:45:32

by Kevin Hilman

[permalink] [raw]
Subject: Re: [PATCH 1/2] driver: serial-omap: move max uart count into generic header file.

Sourav Poddar <[email protected]> writes:

> OMAP_MAX_HSUART_PORTS is moved to omap_serial header file.

Why?

You started to explain it in the cover letter, but a full description
belongs here for the permanent git history.

Kevin

> Cc: Santosh Shilimkar <[email protected]>
> Cc: Felipe Balbi <[email protected]>
> Cc: Rajendra nayak <[email protected]>
> Signed-off-by: Sourav Poddar <[email protected]>
> ---
> drivers/tty/serial/omap-serial.c | 2 --
> include/linux/platform_data/serial-omap.h | 1 +
> 2 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/tty/serial/omap-serial.c b/drivers/tty/serial/omap-serial.c
> index 5722eaf..4c2ba9f 100644
> --- a/drivers/tty/serial/omap-serial.c
> +++ b/drivers/tty/serial/omap-serial.c
> @@ -43,8 +43,6 @@
> #include <linux/pinctrl/consumer.h>
> #include <linux/platform_data/serial-omap.h>
>
> -#define OMAP_MAX_HSUART_PORTS 6
> -
> #define UART_BUILD_REVISION(x, y) (((x) << 8) | (y))
>
> #define OMAP_UART_REV_42 0x0402
> diff --git a/include/linux/platform_data/serial-omap.h b/include/linux/platform_data/serial-omap.h
> index ff9b0aa..92c48c2 100644
> --- a/include/linux/platform_data/serial-omap.h
> +++ b/include/linux/platform_data/serial-omap.h
> @@ -29,6 +29,7 @@
> * is used as console uart.
> */
> #define OMAP_SERIAL_NAME "ttyO"
> +#define OMAP_MAX_HSUART_PORTS 6
>
> struct omap_uart_port_info {
> bool dma_enabled; /* To specify DMA Mode */

2013-03-19 06:01:00

by Sourav Poddar

[permalink] [raw]
Subject: Re: [PATCH 1/2] driver: serial-omap: move max uart count into generic header file.

Hi Kevin,
On Tuesday 19 March 2013 12:15 AM, Kevin Hilman wrote:
> Sourav Poddar<[email protected]> writes:
>
>> OMAP_MAX_HSUART_PORTS is moved to omap_serial header file.
> Why?
>
The patch is a prerequisitive for the next patch in the series, where I am
making use of the max number of uarts, omap device supports. Hence, I
am rearranging the macro from a driver file to a header file. So, it can
be used at
multiple places.
> You started to explain it in the cover letter, but a full description
> belongs here for the permanent git history.
>
Sure, I will add the complete description for this here itself in my next
version.
> Kevin
>
>> Cc: Santosh Shilimkar<[email protected]>
>> Cc: Felipe Balbi<[email protected]>
>> Cc: Rajendra nayak<[email protected]>
>> Signed-off-by: Sourav Poddar<[email protected]>
>> ---
>> drivers/tty/serial/omap-serial.c | 2 --
>> include/linux/platform_data/serial-omap.h | 1 +
>> 2 files changed, 1 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/tty/serial/omap-serial.c b/drivers/tty/serial/omap-serial.c
>> index 5722eaf..4c2ba9f 100644
>> --- a/drivers/tty/serial/omap-serial.c
>> +++ b/drivers/tty/serial/omap-serial.c
>> @@ -43,8 +43,6 @@
>> #include<linux/pinctrl/consumer.h>
>> #include<linux/platform_data/serial-omap.h>
>>
>> -#define OMAP_MAX_HSUART_PORTS 6
>> -
>> #define UART_BUILD_REVISION(x, y) (((x)<< 8) | (y))
>>
>> #define OMAP_UART_REV_42 0x0402
>> diff --git a/include/linux/platform_data/serial-omap.h b/include/linux/platform_data/serial-omap.h
>> index ff9b0aa..92c48c2 100644
>> --- a/include/linux/platform_data/serial-omap.h
>> +++ b/include/linux/platform_data/serial-omap.h
>> @@ -29,6 +29,7 @@
>> * is used as console uart.
>> */
>> #define OMAP_SERIAL_NAME "ttyO"
>> +#define OMAP_MAX_HSUART_PORTS 6
>>
>> struct omap_uart_port_info {
>> bool dma_enabled; /* To specify DMA Mode */