From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f43.google.com (mail-wm1-f43.google.com [209.85.128.43]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 39B9F4C98 for ; Sat, 22 Nov 2025 15:37:35 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.43 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763825858; cv=none; b=qezGFd/E63PuB93fp9IYVPdYUEn67Zn2sFocMdI5VUlOU3wxuZGxEtswAQGUKkjUkUOBnJrdYkyj5lHgL44F6wRNFGTEjEcZ7r8eAeTkLq5O7lFE5kN+qg8vwO/hoyVQTCzxQAp9P+R4dsjODmm3HEildT42HkXXsDOTK+lY4hg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763825858; c=relaxed/simple; bh=lLIUauY1z40WJRqEx/JDgzSpk+cO4SKTkxU1CF/geXM=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=Ebj+9MpJPNfB0nDgrgzVI3oQvW+NYqezfk8a4UPahb9OrimjafrpxssmvGbq0JXFBbOes/ffnggU2AGtHTZ/0FXi9Ks291Bw7tpMNfCv+ECPIqBu/K1m1apxrI+8fra32gfC9/TzWk0pjDoxTd+xk9TcPyeUc7iOzls3Wo/QDFE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=tuxon.dev; spf=pass smtp.mailfrom=tuxon.dev; dkim=pass (2048-bit key) header.d=tuxon.dev header.i=@tuxon.dev header.b=OMpHYCv+; arc=none smtp.client-ip=209.85.128.43 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=tuxon.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=tuxon.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=tuxon.dev header.i=@tuxon.dev header.b="OMpHYCv+" Received: by mail-wm1-f43.google.com with SMTP id 5b1f17b1804b1-477b5e0323bso20960355e9.0 for ; Sat, 22 Nov 2025 07:37:35 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tuxon.dev; s=google; t=1763825854; x=1764430654; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:content-language:from :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=eTyNC1+UYoKVAcGxt2SVHOhb/Uf0xrNXGcZihp5xtts=; b=OMpHYCv+rcOPch6xXjQlAapCJcMnN5daSLCCngJz+RnlGXanWUU+82HfY2rN7GKf9k IzpvSU32sx8jkuihtNK+wtUr5r5RT0F23gHIWRfDb5fG+e2zAS6Q12pUBAQBCfEa1s0w Qv0IWsnTT0KNAOjQQO/EYIP5uicYmX4zsyqFToZjMA6HfZntlJBSo16wvtOa674D5uG5 c2kyV8h3IEVFEpqe8DHu4QRRicMrvPgl/hkWAI3dyC+scl9GzR1hsGsPl+WT0o/11HNq EMOcFWdqmiOlMI2e3FoR+JzleBpZKFOzBOfzEA5tOUkm7eZ+Ot0aZVp1JTGw66j/6xuJ whtA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763825854; x=1764430654; h=content-transfer-encoding:in-reply-to:content-language:from :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=eTyNC1+UYoKVAcGxt2SVHOhb/Uf0xrNXGcZihp5xtts=; b=mHZ44QyvTWVkdiKSznxdHJNJ83RLdBPM35ZRBLRKni1x/tlZp/rCzvEkWRmkgX8XSI pc+OvfpdrFT1/B3v3XSQULhP6x+jcMuFHbKA3jGL6zcwW9xL1nZvkzyytTq+PrU8Uo5d LBHN7gMc7FMLyGFGZM0tRFjxwjWSPpRS4bAocCQO1jh7RyXJB93WqPD4/j31ihP362w1 ItPSbUz/7oGQwoOqqlJzOVR31KI7jhKZfiwq7Bxk05MxAgtr+mT2tR5XGM5X3GByxpgU Thq9kctw5jIzTtbZhyuKm0vH/0ymcU5EgRSsUpzGMArEDIJfhXWgfuEB/rG4JycIi3Lr Hd1g== X-Forwarded-Encrypted: i=1; AJvYcCWMi3ffwY1youI6mIWw6VTdrFLdf3witIZklIlqJvqLeWNVBj7Y8Aep6/3SqKnoqP7Ov3lUgqfswk+Pi0g=@vger.kernel.org X-Gm-Message-State: AOJu0Yxrs/IoRbWuC9ZU03eXKuCfOHsboJB4mDv4pDIW3g8O3frK5EVd L+UfUqLKZZrKXz0xseSmovDGu9Un+8IxPMlavs8buLJkrmaHMHtbWA4myc+6FGwjoD38Z2aDttA KRzLvzPI= X-Gm-Gg: ASbGncsXgz61AOjzMGRggkpJcX6QJ4Lw6JEzDpVfWIawWms/HCsYr7NFbL/7IZFxQ4V hGHI0NJd25VSmWf9luMbuHt+TZjKO/B7LiYUvs1gV1RgsP7HaLY0xMwiRB9JRtXbMnYEA8Dz+qL uUiOy7urMjJzwYrXH0SvZjUDmfrQbbsJtf5XCCkn2y10xL+81tve1jgbyoDps0MAyOsDqKwX2+d Im0KXoR7lDadSw1QP4aR8xpPTQkAgyzC5VAVGogaTOaFWBZ3ckobLBjP6zSCmlc+18+ptrCnC9j JlMgG4utEiCBNZDgbq0eCeOKqqs4BoiIihOX/rLy0CQ3ioh+YC1NHJLGx5CeBKriN+6ipsA+xTc 3k0bi9EvUGv0lAa36KVkl3tbDWUuiH9jZCPDzztnu7aKlgdHYgAr6MpbNQ1TqsSchlLiURGahrH SGoK1yjv7nR2+x8bzGcj93smhjYKmu3g== X-Google-Smtp-Source: AGHT+IGNsCW8GaBWE80eC552ADlIIeLb1DFaNnNWkr5NJ5EoHsoPa7vEmJGXkXIjZpH3a0KG8muD6g== X-Received: by 2002:a05:600c:4f45:b0:45d:5c71:769d with SMTP id 5b1f17b1804b1-477c04db9a0mr64162485e9.8.1763825854228; Sat, 22 Nov 2025 07:37:34 -0800 (PST) Received: from [192.168.50.4] ([82.78.167.134]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-477bf3aef57sm93496025e9.11.2025.11.22.07.37.32 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 22 Nov 2025 07:37:33 -0800 (PST) Message-ID: Date: Sat, 22 Nov 2025 17:37:31 +0200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 2/3] ARM: at91: Move PM init functions to .init_late hook To: Rob Herring , Miquel Raynal , Richard Weinberger , Vignesh Raghavendra , Nicolas Ferre , Alexandre Belloni , Russell King Cc: linux-mtd@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org References: <20251120-at91-probe-v2-0-8c8592f02e8e@kernel.org> <20251120-at91-probe-v2-2-8c8592f02e8e@kernel.org> From: Claudiu Beznea Content-Language: en-US In-Reply-To: <20251120-at91-probe-v2-2-8c8592f02e8e@kernel.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Hi, Rob, On 11/20/25 23:37, Rob Herring wrote: > Move the AT91 PM init functions to .init_late hook to ensure driver > dependencies have probed. > > Cc: Nicolas Ferre > Cc: Alexandre Belloni > Cc: linux-arm-kernel@lists.infradead.org > Signed-off-by: Rob Herring (Arm) 'From: Rob Herring ' != 'Signed-off-by: Rob Herring (Arm) > --- > v2: > - new patch > --- > arch/arm/mach-at91/at91rm9200.c | 3 +-- > arch/arm/mach-at91/at91sam9.c | 3 +-- > arch/arm/mach-at91/sam9x60.c | 3 +-- > arch/arm/mach-at91/sam9x7.c | 3 +-- > arch/arm/mach-at91/sama5.c | 2 +- > arch/arm/mach-at91/sama7.c | 3 +-- > 6 files changed, 6 insertions(+), 11 deletions(-) > > diff --git a/arch/arm/mach-at91/at91rm9200.c b/arch/arm/mach-at91/at91rm9200.c > index 4f8186211619..2ac564eb8bbb 100644 > --- a/arch/arm/mach-at91/at91rm9200.c > +++ b/arch/arm/mach-at91/at91rm9200.c > @@ -17,8 +17,6 @@ > static void __init at91rm9200_dt_device_init(void) > { > of_platform_default_populate(NULL, NULL, NULL); > - > - at91rm9200_pm_init(); > } > > static const char *const at91rm9200_dt_board_compat[] __initconst = { > @@ -27,6 +25,7 @@ static const char *const at91rm9200_dt_board_compat[] __initconst = { > }; > > DT_MACHINE_START(at91rm9200_dt, "Atmel AT91RM9200") > + .init_late = at91rm9200_pm_init, > .init_machine = at91rm9200_dt_device_init, > .dt_compat = at91rm9200_dt_board_compat, > MACHINE_END > diff --git a/arch/arm/mach-at91/at91sam9.c b/arch/arm/mach-at91/at91sam9.c > index 7e572189a5eb..cf07cba4ee5d 100644 > --- a/arch/arm/mach-at91/at91sam9.c > +++ b/arch/arm/mach-at91/at91sam9.c > @@ -17,8 +17,6 @@ > static void __init at91sam9_init(void) > { > of_platform_default_populate(NULL, NULL, NULL); > - > - at91sam9_pm_init(); > } > > static const char *const at91_dt_board_compat[] __initconst = { > @@ -28,6 +26,7 @@ static const char *const at91_dt_board_compat[] __initconst = { > > DT_MACHINE_START(at91sam_dt, "Atmel AT91SAM9") > /* Maintainer: Atmel */ > + .init_late = at91sam9_pm_init, > .init_machine = at91sam9_init, > .dt_compat = at91_dt_board_compat, > MACHINE_END > diff --git a/arch/arm/mach-at91/sam9x60.c b/arch/arm/mach-at91/sam9x60.c > index d8c739d25458..a31beaaeffcd 100644 > --- a/arch/arm/mach-at91/sam9x60.c > +++ b/arch/arm/mach-at91/sam9x60.c > @@ -18,8 +18,6 @@ > static void __init sam9x60_init(void) > { > of_platform_default_populate(NULL, NULL, NULL); > - > - sam9x60_pm_init(); > } > > static const char *const sam9x60_dt_board_compat[] __initconst = { > @@ -29,6 +27,7 @@ static const char *const sam9x60_dt_board_compat[] __initconst = { > > DT_MACHINE_START(sam9x60_dt, "Microchip SAM9X60") > /* Maintainer: Microchip */ > + .init_late = sam9x60_pm_init, > .init_machine = sam9x60_init, > .dt_compat = sam9x60_dt_board_compat, > MACHINE_END > diff --git a/arch/arm/mach-at91/sam9x7.c b/arch/arm/mach-at91/sam9x7.c > index e1ff30b5b09b..0c73f7fefd4f 100644 > --- a/arch/arm/mach-at91/sam9x7.c > +++ b/arch/arm/mach-at91/sam9x7.c > @@ -17,8 +17,6 @@ > static void __init sam9x7_init(void) > { > of_platform_default_populate(NULL, NULL, NULL); > - > - sam9x7_pm_init(); > } > > static const char * const sam9x7_dt_board_compat[] __initconst = { > @@ -28,6 +26,7 @@ static const char * const sam9x7_dt_board_compat[] __initconst = { > > DT_MACHINE_START(sam9x7_dt, "Microchip SAM9X7") > /* Maintainer: Microchip */ > + .init_late = sam9x7_pm_init, > .init_machine = sam9x7_init, > .dt_compat = sam9x7_dt_board_compat, > MACHINE_END > diff --git a/arch/arm/mach-at91/sama5.c b/arch/arm/mach-at91/sama5.c > index bf2b5c6a18c6..efae697ce29c 100644 > --- a/arch/arm/mach-at91/sama5.c > +++ b/arch/arm/mach-at91/sama5.c > @@ -33,7 +33,6 @@ static void __init sama5_secure_cache_init(void) > static void __init sama5_dt_device_init(void) > { > of_platform_default_populate(NULL, NULL, NULL); > - sama5_pm_init(); > } > > static const char *const sama5_dt_board_compat[] __initconst = { > @@ -43,6 +42,7 @@ static const char *const sama5_dt_board_compat[] __initconst = { > > DT_MACHINE_START(sama5_dt, "Atmel SAMA5") > /* Maintainer: Atmel */ > + .init_late = sama5_pm_init, > .init_machine = sama5_dt_device_init, > .dt_compat = sama5_dt_board_compat, > MACHINE_END > diff --git a/arch/arm/mach-at91/sama7.c b/arch/arm/mach-at91/sama7.c > index bd43733ede18..3e28727f2a8e 100644 > --- a/arch/arm/mach-at91/sama7.c > +++ b/arch/arm/mach-at91/sama7.c > @@ -17,8 +17,6 @@ > static void __init sama7_dt_device_init(void) > { > of_platform_default_populate(NULL, NULL, NULL); > - sama7_pm_init(); > -} } was dropped as well. Thank you, Claudiu