> On Mar 23, 2021, at 2:22 AM, Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > > On Thu, Feb 18, 2021 at 06:42:09PM -0800, Alex Nemirovsky wrote: >> From: Jason Li <jason.li@xxxxxxxxxxxxxxxxxx> >> >> This driver supports Cortina Access UART IP integrated >> in most all CAXXXX line of SoCs. Earlycom is also supported >> >> Signed-off-by: Jason Li <jason.li@xxxxxxxxxxxxxxxxxx> >> Reviewed-by: Alex Nemirovsky <alex.nemirovsky@xxxxxxxxxxxxxxxxxx> >> --- >> MAINTAINERS | 5 + >> drivers/tty/serial/Kconfig | 19 + >> drivers/tty/serial/Makefile | 1 + >> drivers/tty/serial/serial_cortina-access.c | 798 +++++++++++++++++++++++++++++ >> include/uapi/linux/serial_core.h | 3 + >> 5 files changed, 826 insertions(+) >> create mode 100644 drivers/tty/serial/serial_cortina-access.c >> >> Change log >> drivers/tty/serial/serial_cortina-access.c >> v3: >> - Remove usage of uintptr_t. Change to pointer to driver's private >> structure instead. > > Is this really a "v3"? The subject lines do not show that, so I'm > totally confused as to what to review and what has been reviewed here. > > Please fix this up and submit a "v4" so we know what is going on :) Could you recommend a method or a tool to update the commit subject id with a version prefix? Currently we are doing a git format-patch and the subject line is automatically created without a version number. Do you just go in manual and edit the resulting patch contents file or do you use a tool to assist in this? > > thanks, > > greg k-h