From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f48.google.com (mail-wr1-f48.google.com [209.85.221.48]) (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 9F5C62FAC18 for ; Wed, 19 Nov 2025 15:07:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.48 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763564853; cv=none; b=eBZ7lYpWj23bjTJn50VJgjB9K9LB8RAchDaWm5rWxL/DHALyRVJb47kOLx7h02UqR5vZ8DecucvG5haDVIiFhjVMSPrXeycuz4LezXwLNAu1DpYqxAkzcxdddYCfQ0uNR/yLoDZsM2ua9D9IzBrO0ql34PrAAv1e+oznA6AffIE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763564853; c=relaxed/simple; bh=zQGn2TBEg53bLFqIRWcH3TNqoIVQRmjshIxUlHg5MAA=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=ZkOALcTEu6EEHyBPsF6tQKrVcqX2ZFjRUiShMkq/8mQBKWKsD3z9AaIgdS6NN9ykk3P5U+5xqJ9Ei8WoBP2MIvH6LaHQZWvTl/8RE2Qq6GXSjLO6lrZnfe1pZwJuF5W6N67je4VheNfSL68qlwKn5aGUDeQXxPM8qI0udJl8yjk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=lbjwLgya; arc=none smtp.client-ip=209.85.221.48 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="lbjwLgya" Received: by mail-wr1-f48.google.com with SMTP id ffacd0b85a97d-42b3b0d76fcso4266582f8f.3 for ; Wed, 19 Nov 2025 07:07:31 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1763564850; x=1764169650; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:organization:autocrypt :content-language:from:references:cc:to:subject:reply-to:user-agent :mime-version:date:message-id:from:to:cc:subject:date:message-id :reply-to; bh=b8MXHRVY48CutCxVOD2Cd8MderTR1vjNZQmBDMhIq4I=; b=lbjwLgyai4O3FlycwQXd9TqAoFjcsAixVhJWm7Tw3H6SVtXX4Rf/JkNSIFnlbBwMbW fdA/Hn88qPYOcMREaxDWeAlEFzYZXF8IRiNSXB3MiR3cvnDgFmqFStPl3N7fmC5fOLNV Rc/IfE4Y2ahk3y8O+7+ItMCa274GAxEukWmac30C5y0T1gY66bjFcZ0dKPt1NgqKFx6k 5KEDAVvKq5+eLO3pudZW2dwFbxpgjk/yqhFi42Xk09PXgIZ85+AO6mBUIZfCL1xqXtH3 0nLks7qtFP/WpIBAt2kID5V47WHWHVOp2RJefhVwOVvVTCNIOWALoSQpXTVNi/+Ne2IA Neqw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763564850; x=1764169650; h=content-transfer-encoding:in-reply-to:organization:autocrypt :content-language:from:references:cc:to:subject:reply-to:user-agent :mime-version:date:message-id:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=b8MXHRVY48CutCxVOD2Cd8MderTR1vjNZQmBDMhIq4I=; b=H52eNlfTQEhxxkLNs134KtB/MjccLIpfBLMAJWVtHFgeCVDmbUdo2s9IPaY8Ej5zh+ ZwaXPWBDOAIS4qTISaxqHvXxuS7CY+w8gezv5+45JpzXvZUO6yZvRWGAC9FrFHH8Bsp2 M2BUEQZvZ6vJDSmKgW8BdrONdmGP2zNr4GoL3fcrupGW2a1fTbhG2wG3AIShP8wLHUXH sPdA8RiS1KH8Q1RpvutXn3AwAd/jadk6gutSfuzfe7e/ANffL88zPnPOpgcA7LTj5x6s 2+6T0bQf+txAb71tjnC4/4HieYap8861ddGZAxdNCl4Y6k/Dv9FQ65sJ+5uk4ineYrN/ nNQA== X-Forwarded-Encrypted: i=1; AJvYcCXhY2SAbnS0nLawkJFBvNAZEEgBN4cB1/MG7t3TVgSXFlw6nwIJi8Uy9EfARGDg1S7IRkOOMqg5IPyNsk4=@vger.kernel.org X-Gm-Message-State: AOJu0Yy9iytgNn89dRcnNxjJl6ykOoiCGTM8qhI4tWD/YM2tFlaMSu/7 DJoAWoZcgsKkZtFSjAtt6ZfHyx5Mq37vbAS9j/mxBKkzrofGFWfWXPtJbe0HC0LFsdI= X-Gm-Gg: ASbGncsgSgge5Ahr+tDzpej4CfRkGrvdvdlPlM6AxyIVENFhyDmu5tP75tksQjnGxSK yulo6lZL0cxhPh5fYJrccfJk0/DbBE7Zn4x3rg6sTKdh1J9c2HSka/EgWj6klh07G3HO+h7HU1G 1r/vxHYBFj7hoKM0qDRwSRrOvKUlqtRGj/6fyezaMhjOXrkjim95Q6MPAwSO1kPfM9pm9BP29x0 ma2XDrkkyvTa2ky7S1NbUg5cmleIT2FDg/L6jrOt8/39sOC6V04RoDLQ+RpJrBIwrE+6GdL0KT/ gkENby3A/nGV+bzgWpeL1bJRhm+frVDjklfe9fJbx7bifcgnvS6uuJNZwjuEGcEwh8L6/aGBMVn 8peyZBT4DrJPRnLP2HwbpgSErb5MLch68t2/f2u0YD8hlcA2ghpf99waUQJizacmFKOqWa/LXi7 qA7LLN3uebytEikzVuXa1pvdaQ0oGhuCqR+dRRvaZCbtixUMeHNCwPo5PjBIYJ8Jo= X-Google-Smtp-Source: AGHT+IHdI5AOlmNDX5m4IHoZ74FO5t2segG/koratGEOW9qb5av5beCTKzXqT4NvIhC8HV6z9TOzRg== X-Received: by 2002:a05:6000:2010:b0:42b:55a1:214f with SMTP id ffacd0b85a97d-42b59340d6dmr20687485f8f.21.1763564849819; Wed, 19 Nov 2025 07:07:29 -0800 (PST) Received: from ?IPV6:2a01:e0a:3d9:2080:263d:925b:a11f:ae36? ([2a01:e0a:3d9:2080:263d:925b:a11f:ae36]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-42b53e91f2dsm39622909f8f.19.2025.11.19.07.07.28 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 19 Nov 2025 07:07:29 -0800 (PST) Message-ID: <461244a1-3bc1-478f-b97f-b0607375e596@linaro.org> Date: Wed, 19 Nov 2025 16:07:28 +0100 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Reply-To: Neil Armstrong Subject: Re: [PATCH] phy: broadcom: bcm63xx-usbh: fix section mismatches To: Johan Hovold , Vinod Koul , Kishon Vijay Abraham I Cc: linux-phy@lists.infradead.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org, =?UTF-8?Q?=C3=81lvaro_Fern=C3=A1ndez_Rojas?= References: <20251017054537.6884-1-johan@kernel.org> From: Neil Armstrong Content-Language: en-US, fr Autocrypt: addr=neil.armstrong@linaro.org; keydata= xsBNBE1ZBs8BCAD78xVLsXPwV/2qQx2FaO/7mhWL0Qodw8UcQJnkrWmgTFRobtTWxuRx8WWP GTjuhvbleoQ5Cxjr+v+1ARGCH46MxFP5DwauzPekwJUD5QKZlaw/bURTLmS2id5wWi3lqVH4 BVF2WzvGyyeV1o4RTCYDnZ9VLLylJ9bneEaIs/7cjCEbipGGFlfIML3sfqnIvMAxIMZrvcl9 qPV2k+KQ7q+aXavU5W+yLNn7QtXUB530Zlk/d2ETgzQ5FLYYnUDAaRl+8JUTjc0CNOTpCeik 80TZcE6f8M76Xa6yU8VcNko94Ck7iB4vj70q76P/J7kt98hklrr85/3NU3oti3nrIHmHABEB AAHNKk5laWwgQXJtc3Ryb25nIDxuZWlsLmFybXN0cm9uZ0BsaW5hcm8ub3JnPsLAkQQTAQoA OwIbIwULCQgHAwUVCgkICwUWAgMBAAIeAQIXgBYhBInsPQWERiF0UPIoSBaat7Gkz/iuBQJk Q5wSAhkBAAoJEBaat7Gkz/iuyhMIANiD94qDtUTJRfEW6GwXmtKWwl/mvqQtaTtZID2dos04 YqBbshiJbejgVJjy+HODcNUIKBB3PSLaln4ltdsV73SBcwUNdzebfKspAQunCM22Mn6FBIxQ GizsMLcP/0FX4en9NaKGfK6ZdKK6kN1GR9YffMJd2P08EO8mHowmSRe/ExAODhAs9W7XXExw UNCY4pVJyRPpEhv373vvff60bHxc1k/FF9WaPscMt7hlkbFLUs85kHtQAmr8pV5Hy9ezsSRa GzJmiVclkPc2BY592IGBXRDQ38urXeM4nfhhvqA50b/nAEXc6FzqgXqDkEIwR66/Gbp0t3+r yQzpKRyQif3OwE0ETVkGzwEIALyKDN/OGURaHBVzwjgYq+ZtifvekdrSNl8TIDH8g1xicBYp QTbPn6bbSZbdvfeQPNCcD4/EhXZuhQXMcoJsQQQnO4vwVULmPGgtGf8PVc7dxKOeta+qUh6+ SRh3vIcAUFHDT3f/Zdspz+e2E0hPV2hiSvICLk11qO6cyJE13zeNFoeY3ggrKY+IzbFomIZY 4yG6xI99NIPEVE9lNBXBKIlewIyVlkOaYvJWSV+p5gdJXOvScNN1epm5YHmf9aE2ZjnqZGoM Mtsyw18YoX9BqMFInxqYQQ3j/HpVgTSvmo5ea5qQDDUaCsaTf8UeDcwYOtgI8iL4oHcsGtUX oUk33HEAEQEAAcLAXwQYAQIACQUCTVkGzwIbDAAKCRAWmrexpM/4rrXiB/sGbkQ6itMrAIfn M7IbRuiSZS1unlySUVYu3SD6YBYnNi3G5EpbwfBNuT3H8//rVvtOFK4OD8cRYkxXRQmTvqa3 3eDIHu/zr1HMKErm+2SD6PO9umRef8V82o2oaCLvf4WeIssFjwB0b6a12opuRP7yo3E3gTCS KmbUuLv1CtxKQF+fUV1cVaTPMyT25Od+RC1K+iOR0F54oUJvJeq7fUzbn/KdlhA8XPGzwGRy 4zcsPWvwnXgfe5tk680fEKZVwOZKIEuJC3v+/yZpQzDvGYJvbyix0lHnrCzq43WefRHI5XTT QbM0WUIBIcGmq38+OgUsMYu4NzLu7uZFAcmp6h8g Organization: Linaro In-Reply-To: <20251017054537.6884-1-johan@kernel.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit On 10/17/25 07:45, Johan Hovold wrote: > Platform drivers can be probed after their init sections have been > discarded (e.g. on probe deferral or manual rebind through sysfs) so the > probe function and match table must not live in init. > > Fixes: 783f6d3dcf35 ("phy: bcm63xx-usbh: Add BCM63xx USBH driver") > Cc: stable@vger.kernel.org # 5.9 > Cc: Álvaro Fernández Rojas > Signed-off-by: Johan Hovold > --- > drivers/phy/broadcom/phy-bcm63xx-usbh.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/phy/broadcom/phy-bcm63xx-usbh.c b/drivers/phy/broadcom/phy-bcm63xx-usbh.c > index 647644de041b..29fd6791bae6 100644 > --- a/drivers/phy/broadcom/phy-bcm63xx-usbh.c > +++ b/drivers/phy/broadcom/phy-bcm63xx-usbh.c > @@ -375,7 +375,7 @@ static struct phy *bcm63xx_usbh_phy_xlate(struct device *dev, > return of_phy_simple_xlate(dev, args); > } > > -static int __init bcm63xx_usbh_phy_probe(struct platform_device *pdev) > +static int bcm63xx_usbh_phy_probe(struct platform_device *pdev) > { > struct device *dev = &pdev->dev; > struct bcm63xx_usbh_phy *usbh; > @@ -432,7 +432,7 @@ static int __init bcm63xx_usbh_phy_probe(struct platform_device *pdev) > return 0; > } > > -static const struct of_device_id bcm63xx_usbh_phy_ids[] __initconst = { > +static const struct of_device_id bcm63xx_usbh_phy_ids[] = { > { .compatible = "brcm,bcm6318-usbh-phy", .data = &usbh_bcm6318 }, > { .compatible = "brcm,bcm6328-usbh-phy", .data = &usbh_bcm6328 }, > { .compatible = "brcm,bcm6358-usbh-phy", .data = &usbh_bcm6358 }, > @@ -443,7 +443,7 @@ static const struct of_device_id bcm63xx_usbh_phy_ids[] __initconst = { > }; > MODULE_DEVICE_TABLE(of, bcm63xx_usbh_phy_ids); > > -static struct platform_driver bcm63xx_usbh_phy_driver __refdata = { > +static struct platform_driver bcm63xx_usbh_phy_driver = { > .driver = { > .name = "bcm63xx-usbh-phy", > .of_match_table = bcm63xx_usbh_phy_ids, Reviewed-by: Neil Armstrong