| Message ID | 20230310200652.19926-1-krzysztof.kozlowski@linaro.org |
|---|---|
| State | Accepted |
| Commit | ef6d10568ca9ca622a867e5d059982251bbccd6b |
| Headers | show |
| Series | power: reset: qcom-pon: drop of_match_ptr for ID table | expand |
Hi, On Fri, Mar 10, 2023 at 09:06:52PM +0100, Krzysztof Kozlowski wrote: > The Qualcomm SoC power-on driver is specific to ARCH_QCOM which depends > on OF thus the driver is OF-only. It's of_device_id table is built > unconditionally, thus of_match_ptr() for ID table does not make sense. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- Thanks, queued to power-supply's for next branch. -- Sebastian > drivers/power/reset/qcom-pon.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/power/reset/qcom-pon.c b/drivers/power/reset/qcom-pon.c > index 16bc01738be9..ebdcfb28c4a0 100644 > --- a/drivers/power/reset/qcom-pon.c > +++ b/drivers/power/reset/qcom-pon.c > @@ -91,7 +91,7 @@ static struct platform_driver pm8916_pon_driver = { > .probe = pm8916_pon_probe, > .driver = { > .name = "pm8916-pon", > - .of_match_table = of_match_ptr(pm8916_pon_id_table), > + .of_match_table = pm8916_pon_id_table, > }, > }; > module_platform_driver(pm8916_pon_driver); > -- > 2.34.1 >
diff --git a/drivers/power/reset/qcom-pon.c b/drivers/power/reset/qcom-pon.c index 16bc01738be9..ebdcfb28c4a0 100644 --- a/drivers/power/reset/qcom-pon.c +++ b/drivers/power/reset/qcom-pon.c @@ -91,7 +91,7 @@ static struct platform_driver pm8916_pon_driver = { .probe = pm8916_pon_probe, .driver = { .name = "pm8916-pon", - .of_match_table = of_match_ptr(pm8916_pon_id_table), + .of_match_table = pm8916_pon_id_table, }, }; module_platform_driver(pm8916_pon_driver);
The Qualcomm SoC power-on driver is specific to ARCH_QCOM which depends on OF thus the driver is OF-only. It's of_device_id table is built unconditionally, thus of_match_ptr() for ID table does not make sense. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- drivers/power/reset/qcom-pon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)