linux-kernel - Re: [PATCH v2] USB: serial: option: adding support for YUGA CLM920-NC5

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20171219102426.GE8917@localhost>
Date:   Tue, 19 Dec 2017 11:24:26 +0100
From:   Johan Hovold <johan@...nel.org>
To:     SZ Lin (林上智) <sz.lin@...a.com>
Cc:     bjorn@...k.no, Taiyi Wu <taiyity.wu@...a.com>,
        Johan Hovold <johan@...nel.org>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] USB: serial: option: adding support for YUGA
 CLM920-NC5

On Tue, Dec 19, 2017 at 11:14:37AM +0100, Johan Hovold wrote:
> On Tue, Dec 19, 2017 at 11:11:43AM +0100, Johan Hovold wrote:
> > 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@...a.com>
> > > Signed-off-by: SZ Lin (林上智) <sz.lin@...a.com>
> > > ---
> > > 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.
> 
> Sorry, I meant "SZ Lin" of course.
> 
> > Care to resend a v3 with your original From and Signed-off-by?

Nevermind, it seems it's just my terminals that can't represent it. I'll
apply the patch as is.

Sorry about the noise.

Johan