From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 31A62C43334 for ; Mon, 27 Jun 2022 12:39:29 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236211AbiF0Mj2 (ORCPT ); Mon, 27 Jun 2022 08:39:28 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42506 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234976AbiF0Mj1 (ORCPT ); Mon, 27 Jun 2022 08:39:27 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CC241DFE; Mon, 27 Jun 2022 05:39:26 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 6766E61004; Mon, 27 Jun 2022 12:39:26 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 47960C3411D; Mon, 27 Jun 2022 12:39:25 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1656333565; bh=ZnWpSOiSOj0QVZ0k9JieiGzuTBhurNcyR6fM1Su+ZT4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=FX2/TwKw5z+EEBKy+Adrchc9L29Buf0efUPFoEuuIxhfulnVktJFgRUi5IkcD+gcF Y8b7uSRKqdjoPSDwwdjD6Zp9ABTB+fawDin9rTcFdBzuotFu3SrdUpAixm4x/u35+b pI0+9eKwRMP5xdmwntydhpnsunVOJu/hYv/YxAew= Date: Mon, 27 Jun 2022 14:39:23 +0200 From: Greg Kroah-Hartman To: Chanho Park Cc: Krzysztof Kozlowski , Jiri Slaby , Alim Akhtar , Hector Martin , Jaewon Kim , Vincent Whitchurch , linux-samsung-soc@vger.kernel.org, linux-serial@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH] tty: serial: samsung_tty: loopback mode support Message-ID: References: <20220627032353.8868-1-chanho61.park@samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220627032353.8868-1-chanho61.park@samsung.com> Precedence: bulk List-ID: X-Mailing-List: linux-samsung-soc@vger.kernel.org On Mon, Jun 27, 2022 at 12:23:53PM +0900, Chanho Park wrote: > Internal loopback mode can be supported by setting > S3C2443_UCON_LOOPBACK bit. The mode & bit can be supported since > s3c2410 and later SoCs. We can test it by linux-serial-test program[1] > with -k option. It will set TIOCM_LOOP mode during test. > > -k, --loopback Use internal hardware loop back I do not understand this line in the changelog text, what is it for? > > [1]: https://github.com/cbrake/linux-serial-test > Signed-off-by: Chanho Park Our tools require a blank line before the signed-off-by line. Did you not get a warning when checking this patch before sending it out? Also, no need for a footnote in a changelog that is so short, just put the link in the body of the text itself. thanks, greg k-h