From patchwork Thu Jan 5 12:47:41 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: 640540 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 64C75C3DA7A for ; Thu, 5 Jan 2023 12:48:00 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232370AbjAEMr6 (ORCPT ); Thu, 5 Jan 2023 07:47:58 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38202 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232952AbjAEMrz (ORCPT ); Thu, 5 Jan 2023 07:47:55 -0500 Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4A12E479E8; Thu, 5 Jan 2023 04:47:54 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1672922874; x=1704458874; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=jXQCTdIu2ZeMuZ/yZOgheaRyF6UCRmg5kST3NNxQhro=; b=LnrnqHmyvMATPd6RpuaCaPW1waiSAUd/+j+NDcNnYHxRPj2aiCsk7yyD 7TBg+KfGRJvDd08gPsl67FUa/0APmPQqVAswGl2tWpX+Y2UoHD/QZ1Z7a p8rfFxkyM2GDxw2X4T0sqPCJ0CP7YaRIagK8B6McWEC7/vxDVA1l7jjLf KeDvY6FYVamJvF/I16GUZ/efX2qW8JhYfDWMvjqTE61GlyHnQ41IUl6Qn g+Ciry2SdAHQeJRPAKEOzNDpEjEy35PWOH0+w4TQvflEXyURQFERomQkS X+Y/QoJNDosoeYszXSsbnqnOnmcN8jUXCtPZ2UXO86+NsBMlvWbMoCZCc A==; X-IronPort-AV: E=McAfee;i="6500,9779,10580"; a="301889805" X-IronPort-AV: E=Sophos;i="5.96,302,1665471600"; d="scan'208";a="301889805" Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 05 Jan 2023 04:47:54 -0800 X-IronPort-AV: E=McAfee;i="6500,9779,10580"; a="984287770" X-IronPort-AV: E=Sophos;i="5.96,302,1665471600"; d="scan'208";a="984287770" Received: from khaunx-mobl1.ger.corp.intel.com (HELO ijarvine-MOBL2.ger.corp.intel.com) ([10.252.35.181]) by fmsmga005-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 05 Jan 2023 04:47:52 -0800 From: =?utf-8?q?Ilpo_J=C3=A4rvinen?= To: linux-serial@vger.kernel.org, Greg Kroah-Hartman , Jiri Slaby , =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= Cc: linux-kernel@vger.kernel.org, =?utf-8?q?Ilpo_J=C3=A4rvinen?= Subject: [PATCH v2 0/3] serial: Separate RT288x/Au1xxx code into own file Date: Thu, 5 Jan 2023 14:47:41 +0200 Message-Id: <20230105124744.105950-1-ilpo.jarvinen@linux.intel.com> X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-serial@vger.kernel.org A non-trivial amount of RT288x/Au1xxx code is encapsulated into ifdeffery in 8250_port / 8250_early and some if/switch UPIO_AU blocks. Create a separate file from them and do a few additional cleanups. I kept the Kconfig entry as bool because the code has somewhat tricky dependency chain (mips arch code and 8250_of driver). It would be nice to make it tristate but I don't know how blocking some invalid =m + =y combinations would be best addressed. It should probably be best done on top of this series independent of the split itself anyway. UPIO_AU could now be removed because it's same as UPIO_MEM for the remaining code path but I'm unsure if that's allowed (is the port iotype part of stable ABI or not)? v2: - Define register map array lengths explicitly to avoid creating declaration trap. Ilpo Järvinen (3): serial: 8250: RT288x/Au1xxx code away from core serial: 8250_rt288x: Name non-standard divisor latch reg serial: 8250_rt288x: Remove unnecessary UART_REG_UNMAPPED arch/mips/alchemy/common/platform.c | 10 +- drivers/tty/serial/8250/8250_core.c | 4 + drivers/tty/serial/8250/8250_early.c | 21 ---- drivers/tty/serial/8250/8250_of.c | 4 +- drivers/tty/serial/8250/8250_port.c | 78 --------------- drivers/tty/serial/8250/8250_rt288x.c | 135 ++++++++++++++++++++++++++ drivers/tty/serial/8250/Makefile | 1 + include/linux/serial_8250.h | 14 ++- 8 files changed, 163 insertions(+), 104 deletions(-) create mode 100644 drivers/tty/serial/8250/8250_rt288x.c