Date: Tue, 19 Dec 2017 11:14:37 +0100
From: Johan Hovold <johan@kernel.org>
To: SZ Lin =?utf-8?B?KOael+S4iuaZuik=?= <sz.lin@moxa.com>
Cc: bjorn@mork.no, Taiyi Wu <taiyity.wu@moxa.com>,
        Johan Hovold <johan@kernel.org>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2] USB: serial: option: adding support for YUGA
 CLM920-NC5
Message-ID: <20171219101437.GD8917@localhost>
References: <20171219094032.24077-1-sz.lin@moxa.com>
 <20171219101143.GC8917@localhost>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20171219101143.GC8917@localhost>
User-Agent: Mutt/1.7.2 (2016-11-26)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 919
Lines: 32

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@moxa.com>
> > Signed-off-by: SZ Lin (林上智) <sz.lin@moxa.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?
 
Thanks,
Johan