From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1752610AbdFMKBK (ORCPT <rfc822;w@1wt.eu>);
        Tue, 13 Jun 2017 06:01:10 -0400
Received: from mail.linuxfoundation.org ([140.211.169.12]:55424 "EHLO
        mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1752522AbdFMKBI (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Tue, 13 Jun 2017 06:01:08 -0400
Date: Tue, 13 Jun 2017 12:01:00 +0200
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Johan Hovold <johan@kernel.org>
Cc: Alan Cox <gnomes@lxorguk.ukuu.org.uk>, Jiri Slaby <jslaby@suse.com>,
        Tony Luck <tony.luck@intel.com>, Fenghua Yu <fenghua.yu@intel.com>,
        Samuel Ortiz <samuel@sortiz.org>,
        "David S. Miller" <davem@davemloft.net>, linux-kernel@vger.kernel.org,
        linux-usb@vger.kernel.org, linux-serial@vger.kernel.org
Subject: Re: [PATCH 8/9] tty: ircomm: remove dead and broken ioctl code
Message-ID: <20170613100100.GA1718@kroah.com>
References: <20170606105441.689-1-johan@kernel.org>
 <20170606105441.689-9-johan@kernel.org>
 <20170606122704.3e328125@lxorguk.ukuu.org.uk>
 <20170613094841.GA31846@kroah.com>
 <20170613095714.GJ10775@localhost>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20170613095714.GJ10775@localhost>
User-Agent: Mutt/1.8.3 (2017-05-23)
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Tue, Jun 13, 2017 at 11:57:14AM +0200, Johan Hovold wrote:
> On Tue, Jun 13, 2017 at 11:48:41AM +0200, Greg Kroah-Hartman wrote:
> > On Tue, Jun 06, 2017 at 12:27:04PM +0100, Alan Cox wrote:
> > > On Tue,  6 Jun 2017 12:54:40 +0200
> > > Johan Hovold <johan@kernel.org> wrote:
> > > 
> > > > Remove three ifdefed and broken implementations of TIOCSSERIAL and
> > > > TIOCGICOUNT, and parity handling in set_termios which had suffered
> > > > severe bit rot.
> > > 
> > > I would be amazed if the IRDA code still works. It's not been tested
> > > properly for years and it never followed the tty rules properly in the
> > > first place - so this looks good, although moving IRDA into staging
> > > and /dev/null would IMHO be far better.
> > > 
> > > IRDA is dead, and IR remotes are handled via completely different code.
> > 
> > I agree, and we should move it into staging and out of the tree, I'll
> > send netdev some patches for that...
> 
> Sounds good, but please consider applying this patch before moving to
> staging as people have been modifying this dead code as part of
> kernel-wide updates even though hasn't even compiled for quite some
> time.

I already applied it :)

I'll wait for 4.13-rc1 before doing it (if I remember...)

thanks,

greg k-h