All of lore.kernel.org
 help / color / mirror / Atom feed
From: Giulio Benetti <giulio.benetti@micronovasrl.com>
To: Eric Tremblay <etremblay@distech-controls.com>
Cc: gregkh@linuxfoundation.org, jslaby@suse.com,
	andriy.shevchenko@linux.intel.com, matwey.kornilov@gmail.com,
	lukas@wunner.de, linux-serial@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	christoph.muellner@theobroma-systems.com, heiko@sntech.de,
	heiko.stuebner@theobroma-systems.com
Subject: Re: [PATCH 0/3] Handle UART without interrupt on TEMT using em485
Date: Fri, 29 Jan 2021 01:12:20 +0100	[thread overview]
Message-ID: <5FC36FF8-8F09-4B82-92C0-BE5E0AA2C117@micronovasrl.com> (raw)

Hi Eric,

> Il giorno 29 gen 2021, alle ore 00:37, Eric Tremblay <etremblay@distech-controls.com> ha scritto:
> 
> The series is mainly about the support of 8250 UART without TEMT
> interrupt. I saw that there was some development in the past but
> it was never merged. Since the last discussion were quite some
> time ago, I was not sure if I should post a v4 over the
> last v3 or start from scratch so I decided to post a new patch. Please
> advice if I should have done the reverse.

Please keep my and Heiko’s SoB and add your SoB too describing between [ ] what Heiko has done and what you’ve done. 
For example:
SoB: Giulio
SoB: Heiko
[Heiko: ...]
SoB Eric
[Eric: improved timeout etc.]

> 
> The approach is a little different from the last proposed patch which was
> doing using a polling at 100us. I tought that it could be really long on 
> some fast baudrate and really not that long on slow baudrate. The current
> approach is to calculate the time of a bytes when the settings are changed.
> When we get the interrupt for the empty FIFO, it's the longer it can take
> to empty the shift register.

Good idea.

> 
> The other two patches are to use that features with the PORT_16550A_FSL64
> found on some chip like the LS1043A.

Do you mind to add my 8250_dw patch that was originally part of this patchset? I had to send it
soon, you’ve preceded me :-)

Thank you
Best regards
Giulio

> 
> Thanks
> 
> Eric Tremblay (3):
> serial: 8250: Handle UART without interrupt on TEMT using em485
> serial: 8250: add compatible for fsl,16550-FIFO64
> serial: 8250: remove UART_CAP_TEMT on PORT_16550A_FSL64
> 
> drivers/tty/serial/8250/8250.h            |  1 +
> drivers/tty/serial/8250/8250_bcm2835aux.c |  2 +-
> drivers/tty/serial/8250/8250_of.c         |  5 ++
> drivers/tty/serial/8250/8250_omap.c       |  2 +-
> drivers/tty/serial/8250/8250_port.c       | 89 ++++++++++++++++++++++-
> include/linux/serial_8250.h               |  2 +
> 6 files changed, 98 insertions(+), 3 deletions(-)
> 
> -- 
> 2.17.1


             reply	other threads:[~2021-01-29  0:13 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-29  0:12 Giulio Benetti [this message]
2021-02-02  0:02 ` [PATCH 0/3] Handle UART without interrupt on TEMT using em485 Eric Tremblay
2021-02-02  0:28   ` Giulio Benetti
  -- strict thread matches above, loose matches on Subject: below --
2021-01-28 23:36 Eric Tremblay

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5FC36FF8-8F09-4B82-92C0-BE5E0AA2C117@micronovasrl.com \
    --to=giulio.benetti@micronovasrl.com \
    --cc=andriy.shevchenko@linux.intel.com \
    --cc=christoph.muellner@theobroma-systems.com \
    --cc=etremblay@distech-controls.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=heiko.stuebner@theobroma-systems.com \
    --cc=heiko@sntech.de \
    --cc=jslaby@suse.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-serial@vger.kernel.org \
    --cc=lukas@wunner.de \
    --cc=matwey.kornilov@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.