PM / Domains: cleanup: rename gpd -> genpd in debugfs interface
Subsystems (4)
PATCH 1/1 PM / Domains: cleanup: rename gpd -> genpd in debugfs interface
From: Kevin Hilman <[email protected]>
To keep consisitency with the rest of the file, use 'genpd' as the name of the 'struct generic_pm_domain' pointer instead of 'gpd'.
This is just a rename, no functional changes.
--- drivers/base/power/domain.c | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-)
drivers/base/power/domain.c +12 -12
@@ -2242,7 +2242,7 @@ static void rtpm_status_str(struct seq_file *s, struct device *dev) }static int pm_genpd_summary_one (struct seq_file *s, -struct generic_pm_domain *gpd) +struct generic_pm_domain *genpd) {static const char *const status_lookup[] = { [GPD_STATE_ACTIVE ] ="on" ,
@@ -2256,26 +2256,26 @@ static int pm_genpd_summary_one(struct seq_file *s,struct gpd_link *link;int ret; - ret =mutex_lock_interruptible (&gpd->lock); + ret =mutex_lock_interruptible (&genpd->lock);if (ret)return -ERESTARTSYS ; -if (WARN_ON (gpd->status >=ARRAY_SIZE (status_lookup))) +if (WARN_ON (genpd->status >=ARRAY_SIZE (status_lookup))) goto exit; -seq_printf (s,"%-30s %-15s " , gpd->name, status_lookup[gpd->status]); +seq_printf (s,"%-30s %-15s " , genpd->name, status_lookup[genpd->status]);/* * Modifications on the list require holding locks on both * master and slave, so we are safe. -* Also gpd->name is immutable. +* Also genpd->name is immutable. */ -list_for_each_entry (link, &gpd->master_links, master_node) { +list_for_each_entry (link, &genpd->master_links, master_node) {seq_printf (s,"%s" , link->slave->name); -if (!list_is_last (&link->master_node, &gpd->master_links)) +if (!list_is_last (&link->master_node, &genpd->master_links))seq_puts (s,", " ); } -list_for_each_entry (pm_data, &gpd->dev_list, list_node) { +list_for_each_entry (pm_data, &genpd->dev_list, list_node) { kobj_path =kobject_get_path (&pm_data->dev->kobj,GFP_KERNEL );if (kobj_path ==NULL )continue ;
@@ -2287,14 +2287,14 @@ static int pm_genpd_summary_one(struct seq_file *s,seq_puts (s,"\n" ); exit: -mutex_unlock (&gpd->lock); +mutex_unlock (&genpd->lock);return 0 ; }static int pm_genpd_summary_show (struct seq_file *s,void *data) { -struct generic_pm_domain *gpd; +struct generic_pm_domain *genpd;int ret =0 ;seq_puts (s," domain status slaves\n" );
@@ -2305,8 +2305,8 @@ static int pm_genpd_summary_show(struct seq_file *s, void *data)if (ret)return -ERESTARTSYS ; -list_for_each_entry (gpd, &gpd_list, gpd_list_node) { - ret =pm_genpd_summary_one (s, gpd); +list_for_each_entry (genpd, &gpd_list, gpd_list_node) { + ret =pm_genpd_summary_one (s, genpd);if (ret)break ; }
Signature · 1 line
2.1.3
Quoted 8 lines · Kevin Hilman
From: Kevin Hilman <[email protected]>
To keep consisitency with the rest of the file, use 'genpd' as the name of the 'struct
generic_pm_domain' pointer instead of 'gpd'.This is just a rename, no functional changes.
Signed-off-by Kevin Hilman <[email protected]>
Signature · 2 lines
(english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
Quoted 8 lines · Kevin Hilman
From: Kevin Hilman <[email protected]>
To keep consisitency with the rest of the file, use 'genpd' as the name of the 'struct
generic_pm_domain' pointer instead of 'gpd'.This is just a rename, no functional changes.
Signed-off-by Kevin Hilman <[email protected]>
Quoted 82 lines
--- drivers/base/power/domain.c | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-)drivers/base/power/domain.c +12
-13@@ -2242,7 +2242,7 @@ static void rtpm_status_str(struct seq_file *s, struct device *dev) }static int pm_genpd_summary_one (struct seq_file *s, -struct generic_pm_domain *gpd) +struct generic_pm_domain *genpd) {static const char *const status_lookup[] = { [GPD_STATE_ACTIVE ] ="on" ,@@ -2256,26 +2256,26 @@ static int pm_genpd_summary_one(struct seq_file *s,struct gpd_link *link;int ret; - ret =mutex_lock_interruptible (&gpd->lock); + ret =mutex_lock_interruptible (&genpd->lock);if (ret)return -ERESTARTSYS ; -if (WARN_ON (gpd->status >=ARRAY_SIZE (status_lookup))) +if (WARN_ON (genpd->status >=ARRAY_SIZE (status_lookup))) goto exit; -seq_printf (s,"%-30s %-15s " , gpd->name, status_lookup[gpd->status]); +seq_printf (s,"%-30s %-15s " , genpd->name, status_lookup[genpd->status]);/* * Modifications on the list require holding locks on both * master and slave, so we are safe. -* Also gpd->name is immutable. +* Also genpd->name is immutable. */ -list_for_each_entry (link, &gpd->master_links, master_node) { +list_for_each_entry (link, &genpd->master_links, master_node) {seq_printf (s,"%s" , link->slave->name); -if (!list_is_last (&link->master_node, &gpd->master_links)) +if (!list_is_last (&link->master_node, &genpd->master_links))seq_puts (s,", " ); } -list_for_each_entry (pm_data, &gpd->dev_list, list_node) { +list_for_each_entry (pm_data, &genpd->dev_list, list_node) { kobj_path =kobject_get_path (&pm_data->dev->kobj,GFP_KERNEL );if (kobj_path ==NULL )continue ;@@ -2287,14 +2287,14 @@ static int pm_genpd_summary_one(struct seq_file *s,seq_puts (s,"\n" ); exit: -mutex_unlock (&gpd->lock); +mutex_unlock (&genpd->lock);return 0 ; }static int pm_genpd_summary_show (struct seq_file *s,void *data) { -struct generic_pm_domain *gpd; +struct generic_pm_domain *genpd;int ret =0 ;seq_puts (s," domain status slaves\n" );@@ -2305,8 +2305,8 @@ static int pm_genpd_summary_show(struct seq_file *s, void *data)if (ret)return -ERESTARTSYS ; -list_for_each_entry (gpd, &gpd_list, gpd_list_node) { - ret =pm_genpd_summary_one (s, gpd); +list_for_each_entry (genpd, &gpd_list, gpd_list_node) { + ret =pm_genpd_summary_one (s, genpd);if (ret)break ; } --2.1.3
Rafael J. Wysocki
AppliedQuoted 8 lines · Kevin Hilman
From: Kevin Hilman <[email protected]>
To keep consisitency with the rest of the file, use 'genpd' as the name of the 'struct
generic_pm_domain' pointer instead of 'gpd'.This is just a rename, no functional changes.
Signed-off-by Kevin Hilman <[email protected]>
Applied, thanks!
Signature · 2 lines
I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center.