[U-Boot] [PATCH v4 1/3] S5P: serial: Use the inline function instead of static value

[U-Boot] [PATCH v4 1/3] S5P: serial: Use the inline function instead of static value

[Minkyu Kang]

Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>
---
 arch/arm/include/asm/arch-s5pc1xx/uart.h |    5 ++++-
 drivers/serial/serial_s5p.c              |    2 +-
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/arch/arm/include/asm/arch-s5pc1xx/uart.h b/arch/arm/include/asm/arch-s5pc1xx/uart.h
index f6eeab4..1c56739 100644
--- a/arch/arm/include/asm/arch-s5pc1xx/uart.h
+++ b/arch/arm/include/asm/arch-s5pc1xx/uart.h
@@ -48,7 +48,10 @@ struct s5p_uart {
 	unsigned char	res3[0x3d0];
 };
 
-static int use_divslot = 1;
+static inline int s5p_uart_divslot(void)
+{
+	return 1;
+}
 
 #endif	/* __ASSEMBLY__ */
 
diff --git a/drivers/serial/serial_s5p.c b/drivers/serial/serial_s5p.c
index 9c1cbf4..f1ffa29 100644
--- a/drivers/serial/serial_s5p.c
+++ b/drivers/serial/serial_s5p.c
@@ -72,7 +72,7 @@ void serial_setbrg_dev(const int dev_index)
 
 	writel(val / 16 - 1, &uart->ubrdiv);
 
-	if (use_divslot)
+	if (s5p_uart_divslot())
 		writew(udivslot[val % 16], &uart->rest.slot);
 	else
 		writeb(val % 16, &uart->rest.value);
-- 
1.7.1

[U-Boot] [PATCH v4 1/3] S5P: serial: Use the inline function instead of static value

[Minkyu Kang]

On 24 January 2011 16:33, Minkyu Kang <mk7.kang@samsung.com> wrote:
> Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>
> ---
> ?arch/arm/include/asm/arch-s5pc1xx/uart.h | ? ?5 ++++-
> ?drivers/serial/serial_s5p.c ? ? ? ? ? ? ?| ? ?2 +-
> ?2 files changed, 5 insertions(+), 2 deletions(-)
>
applied to u-boot-samsung

Minkyu Kang
-- 
from. prom.
www.promsoft.net

[U-Boot] [PATCH v4 1/3] S5P: serial: Use the inline function instead of static value

[Graeme Russ]

Hi Minkyu,

On Mon, Jan 24, 2011 at 6:33 PM, Minkyu Kang <mk7.kang@samsung.com> wrote:
> Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>
> ---
> ?arch/arm/include/asm/arch-s5pc1xx/uart.h | ? ?5 ++++-
> ?drivers/serial/serial_s5p.c ? ? ? ? ? ? ?| ? ?2 +-
> ?2 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/include/asm/arch-s5pc1xx/uart.h b/arch/arm/include/asm/arch-s5pc1xx/uart.h
> index f6eeab4..1c56739 100644
> --- a/arch/arm/include/asm/arch-s5pc1xx/uart.h
> +++ b/arch/arm/include/asm/arch-s5pc1xx/uart.h
> @@ -48,7 +48,10 @@ struct s5p_uart {
> ? ? ? ?unsigned char ? res3[0x3d0];
> ?};
>
> -static int use_divslot = 1;
> +static inline int s5p_uart_divslot(void)
> +{
> + ? ? ? return 1;
> +}
>
> ?#endif /* __ASSEMBLY__ */
>
> diff --git a/drivers/serial/serial_s5p.c b/drivers/serial/serial_s5p.c
> index 9c1cbf4..f1ffa29 100644
> --- a/drivers/serial/serial_s5p.c
> +++ b/drivers/serial/serial_s5p.c
> @@ -72,7 +72,7 @@ void serial_setbrg_dev(const int dev_index)
>
> ? ? ? ?writel(val / 16 - 1, &uart->ubrdiv);
>
> - ? ? ? if (use_divslot)
> + ? ? ? if (s5p_uart_divslot())
> ? ? ? ? ? ? ? ?writew(udivslot[val % 16], &uart->rest.slot);
> ? ? ? ?else
> ? ? ? ? ? ? ? ?writeb(val % 16, &uart->rest.value);

Just for my own curiosity, what is the reason behind this?

I would have thought a #define would even work quite nicely.

Regards,

Graeme