Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1386880

Re: [PATCH 2/2] tty: serial: msm_serial add info message

From Stephen Boyd <sboyd@codeaurora.org>
Newsgroups linux.kernel
Subject Re: [PATCH 2/2] tty: serial: msm_serial add info message
Date 2016-04-25 22:50 +0200
Message-ID <rrVCa-3B8-27@gated-at.bofh.it> (permalink)
References <rr9nP-6Ka-15@gated-at.bofh.it> <rr9nQ-6Ka-27@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 04/23, Frank Rowand wrote:
> From: Frank Rowand <frank.rowand@am.sony.com>
> 
> Failure to enable DMA by the msm_serial driver is silent.
> Add a message to report the failure.
> 
> Signed-off-by: Frank Rowand <frank.rowand@am.sony.com>
> ---
>  drivers/tty/serial/msm_serial.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> Index: b/drivers/tty/serial/msm_serial.c
> ===================================================================
> --- a/drivers/tty/serial/msm_serial.c
> +++ b/drivers/tty/serial/msm_serial.c
> @@ -170,6 +170,7 @@ rel_tx:
>  	dma_release_channel(dma->chan);
>  no_tx:
>  	memset(dma, 0, sizeof(*dma));
> +	dev_info(dev, "msm_serial: DMA not enabled\n");
>  }
>  

Wouldn't this print twice for TX and RX channels? I'd prefer we
not print anything when this driver probes, just because it's a
bunch of log spam that we don't really need.

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 0/2] tty: serial: msm_serial regression and add info message Frank Rowand <frowand.list@gmail.com> - 2016-04-23 19:20 +0200
  [PATCH 2/2] tty: serial: msm_serial add info message Frank Rowand <frowand.list@gmail.com> - 2016-04-23 19:20 +0200
    Re: [PATCH 2/2] tty: serial: msm_serial add info message Stephen Boyd <sboyd@codeaurora.org> - 2016-04-25 22:50 +0200
      Re: [PATCH 2/2] tty: serial: msm_serial add info message Frank Rowand <frowand.list@gmail.com> - 2016-04-25 23:40 +0200
        Re: [PATCH 2/2] tty: serial: msm_serial add info message Stephen Boyd <sboyd@codeaurora.org> - 2016-04-25 23:40 +0200
          Re: [PATCH 2/2] tty: serial: msm_serial add info message Frank Rowand <frowand.list@gmail.com> - 2016-04-26 02:50 +0200

csiph-web