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 AEC96C433FE for ; Tue, 22 Nov 2022 20:49:51 +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-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:References :In-Reply-To:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=WlOx7NJbYmQLh40v7MdZL2FEmQMMqhvKOWCt9arLZJk=; b=pGFYHTB23xoYDF Vf7NnpduU3Bz0CUXoZQT5+PUVspzcpEpLkLFh7nQgXdxzvDVk9DfpWL/8UT/zxVGvdLiG79EB4e6A Yc244G0uLl5P4gPNWaHFi7dVj+rNhu6ElUafiTzBw4HJ5PPWTcaaC75grHaiQFt/PYAHzeBvm3xfy P3etj2a9p6gYXszduxuqyi3oW+3NF/Tq4yIexXQFHJMSPPZncWfSEeTLT69FuEM6Edau3tVHJqy5p s+3D79upnSgMPJLtERledPCPm/hk8UHVWod6Gin+i2OrKpvMTp5oIkWdgEZR2smdAJm0Rq7Cro0cH xA/L+ufu5QQCo4AmXd2Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oxaC1-00BtCw-Ki; Tue, 22 Nov 2022 20:48:33 +0000 Received: from mail-pg1-x52c.google.com ([2607:f8b0:4864:20::52c]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oxaBy-00BtCF-JP for linux-arm-kernel@lists.infradead.org; Tue, 22 Nov 2022 20:48:32 +0000 Received: by mail-pg1-x52c.google.com with SMTP id n17so15048999pgh.9 for ; Tue, 22 Nov 2022 12:48:28 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20210112.gappssmtp.com; s=20210112; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:from:to:cc:subject:date:message-id:reply-to; bh=qQ4il4WCPkNns9UB7hYX0PSzg5JhITm8H9TsWBohCps=; b=1fnX0Z4rNDw3WqW7vxA09Nvzwl604ChRAQDY326MTPWvgOWsIg9zOYQXUVcDfRFQcG GMK2za3OoxZSU4iAbDaPm2HH/xuJUYw+xIAfsHc4Tg5QCr5CvenylZaIg8MP3H5F4q7c ByHy6lQP98I0X6BkktI1gt2TubmQmdioHQoqYsULG/DAkrxYa6RF8pm0YOxo7OvTx5j0 EDQk+ZkyYNY59jtwVsGHRGlCxmeHciOt8Efxwa5lJVI1DHjM5PQQ++0XcNlkwwz6LEKk b7DKgD6oD99TIw+f+o7HSMe2KeRXVUu4XZuB/sXSzwsZA1G8BWMrS2oacO8E/aHIPjYl K/Rw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=qQ4il4WCPkNns9UB7hYX0PSzg5JhITm8H9TsWBohCps=; b=hXCGKFHexHoErwVjw+LZNneXqS09l7EWaC1/Fo2tusOBhDA7SUiHIj95YUra2Coclb N8iomeiKTqAdGr4FtAAaklyAgDWrF4E9ZE4pXfJqai/vA10pAHco4QyoqR4FRR51AAIj jHLecarDtitRWsZs1fKUptQuQDsGQbAS0yA7inb4Fqmf+ST6BjNgNp+AY2nCSH7NCupy pMM2WituAQikzFc6/CfRwryBXue185nD9Jd/Iih3d2ePgjJePwVRt1cb1myco6AQXvI2 GFLtPYoxMqjvbMR4g4vMXuSD5BYybnmu9TN14mEUCwyLakqL1dia5M78DbwKzevkO3O7 LpYA== X-Gm-Message-State: ANoB5pmyUN0bo0HzMs0InDIYlKxZBFyRnVWIU9SyJhy0YTHeGrMAiDDp GWrrgQjoZWxF2f8Qf/WOkNKZzQ== X-Google-Smtp-Source: AA0mqf4kN1U4OHLHbJ789kCmBZLZ4vW6PKsHiJnp5W2dY4sK+ojJihZwT+uTMhpuv+/LcZAoirBvZQ== X-Received: by 2002:a05:6a00:16c6:b0:573:65d4:a104 with SMTP id l6-20020a056a0016c600b0057365d4a104mr16579849pfc.85.1669150107446; Tue, 22 Nov 2022 12:48:27 -0800 (PST) Received: from localhost ([75.172.139.56]) by smtp.gmail.com with ESMTPSA id a13-20020aa794ad000000b0056ba02feda1sm11442823pfl.94.2022.11.22.12.48.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 22 Nov 2022 12:48:26 -0800 (PST) From: Kevin Hilman To: Nishanth Menon , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, nfrayer@baylibre.com, Santosh Shilimkar Cc: Nishanth Menon , glaroque@baylibre.com Subject: Re: [PATCH v6 0/2] soc: ti: k3-socinfo: Add module build support to the k3 socinfo driver In-Reply-To: <166906774215.20855.4956750581421635106.b4-ty@ti.com> References: <20221117095419.171906-1-nfrayer@baylibre.com> <166906774215.20855.4956750581421635106.b4-ty@ti.com> Date: Tue, 22 Nov 2022 12:48:26 -0800 Message-ID: <7ha64iaf6t.fsf@baylibre.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221122_124830_659275_AB36E0D5 X-CRM114-Status: GOOD ( 17.98 ) 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-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Nishanth, Nishanth Menon writes: > Hi Nicolas Frayer, > > On Thu, 17 Nov 2022 10:54:17 +0100, Nicolas Frayer wrote: >> In order for the TI K3 SoC info driver to be built as a module, the >> following changes have been made: >> - Converted memory allocations to devm and added the remove callback >> - Added necessary code to build the driver as a module >> >> v2->v3: >> dropped module conversion part of this series while other driver >> dependencies on socinfo are worked out. >> A dependency issue is introduced by changing subsys_initcall() >> to module_platform_driver(). Some drivers using the socinfo information >> probe before the socinfo driver itself and it makes their probe fail. >> >> [...] > > I have applied the following to branch ti-drivers-soc-next on [1]. > Thank you! Sorry for the churn here, but I think these need to be reverted. We've discoverd some more complexities/issues when socinfo is modular, and I think the socinfo core code needs some work before this can actually work reliably for all socinfo users. Basically, I think we need to finish converting all the socinfo users to modules before we can be sure of all the corner cases when socinfo itself is a module. Sorry about this churn, but I think it's better to be a bit conservative here and revert before this lands in mainline. Thanks, Kevin _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel 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 98AEAC4332F for ; Tue, 22 Nov 2022 20:48:52 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234559AbiKVUsu (ORCPT ); Tue, 22 Nov 2022 15:48:50 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55938 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235128AbiKVUsa (ORCPT ); Tue, 22 Nov 2022 15:48:30 -0500 Received: from mail-pg1-x52a.google.com (mail-pg1-x52a.google.com [IPv6:2607:f8b0:4864:20::52a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 444ED1C9 for ; Tue, 22 Nov 2022 12:48:28 -0800 (PST) Received: by mail-pg1-x52a.google.com with SMTP id 6so15051686pgm.6 for ; Tue, 22 Nov 2022 12:48:28 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20210112.gappssmtp.com; s=20210112; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:from:to:cc:subject:date:message-id:reply-to; bh=qQ4il4WCPkNns9UB7hYX0PSzg5JhITm8H9TsWBohCps=; b=1fnX0Z4rNDw3WqW7vxA09Nvzwl604ChRAQDY326MTPWvgOWsIg9zOYQXUVcDfRFQcG GMK2za3OoxZSU4iAbDaPm2HH/xuJUYw+xIAfsHc4Tg5QCr5CvenylZaIg8MP3H5F4q7c ByHy6lQP98I0X6BkktI1gt2TubmQmdioHQoqYsULG/DAkrxYa6RF8pm0YOxo7OvTx5j0 EDQk+ZkyYNY59jtwVsGHRGlCxmeHciOt8Efxwa5lJVI1DHjM5PQQ++0XcNlkwwz6LEKk b7DKgD6oD99TIw+f+o7HSMe2KeRXVUu4XZuB/sXSzwsZA1G8BWMrS2oacO8E/aHIPjYl K/Rw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=qQ4il4WCPkNns9UB7hYX0PSzg5JhITm8H9TsWBohCps=; b=5Wp+m7L8G2cKqLNe/qN1/ZD6s+5zmAJwcBJZ7E/y22uRZvvZc/1/HELucaXZ4nB2RP 5PRlGB3HmUZ3zyihhaIWtajm5mkdxdxEUOnj0i29oMfKPKAuR+HngRX+3ihRYNvu919l H0k3Rm+hJfVTx9ZmSXBlQwExIIe9oeSxNwnnlfmSoE1Hk0oq6/ltD5pc5hoT9oIeFZFT Lc717UEEkHbOjrGEZmZ8DYmtQ4utTtmpV9mRfVtjpxtD6cCXedzrYoIfSdsn3U9xyao+ GSgvamCxW8Hna6m43VZcS079Tvjn6rSvnIigv4Tnh0Aa8oLMHmXu8xptFAN+B6rlL5GM mObw== X-Gm-Message-State: ANoB5pmfFHbFYr2FXQ7hShyWQPQyz4Nqi7gbWAtJoDYsL++2kycwG7Rs oLCwoyamWUn7nNxhcWCzhqGkTQ== X-Google-Smtp-Source: AA0mqf4kN1U4OHLHbJ789kCmBZLZ4vW6PKsHiJnp5W2dY4sK+ojJihZwT+uTMhpuv+/LcZAoirBvZQ== X-Received: by 2002:a05:6a00:16c6:b0:573:65d4:a104 with SMTP id l6-20020a056a0016c600b0057365d4a104mr16579849pfc.85.1669150107446; Tue, 22 Nov 2022 12:48:27 -0800 (PST) Received: from localhost ([75.172.139.56]) by smtp.gmail.com with ESMTPSA id a13-20020aa794ad000000b0056ba02feda1sm11442823pfl.94.2022.11.22.12.48.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 22 Nov 2022 12:48:26 -0800 (PST) From: Kevin Hilman To: Nishanth Menon , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, nfrayer@baylibre.com, Santosh Shilimkar Cc: Nishanth Menon , glaroque@baylibre.com Subject: Re: [PATCH v6 0/2] soc: ti: k3-socinfo: Add module build support to the k3 socinfo driver In-Reply-To: <166906774215.20855.4956750581421635106.b4-ty@ti.com> References: <20221117095419.171906-1-nfrayer@baylibre.com> <166906774215.20855.4956750581421635106.b4-ty@ti.com> Date: Tue, 22 Nov 2022 12:48:26 -0800 Message-ID: <7ha64iaf6t.fsf@baylibre.com> MIME-Version: 1.0 Content-Type: text/plain Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Nishanth, Nishanth Menon writes: > Hi Nicolas Frayer, > > On Thu, 17 Nov 2022 10:54:17 +0100, Nicolas Frayer wrote: >> In order for the TI K3 SoC info driver to be built as a module, the >> following changes have been made: >> - Converted memory allocations to devm and added the remove callback >> - Added necessary code to build the driver as a module >> >> v2->v3: >> dropped module conversion part of this series while other driver >> dependencies on socinfo are worked out. >> A dependency issue is introduced by changing subsys_initcall() >> to module_platform_driver(). Some drivers using the socinfo information >> probe before the socinfo driver itself and it makes their probe fail. >> >> [...] > > I have applied the following to branch ti-drivers-soc-next on [1]. > Thank you! Sorry for the churn here, but I think these need to be reverted. We've discoverd some more complexities/issues when socinfo is modular, and I think the socinfo core code needs some work before this can actually work reliably for all socinfo users. Basically, I think we need to finish converting all the socinfo users to modules before we can be sure of all the corner cases when socinfo itself is a module. Sorry about this churn, but I think it's better to be a bit conservative here and revert before this lands in mainline. Thanks, Kevin