Re: [PATCH RFC 01/10] ARM: s3c64xx: pm: Use name field of generic_pm_domain

2014-01-12 Thread Tomasz Figa
On 12.01.2014 20:20, Mark Brown wrote: On Sat, Jan 11, 2014 at 08:42:43PM +0100, Tomasz Figa wrote: static struct s3c64xx_pm_domain s3c64xx_pm_irom = { - .name = "IROM", .ena = S3C64XX_NORMALCFG_IROM_ON, .pd = { + .name = "domain_irom", This is nitpicki

Re: [PATCH RFC 01/10] ARM: s3c64xx: pm: Use name field of generic_pm_domain

2014-01-12 Thread Mark Brown
On Sun, Jan 12, 2014 at 08:03:50PM +0100, Tomasz Figa wrote: > Also, there are multiple devices in most domains, so that would > rather end up being "Domain_I (JPEG, Cam I/F)", "Domain_P (2D, TV > Enc., Scaler)", "Domain_F (Rot, Post, LCD)". It's not just that there's multiple devices either, it'

Re: [PATCH RFC 01/10] ARM: s3c64xx: pm: Use name field of generic_pm_domain

2014-01-12 Thread Mark Brown
On Sat, Jan 11, 2014 at 08:42:43PM +0100, Tomasz Figa wrote: > static struct s3c64xx_pm_domain s3c64xx_pm_irom = { > - .name = "IROM", > .ena = S3C64XX_NORMALCFG_IROM_ON, > .pd = { > + .name = "domain_irom", This is nitpicking a bit but are you sure this is actually a

Re: [PATCH RFC 01/10] ARM: s3c64xx: pm: Use name field of generic_pm_domain

2014-01-12 Thread Tomasz Figa
On 12.01.2014 19:53, Pavel Machek wrote: Hi! Could we get some more descriptive names for the domains? They are listed like this in user's manual, e.g. DOMAIN_F, DOMAIN_I, DOMAIN_G, etc. I guessed so. So the manual sucks. Would it be feasible to get it more descriptive (like "Domain_G (GPU)

Re: [PATCH RFC 01/10] ARM: s3c64xx: pm: Use name field of generic_pm_domain

2014-01-12 Thread Pavel Machek
Hi! > >Could we get some more descriptive names for the domains? > > They are listed like this in user's manual, e.g. DOMAIN_F, DOMAIN_I, > DOMAIN_G, etc. I guessed so. So the manual sucks. Would it be feasible to get it more descriptive (like "Domain_G (GPU)")?

Re: [PATCH RFC 01/10] ARM: s3c64xx: pm: Use name field of generic_pm_domain

2014-01-12 Thread Tomasz Figa
Hi Pavel, On 12.01.2014 12:47, Pavel Machek wrote: On Sat 2014-01-11 20:42:43, Tomasz Figa wrote: This patch removes name field from private s3c64xx_pm_domain struct and moves domain name to dedicated field of generic_pm_domain struct. When at it, beautify the names a bit, since they are used

Re: [PATCH RFC 01/10] ARM: s3c64xx: pm: Use name field of generic_pm_domain

2014-01-12 Thread Pavel Machek
On Sat 2014-01-11 20:42:43, Tomasz Figa wrote: > This patch removes name field from private s3c64xx_pm_domain struct and > moves domain name to dedicated field of generic_pm_domain struct. > > When at it, beautify the names a bit, since they are used by genpd core > as message prefixes. > > Signe

[PATCH RFC 01/10] ARM: s3c64xx: pm: Use name field of generic_pm_domain

2014-01-11 Thread Tomasz Figa
This patch removes name field from private s3c64xx_pm_domain struct and moves domain name to dedicated field of generic_pm_domain struct. When at it, beautify the names a bit, since they are used by genpd core as message prefixes. Signed-off-by: Tomasz Figa --- arch/arm/mach-s3c64xx/pm.c | 19 +