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 A2460C43334 for ; Fri, 8 Jul 2022 14:20:12 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236977AbiGHOUM (ORCPT ); Fri, 8 Jul 2022 10:20:12 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44916 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236547AbiGHOUL (ORCPT ); Fri, 8 Jul 2022 10:20:11 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7137D24F2F; Fri, 8 Jul 2022 07:20:06 -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 88B1462831; Fri, 8 Jul 2022 14:20:05 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8E400C341C0; Fri, 8 Jul 2022 14:20:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1657290005; bh=bnnMOD+hVTtfzjkgSbWTdqdFyQ813Gs/UhdaiJL80eU=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Llpr1fGuCLViKrjtK5Czp26O1u18WarIs8xUC+Vl4YIqkb1sBLEXzFhyGBcz4Ni6B mPWVUPQKxDWiR1FI6lNF/Mn6VF5ltqji9767/q1WmbaZDxOTLeEM2mHuGRICaARAIo 2YAkxWoSYmWTCIJzUCu7YFmQtH2ik8B3BIksJatqsRBOQ7cCusf1d4GabslcbHOFkS Ic0ao6rIOsT8olsJgRZfETY0pwChDGxSFA92nd1BFc9+hQmCtE2IYRrcW0HhgPg5qa EgLsUxH5de88mknx2bnahbiMq9Rp9xGu9F4KOKWqCZdWTv2Dl7uS/ctLVMi2yd9dLP prKblrFK3mmcg== Received: by pali.im (Postfix) id 4BC237D1; Fri, 8 Jul 2022 16:20:01 +0200 (CEST) Date: Fri, 8 Jul 2022 16:20:01 +0200 From: Pali =?utf-8?B?Um9ow6Fy?= To: Greg Kroah-Hartman Cc: Andy Shevchenko , Jiri Slaby , Johan Hovold , Marek =?utf-8?B?QmVow7pu?= , "open list:SERIAL DRIVERS" , Linux Kernel Mailing List Subject: Re: [PATCH 0/3] serial: Fix support for UPF_SPD_* flags Message-ID: <20220708142001.7fciqnfckznlvsa4@pali> References: <20220321163055.4058-1-pali@kernel.org> <20220707084840.jvsstvyx2ul5ltb6@pali> <20220708132621.4v2es73h52aq3izn@pali> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: NeoMutt/20180716 Precedence: bulk List-ID: X-Mailing-List: linux-serial@vger.kernel.org On Friday 08 July 2022 15:51:01 Greg Kroah-Hartman wrote: > On Fri, Jul 08, 2022 at 03:26:21PM +0200, Pali Rohár wrote: > > On Friday 08 July 2022 15:07:43 Greg Kroah-Hartman wrote: > > > On Thu, Jul 07, 2022 at 10:48:40AM +0200, Pali Rohár wrote: > > > > On Friday 22 April 2022 16:28:06 Greg Kroah-Hartman wrote: > > > > > On Tue, Mar 22, 2022 at 04:29:08PM +0200, Andy Shevchenko wrote: > > > > > > On Mon, Mar 21, 2022 at 11:07 PM Pali Rohár wrote: > > > > > > > > > > > > > > Support for UPF_SPD_* flags is currently broken in more drivers for two > > > > > > > reasons. First one is that uart_update_timeout() function does not > > > > > > > > > > > > the uart_update_timeout() > > > > > > > > > > > > > calculate timeout for UPF_SPD_CUST flag correctly. Second reason is that > > > > > > > userspace termios structre is modified by most drivers after each > > > > > > > > > > > > structure > > > > > > > > > > > > ... > > > > > > > > > > > > > (error handling was ommited for simplification) > > > > > > > > > > > > omitted > > > > > > > > > > > > > After calling set_active_spd_cust_baud() function SPD custom divisor > > > > > > > should be active and therefore is_spd_cust_active() should return true. > > > > > > > > > > > > > > But it is not active (cfgetospeed does not return B38400) and this patch > > > > > > > series should fix it. I have tested it with 8250 driver. > > > > > > > > > > > > drivers > > > > > > > > > > > > > Originally Johan Hovold reported that there may be issue with these > > > > > > > ASYNC_SPD_FLAGS in email: > > > > > > > https://lore.kernel.org/linux-serial/20211007133146.28949-1-johan@kernel.org/ > > > > > > > > > > > > > > > > > > > > > Johan, Greg, could you please test these patches if there is not any > > > > > > > regression? > > > > > > > > > > > > I'm wondering why we are still supporting this ugly hack? > > > > > > Doesn't BOTHER work for you? > > > > > > > > > > Yes, I too do not want to add more support for these old flags. If they > > > > > have not been working, let's not add support for them as obviously no > > > > > one is using them. Let's try to remove them if at all possible. > > > > > > > > Well, it works partially. For more drivers SET method is working, but > > > > GET method returns incorrect value. If your userspace application is > > > > written in a way that does not retrieve from kernel current settings > > > > then it has big probability that application works. > > > > > > I do not understand, sorry, what do you mean by this? > > > > I mean that SET methods are working, GET methods not. In this case SET > > done via ioctl(TIOCSSERIAL) and GET via ioctl(TIOCGSERIAL). > > > > > And as you are responding to a months-old thread, I am totally lost, and > > > don't even know what the patch here was... > > > > > > > So, do you really want to remove support for these old flags completely? > > > > That would of course break above applications. > > > > > > I'm not saying remove them, I'm saying let us not add any more > > > dependancies on them in order to keep new applications from ever wanting > > > to use them. > > > > Last time you wrote to remove them. Now saying not to remove them. So I > > do not understand you now. > > I'm sorry, I am totally lost. So look what you have wrote? Who is lost here is me. > How about starting over and resubmitting > the changes you want and we can go from there. > > greg k-h What to resubmit? I do not understand you. In case you lost emails or accidentally removed them, you can look at them in archive, not? I hope that you do not want me to copy+paste all existing patches with all your quotes on them which you wrote into new emails.