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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 37E10C433EF for ; Wed, 11 May 2022 11:02:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:References:Cc:To:Subject:From: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=ZVchb13eHIeyF7Jr+98IiX9EpQ56uxPdLQVOdYdegN8=; b=19A6L14xsi5EmI zlzkMRvvgMRM3UKVmp9rDD3RcEoHS8Y9jjzgL1JPZBHKlKjXwtkycoNpDYEmwz0jei7KqaQg7wwtS sqRS3TWt9v8trOGrL194a4fwQ2KPZlsY6TP3EqnmXAGSaaAa4KIAN9ansIXbdbIfqKD5Md8Y/8jtr 8V/36QI4axA4c0EJaT4pwWM0ggDB67yr3anFqMCSWNqq2Xukp1m38HNAA7nSydAeFnPt5FhcbSZ09 qI4XaFJj/5RgYWK3Gihs9ZqS4DpQgYK3yAgIAFyT7WzKw4ggmLOAk0Q70LUKHfkwqcH7/tTR12Fjt 5C3oQmDINPgYNNzf6eQw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nok5P-006S6D-Pe; Wed, 11 May 2022 11:00:55 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nok5M-006S5T-Kv for linux-arm-kernel@lists.infradead.org; Wed, 11 May 2022 11:00:54 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 73080ED1; Wed, 11 May 2022 04:00:49 -0700 (PDT) Received: from [192.168.99.12] (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 7F3773F73D; Wed, 11 May 2022 04:00:48 -0700 (PDT) Message-ID: <6d368f6a-6105-e1e8-2b71-38c3d153092a@foss.arm.com> Date: Wed, 11 May 2022 12:00:43 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.9.0 From: Carsten Haitzler Subject: Re: [PATCH] arm64: defconfig: Enable modules for arm displays To: Mark Brown , Liviu Dudau Cc: Sudeep Holla , carsten.haitzler@arm.com, soc@kernel.org, linux-arm-kernel@lists.infradead.org, Robin Murphy References: <20220509125510.152114-1-carsten.haitzler@foss.arm.com> <20220510111545.uqatdfthc6lv5p7i@bogus> <02ce0958-7e19-3315-a39f-548cf41e1f0b@foss.arm.com> Content-Language: en-US Organization: Arm Ltd. In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220511_040052_788207_54EC1021 X-CRM114-Status: GOOD ( 19.29 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 5/10/22 17:31, Mark Brown wrote: > On Tue, May 10, 2022 at 05:19:04PM +0100, Liviu Dudau wrote: >> On Tue, May 10, 2022 at 03:05:21PM +0100, Carsten Haitzler wrote: > >>> is it the: > >>> [ 7.471809] tda9950 0-0034: driver requires an interrupt > >>> that you're talking about Liviu? > >> Yes, that is the warning. > > That doesn't look too worrying TBH, if it's causing confusion we should Looks un-worrying to me. It (to my eyes) looks like an overzealous driver that should silently fail to load if no interrupt can be found as obviously the hardware can't exist, or the DT is misconfigured badly enough so it can never be driven until it's fixed. At best this is some info or so level printk to turn on in debugging your DT or something. > just either fix the tda998x driver to not instantiate the tda9950 unless > it has an interrupt or lower the severity of the log message in the > tda9950 code. It's certanly not something that should block use in a > defconfig. I agree. > Ideally we'd not be using the wildcard compatible in the tda998x and > would know if the CEC functionality was there but I think that ship has > disappeared over the horizon at this point, and in any case someone > might just not want to support CEC for some reason. Right now the best "bang for buck" change would be to lower than complaint to an info level so s/dev_err/dev_info/ or maybe dev_notice there in tda9950.c for that specific complaint. Any preferences? _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel