Re: [PATCH v2] thermal: consistently use int for temperatures

2015-07-25 Thread Pavel Machek
On Fri 2015-07-24 15:49:41, Guenter Roeck wrote: > On 07/24/2015 03:11 PM, Pavel Machek wrote: > >On Fri 2015-07-24 06:59:26, Guenter Roeck wrote: > >>On 07/23/2015 11:29 PM, Sascha Hauer wrote: > >>>On Thu, Jul 23, 2015 at 02:07:59PM +0200, Pavel Machek wrote: &g

Re: [PATCH v2] thermal: consistently use int for temperatures

2015-07-24 Thread Pavel Machek
On Fri 2015-07-24 06:59:26, Guenter Roeck wrote: > On 07/23/2015 11:29 PM, Sascha Hauer wrote: > >On Thu, Jul 23, 2015 at 02:07:59PM +0200, Pavel Machek wrote: > >>On Tue 2015-07-21 09:21:32, Sascha Hauer wrote: > >>>The thermal code uses int, long and unsigned

Re: [PATCH v2] thermal: consistently use int for temperatures

2015-07-23 Thread Pavel Machek
On Tue 2015-07-21 09:21:32, Sascha Hauer wrote: > The thermal code uses int, long and unsigned long for temperatures > in different places. > > Using an unsigned type limits the thermal framework to positive > temperatures without need. Also several drivers currently will report > temperatures nea

Re: [PATCH] Show proper respect for Heinrich Hertz by using the correct unit for frequency

2015-06-23 Thread Pavel Machek
Hi! > The SI unit of frequency is Hertz, named after Heinrich Hertz, and is > given the symbol "Hz" to denote this. "hz" is not the unit of frequency, > and is in fact meaningless. > > Fix arch/arm to correctly use "Hz", thereby acknowledging Heinrich Hertz > contribution to the modern world. >

Re: [PATCH 11/11] dt: Exynos: add Snow SPI NOR node.

2015-06-17 Thread Pavel Machek
On Thu 2015-06-04 17:20:54, Marek Vasut wrote: > On Thursday, June 04, 2015 at 04:04:18 AM, Krzysztof Kozlowski wrote: > > On 04.06.2015 06:26, Michal Suchanek wrote: > > > The Snow has onboard SPI NOR flash which contains the bootloader. > > > > > > Add DT node for this flash chip. The flash is r

Re: [PATCH 1/8] PM / Domains: Rename __pm_genpd_alloc|free_dev_data()

2015-01-28 Thread Pavel Machek
me rules: > > Internal functions: > genpd_* > _genpd_* > __genpd_* > > External functions: > pm_genpd_* > _pm_genpd_* > __pm_genpd_* > > Signed-off-by: Ulf Hansson Acked-by: Pavel Machek -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictu

Re: [PATCH 2/8] PM / Domains: Remove reference counting for the generic_pm_domain_data

2015-01-28 Thread Pavel Machek
f Hansson Acked-by: Pavel Machek -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@

Re: [PATCH v3 0/9] PM / Domains: Fix race conditions during boot

2014-11-13 Thread Pavel Machek
Hi! > 2) There are no requirements for arch/platforms/drivers to work in both cases > CONFIG_PM_RUNTIME=y/n. But they must be built without errors/warning for both > cases. I don't think where you got that. They are config options, they should work. It is prefer not to compile than to compile bu

Re: [PATCH] PM / Domains: Change prototype for the ->attach_dev() callback

2014-10-30 Thread Pavel Machek
On Thu 2014-10-30 13:02:49, Ulf Hansson wrote: > Convert the prototype to return and int. This is just an initial step, > needed to support error handling. > > Signed-off-by: Ulf Hansson Acked-by: Pavel Machek > --- > > This patch is intended as fix for 3.18 rc[n]. Why?

Re: Powering down unused PM domains (was: Re: [PATCH 0/4] PM / Domains: Fix race conditions during boot)

2014-10-10 Thread Pavel Machek
Hi! > > While operation A2 is PM domain-centric (it walks the list of genpd > > domains), > > A3 and B3 are device-centric (A3 operates on one specific device, B3 walks > > the > > list of devices). > > Hence B3 never touches PM domains that don't contain any devices[*]. > > So these PM domains

Re: [PATCHv10 0/5] ARM: remove the sub-node and deprecate supports-highspeed property for dwmmc.

2014-08-27 Thread Pavel Machek
On Wed 2014-08-27 12:11:55, Jaehoon Chung wrote: > Hi, > > On 08/26/2014 07:19 PM, Pavel Machek wrote: > > Hi! > > > >>>> Would you elaborate? > >>>> > >>>> If I have a device like a phone, I may want to put one "slo

Re: [PATCHv10 0/5] ARM: remove the sub-node and deprecate supports-highspeed property for dwmmc.

2014-08-26 Thread Pavel Machek
Hi! > >> Would you elaborate? > >> > >> If I have a device like a phone, I may want to put one "slot" inside > >> phone for basic system, and offer second slot for user expansion > >> (initially empty). > > > > if multiple slot is supported, then a mmcqd should be processing for > > multiple slo

Re: [PATCHv10 0/5] ARM: remove the sub-node and deprecate supports-highspeed property for dwmmc.

2014-08-25 Thread Pavel Machek
On Mon 2014-08-25 20:28:21, Jaehoon Chung wrote: > On 08/25/2014 08:21 PM, Pavel Machek wrote: > > On Thu 2014-08-07 16:37:57, Jaehoon Chung wrote: > >> Since used the mmc_of_parse(), didn't parse the sub-node. > >> So we can remove the sub-node, because almost SoC

Re: [PATCHv10 0/5] ARM: remove the sub-node and deprecate supports-highspeed property for dwmmc.

2014-08-25 Thread Pavel Machek
On Thu 2014-08-07 16:37:57, Jaehoon Chung wrote: > Since used the mmc_of_parse(), didn't parse the sub-node. > So we can remove the sub-node, because almost SoC used the only one card per > a host. > And supports-highspeed can be replaced with "cap-mmc/sd-highspeed" >property. Would it be better

Re: [PATCHv6 3/5] ARM: dts: socfpga: unuse the slot-node and deprecated the supports-highspeed for dw-mmc

2014-08-25 Thread Pavel Machek
On Fri 2014-07-25 10:11:51, Jaehoon Chung wrote: > dw-mmc controller can support multiple slots. > But, there are no use-cases anywhere. So we don't need to support the > slot-node for dw-mmc controller. > And "supports-highspeed" property in dw-mmc is deprecated. > "supports-highspeed" property ca

Re: [PATCH 1/2] serial: samsung: Move uart_register_driver call to device probe

2014-01-26 Thread Pavel Machek
On Thu 2014-01-23 19:36:33, Mark Brown wrote: > On Thu, Jan 23, 2014 at 07:47:56PM +0100, Tomasz Figa wrote: > > On 23.01.2014 19:40, Mark Brown wrote: > > > >We'd need to leave it user selectable rather than enabling it for ARM, > > >the whole reason this got noticed is that people are trying to

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 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

Re: [PATCH] usb: phy: samsung: Introducing usb phy driver for hsotg

2012-10-20 Thread Pavel Machek
Hi! > > +#define S3C_PHYPWR (0x00) ... > > +#define S3C_PHYCLK (0x04) > > + > > +#define S3C_PHYCLK_MODE_SERIAL (1 << 6) > > +#define S3C_PHYCLK_EXT_OSC (1 << 5) > > +#define S3C_PHYCLK_COMMON_ON_N

Re: [PATCH 0/3] ARM: Removing ARCH_S5P6442

2011-05-17 Thread Pavel Machek
On Thu 2011-05-12 10:08:06, Kukjin Kim wrote: > This patch remvoes mach-s5p6442 directory for ARCH_S5P6442 in arch/arm. Why remove it? No users? Broken? ...? -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html --

Re: [PATCH 2/4] ARM: SAMSUNG: Rename s3c24xx touchscreen interface to s3c

2010-01-31 Thread Pavel Machek
On Tue 2010-01-26 14:55:58, Ben Dooks wrote: > On Fri, Jan 22, 2010 at 11:47:11PM +0100, Maurus Cuelenaere wrote: > > ARM: SAMSUNG: Rename s3c24xx touchscreen interface to s3c > > > > Signed-off-by: Maurus Cuelenaere > > Renaming for naming sake is in my view just a waste of diff, unless > anyon

Re: [patch 1/1] TOUCHSCREEN: S3C24XX touchscreen driver from Arnaud Patard.

2009-11-26 Thread Pavel Machek
Hi! > >>> +/* Per-touchscreen data. */ > >>> + > >>> +/** > >>> + * struct s3c2410ts - driver touchscreen state. > >>> + * @client: The ADC client we registered with the core driver. > >>> + * @dev: The device we are bound to. > >>> + * @input: The input device we registered with the input subsyst