From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.hugovil.com (mail.hugovil.com [162.243.120.170]) (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 BBCE7219E8F; Thu, 9 Jan 2025 14:43:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=162.243.120.170 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736433794; cv=none; b=abvEoVKYKkhdFNjOzJ+Z/qA/c0lXomD5TrTwjteOUUOl8qrDEuvijiV10Z2oL/3f6unPlX9fFgdajYt/3acJoCNo9hOoJKKYv5zweTehYXO524fsNFbaZB/R4REvYHkfClt9++IxnBcioDz1lPHj7WP5eFLOrjh+2q0S1NxeipY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736433794; c=relaxed/simple; bh=0KqcZfWV5+pbBzCM6o36XeWrJStQxVyG5TedUi5+s4w=; h=Date:From:To:Cc:Message-Id:In-Reply-To:References:Mime-Version: Content-Type:Subject; b=TMNe+0afOr9wWsH6HIS/eRXpVL7yxgyDq00D13R8RVC4jfP2JwwaNl7VE7ya1Xes6rQpxweTXve4fSyW9Ld0HX5mQ5aW+IBTFJ2eCNPpdZ1rETwNT1VTY08azHuecOD7w2LS03tJMBQXCc8yFI1R199YmGPtlseFNqWmBVN8vPM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=hugovil.com; spf=pass smtp.mailfrom=hugovil.com; dkim=pass (1024-bit key) header.d=hugovil.com header.i=@hugovil.com header.b=R9YduRrN; arc=none smtp.client-ip=162.243.120.170 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=hugovil.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=hugovil.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=hugovil.com header.i=@hugovil.com header.b="R9YduRrN" DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=hugovil.com ; s=x; h=Subject:Content-Transfer-Encoding:Mime-Version:Message-Id:Cc:To:From :Date:subject:date:message-id:reply-to; bh=5D2Irp/LDipB7s2VJEUMGWiFyWW8CK6K/zuc0+qnBtg=; b=R9YduRrNsikfVdC06kZfZfd0Xp 3qojyCZVzKhXfOAAvqHtI0wjk9+6/1rKQl6hvkPcQaG2ceTQy0WPnZ8eUBqA/PZT6pOmGzGmZnvBt zVFdoWmqw1IJbokGrDVVKflSYZd3i3vu+yRY8ifaQljunEdJkBl+nNbBQZfsaRDpMhPQ=; Received: from modemcable168.174-80-70.mc.videotron.ca ([70.80.174.168]:33694 helo=pettiford.lan) by mail.hugovil.com with esmtpa (Exim 4.92) (envelope-from ) id 1tVtkW-00013s-8Q; Thu, 09 Jan 2025 09:43:05 -0500 Date: Thu, 9 Jan 2025 09:43:03 -0500 From: Hugo Villeneuve To: Andre Werner Cc: gregkh@linuxfoundation.org, jirislaby@kernel.org, hvilleneuve@dimonoff.com, andy@kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org, lech.perczak@camlingroup.com, krzk+dt@kernel.org, conor+dt@kernel.org, robh@kernel.org Message-Id: <20250109094303.9c1917e2e8704ff265dac4e2@hugovil.com> In-Reply-To: <20250109093834.1013025-2-andre.werner@systec-electronic.com> References: <20250109093834.1013025-1-andre.werner@systec-electronic.com> <20250109093834.1013025-2-andre.werner@systec-electronic.com> X-Mailer: Sylpheed 3.8.0beta1 (GTK+ 2.24.33; x86_64-pc-linux-gnu) 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-Transfer-Encoding: 7bit X-SA-Exim-Connect-IP: 70.80.174.168 X-SA-Exim-Mail-From: hugo@hugovil.com X-Spam-Level: X-Spam-Report: * -1.0 ALL_TRUSTED Passed through trusted hosts only via SMTP * -0.1 NICE_REPLY_A Looks like a legit reply (A) Subject: Re: [PATCH v4 2/2] serial: sc16is7xx: Add polling mode if no IRQ pin is available X-SA-Exim-Version: 4.2.1 (built Wed, 08 May 2019 21:11:16 +0000) X-SA-Exim-Scanned: Yes (on mail.hugovil.com) Hi Andre, On Thu, 9 Jan 2025 10:38:34 +0100 Andre Werner wrote: > Fall back to polling mode if no interrupt is configured because there > is no possibility to connect the interrupt pin. > If "interrupts" property is missing in devicetree the driver > uses a delayed worker to pull the state of interrupt status registers. > > Signed-off-by: Andre Werner > --- > V2: > - Change warning for polling mode to debug log entry > - Correct typo: Resuse -> Reuse > - Format define with missing tabs for SC16IS7XX_POLL_PERIOD > - Format struct declaration sc16is7xx_one_config with missing tabs for polling and shutdown > - Adapt dtbinding with new polling feature > V3: > - Use suffix with units and drop a comment SC16IS7XX_POLL_PERIOD_MS. Sorry for that miss. > - Make Kernel lowercase. > V4: > - Reword commit messages for better understanding. > - Remove 'shutdown' property for canceling delayed worker. > - Rename worker function: sc16is7xx_transmission_poll -> sc16is7xx_poll_proc > - Unify argument for worker functions: kthread_work *work -> kthread_work *ws > --- > drivers/tty/serial/sc16is7xx.c | 38 ++++++++++++++++++++++++++++++++++ > 1 file changed, 38 insertions(+) > > diff --git a/drivers/tty/serial/sc16is7xx.c b/drivers/tty/serial/sc16is7xx.c > index a3093e09309f..6645cab2ae5f 100644 > --- a/drivers/tty/serial/sc16is7xx.c > +++ b/drivers/tty/serial/sc16is7xx.c > @@ -314,6 +314,7 @@ > #define SC16IS7XX_FIFO_SIZE (64) > #define SC16IS7XX_GPIOS_PER_BANK 4 > > +#define SC16IS7XX_POLL_PERIOD_MS 10 > #define SC16IS7XX_RECONF_MD BIT(0) > #define SC16IS7XX_RECONF_IER BIT(1) > #define SC16IS7XX_RECONF_RS485 BIT(2) > @@ -348,6 +349,8 @@ struct sc16is7xx_port { > u8 mctrl_mask; > struct kthread_worker kworker; > struct task_struct *kworker_task; > + struct kthread_delayed_work poll_work; > + bool polling; > struct sc16is7xx_one p[]; > }; > > @@ -861,6 +864,18 @@ static irqreturn_t sc16is7xx_irq(int irq, void *dev_id) > return IRQ_HANDLED; > } > > +static void sc16is7xx_poll_proc(struct kthread_work *ws) > +{ > + struct sc16is7xx_port *s = container_of(ws, struct sc16is7xx_port, poll_work.work); > + > + /* Reuse standard IRQ handler. Interrupt ID is unused in this context. */ > + sc16is7xx_irq(0, s); > + > + /* Setup delay based on SC16IS7XX_POLL_PERIOD_MS */ > + kthread_queue_delayed_work(&s->kworker, &s->poll_work, > + msecs_to_jiffies(SC16IS7XX_POLL_PERIOD_MS)); > +} > + > static void sc16is7xx_tx_proc(struct kthread_work *ws) > { > struct uart_port *port = &(to_sc16is7xx_one(ws, tx_work)->port); > @@ -1149,6 +1164,7 @@ static int sc16is7xx_config_rs485(struct uart_port *port, struct ktermios *termi > static int sc16is7xx_startup(struct uart_port *port) > { > struct sc16is7xx_one *one = to_sc16is7xx_one(port, port); > + struct sc16is7xx_port *s = dev_get_drvdata(port->dev); > unsigned int val; > unsigned long flags; > > @@ -1211,6 +1227,10 @@ static int sc16is7xx_startup(struct uart_port *port) > sc16is7xx_enable_ms(port); > uart_port_unlock_irqrestore(port, flags); > > + if (s->polling) > + kthread_queue_delayed_work(&s->kworker, &s->poll_work, > + msecs_to_jiffies(SC16IS7XX_POLL_PERIOD_MS)); > + > return 0; > } > > @@ -1232,6 +1252,9 @@ static void sc16is7xx_shutdown(struct uart_port *port) > > sc16is7xx_power(port, 0); > > + if (s->polling) > + kthread_cancel_delayed_work_sync(&s->poll_work); > + > kthread_flush_worker(&s->kworker); > } > > @@ -1537,6 +1560,12 @@ int sc16is7xx_probe(struct device *dev, const struct sc16is7xx_devtype *devtype, > > /* Always ask for fixed clock rate from a property. */ > device_property_read_u32(dev, "clock-frequency", &uartclk); You are still missing a blank line here to better separate logical blocks (polling is not related to clock-frequency). > + s->polling = !device_property_present(dev, "interrupts"); > + if (s->polling) { > + dev_dbg(dev, > + "No interrupt pin definition, falling back to polling mode\n"); > + irq = 0; > + } > > s->clk = devm_clk_get_optional(dev, NULL); > if (IS_ERR(s->clk)) > @@ -1665,6 +1694,12 @@ int sc16is7xx_probe(struct device *dev, const struct sc16is7xx_devtype *devtype, > goto out_ports; > #endif > > + if (s->polling) { > + /* Initialize kernel thread for polling */ > + kthread_init_delayed_work(&s->poll_work, sc16is7xx_poll_proc); > + return 0; > + } > + > /* > * Setup interrupt. We first try to acquire the IRQ line as level IRQ. > * If that succeeds, we can allow sharing the interrupt as well. > @@ -1724,6 +1759,9 @@ void sc16is7xx_remove(struct device *dev) > sc16is7xx_power(&s->p[i].port, 0); > } > > + if (s->polling) > + kthread_cancel_delayed_work_sync(&s->poll_work); > + > kthread_flush_worker(&s->kworker); > kthread_stop(s->kworker_task); > > -- > 2.47.1 > > > -- Hugo Villeneuve