[PATCH 2/2] tty: serial: fsl_lpuart: add 32-bit register interface support

Jingchang Lu jingchang.lu at freescale.com
Mon Jul 14 03:16:12 PDT 2014



>-----Original Message-----
>From: Greg KH [mailto:gregkh at linuxfoundation.org]
>Sent: Saturday, July 12, 2014 8:51 AM
>To: Lu Jingchang-B35083
>Cc: linux-serial at vger.kernel.org; linux-arm-kernel at lists.infradead.org
>Subject: Re: [PATCH 2/2] tty: serial: fsl_lpuart: add 32-bit register
>interface support
>
>On Fri, Jul 11, 2014 at 01:41:42PM +0800, Jingchang Lu wrote:
>> This add the 32-bit register version LPUART support with big-endian
>> byte order.
>>
>> Signed-off-by: Jingchang Lu <jingchang.lu at freescale.com>
>> ---
>>  drivers/tty/serial/fsl_lpuart.c | 653
>> +++++++++++++++++++++++++++++++++++++++-
>>  1 file changed, 643 insertions(+), 10 deletions(-)
>
>You didn't even test build this patch:
>
>  CC [M]  drivers/tty/serial/fsl_lpuart.o
>drivers/tty/serial/fsl_lpuart.c: In function ‘lpuart_suspend’:
>drivers/tty/serial/fsl_lpuart.c:1856:22: error: ‘lpuart32_reg’ undeclared
>(first use in this function)
>   uart_suspend_port(&lpuart32_reg, &sport->port);
>                      ^
>
>No quicker way to make me grumpy...
>
>Please take some time to do this right.
>
>greg k-h

Sorry for the mistake, I only test the functionality without option of CONFIG_PM_SLEEP,
I will fix that and send the patch again. Thanks.



Best Regards,
Jingchang





More information about the linux-arm-kernel mailing list