Received: by 2002:ab2:60d1:0:b0:1f7:5705:b850 with SMTP id i17csp1299732lqm; Thu, 2 May 2024 10:23:01 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCV0uWnx1Mlg720D985I8tDWU1WEZaMe8TNb0llHbeBl49+20p9BPKByAD9aNq+4bFQhUeFSyKXsK1eXgqSic1IGGnIY6qg4/vQNNcHJYg== X-Google-Smtp-Source: AGHT+IFJRyqI9RPoF5zQv3I94LtLfEhrHArVKqyBocLNiTaLFpgHJMn2Rk2Xk9NCAe3taBfvDQqN X-Received: by 2002:a05:6a21:9989:b0:1ad:92e0:f5d7 with SMTP id ve9-20020a056a21998900b001ad92e0f5d7mr402534pzb.46.1714670581018; Thu, 02 May 2024 10:23:01 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1714670580; cv=pass; d=google.com; s=arc-20160816; b=Rynfh0f/9Fro43GZOiTJliYQ8wl03vuPASoYA0TysXOvkrx8/ZsV1FqA6/6zaLvMcu RoOopSkvfXbhX0FjhZ9DS0QiC72MHCzs6VH6gShyl1VMW/E4mQRupjcJrKgO6sU5FAJp 166XHrqNA6bCB1gyRrFarF7uxTh5QOF59QAP+D7LTbTnxe57oLu72c/ckW4ylwRxGvzJ kIBpwrO2aCxZydCrxNA0zPNNCswN3iRtxdcKypgor30hnB3q3m926oB/vpN0sk4r9c+Y nMASVSJJSBn1OFkIhNVEiv4hptR4S3kZl+0U1T8ipAnA/SHoHxDuK4rQAhU7yfkWyNyL ugxA== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=organization:in-reply-to:content-disposition:mime-version :list-unsubscribe:list-subscribe:list-id:precedence:references :message-id:subject:cc:to:from:date:dkim-signature; bh=a3AE1MwTSFgrChDmZMBhnL5UM7ggiAG196kK3865aVw=; fh=rm3Z9XMSoddT5ih1CDtqeoCdFJOVuVQRhl1tAMPjVgE=; b=c2nLt0pL3dTxraPeOH1l3V5aqSUsD8PuNNznmAHyCuwDf0hHSwL0eiQAqcFoBiiFM1 0QKjtVhJD+c4nYBWhDV+dgT77DtbkdAkbPpDZIL3aVv00Ev03zAl/U1TioFFmsP/3jg3 tmcOdjH9F7ejl4t0PCShDJJePnAVwpqwbG8SA9xqyHFjGVdTWfLiObMTcdOho/KxL1Qo u2bpNtoVGDrO+OLzxiXE9yFxyAk5SJFCvRmOut5txhWAXdw7nHwEcSAtQczJ2tv91xp+ GsxR297FviWzpCYp4mCGeIM3uDWKTDdJf90Yx0HtnXDsq1GEnmrq6vuz+WXOxQWu57Wr ko7w==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b="B/IKTubw"; arc=pass (i=1 dkim=pass dkdomain=intel.com dmarc=pass fromdomain=linux.intel.com); spf=pass (google.com: domain of linux-kernel+bounces-166776-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-166776-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [139.178.88.99]) by mx.google.com with ESMTPS id bx33-20020a056a02052100b005e4a4aa2528si1545711pgb.48.2024.05.02.10.23.00 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 02 May 2024 10:23:00 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-166776-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99; Authentication-Results: mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b="B/IKTubw"; arc=pass (i=1 dkim=pass dkdomain=intel.com dmarc=pass fromdomain=linux.intel.com); spf=pass (google.com: domain of linux-kernel+bounces-166776-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-166776-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 9FBB0281AA9 for ; Thu, 2 May 2024 17:23:00 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id E7F4016FF3D; Thu, 2 May 2024 17:22:54 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="B/IKTubw" Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 91E8816D4C0; Thu, 2 May 2024 17:22:52 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.175.65.17 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1714670574; cv=none; b=DAYp9Pq3nyxQGPMU3wChAM5cSUqep6k7vjoN11LtHOXo3/gzT3C+E9VlQhOMUnY06ye0nJhGymYxtH9f9sr7E8lGGJ5iDKjL821s/zlKkFlA8I5spRd40twbSx/7UUAvqDLWib0oAfjgUbBbDEVAvy0CWFxwph4n6k7pDSkYlQA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1714670574; c=relaxed/simple; bh=xxnRKRRsdiWLKlOgW8fmndRPG/Gun2yKWKn7YZ5of50=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=qnXIadmIHEBQinxCRiYvQaraXyoiQBTghVbnQsd3GHxVfmX2RVug6yRlhrFHXBY5O5vPsaX4EyHXxQnTlm2EDN8iEKDysYcz+l3lA3xPs0wABeGSjkxiu+qVlT8A5YpTD/+2A5pljLIgmx29Tid+VhaSLNzayMAoIRU1boEBakU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com; spf=none smtp.mailfrom=linux.intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=B/IKTubw; arc=none smtp.client-ip=198.175.65.17 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=linux.intel.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1714670572; x=1746206572; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=xxnRKRRsdiWLKlOgW8fmndRPG/Gun2yKWKn7YZ5of50=; b=B/IKTubwUDyIkmrSD+oGIPGtSgkysy0lGJMPjO2U7M8Wvq0V+tmOapGN m+vIvKlUEB/SAolxb+KjFoR8ntohzZ8+aWDCjieI7y5m1r1aROIp8npa5 NK6JCugqJ2sM6juvKuM9Qe38bC7DbtTA6wf9NxIeiEkRIiCWyk9nfnt7t lcpeVd+xROUsK2faVNQ46XBCajEL/o1oGAs2nW+94xcXxDuxyNENYIjHb C2EofbcNR39DjPDQtVjSMV0MhM/0zXwWbXcB6huiBdBcFoIA1CTVIJbZ9 RwTHk4SZKC2bKDmU25CE7IA/2TEcLm8TrnG/T+Lhfrbp6o8znu8sppWAQ w==; X-CSE-ConnectionGUID: 67blkTPYSyu7Imb1EFavCA== X-CSE-MsgGUID: wIu9iSBRTWGw/UQrn9wrCw== X-IronPort-AV: E=McAfee;i="6600,9927,11062"; a="10577993" X-IronPort-AV: E=Sophos;i="6.07,247,1708416000"; d="scan'208";a="10577993" Received: from fmviesa007.fm.intel.com ([10.60.135.147]) by orvoesa109.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 May 2024 10:22:52 -0700 X-CSE-ConnectionGUID: UDIwUJVaSWKh+oGahgMqJg== X-CSE-MsgGUID: 5lm7PBVdS4K5zd38mdUaCA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.07,247,1708416000"; d="scan'208";a="27199840" Received: from smile.fi.intel.com ([10.237.72.54]) by fmviesa007.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 May 2024 10:22:51 -0700 Received: from andy by smile.fi.intel.com with local (Exim 4.97) (envelope-from ) id 1s2a8u-00000003Ojo-0Jhe; Thu, 02 May 2024 20:22:48 +0300 Date: Thu, 2 May 2024 20:22:47 +0300 From: Andy Shevchenko To: Parker Newman Cc: Parker Newman , linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org, Greg Kroah-Hartman , Jiri Slaby Subject: Re: [PATCH v1 00/13] serial: 8250_exar: Clean up the driver Message-ID: References: <20240502144626.2716994-1-andriy.shevchenko@linux.intel.com> <20240502114645.4445b3da@SWDEV2.connecttech.local> <20240502120840.02c65f30@SWDEV2.connecttech.local> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240502120840.02c65f30@SWDEV2.connecttech.local> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo On Thu, May 02, 2024 at 12:08:40PM -0400, Parker Newman wrote: > On Thu, 2 May 2024 19:01:01 +0300 > Andy Shevchenko wrote: > > On Thu, May 02, 2024 at 11:46:45AM -0400, Parker Newman wrote: > > > On Thu, 2 May 2024 17:43:54 +0300 > > > Andy Shevchenko wrote: > > > > > > > After a rework for CONNTECH was done, the driver may need a bit of > > > > love in order to become less verbose (in terms of indentation and > > > > code duplication) and hence easier to read. > > > > > > > > This clean up series fixes a couple of (not so critical) issues and > > > > cleans up the recently added code. No functional change indented by > > > > the cleaning up part. > > > > > > Just an FYI I submitted a patch series that fixed several of these issues but I > > > think it fell through the cracks (I know everyone is very busy!). > > > > > > Link: https://lore.kernel.org/linux-serial/cover.1713533298.git.pnewman@connecttech.com/ > > > > > > I believe my previous patch series is no longer required. This one fixes > > > everything. > > > > I haven't noticed that, if it contains duplicated patches, I may replace mine > > with yours if you insist. > > > > In any case it's better to reply there that you prefer this series to be > > applied, so Greg will not pick it up. > > > > I do not have a preference. I am fine with using yours if it is easier on > the maintainers. Up to you, there is no issue to take your patches in case they are the same (or quite similar) as mine. I can pick them up, just tell me if you want this to happen with a list of the patches (as mail Message-Id). > I will send a reply on my previous series that it is not needed and link > to this. I am new to the mailing lists so I didn't know what the procedure > is for this situation. It's not about mailing lists, it's just a common sense. > Thanks for the fixes :), You are welcome! -- With Best Regards, Andy Shevchenko