From patchwork Fri Sep 15 09:43:35 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Ilpo_J=C3=A4rvinen?= X-Patchwork-Id: 140375 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:612c:172:b0:3f2:4152:657d with SMTP id h50csp976180vqi; Fri, 15 Sep 2023 04:38:12 -0700 (PDT) X-Google-Smtp-Source: AGHT+IEiP2jprTf0ypymV9ezEFvX6upGyDMhNWVraQlSOzbCy5Xz7eoEBVBdjnBEkCWJ21dRhg/j X-Received: by 2002:a05:6358:9143:b0:142:fba8:dcfe with SMTP id r3-20020a056358914300b00142fba8dcfemr1777082rwr.16.1694777892671; Fri, 15 Sep 2023 04:38:12 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1694777892; cv=none; d=google.com; s=arc-20160816; b=H5GWi0suatbYVZ4TaOc28MGwQzDf4Z6C1saoqvPvSWqcct54WLyXnnn1iyAMY+GGnM TEBH/DnQtRG5GnI3Z/7Vqtf24Zf6tratD9UmTNd9b3Rm1P3RVLMpW1EKjbUpVnYmKblL WMRoo8uzdTkQaqczMbxf9FGIRQ0e3GNdFA8WIKkmLl44mEKCkwTW1smnaCsDZ+OdXWDM sj67XYv/MSWAsUwXc/No2r4fwCFxpeuQvz7zdSrgs9pIdJWUfI0ZAxXFukJ/RXX3xZiE /qJoS5v+WvNUl3nmg4AIKmxtnBiPMy39FCuVR3p6Vks2w2tIHs9mhCJZQnvtr/+NeTFm XAxw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=I43AkdlkrtAjZmv1PQUNPWwt3tlBX+xg0OUX7mN20FA=; fh=s/lNR7Eg0A2kOzNwIJ6Zp0rhs5S/2sZrFfkQcCxEeL8=; b=N0/e26+UhVXrxh7BFiVox7ovOSqNhiuKdmcoMwOa6GSmVqTBlZm7COTGcTuU1s0kPb 4F75CZaITxqk+YljMWFxJTHgHgwt/XCjvWHfIZF1YO2KTmecTU/cc0CyrAzTHZGmswl6 KYNaxD06vSn7CQjPVI8AgWLd/JqqW+kaax6IutzG3i1hiWCib6SU4dwhe36X1ty+FKZD yZD7uIFt/u88aMQoEY1yU6yOJnFnCafsRz93BthtpOzFLhHuPSzQIZ/vTX6yvM2Ipz6T pINlm9NmA0veb1izgVwSaUtydvgWQ0gDZ3HcUkDihs8mqefsaXXVn9PsuR1xYvVLcfQw Vj6g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=PbQtS8O2; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.31 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from morse.vger.email (morse.vger.email. [23.128.96.31]) by mx.google.com with ESMTPS id h71-20020a63834a000000b00573fc71f6d6si3213530pge.6.2023.09.15.04.38.12 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 15 Sep 2023 04:38:12 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.31 as permitted sender) client-ip=23.128.96.31; Authentication-Results: mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=PbQtS8O2; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.31 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by morse.vger.email (Postfix) with ESMTP id CE8CA80E6C4E; Fri, 15 Sep 2023 02:46:38 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at morse.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234007AbjIOJqK (ORCPT + 32 others); Fri, 15 Sep 2023 05:46:10 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60362 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233825AbjIOJqD (ORCPT ); Fri, 15 Sep 2023 05:46:03 -0400 Received: from mgamail.intel.com (mgamail.intel.com [192.55.52.115]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id ABB09270A; Fri, 15 Sep 2023 02:44:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1694771095; x=1726307095; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=/AlIt0LhwQnYsaDqrP7kQEj98W3VwaOUUdLbofv2N8w=; b=PbQtS8O2xuSGYhYktmFIxrkQ0sbVJ7ptm8hdS6pWGwI9VF6EtOugrl5D 9XlJ6xV1A610KExEolZ8F7/4rfgDcOtJC8MyuvsmDqQFYr3yXu9V8Wqux PVd2UjCitkBRbFNJOOaDIGzollXzDn013P/8rjrnbLGuhGKSheMfZHqIV HSTm2bx2dUK75VFmvSnm4APC0UnE9UKutd1yHwMu4LciAU0AVkM4r+dwW SIIQ06gp48WlJc7tKjvKQpOncnBp2aBBzsbzVUy7g9f85yTKwP7QqNnmK LdaQXTR/H5qvtLkXT8kacoHVfE7zJHoIUb0H2UTRE3mq5/+7J31gs7+x+ A==; X-IronPort-AV: E=McAfee;i="6600,9927,10833"; a="379128055" X-IronPort-AV: E=Sophos;i="6.02,148,1688454000"; d="scan'208";a="379128055" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Sep 2023 02:43:45 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10833"; a="774245912" X-IronPort-AV: E=Sophos;i="6.02,148,1688454000"; d="scan'208";a="774245912" Received: from srdoo-mobl1.ger.corp.intel.com (HELO ijarvine-mobl2.ger.corp.intel.com) ([10.252.38.99]) by orsmga008-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Sep 2023 02:43:43 -0700 From: =?utf-8?q?Ilpo_J=C3=A4rvinen?= To: Greg Kroah-Hartman , Jiri Slaby , linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org Cc: =?utf-8?q?Ilpo_J=C3=A4rvinen?= Subject: [PATCH 1/2] serial: 8250_mid: Remove 8250_pci usage Date: Fri, 15 Sep 2023 12:43:35 +0300 Message-Id: <20230915094336.13278-1-ilpo.jarvinen@linux.intel.com> X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on morse.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (morse.vger.email [0.0.0.0]); Fri, 15 Sep 2023 02:46:38 -0700 (PDT) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1777103423277828699 X-GMAIL-MSGID: 1777103423277828699 8250_mid uses FL_*BASE* from linux/8250_pci.h and nothing else. The code can be simplified by directly defining BARs within the driver instead. Signed-off-by: Ilpo Järvinen --- drivers/tty/serial/8250/8250_mid.c | 18 +++++++----------- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git a/drivers/tty/serial/8250/8250_mid.c b/drivers/tty/serial/8250/8250_mid.c index 2cc78a4bf7a1..8ec03863606e 100644 --- a/drivers/tty/serial/8250/8250_mid.c +++ b/drivers/tty/serial/8250/8250_mid.c @@ -12,7 +12,6 @@ #include #include -#include #include "8250.h" @@ -32,9 +31,9 @@ struct mid8250; struct mid8250_board { - unsigned int flags; unsigned long freq; unsigned int base_baud; + unsigned int bar; int (*setup)(struct mid8250 *, struct uart_port *p); void (*exit)(struct mid8250 *); }; @@ -169,7 +168,6 @@ static int dnv_setup(struct mid8250 *mid, struct uart_port *p) { struct hsu_dma_chip *chip = &mid->dma_chip; struct pci_dev *pdev = to_pci_dev(p->dev); - unsigned int bar = FL_GET_BASE(mid->board->flags); int ret; pci_set_master(pdev); @@ -183,7 +181,7 @@ static int dnv_setup(struct mid8250 *mid, struct uart_port *p) chip->dev = &pdev->dev; chip->irq = pci_irq_vector(pdev, 0); chip->regs = p->membase; - chip->length = pci_resource_len(pdev, bar); + chip->length = pci_resource_len(pdev, mid->board->bar); chip->offset = DNV_DMA_CHAN_OFFSET; /* Falling back to PIO mode if DMA probing fails */ @@ -291,7 +289,6 @@ static int mid8250_probe(struct pci_dev *pdev, const struct pci_device_id *id) { struct uart_8250_port uart; struct mid8250 *mid; - unsigned int bar; int ret; ret = pcim_enable_device(pdev); @@ -303,7 +300,6 @@ static int mid8250_probe(struct pci_dev *pdev, const struct pci_device_id *id) return -ENOMEM; mid->board = (struct mid8250_board *)id->driver_data; - bar = FL_GET_BASE(mid->board->flags); memset(&uart, 0, sizeof(struct uart_8250_port)); @@ -316,8 +312,8 @@ static int mid8250_probe(struct pci_dev *pdev, const struct pci_device_id *id) uart.port.flags = UPF_SHARE_IRQ | UPF_FIXED_PORT | UPF_FIXED_TYPE; uart.port.set_termios = mid8250_set_termios; - uart.port.mapbase = pci_resource_start(pdev, bar); - uart.port.membase = pcim_iomap(pdev, bar, 0); + uart.port.mapbase = pci_resource_start(pdev, mid->board->bar); + uart.port.membase = pcim_iomap(pdev, mid->board->bar, 0); if (!uart.port.membase) return -ENOMEM; @@ -353,25 +349,25 @@ static void mid8250_remove(struct pci_dev *pdev) } static const struct mid8250_board pnw_board = { - .flags = FL_BASE0, .freq = 50000000, .base_baud = 115200, + .bar = 0, .setup = pnw_setup, .exit = pnw_exit, }; static const struct mid8250_board tng_board = { - .flags = FL_BASE0, .freq = 38400000, .base_baud = 1843200, + .bar = 0, .setup = tng_setup, .exit = tng_exit, }; static const struct mid8250_board dnv_board = { - .flags = FL_BASE1, .freq = 133333333, .base_baud = 115200, + .bar = 1, .setup = dnv_setup, .exit = dnv_exit, }; From patchwork Fri Sep 15 09:43:36 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Ilpo_J=C3=A4rvinen?= X-Patchwork-Id: 140792 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:612c:172:b0:3f2:4152:657d with SMTP id h50csp1393619vqi; Fri, 15 Sep 2023 17:08:02 -0700 (PDT) X-Google-Smtp-Source: AGHT+IH38XCwXYoJfGhQOyf2Mi4CfB99PjlIRWOO56CQgxs+h4R/o4mrWsXYmvwZPnTbJK9sL5dT X-Received: by 2002:a05:6358:2806:b0:140:fbfe:d941 with SMTP id k6-20020a056358280600b00140fbfed941mr3126939rwb.20.1694822881970; Fri, 15 Sep 2023 17:08:01 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1694822881; cv=none; d=google.com; s=arc-20160816; b=yc0jsNUcU7vOC9mu9dxexEcQZhag8dKsIpKFZzFKmLRTBFAymq3cB17OAj+IruEJ/8 dDD/ewdtsGNirSQQeLs9+3plCly0ABhBqkOYj/zLgDqxaolaNWaLPUzyGaCenz/5QnUU QOyfg7yOvDUYiLIKQgfHw/UW4xSRdT+eiS7zERK13kwYPi2oA7vJQsQ+Plh/vK2CDDHH sT7bd33+inz650cvYMqhpb6rDFV+28CiJYOvn3yeBfiTsyD4OdRO4kUdzce4Ur4B03li 4KEAHQVavrJ158LoiGnBMdnF8uYVlreKsYVGum++EqPBICaqtBzs9XJK7Co6M7saOiNh ZLiw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=PxkgmB54g+fwImXeID5Gr57SahR7UOUNmrNrdWZrDi8=; fh=Qo7YC0TKhshZNMpna9JHC6TNxaj0284/ioP1D6v5Lys=; b=UgL0bDyyf/lMHNMJNVkIoA8XqFBgAxkFekofLVfgeD3Is+p1lZGPAc3gbd4ia1YvRi 82RMjGCg1iebE/lS9wmtl4Qg4Ly4tkzIm2R8bDV2/PMx4uw+aFu/M9yRd4xj5h9wjQSD pN6zy5zP6hysrbwiS6H31VhY0fN9+kWSfZwMIwp5aXYgJUP91RIkRr62iFSiib4Avgvc i82dX1yGBgPpDBg1O5W0EPXWgCE/4qxJ9ica1wX0cLa7mW8K5uel1pt7oKuOgOcGZ3p3 /Vefn92e43hyLAlQaT107+cXgQTuAnEfH/Cn8jAIdNMZuFBv/zu0Q8iU2WqcEFscGdR5 3s/g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=d+SiNYew; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.38 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from fry.vger.email (fry.vger.email. [23.128.96.38]) by mx.google.com with ESMTPS id j73-20020a638b4c000000b00574057d7c19si3892130pge.220.2023.09.15.17.08.01 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 15 Sep 2023 17:08:01 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.38 as permitted sender) client-ip=23.128.96.38; Authentication-Results: mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=d+SiNYew; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.38 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by fry.vger.email (Postfix) with ESMTP id F1E9680698D2; Fri, 15 Sep 2023 02:48:04 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at fry.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233894AbjIOJr7 (ORCPT + 32 others); Fri, 15 Sep 2023 05:47:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56798 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233837AbjIOJr5 (ORCPT ); Fri, 15 Sep 2023 05:47:57 -0400 Received: from mgamail.intel.com (mgamail.intel.com [192.55.52.115]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9BAD23ABD; Fri, 15 Sep 2023 02:45:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1694771100; x=1726307100; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=28Q6FS8KH3B7iBp/MzaF66/guYlDU+uUua9Jud0/lCE=; b=d+SiNYew3NOfxT6auHt4JFVaFjAyaHhhAOTtFX+HMz+wI5f2GmRdC9rK rZ005imGoUYnKJp65OEBgydLVYoj+ET+bpyYRiASMlmcmJSg2Ecz+IWNa NNSVzaMbG1WyU0fynpDgpAo/BXm+L2P4ZvEhZjWil497ZcxHwDdEamm5i kuwZYqfr5JnsF5WZCx4nKrnoB/gU2fR9l+tU/gdGd7v1Dn7G+VJJUNLoJ oJMXFaWmT9C4x5kd5W0v30fZQYsTMoB5cdGdxDtXbeM+V1cu9q7gPE7Ni 4xnv99DKDGil3rqNfLRzBN3+2cl5yYVqEEVbjrYGG4HUj7WHKnA40yRTt g==; X-IronPort-AV: E=McAfee;i="6600,9927,10833"; a="379128092" X-IronPort-AV: E=Sophos;i="6.02,148,1688454000"; d="scan'208";a="379128092" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Sep 2023 02:43:48 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10833"; a="774245921" X-IronPort-AV: E=Sophos;i="6.02,148,1688454000"; d="scan'208";a="774245921" Received: from srdoo-mobl1.ger.corp.intel.com (HELO ijarvine-mobl2.ger.corp.intel.com) ([10.252.38.99]) by orsmga008-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Sep 2023 02:43:46 -0700 From: =?utf-8?q?Ilpo_J=C3=A4rvinen?= To: Greg Kroah-Hartman , Jiri Slaby , linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org Cc: =?utf-8?q?Ilpo_J=C3=A4rvinen?= , Matthew Howell Subject: [PATCH 2/2] tty: serial: 8250_exar: Does not use anything from 8250_pci Date: Fri, 15 Sep 2023 12:43:36 +0300 Message-Id: <20230915094336.13278-2-ilpo.jarvinen@linux.intel.com> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20230915094336.13278-1-ilpo.jarvinen@linux.intel.com> References: <20230915094336.13278-1-ilpo.jarvinen@linux.intel.com> MIME-Version: 1.0 X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on fry.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (fry.vger.email [0.0.0.0]); Fri, 15 Sep 2023 02:48:05 -0700 (PDT) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1777150598262454277 X-GMAIL-MSGID: 1777150598262454277 8250_exar includes linux/8250_pci.h and depends on SERIAL_8250_PCI. Neither is necessary so this patch removes the include and changes the depends on to SERIAL_8250 && PCI (taken from SERIAL_8250_PCI). Signed-off-by: Ilpo Järvinen --- Cc: Matthew Howell drivers/tty/serial/8250/8250_exar.c | 1 - drivers/tty/serial/8250/Kconfig | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/tty/serial/8250/8250_exar.c b/drivers/tty/serial/8250/8250_exar.c index 077c3ba3539e..2f9813429278 100644 --- a/drivers/tty/serial/8250/8250_exar.c +++ b/drivers/tty/serial/8250/8250_exar.c @@ -18,7 +18,6 @@ #include #include #include -#include #include #include diff --git a/drivers/tty/serial/8250/Kconfig b/drivers/tty/serial/8250/Kconfig index ee17cf5c44c6..d91924cb9b21 100644 --- a/drivers/tty/serial/8250/Kconfig +++ b/drivers/tty/serial/8250/Kconfig @@ -148,7 +148,7 @@ config SERIAL_8250_PCI config SERIAL_8250_EXAR tristate "8250/16550 Exar/Commtech PCI/PCIe device support" - depends on SERIAL_8250_PCI + depends on SERIAL_8250 && PCI default SERIAL_8250 help This builds support for XR17C1xx, XR17V3xx and some Commtech