From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Subject: Re: [PATCH v2 5/9] pps: Move timestamp read into PPS code proper Date: Wed, 13 Feb 2013 10:16:28 -0800 Message-ID: <20130213181628.GA17367@kroah.com> References: <31e24f360b1715c262c05d6708353e292640b3a8.1360677367.git.linux@horizon.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <31e24f360b1715c262c05d6708353e292640b3a8.1360677367.git.linux@horizon.com> Sender: linux-kernel-owner@vger.kernel.org To: George Spelvin Cc: linux-serial@vger.kernel.org, peter@hurleysoftware.com, linux-kernel@vger.kernel.org, giometti@linux.it List-Id: linux-serial@vger.kernel.org On Tue, Feb 12, 2013 at 02:00:43AM -0500, George Spelvin wrote: > The PPS (Pulse-Per-Second) line discipline has developed a number of > unhealthy attachments to core tty data and functions, ultimately leading > to its breakage. > > The previous patches fixed the crashing. This one reduces coupling further > by eliminating the timestamp parameter from the dcd_change ldisc method. > This reduces header file linkage and makes the extension more generic, > and the timestamp read is delayed only slightly, from just before the > ldisc->ops->dcd_change method call to just after. > > Fix attendant build breakage in > drivers/tty/n_tty.c > drivers/tty/tty_buffer.c > drivers/staging/speakup/selection.c I also had to fix up: drivers/staging/dgrp/dgrp_*.c to get things to build properly. thanks, greg k-h