Skip to content

Commit e4ab1bf

Browse files
KwonTae-youngsre
authored andcommitted
power: reset: qcom-pon: Rename variables to use generic naming
The qcom-pon driver was originally implemented for the PM8916 PMIC, and as a result, several internal variable names still refer to 'pm8916'. However, the driver has since been extended to support other PMICs as well. This patch renames those variables to use more generic and consistent names, improving clarity and reducing confusion for non-PM8916 devices. Signed-off-by: Taeyoung Kwon <[email protected]> Reviewed-by: Dmitry Baryshkov <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Sebastian Reichel <[email protected]>
1 parent 202ac22 commit e4ab1bf

File tree

1 file changed

+15
-15
lines changed

1 file changed

+15
-15
lines changed

drivers/power/reset/qcom-pon.c

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -19,19 +19,19 @@
1919

2020
#define NO_REASON_SHIFT 0
2121

22-
struct pm8916_pon {
22+
struct qcom_pon {
2323
struct device *dev;
2424
struct regmap *regmap;
2525
u32 baseaddr;
2626
struct reboot_mode_driver reboot_mode;
2727
long reason_shift;
2828
};
2929

30-
static int pm8916_reboot_mode_write(struct reboot_mode_driver *reboot,
30+
static int qcom_pon_reboot_mode_write(struct reboot_mode_driver *reboot,
3131
unsigned int magic)
3232
{
33-
struct pm8916_pon *pon = container_of
34-
(reboot, struct pm8916_pon, reboot_mode);
33+
struct qcom_pon *pon = container_of
34+
(reboot, struct qcom_pon, reboot_mode);
3535
int ret;
3636

3737
ret = regmap_update_bits(pon->regmap,
@@ -44,9 +44,9 @@ static int pm8916_reboot_mode_write(struct reboot_mode_driver *reboot,
4444
return ret;
4545
}
4646

47-
static int pm8916_pon_probe(struct platform_device *pdev)
47+
static int qcom_pon_probe(struct platform_device *pdev)
4848
{
49-
struct pm8916_pon *pon;
49+
struct qcom_pon *pon;
5050
long reason_shift;
5151
int error;
5252

@@ -72,7 +72,7 @@ static int pm8916_pon_probe(struct platform_device *pdev)
7272
if (reason_shift != NO_REASON_SHIFT) {
7373
pon->reboot_mode.dev = &pdev->dev;
7474
pon->reason_shift = reason_shift;
75-
pon->reboot_mode.write = pm8916_reboot_mode_write;
75+
pon->reboot_mode.write = qcom_pon_reboot_mode_write;
7676
error = devm_reboot_mode_register(&pdev->dev, &pon->reboot_mode);
7777
if (error) {
7878
dev_err(&pdev->dev, "can't register reboot mode\n");
@@ -85,24 +85,24 @@ static int pm8916_pon_probe(struct platform_device *pdev)
8585
return devm_of_platform_populate(&pdev->dev);
8686
}
8787

88-
static const struct of_device_id pm8916_pon_id_table[] = {
88+
static const struct of_device_id qcom_pon_id_table[] = {
8989
{ .compatible = "qcom,pm8916-pon", .data = (void *)GEN1_REASON_SHIFT },
9090
{ .compatible = "qcom,pm8941-pon", .data = (void *)NO_REASON_SHIFT },
9191
{ .compatible = "qcom,pms405-pon", .data = (void *)GEN1_REASON_SHIFT },
9292
{ .compatible = "qcom,pm8998-pon", .data = (void *)GEN2_REASON_SHIFT },
9393
{ .compatible = "qcom,pmk8350-pon", .data = (void *)GEN2_REASON_SHIFT },
9494
{ }
9595
};
96-
MODULE_DEVICE_TABLE(of, pm8916_pon_id_table);
96+
MODULE_DEVICE_TABLE(of, qcom_pon_id_table);
9797

98-
static struct platform_driver pm8916_pon_driver = {
99-
.probe = pm8916_pon_probe,
98+
static struct platform_driver qcom_pon_driver = {
99+
.probe = qcom_pon_probe,
100100
.driver = {
101-
.name = "pm8916-pon",
102-
.of_match_table = pm8916_pon_id_table,
101+
.name = "qcom-pon",
102+
.of_match_table = qcom_pon_id_table,
103103
},
104104
};
105-
module_platform_driver(pm8916_pon_driver);
105+
module_platform_driver(qcom_pon_driver);
106106

107-
MODULE_DESCRIPTION("pm8916 Power On driver");
107+
MODULE_DESCRIPTION("Qualcomm Power On driver");
108108
MODULE_LICENSE("GPL v2");

0 commit comments

Comments
 (0)