From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752535AbdFMJzc (ORCPT ); Tue, 13 Jun 2017 05:55:32 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:55286 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752081AbdFMJz3 (ORCPT ); Tue, 13 Jun 2017 05:55:29 -0400 Date: Tue, 13 Jun 2017 11:55:11 +0200 From: Greg Kroah-Hartman To: Johan Hovold Cc: Jiri Slaby , Tony Luck , Fenghua Yu , Samuel Ortiz , "David S. Miller" , linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org, linux-serial@vger.kernel.org Subject: Re: [PATCH 0/9] tty: drop broken alt-speed handling Message-ID: <20170613095511.GA15192@kroah.com> References: <20170606105441.689-1-johan@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170606105441.689-1-johan@kernel.org> User-Agent: Mutt/1.8.3 (2017-05-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jun 06, 2017 at 12:54:32PM +0200, Johan Hovold wrote: > Setting an alt-speed using TIOCSSERIAL and SPD flags has been deprecated > since v2.1.69 and has been broken for all tty drivers but serial-core > since v3.10 and commit 6865ff222cca ("TTY: do not warn about setting > speed via SPD_*") without anyone noticing (for four years). > > Instead of reverting the offending commit, lets get rid of this legacy > code while adding back a warning about the SPD flags being deprecated to > the drivers that once implemented it. Note that drivers implementing > SPD_CUST will continue to support using a custom divisor. > > Also note that serial-core did not rely on the TTY layer for SPD > handling and continues to support it while warning about deprecation > (since 2003 at least). > > Greg, I suggest you take it all trough the TTY tree even if merging > through multiple trees and applying the final patch once everything is > upstream would be an option. Also the irda clean up does not depend on > the rest of series as the code implementing SPD handling was ifdefed > out. I've taken these all through my tty tree, nice work, thanks for the cleanups. greg k-h