From nobody Mon Feb 9 14:37:26 2026 Received: from mslow1.mail.gandi.net (mslow1.mail.gandi.net [217.70.178.240]) (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 AE4D11FA167; Mon, 2 Dec 2024 10:39:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.70.178.240 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733135995; cv=none; b=rBau1dFDmTnQbbcARXYwn02b6S/dFxivptj4p4x+KkrOTaOh7PipZJ3rSEXpopTPKNG7lA89RQY+fVTb4nlNGu7fpsla0ztVKKB3oPoK5aA69bq7M5Z/CsRGpAv3PC223kR4g9aJ+5rO9kdmN2hug3HARIc+eD4QVJlQAAlOu/A= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733135995; c=relaxed/simple; bh=eK6yjxsDp6WwhycvLzAI+Os6thQ0t/rKpZCVkatl/kk=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:To:Cc; b=Q992UL1krNT2E/K76tj3LFLYKiiVvLu2w4n9/ZLY/5YcaUDH2Nl6NRZFYDoDkjufrJgWIrLJW67lhKtstFKUKrIEMcW+3sb8cwvLzA6ns0EcH7Cugg3GlxchGz1Dx9A6cdWp7kyNyIpEFnMt8LEm4VSt3x5jacmQAfni91JsS/8= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=yoseli.org; spf=none smtp.mailfrom=yoseli.org; dkim=pass (2048-bit key) header.d=yoseli.org header.i=@yoseli.org header.b=WA6WwHq2; arc=none smtp.client-ip=217.70.178.240 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=yoseli.org Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=yoseli.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=yoseli.org header.i=@yoseli.org header.b="WA6WwHq2" Received: from relay9-d.mail.gandi.net (unknown [217.70.183.199]) by mslow1.mail.gandi.net (Postfix) with ESMTP id 706E9C2164; Mon, 2 Dec 2024 10:34:31 +0000 (UTC) Received: by mail.gandi.net (Postfix) with ESMTPSA id 66A5FFF806; Mon, 2 Dec 2024 10:34:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yoseli.org; s=gm1; t=1733135663; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=ztHX/ZoXirD5Kwovsqs5OlEVZDsoMYIef909rL/emsY=; b=WA6WwHq2Mm8ChwQDDMNvUQ7sRyt0VyTLOPMiu0PMpEifDUXMlX3vpD4XL18kFry79JEP8G g2QOPe5+8IV2hqunDDkxLPW7vt8QmZlZkWfs5isiURY10+476/cM3gynYiCo5ptuPOoQRo DVuli/nhCV3waV5kgn5sc8rBGdE7ER/FE6Q8/ZvT4yEs2xKT3AarbBRvZ8Ftq5Y77qEd6s dHlnj2ad8wqPK08FoKc1atxakoyGhVaGdkAWLAvaYM14T7BXC6aztKDMoGhKSEB5e8zlaF OUvjYvKYoQm8jriJrZgafjxjNvOmLd0/OVdjfGaINrEZ3TOr1XRfED1bQlmuIg== From: Jean-Michel Hautbois Date: Mon, 02 Dec 2024 11:34:18 +0100 Subject: [PATCH] m68k: coldfire: Support resources for UART 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="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20241202-m5441x_uart_resource-v1-1-6b28cb295fb5@yoseli.org> X-B4-Tracking: v=1; b=H4sIACmNTWcC/x2MSQqAMBDAviJztmAXFf2KSNE66hxcmKoIpX+3e AnkkATwyIQe2iwA40Oejj2JzDNw67AvKGhKDqpQRiaIrTRGvvYe+LKM/rjZoahVM+rCyVJXGlJ 6Ms70/tuuj/ED3Tcxj2YAAAA= X-Change-ID: 20241202-m5441x_uart_resource-729b30c15363 To: Greg Ungerer , Geert Uytterhoeven , Greg Kroah-Hartman , Jiri Slaby Cc: linux-m68k@lists.linux-m68k.org, linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org, Jean-Michel Hautbois X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=ed25519-sha256; t=1733135661; l=6941; i=jeanmichel.hautbois@yoseli.org; s=20240925; h=from:subject:message-id; bh=eK6yjxsDp6WwhycvLzAI+Os6thQ0t/rKpZCVkatl/kk=; b=gEPFeyR1Efxg+G/L3sujYuDGlqvtW69oVm91xDeG5DM4gcJgE7bG5Sh1qC0o8UljZX/QdSXEX Vhi81Ak/Fm0Dkrjf+DxLMRJQmQx7fK0ujm8RkPm+48FNPczDJKAfAkx X-Developer-Key: i=jeanmichel.hautbois@yoseli.org; a=ed25519; pk=MsMTVmoV69wLIlSkHlFoACIMVNQFyvJzvsJSQsn/kq4= X-GND-Sasl: jeanmichel.hautbois@yoseli.org In order to use the eDMA channels for UART, the mcf_platform_uart needs to be changed. Instead of adding another custom member for the structure, use a resource tree in a platform_device per UART. It then makes it possible to have a device named like "mcfuart.N" with N the UART number. Later, adding the dma channel in the mcf tty driver will also be more straightfoward. Signed-off-by: Jean-Michel Hautbois --- arch/m68k/coldfire/device.c | 96 ++++++++++++++---------------------------= ---- drivers/tty/serial/mcf.c | 69 +++++++++++++++++++------------- 2 files changed, 70 insertions(+), 95 deletions(-) diff --git a/arch/m68k/coldfire/device.c b/arch/m68k/coldfire/device.c index b6958ec2a220cf91a78a14fc7fa18749451412f7..fd7d0b0ce7eb2970cb8ffe33589= fe8d7e88c268d 100644 --- a/arch/m68k/coldfire/device.c +++ b/arch/m68k/coldfire/device.c @@ -24,73 +24,35 @@ #include #include =20 -/* - * All current ColdFire parts contain from 2, 3, 4 or 10 UARTS. - */ -static struct mcf_platform_uart mcf_uart_platform_data[] =3D { - { - .mapbase =3D MCFUART_BASE0, - .irq =3D MCF_IRQ_UART0, - }, - { - .mapbase =3D MCFUART_BASE1, - .irq =3D MCF_IRQ_UART1, - }, -#ifdef MCFUART_BASE2 - { - .mapbase =3D MCFUART_BASE2, - .irq =3D MCF_IRQ_UART2, - }, -#endif -#ifdef MCFUART_BASE3 - { - .mapbase =3D MCFUART_BASE3, - .irq =3D MCF_IRQ_UART3, - }, -#endif -#ifdef MCFUART_BASE4 - { - .mapbase =3D MCFUART_BASE4, - .irq =3D MCF_IRQ_UART4, - }, -#endif -#ifdef MCFUART_BASE5 - { - .mapbase =3D MCFUART_BASE5, - .irq =3D MCF_IRQ_UART5, - }, -#endif -#ifdef MCFUART_BASE6 - { - .mapbase =3D MCFUART_BASE6, - .irq =3D MCF_IRQ_UART6, - }, -#endif -#ifdef MCFUART_BASE7 - { - .mapbase =3D MCFUART_BASE7, - .irq =3D MCF_IRQ_UART7, +static u64 mcf_uart_mask =3D DMA_BIT_MASK(32); + +static struct resource mcf_uart0_resource[] =3D { + [0] =3D { + .start =3D MCFUART_BASE0, + .end =3D MCFUART_BASE0 + 0x3fff, + .flags =3D IORESOURCE_MEM, }, -#endif -#ifdef MCFUART_BASE8 - { - .mapbase =3D MCFUART_BASE8, - .irq =3D MCF_IRQ_UART8, + [1] =3D { + .start =3D 2, + .end =3D 3, + .flags =3D IORESOURCE_DMA, }, -#endif -#ifdef MCFUART_BASE9 - { - .mapbase =3D MCFUART_BASE9, - .irq =3D MCF_IRQ_UART9, + [2] =3D { + .start =3D MCF_IRQ_UART0, + .end =3D MCF_IRQ_UART0, + .flags =3D IORESOURCE_IRQ, }, -#endif - { }, }; =20 -static struct platform_device mcf_uart =3D { +static struct platform_device mcf_uart0 =3D { .name =3D "mcfuart", .id =3D 0, - .dev.platform_data =3D mcf_uart_platform_data, + .num_resources =3D ARRAY_SIZE(mcf_uart0_resource), + .resource =3D mcf_uart0_resource, + .dev =3D { + .dma_mask =3D &mcf_uart_mask, + .coherent_dma_mask =3D DMA_BIT_MASK(32), + }, }; =20 #ifdef MCFFEC_BASE0 @@ -485,12 +447,12 @@ static struct platform_device mcf_i2c5 =3D { static const struct dma_slave_map mcf_edma_map[] =3D { { "dreq0", "rx-tx", MCF_EDMA_FILTER_PARAM(0) }, { "dreq1", "rx-tx", MCF_EDMA_FILTER_PARAM(1) }, - { "uart.0", "rx", MCF_EDMA_FILTER_PARAM(2) }, - { "uart.0", "tx", MCF_EDMA_FILTER_PARAM(3) }, - { "uart.1", "rx", MCF_EDMA_FILTER_PARAM(4) }, - { "uart.1", "tx", MCF_EDMA_FILTER_PARAM(5) }, - { "uart.2", "rx", MCF_EDMA_FILTER_PARAM(6) }, - { "uart.2", "tx", MCF_EDMA_FILTER_PARAM(7) }, + { "mcfuart.0", "rx", MCF_EDMA_FILTER_PARAM(2) }, + { "mcfuart.0", "tx", MCF_EDMA_FILTER_PARAM(3) }, + { "mcfuart.1", "rx", MCF_EDMA_FILTER_PARAM(4) }, + { "mcfuart.1", "tx", MCF_EDMA_FILTER_PARAM(5) }, + { "mcfuart.2", "rx", MCF_EDMA_FILTER_PARAM(6) }, + { "mcfuart.2", "tx", MCF_EDMA_FILTER_PARAM(7) }, { "timer0", "rx-tx", MCF_EDMA_FILTER_PARAM(8) }, { "timer1", "rx-tx", MCF_EDMA_FILTER_PARAM(9) }, { "timer2", "rx-tx", MCF_EDMA_FILTER_PARAM(10) }, @@ -623,7 +585,7 @@ static struct platform_device mcf_flexcan0 =3D { #endif /* MCFFLEXCAN_SIZE */ =20 static struct platform_device *mcf_devices[] __initdata =3D { - &mcf_uart, + &mcf_uart0, #ifdef MCFFEC_BASE0 &mcf_fec0, #endif diff --git a/drivers/tty/serial/mcf.c b/drivers/tty/serial/mcf.c index 93e7dda4d39acd23daf8c0d4c29ac8d666f263c5..07b8decfdb6005f0265dd130765= e45c3fd1715eb 100644 --- a/drivers/tty/serial/mcf.c +++ b/drivers/tty/serial/mcf.c @@ -570,31 +570,46 @@ static struct uart_driver mcf_driver =3D { =20 static int mcf_probe(struct platform_device *pdev) { - struct mcf_platform_uart *platp =3D dev_get_platdata(&pdev->dev); struct uart_port *port; - int i; - - for (i =3D 0; ((i < MCF_MAXPORTS) && (platp[i].mapbase)); i++) { - port =3D &mcf_ports[i].port; - - port->line =3D i; - port->type =3D PORT_MCF; - port->mapbase =3D platp[i].mapbase; - port->membase =3D (platp[i].membase) ? platp[i].membase : - (unsigned char __iomem *) platp[i].mapbase; - port->dev =3D &pdev->dev; - port->iotype =3D SERIAL_IO_MEM; - port->irq =3D platp[i].irq; - port->uartclk =3D MCF_BUSCLK; - port->ops =3D &mcf_uart_ops; - port->flags =3D UPF_BOOT_AUTOCONF; - port->rs485_config =3D mcf_config_rs485; - port->rs485_supported =3D mcf_rs485_supported; - port->has_sysrq =3D IS_ENABLED(CONFIG_SERIAL_MCF_CONSOLE); - - uart_add_one_port(&mcf_driver, port); + struct mcf_uart *pp; + struct resource *res; + void __iomem *base; + int id =3D pdev->id; + + if (id =3D=3D -1 || id >=3D MCF_MAXPORTS) { + dev_err(&pdev->dev, "uart%d out of range\n", + id); + return -EINVAL; } =20 + port =3D &mcf_ports[id].port; + port->line =3D id; + + base =3D devm_platform_get_and_ioremap_resource(pdev, 0, &res); + if (IS_ERR(base)) + return PTR_ERR(base); + + port->mapbase =3D res->start; + port->membase =3D base; + + port->irq =3D platform_get_irq(pdev, 0); + if (port->irq < 0) + return port->irq; + + port->type =3D PORT_MCF; + port->dev =3D &pdev->dev; + port->iotype =3D SERIAL_IO_MEM; + port->uartclk =3D MCF_BUSCLK; + port->ops =3D &mcf_uart_ops; + port->flags =3D UPF_BOOT_AUTOCONF; + port->rs485_config =3D mcf_config_rs485; + port->rs485_supported =3D mcf_rs485_supported; + port->has_sysrq =3D IS_ENABLED(CONFIG_SERIAL_MCF_CONSOLE); + + pp =3D container_of(port, struct mcf_uart, port); + + uart_add_one_port(&mcf_driver, port); + return 0; } =20 @@ -603,13 +618,11 @@ static int mcf_probe(struct platform_device *pdev) static void mcf_remove(struct platform_device *pdev) { struct uart_port *port; - int i; + int id =3D pdev->id; =20 - for (i =3D 0; (i < MCF_MAXPORTS); i++) { - port =3D &mcf_ports[i].port; - if (port) - uart_remove_one_port(&mcf_driver, port); - } + port =3D &mcf_ports[id].port; + if (port) + uart_remove_one_port(&mcf_driver, port); } =20 /*************************************************************************= ***/ --- base-commit: e457f18d7f25288d143c1fe024a620d0b15caec1 change-id: 20241202-m5441x_uart_resource-729b30c15363 Best regards, --=20 Jean-Michel Hautbois