From: Inochi Amaoto <inochiama@gmail.com>
To: Hans de Goede <hansg@kernel.org>,
Damien Le Moal <dlemoal@kernel.org>,
Niklas Cassel <cassel@kernel.org>,
Andrzej Hajda <andrzej.hajda@intel.com>,
Neil Armstrong <neil.armstrong@linaro.org>,
Robert Foss <rfoss@kernel.org>,
Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
Jonas Karlman <jonas@kwiboo.se>,
Jernej Skrabec <jernej.skrabec@gmail.com>,
Luca Ceresoli <luca.ceresoli@bootlin.com>,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Maxime Ripard <mripard@kernel.org>,
Thomas Zimmermann <tzimmermann@suse.de>,
David Airlie <airlied@gmail.com>, Simona Vetter <simona@ffwll.ch>,
Minas Harutyunyan <hminas@synopsys.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Thinh Nguyen <Thinh.Nguyen@synopsys.com>,
Vinod Koul <vkoul@kernel.org>,
Manivannan Sadhasivam <mani@kernel.org>,
Damon Ding <damon.ding@rock-chips.com>,
Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>,
Heiko Stuebner <heiko@sntech.de>
Cc: Inochi Amaoto <inochiama@gmail.com>,
linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org,
dri-devel@lists.freedesktop.org, linux-usb@vger.kernel.org,
linux-phy@lists.infradead.org, Yixun Lan <dlan@gentoo.org>,
Longbin Li <looong.bin@gmail.com>
Subject: [RFC PATCH 1/5] phy: core: Use EOPNOTSUPP for disabled blob return value
Date: Tue, 22 Sep 2026 10:47:16 +0800 [thread overview]
Message-ID: <20260922024724.191412-2-inochiama@gmail.com> (raw)
In-Reply-To: <20260922024724.191412-1-inochiama@gmail.com>
Currently, the phy subsystem uses ENOSYS for dummy blob functions,
it does not reflect the function state correctly. As kernel already
has EOPNOTSUPP for disabled blob function, replace ENOSYS with
EOPNOTSUPP for a clear return value.
Signed-off-by: Inochi Amaoto <inochiama@gmail.com>
---
include/linux/phy/phy.h | 52 ++++++++++++++++++++---------------------
1 file changed, 26 insertions(+), 26 deletions(-)
diff --git a/include/linux/phy/phy.h b/include/linux/phy/phy.h
index ea47975e288a..3e84b54a4b03 100644
--- a/include/linux/phy/phy.h
+++ b/include/linux/phy/phy.h
@@ -314,14 +314,14 @@ static inline int phy_pm_runtime_get(struct phy *phy)
{
if (!phy)
return 0;
- return -ENOSYS;
+ return -EOPNOTSUPP;
}
static inline int phy_pm_runtime_get_sync(struct phy *phy)
{
if (!phy)
return 0;
- return -ENOSYS;
+ return -EOPNOTSUPP;
}
static inline void phy_pm_runtime_put(struct phy *phy)
@@ -332,35 +332,35 @@ static inline int phy_pm_runtime_put_sync(struct phy *phy)
{
if (!phy)
return 0;
- return -ENOSYS;
+ return -EOPNOTSUPP;
}
static inline int phy_init(struct phy *phy)
{
if (!phy)
return 0;
- return -ENOSYS;
+ return -EOPNOTSUPP;
}
static inline int phy_exit(struct phy *phy)
{
if (!phy)
return 0;
- return -ENOSYS;
+ return -EOPNOTSUPP;
}
static inline int phy_power_on(struct phy *phy)
{
if (!phy)
return 0;
- return -ENOSYS;
+ return -EOPNOTSUPP;
}
static inline int phy_power_off(struct phy *phy)
{
if (!phy)
return 0;
- return -ENOSYS;
+ return -EOPNOTSUPP;
}
static inline int phy_set_mode_ext(struct phy *phy, enum phy_mode mode,
@@ -368,7 +368,7 @@ static inline int phy_set_mode_ext(struct phy *phy, enum phy_mode mode,
{
if (!phy)
return 0;
- return -ENOSYS;
+ return -EOPNOTSUPP;
}
#define phy_set_mode(phy, mode) \
@@ -397,35 +397,35 @@ static inline int phy_reset(struct phy *phy)
{
if (!phy)
return 0;
- return -ENOSYS;
+ return -EOPNOTSUPP;
}
static inline int phy_calibrate(struct phy *phy)
{
if (!phy)
return 0;
- return -ENOSYS;
+ return -EOPNOTSUPP;
}
static inline int phy_notify_connect(struct phy *phy, int index)
{
if (!phy)
return 0;
- return -ENOSYS;
+ return -EOPNOTSUPP;
}
static inline int phy_notify_disconnect(struct phy *phy, int index)
{
if (!phy)
return 0;
- return -ENOSYS;
+ return -EOPNOTSUPP;
}
static inline int phy_notify_state(struct phy *phy, union phy_notify state)
{
if (!phy)
return 0;
- return -ENOSYS;
+ return -EOPNOTSUPP;
}
static inline int phy_configure(struct phy *phy,
@@ -434,7 +434,7 @@ static inline int phy_configure(struct phy *phy,
if (!phy)
return 0;
- return -ENOSYS;
+ return -EOPNOTSUPP;
}
static inline int phy_validate(struct phy *phy, enum phy_mode mode, int submode,
@@ -443,12 +443,12 @@ static inline int phy_validate(struct phy *phy, enum phy_mode mode, int submode,
if (!phy)
return 0;
- return -ENOSYS;
+ return -EOPNOTSUPP;
}
static inline int phy_get_bus_width(struct phy *phy)
{
- return -ENOSYS;
+ return -EOPNOTSUPP;
}
static inline void phy_set_bus_width(struct phy *phy, int bus_width)
@@ -458,12 +458,12 @@ static inline void phy_set_bus_width(struct phy *phy, int bus_width)
static inline struct phy *phy_get(struct device *dev, const char *string)
{
- return ERR_PTR(-ENOSYS);
+ return ERR_PTR(-EOPNOTSUPP);
}
static inline struct phy *devm_phy_get(struct device *dev, const char *string)
{
- return ERR_PTR(-ENOSYS);
+ return ERR_PTR(-EOPNOTSUPP);
}
static inline struct phy *devm_phy_optional_get(struct device *dev,
@@ -476,7 +476,7 @@ static inline struct phy *devm_of_phy_get(struct device *dev,
struct device_node *np,
const char *con_id)
{
- return ERR_PTR(-ENOSYS);
+ return ERR_PTR(-EOPNOTSUPP);
}
static inline struct phy *devm_of_phy_optional_get(struct device *dev,
@@ -490,7 +490,7 @@ static inline struct phy *devm_of_phy_get_by_index(struct device *dev,
struct device_node *np,
int index)
{
- return ERR_PTR(-ENOSYS);
+ return ERR_PTR(-EOPNOTSUPP);
}
static inline void of_phy_put(struct phy *phy)
@@ -507,27 +507,27 @@ static inline void devm_phy_put(struct device *dev, struct phy *phy)
static inline struct phy *of_phy_get(struct device_node *np, const char *con_id)
{
- return ERR_PTR(-ENOSYS);
+ return ERR_PTR(-EOPNOTSUPP);
}
static inline struct phy *of_phy_simple_xlate(struct device *dev,
const struct of_phandle_args *args)
{
- return ERR_PTR(-ENOSYS);
+ return ERR_PTR(-EOPNOTSUPP);
}
static inline struct phy *phy_create(struct device *dev,
struct device_node *node,
const struct phy_ops *ops)
{
- return ERR_PTR(-ENOSYS);
+ return ERR_PTR(-EOPNOTSUPP);
}
static inline struct phy *devm_phy_create(struct device *dev,
struct device_node *node,
const struct phy_ops *ops)
{
- return ERR_PTR(-ENOSYS);
+ return ERR_PTR(-EOPNOTSUPP);
}
static inline void phy_destroy(struct phy *phy)
@@ -543,7 +543,7 @@ static inline struct phy_provider *__of_phy_provider_register(
struct phy * (*of_xlate)(struct device *dev,
const struct of_phandle_args *args))
{
- return ERR_PTR(-ENOSYS);
+ return ERR_PTR(-EOPNOTSUPP);
}
static inline struct phy_provider *__devm_of_phy_provider_register(struct device
@@ -551,7 +551,7 @@ static inline struct phy_provider *__devm_of_phy_provider_register(struct device
struct phy * (*of_xlate)(struct device *dev,
const struct of_phandle_args *args))
{
- return ERR_PTR(-ENOSYS);
+ return ERR_PTR(-EOPNOTSUPP);
}
static inline void of_phy_provider_unregister(struct phy_provider *phy_provider)
--
2.55.0
next prev parent reply other threads:[~2026-09-22 2:48 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-22 2:47 [RFC PATCH 0/5] " Inochi Amaoto
2026-09-22 2:47 ` Inochi Amaoto [this message]
2026-09-22 5:35 ` [RFC PATCH 1/5] " Greg Kroah-Hartman
2026-09-22 6:10 ` Inochi Amaoto
2026-09-22 6:16 ` Greg Kroah-Hartman
2026-09-22 6:36 ` Inochi Amaoto
2026-09-22 2:47 ` [RFC PATCH 2/5] ata: libahci_platform: Adapt the return value changed for disabled blob Inochi Amaoto
2026-09-22 3:23 ` Damien Le Moal
2026-09-22 6:10 ` Inochi Amaoto
2026-09-22 2:47 ` [RFC PATCH 3/5] usb: dwc2: " Inochi Amaoto
2026-09-22 2:47 ` [RFC PATCH 4/5] usb: dwc3: core: " Inochi Amaoto
2026-09-22 5:34 ` Greg Kroah-Hartman
2026-09-22 6:11 ` Inochi Amaoto
2026-09-22 2:47 ` [RFC PATCH 5/5] drm: bridge: analogix/dp: " Inochi Amaoto
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260922024724.191412-2-inochiama@gmail.com \
--to=inochiama@gmail.com \
--cc=Laurent.pinchart@ideasonboard.com \
--cc=Thinh.Nguyen@synopsys.com \
--cc=airlied@gmail.com \
--cc=andrzej.hajda@intel.com \
--cc=cassel@kernel.org \
--cc=damon.ding@rock-chips.com \
--cc=dlan@gentoo.org \
--cc=dlemoal@kernel.org \
--cc=dmitry.baryshkov@oss.qualcomm.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=gregkh@linuxfoundation.org \
--cc=hansg@kernel.org \
--cc=heiko@sntech.de \
--cc=hminas@synopsys.com \
--cc=jernej.skrabec@gmail.com \
--cc=jonas@kwiboo.se \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-phy@lists.infradead.org \
--cc=linux-usb@vger.kernel.org \
--cc=looong.bin@gmail.com \
--cc=luca.ceresoli@bootlin.com \
--cc=maarten.lankhorst@linux.intel.com \
--cc=mani@kernel.org \
--cc=mripard@kernel.org \
--cc=neil.armstrong@linaro.org \
--cc=rfoss@kernel.org \
--cc=simona@ffwll.ch \
--cc=tzimmermann@suse.de \
--cc=vkoul@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®