RE: [PATCH v2] USB: serial: option: adding support for YUGA CLM920-NC5

From: SZ Lin (æäæ)
Date: Tue Dec 19 2017 - 05:21:40 EST


> -----Original Message-----
> From: Johan Hovold [mailto:jhovold@xxxxxxxxx] On Behalf Of Johan Hovold
> Sent: Tuesday, December 19, 2017 6:12 PM
> To: SZ Lin (æäæ)
> Cc: bjorn@xxxxxxx; Taiyi TY Wu (åææ); Johan Hovold; Greg Kroah-Hartman;
> linux-usb@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH v2] USB: serial: option: adding support for YUGA CLM920-NC5
>
> On Tue, Dec 19, 2017 at 05:40:32PM +0800, SZ Lin (æäæ) wrote:
> > This patch adds support for YUGA CLM920-NC5 PID 0x9625 USB modem to
> > option driver.
> >
> > Interface layout:
> > 0: QCDM/DIAG
> > 1: ADB
> > 2: MODEM
> > 3: AT
> > 4: RMNET
> >
> > Signed-off-by: Taiyi Wu <taiyity.wu@xxxxxxxx>
> > Signed-off-by: SZ Lin (æäæ) <sz.lin@xxxxxxxx>
> > ---
> > Changes from v1:
> > - Fix typo
> > - Remove QCDM/DIAG interface in blacklist
>
> Thanks for the v2. Patch looks good now, but it seems your name cannot be represented in
> UTF-8 (or my setup is broken).
>
> I see you have used the latin version "SF Lin" in the past so that should be fine. I was just
> asking to make sure "SF" wasn't an acronym or nickname.
>
> Care to resend a v3 with your original From and Signed-off-by?
>

SZ Lin (æäæ) is my legal name, but it causes garbled issue in some environment.
Therefore, I usually use SZ Lin as my legal name in patch.

> Thanks,
> Johan