From mboxrd@z Thu Jan 1 00:00:00 1970 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 4918EC433F5 for ; Thu, 17 Feb 2022 12:58:34 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237431AbiBQM6q (ORCPT ); Thu, 17 Feb 2022 07:58:46 -0500 Received: from mxb-00190b01.gslb.pphosted.com ([23.128.96.19]:54896 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S240578AbiBQM6p (ORCPT ); Thu, 17 Feb 2022 07:58:45 -0500 Received: from muru.com (muru.com [72.249.23.125]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 0582E28421A for ; Thu, 17 Feb 2022 04:58:30 -0800 (PST) Received: from localhost (localhost [127.0.0.1]) by muru.com (Postfix) with ESMTPS id BD95D80E4; Thu, 17 Feb 2022 12:57:46 +0000 (UTC) Date: Thu, 17 Feb 2022 14:58:28 +0200 From: Tony Lindgren To: Romain Naour Cc: linux-omap@vger.kernel.org Subject: Re: AM5749: tty serial 8250 omap driver crash Message-ID: References: <9f0cb8f9-ac94-eb50-266a-95df521a7360@smile.fr> <114d59cb-bbea-6298-c346-3f50f04ab2a5@smile.fr> <77a24941-4c46-8d78-391a-d3d1018f311a@smile.fr> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <77a24941-4c46-8d78-391a-d3d1018f311a@smile.fr> Precedence: bulk List-ID: X-Mailing-List: linux-omap@vger.kernel.org * Romain Naour [220217 09:09]: > On u-boot devicetree the uart4 node is missing, but we don't plan to use the gps > from uboot :) > Should I add the uart4 node? There should be no need for that, kernel should be able to initialize it properly no matter what the state is from the bootloader. > Since removing the uart quirk had some other side effect, I did a shameless hack > in 8250_omap.c to disable autosuspend. > > - pm_runtime_put_autosuspend(port->dev); > + pm_runtime_dont_use_autosuspend(port->dev); > > With that the uart is always up and running. Yes but note that 8250_omap autosuspend does not do anything unless the timeouts are manually enabled by the userspace. They are initialized to -1 during probe. > > The test script I posted does that for all the uarts, probably best not > > to do that until the other issues are sorted out :) If so, maybe the issue > > on close is that the uart has already autoidled. > > Indeed. But I'm not sure why the autosuspend is enabled by default? See above, it's always been initialized to -1 by default so it won't do anything. But if you ran the test script I posted, autosuspend timeout got enabled for all the uarts. But maybe the issue you posted is yet another issue that I don't quite understand yet. To me it seems we should always have runtime PM functions enable the serial port to usable state and get rid of the conditional enable for probe and dma. Regards, Tony