All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 1/9] units: Add the HZ macros
@ 2021-02-24 14:42 Daniel Lezcano
  2021-02-24 14:42 ` [PATCH v2 2/9] thermal/drivers/devfreq_cooling: Use " Daniel Lezcano
                   ` (11 more replies)
  0 siblings, 12 replies; 37+ messages in thread
From: Daniel Lezcano @ 2021-02-24 14:42 UTC (permalink / raw)
  To: rafael, andriy.shevchenko
  Cc: linux-kernel, Christian Eggers, Rafael J. Wysocki, Lukasz Luba

The macros for the unit conversion for frequency are duplicated in
different places.

Provide these macros in the 'units' header, so they can be reused.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Reviewed-by: Christian Eggers <ceggers@arri.de>
Reviewed-by: Andy Shevchenko <Andy.Shevchenko@gmail.com>
---
 include/linux/units.h | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/include/linux/units.h b/include/linux/units.h
index dcc30a53fa93..218ec0d314b6 100644
--- a/include/linux/units.h
+++ b/include/linux/units.h
@@ -4,6 +4,10 @@
 
 #include <linux/math.h>
 
+#define HZ_PER_KHZ		1000L
+#define KHZ_PER_MHZ		1000L
+#define HZ_PER_MHZ		1000000L
+
 #define MILLIWATT_PER_WATT	1000L
 #define MICROWATT_PER_MILLIWATT	1000L
 #define MICROWATT_PER_WATT	1000000L
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 37+ messages in thread

* [PATCH v2 2/9] thermal/drivers/devfreq_cooling: Use HZ macros
  2021-02-24 14:42 [PATCH v2 1/9] units: Add the HZ macros Daniel Lezcano
@ 2021-02-24 14:42 ` Daniel Lezcano
  2021-02-24 14:42 ` [PATCH v2 3/9] devfreq: " Daniel Lezcano
                   ` (10 subsequent siblings)
  11 siblings, 0 replies; 37+ messages in thread
From: Daniel Lezcano @ 2021-02-24 14:42 UTC (permalink / raw)
  To: rafael, andriy.shevchenko
  Cc: linux-kernel, Zhang Rui, Amit Kucheria, open list:THERMAL

HZ unit conversion macros are available in units.h, use them and
remove the duplicate definition.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Reviewed-by: Christian Eggers <ceggers@arri.de>
---
 drivers/thermal/devfreq_cooling.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/thermal/devfreq_cooling.c b/drivers/thermal/devfreq_cooling.c
index fed3121ff2a1..fa5b8b0c7604 100644
--- a/drivers/thermal/devfreq_cooling.c
+++ b/drivers/thermal/devfreq_cooling.c
@@ -19,10 +19,10 @@
 #include <linux/pm_opp.h>
 #include <linux/pm_qos.h>
 #include <linux/thermal.h>
+#include <linux/units.h>
 
 #include <trace/events/thermal.h>
 
-#define HZ_PER_KHZ		1000
 #define SCALE_ERROR_MITIGATION	100
 
 static DEFINE_IDA(devfreq_ida);
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 37+ messages in thread

* [PATCH v2 3/9] devfreq: Use HZ macros
  2021-02-24 14:42 [PATCH v2 1/9] units: Add the HZ macros Daniel Lezcano
  2021-02-24 14:42 ` [PATCH v2 2/9] thermal/drivers/devfreq_cooling: Use " Daniel Lezcano
@ 2021-02-24 14:42 ` Daniel Lezcano
  2021-02-26  6:08   ` Chanwoo Choi
  2021-02-24 14:42 ` [PATCH v2 4/9] iio/drivers/as73211: " Daniel Lezcano
                   ` (9 subsequent siblings)
  11 siblings, 1 reply; 37+ messages in thread
From: Daniel Lezcano @ 2021-02-24 14:42 UTC (permalink / raw)
  To: rafael, andriy.shevchenko
  Cc: linux-kernel, Chanwoo Choi, MyungJoo Ham, Kyungmin Park,
	open list:DEVICE FREQUENCY (DEVFREQ)

HZ unit conversion macros are available in units.h, use them and
remove the duplicate definition.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Reviewed-by: Christian Eggers <ceggers@arri.de>
Acked-by: Chanwoo Choi <cw00.choi@samsung.com>
---
 drivers/devfreq/devfreq.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/devfreq/devfreq.c b/drivers/devfreq/devfreq.c
index bf3047896e41..b6d63f02d293 100644
--- a/drivers/devfreq/devfreq.c
+++ b/drivers/devfreq/devfreq.c
@@ -26,6 +26,7 @@
 #include <linux/hrtimer.h>
 #include <linux/of.h>
 #include <linux/pm_qos.h>
+#include <linux/units.h>
 #include "governor.h"
 
 #define CREATE_TRACE_POINTS
@@ -33,7 +34,6 @@
 
 #define IS_SUPPORTED_FLAG(f, name) ((f & DEVFREQ_GOV_FLAG_##name) ? true : false)
 #define IS_SUPPORTED_ATTR(f, name) ((f & DEVFREQ_GOV_ATTR_##name) ? true : false)
-#define HZ_PER_KHZ	1000
 
 static struct class *devfreq_class;
 static struct dentry *devfreq_debugfs;
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 37+ messages in thread

* [PATCH v2 4/9] iio/drivers/as73211: Use HZ macros
  2021-02-24 14:42 [PATCH v2 1/9] units: Add the HZ macros Daniel Lezcano
  2021-02-24 14:42 ` [PATCH v2 2/9] thermal/drivers/devfreq_cooling: Use " Daniel Lezcano
  2021-02-24 14:42 ` [PATCH v2 3/9] devfreq: " Daniel Lezcano
@ 2021-02-24 14:42 ` Daniel Lezcano
  2021-03-14 16:59   ` Jonathan Cameron
  2021-02-24 14:42 ` [PATCH v2 5/9] hwmon/drivers/mr75203: " Daniel Lezcano
                   ` (8 subsequent siblings)
  11 siblings, 1 reply; 37+ messages in thread
From: Daniel Lezcano @ 2021-02-24 14:42 UTC (permalink / raw)
  To: rafael, andriy.shevchenko
  Cc: linux-kernel, Christian Eggers, Jonathan Cameron,
	Lars-Peter Clausen, Peter Meerwald-Stadler,
	open list:AMS AS73211 DRIVER

HZ unit conversion macros are available in units.h, use them and
remove the duplicate definition.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Reviewed-by: Christian Eggers <ceggers@arri.de>
---
 drivers/iio/light/as73211.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/iio/light/as73211.c b/drivers/iio/light/as73211.c
index 7b32dfaee9b3..3ba2378df3dd 100644
--- a/drivers/iio/light/as73211.c
+++ b/drivers/iio/light/as73211.c
@@ -24,8 +24,7 @@
 #include <linux/module.h>
 #include <linux/mutex.h>
 #include <linux/pm.h>
-
-#define HZ_PER_KHZ 1000
+#include <linux/units.h>
 
 #define AS73211_DRV_NAME "as73211"
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 37+ messages in thread

* [PATCH v2 5/9] hwmon/drivers/mr75203: Use HZ macros
  2021-02-24 14:42 [PATCH v2 1/9] units: Add the HZ macros Daniel Lezcano
                   ` (2 preceding siblings ...)
  2021-02-24 14:42 ` [PATCH v2 4/9] iio/drivers/as73211: " Daniel Lezcano
@ 2021-02-24 14:42 ` Daniel Lezcano
  2021-03-11 21:44   ` Guenter Roeck
  2021-02-24 14:42 ` [PATCH v2 6/9] iio/drivers/hid-sensor: " Daniel Lezcano
                   ` (7 subsequent siblings)
  11 siblings, 1 reply; 37+ messages in thread
From: Daniel Lezcano @ 2021-02-24 14:42 UTC (permalink / raw)
  To: rafael, andriy.shevchenko
  Cc: linux-kernel, Jean Delvare, Guenter Roeck, open list:HARDWARE MONITORING

HZ unit conversion macros are available in units.h, use them and
remove the duplicate definition.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Reviewed-by: Christian Eggers <ceggers@arri.de>
---
 drivers/hwmon/mr75203.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/hwmon/mr75203.c b/drivers/hwmon/mr75203.c
index 18da5a25e89a..868243dba1ee 100644
--- a/drivers/hwmon/mr75203.c
+++ b/drivers/hwmon/mr75203.c
@@ -17,6 +17,7 @@
 #include <linux/property.h>
 #include <linux/regmap.h>
 #include <linux/reset.h>
+#include <linux/units.h>
 
 /* PVT Common register */
 #define PVT_IP_CONFIG	0x04
@@ -37,7 +38,6 @@
 #define CLK_SYNTH_EN		BIT(24)
 #define CLK_SYS_CYCLES_MAX	514
 #define CLK_SYS_CYCLES_MIN	2
-#define HZ_PER_MHZ		1000000L
 
 #define SDIF_DISABLE	0x04
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 37+ messages in thread

* [PATCH v2 6/9] iio/drivers/hid-sensor: Use HZ macros
  2021-02-24 14:42 [PATCH v2 1/9] units: Add the HZ macros Daniel Lezcano
                   ` (3 preceding siblings ...)
  2021-02-24 14:42 ` [PATCH v2 5/9] hwmon/drivers/mr75203: " Daniel Lezcano
@ 2021-02-24 14:42 ` Daniel Lezcano
  2021-03-14 17:00   ` Jonathan Cameron
  2021-02-24 14:42 ` [PATCH v2 7/9] i2c/drivers/ov02q10: " Daniel Lezcano
                   ` (6 subsequent siblings)
  11 siblings, 1 reply; 37+ messages in thread
From: Daniel Lezcano @ 2021-02-24 14:42 UTC (permalink / raw)
  To: rafael, andriy.shevchenko
  Cc: linux-kernel, Jonathan Cameron, Lars-Peter Clausen,
	Peter Meerwald-Stadler, open list:IIO SUBSYSTEM AND DRIVERS

HZ unit conversion macros are available in units.h, use them and
remove the duplicate definition.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
 drivers/iio/common/hid-sensors/hid-sensor-attributes.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/iio/common/hid-sensors/hid-sensor-attributes.c b/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
index 442ff787f7af..dda3b67e494f 100644
--- a/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
+++ b/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
@@ -11,13 +11,12 @@
 #include <linux/kernel.h>
 #include <linux/slab.h>
 #include <linux/time.h>
+#include <linux/units.h>
 
 #include <linux/hid-sensor-hub.h>
 #include <linux/iio/iio.h>
 #include <linux/iio/sysfs.h>
 
-#define HZ_PER_MHZ	1000000L
-
 static struct {
 	u32 usage_id;
 	int unit; /* 0 for default others from HID sensor spec */
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 37+ messages in thread

* [PATCH v2 7/9] i2c/drivers/ov02q10: Use HZ macros
  2021-02-24 14:42 [PATCH v2 1/9] units: Add the HZ macros Daniel Lezcano
                   ` (4 preceding siblings ...)
  2021-02-24 14:42 ` [PATCH v2 6/9] iio/drivers/hid-sensor: " Daniel Lezcano
@ 2021-02-24 14:42 ` Daniel Lezcano
  2021-02-24 14:42   ` Daniel Lezcano
                   ` (5 subsequent siblings)
  11 siblings, 0 replies; 37+ messages in thread
From: Daniel Lezcano @ 2021-02-24 14:42 UTC (permalink / raw)
  To: rafael, andriy.shevchenko
  Cc: linux-kernel, Dongchun Zhu, Mauro Carvalho Chehab,
	open list:OMNIVISION OV02A10 SENSOR DRIVER

HZ unit conversion macros are available in units.h, use them and
remove the duplicate definition.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
 drivers/media/i2c/ov02a10.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/i2c/ov02a10.c b/drivers/media/i2c/ov02a10.c
index 8683ffd3287a..59a34e59774e 100644
--- a/drivers/media/i2c/ov02a10.c
+++ b/drivers/media/i2c/ov02a10.c
@@ -9,6 +9,7 @@
 #include <linux/module.h>
 #include <linux/pm_runtime.h>
 #include <linux/regulator/consumer.h>
+#include <linux/units.h>
 #include <media/media-entity.h>
 #include <media/v4l2-async.h>
 #include <media/v4l2-ctrls.h>
@@ -64,7 +65,6 @@
 /* Test pattern control */
 #define OV02A10_REG_TEST_PATTERN			0xb6
 
-#define HZ_PER_MHZ					1000000L
 #define OV02A10_LINK_FREQ_390MHZ			(390 * HZ_PER_MHZ)
 #define OV02A10_ECLK_FREQ				(24 * HZ_PER_MHZ)
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 37+ messages in thread

* [PATCH v2 8/9] mtd/drivers/nand: Use HZ macros
  2021-02-24 14:42 [PATCH v2 1/9] units: Add the HZ macros Daniel Lezcano
@ 2021-02-24 14:42   ` Daniel Lezcano
  2021-02-24 14:42 ` [PATCH v2 3/9] devfreq: " Daniel Lezcano
                     ` (10 subsequent siblings)
  11 siblings, 0 replies; 37+ messages in thread
From: Daniel Lezcano @ 2021-02-24 14:42 UTC (permalink / raw)
  To: rafael, andriy.shevchenko
  Cc: linux-kernel, Miquel Raynal, Richard Weinberger,
	Vignesh Raghavendra, Martin Blumenstingl,
	Ramuthevar Vadivel Murugan, open list:NAND FLASH SUBSYSTEM

HZ unit conversion macros are available in units.h, use them and
remove the duplicate definition.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
 drivers/mtd/nand/raw/intel-nand-controller.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mtd/nand/raw/intel-nand-controller.c b/drivers/mtd/nand/raw/intel-nand-controller.c
index a304fda5d1fa..8d00191a4815 100644
--- a/drivers/mtd/nand/raw/intel-nand-controller.c
+++ b/drivers/mtd/nand/raw/intel-nand-controller.c
@@ -20,6 +20,7 @@
 #include <linux/sched.h>
 #include <linux/slab.h>
 #include <linux/types.h>
+#include <linux/units.h>
 #include <asm/unaligned.h>
 
 #define EBU_CLC			0x000
@@ -102,7 +103,6 @@
 
 #define MAX_CS	2
 
-#define HZ_PER_MHZ	1000000L
 #define USEC_PER_SEC	1000000L
 
 struct ebu_nand_cs {
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 37+ messages in thread

* [PATCH v2 8/9] mtd/drivers/nand: Use HZ macros
@ 2021-02-24 14:42   ` Daniel Lezcano
  0 siblings, 0 replies; 37+ messages in thread
From: Daniel Lezcano @ 2021-02-24 14:42 UTC (permalink / raw)
  To: rafael, andriy.shevchenko
  Cc: Vignesh Raghavendra, Martin Blumenstingl, Richard Weinberger,
	linux-kernel, Ramuthevar Vadivel Murugan,
	open list:NAND FLASH SUBSYSTEM, Miquel Raynal

HZ unit conversion macros are available in units.h, use them and
remove the duplicate definition.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
 drivers/mtd/nand/raw/intel-nand-controller.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mtd/nand/raw/intel-nand-controller.c b/drivers/mtd/nand/raw/intel-nand-controller.c
index a304fda5d1fa..8d00191a4815 100644
--- a/drivers/mtd/nand/raw/intel-nand-controller.c
+++ b/drivers/mtd/nand/raw/intel-nand-controller.c
@@ -20,6 +20,7 @@
 #include <linux/sched.h>
 #include <linux/slab.h>
 #include <linux/types.h>
+#include <linux/units.h>
 #include <asm/unaligned.h>
 
 #define EBU_CLC			0x000
@@ -102,7 +103,6 @@
 
 #define MAX_CS	2
 
-#define HZ_PER_MHZ	1000000L
 #define USEC_PER_SEC	1000000L
 
 struct ebu_nand_cs {
-- 
2.17.1


______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

^ permalink raw reply related	[flat|nested] 37+ messages in thread

* [PATCH v2 9/9] phy/drivers/stm32: Use HZ macros
  2021-02-24 14:42 [PATCH v2 1/9] units: Add the HZ macros Daniel Lezcano
@ 2021-02-24 14:42   ` Daniel Lezcano
  2021-02-24 14:42 ` [PATCH v2 3/9] devfreq: " Daniel Lezcano
                     ` (10 subsequent siblings)
  11 siblings, 0 replies; 37+ messages in thread
From: Daniel Lezcano @ 2021-02-24 14:42 UTC (permalink / raw)
  To: rafael, andriy.shevchenko
  Cc: linux-kernel, Kishon Vijay Abraham I, Vinod Koul,
	Maxime Coquelin, Alexandre Torgue, Amelie Delaunay,
	Etienne Carriere, Chunfeng Yun,
	moderated list:ARM/STM32 ARCHITECTURE,
	moderated list:ARM/STM32 ARCHITECTURE

HZ unit conversion macros are available in units.h, use them and
remove the duplicate definition.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
 drivers/phy/st/phy-stm32-usbphyc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/phy/st/phy-stm32-usbphyc.c b/drivers/phy/st/phy-stm32-usbphyc.c
index a54317e96c41..02dd12bb4692 100644
--- a/drivers/phy/st/phy-stm32-usbphyc.c
+++ b/drivers/phy/st/phy-stm32-usbphyc.c
@@ -14,6 +14,7 @@
 #include <linux/of_platform.h>
 #include <linux/phy/phy.h>
 #include <linux/reset.h>
+#include <linux/units.h>
 
 #define STM32_USBPHYC_PLL	0x0
 #define STM32_USBPHYC_MISC	0x8
@@ -48,7 +49,6 @@ static const char * const supplies_names[] = {
 #define PLL_FVCO_MHZ		2880
 #define PLL_INFF_MIN_RATE_HZ	19200000
 #define PLL_INFF_MAX_RATE_HZ	38400000
-#define HZ_PER_MHZ		1000000L
 
 struct pll_params {
 	u8 ndiv;
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 37+ messages in thread

* [PATCH v2 9/9] phy/drivers/stm32: Use HZ macros
@ 2021-02-24 14:42   ` Daniel Lezcano
  0 siblings, 0 replies; 37+ messages in thread
From: Daniel Lezcano @ 2021-02-24 14:42 UTC (permalink / raw)
  To: rafael, andriy.shevchenko
  Cc: Etienne Carriere, Amelie Delaunay, Alexandre Torgue,
	linux-kernel, Kishon Vijay Abraham I, Vinod Koul,
	Maxime Coquelin, Chunfeng Yun,
	moderated list:ARM/STM32 ARCHITECTURE,
	moderated list:ARM/STM32 ARCHITECTURE

HZ unit conversion macros are available in units.h, use them and
remove the duplicate definition.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
 drivers/phy/st/phy-stm32-usbphyc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/phy/st/phy-stm32-usbphyc.c b/drivers/phy/st/phy-stm32-usbphyc.c
index a54317e96c41..02dd12bb4692 100644
--- a/drivers/phy/st/phy-stm32-usbphyc.c
+++ b/drivers/phy/st/phy-stm32-usbphyc.c
@@ -14,6 +14,7 @@
 #include <linux/of_platform.h>
 #include <linux/phy/phy.h>
 #include <linux/reset.h>
+#include <linux/units.h>
 
 #define STM32_USBPHYC_PLL	0x0
 #define STM32_USBPHYC_MISC	0x8
@@ -48,7 +49,6 @@ static const char * const supplies_names[] = {
 #define PLL_FVCO_MHZ		2880
 #define PLL_INFF_MIN_RATE_HZ	19200000
 #define PLL_INFF_MAX_RATE_HZ	38400000
-#define HZ_PER_MHZ		1000000L
 
 struct pll_params {
 	u8 ndiv;
-- 
2.17.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 1/9] units: Add the HZ macros
  2021-02-24 14:42 [PATCH v2 1/9] units: Add the HZ macros Daniel Lezcano
                   ` (7 preceding siblings ...)
  2021-02-24 14:42   ` Daniel Lezcano
@ 2021-02-24 14:52 ` Daniel Lezcano
  2021-02-25 11:22 ` Andy Shevchenko
                   ` (2 subsequent siblings)
  11 siblings, 0 replies; 37+ messages in thread
From: Daniel Lezcano @ 2021-02-24 14:52 UTC (permalink / raw)
  To: rafael, andriy.shevchenko
  Cc: linux-kernel, Christian Eggers, Rafael J. Wysocki, Lukasz Luba

On Wed, Feb 24, 2021 at 03:42:11PM +0100, Daniel Lezcano wrote:
> The macros for the unit conversion for frequency are duplicated in
> different places.
> 
> Provide these macros in the 'units' header, so they can be reused.
> 
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> Reviewed-by: Christian Eggers <ceggers@arri.de>
> Reviewed-by: Andy Shevchenko <Andy.Shevchenko@gmail.com>
> ---

Is it possible to have these patches merged through the linux-pm tree ?




-- 

 <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 1/9] units: Add the HZ macros
  2021-02-24 14:42 [PATCH v2 1/9] units: Add the HZ macros Daniel Lezcano
                   ` (8 preceding siblings ...)
  2021-02-24 14:52 ` [PATCH v2 1/9] units: Add the " Daniel Lezcano
@ 2021-02-25 11:22 ` Andy Shevchenko
  2021-03-19  8:08   ` Daniel Lezcano
  2021-05-10 12:59 ` Andy Shevchenko
  2021-07-12 16:34 ` Andy Shevchenko
  11 siblings, 1 reply; 37+ messages in thread
From: Andy Shevchenko @ 2021-02-25 11:22 UTC (permalink / raw)
  To: Daniel Lezcano
  Cc: rafael, linux-kernel, Christian Eggers, Rafael J. Wysocki, Lukasz Luba

On Wed, Feb 24, 2021 at 03:42:11PM +0100, Daniel Lezcano wrote:
> The macros for the unit conversion for frequency are duplicated in
> different places.
> 
> Provide these macros in the 'units' header, so they can be reused.

For the all that have not been tagged:
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

Thanks!

> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> Reviewed-by: Christian Eggers <ceggers@arri.de>
> Reviewed-by: Andy Shevchenko <Andy.Shevchenko@gmail.com>
> ---
>  include/linux/units.h | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/include/linux/units.h b/include/linux/units.h
> index dcc30a53fa93..218ec0d314b6 100644
> --- a/include/linux/units.h
> +++ b/include/linux/units.h
> @@ -4,6 +4,10 @@
>  
>  #include <linux/math.h>
>  
> +#define HZ_PER_KHZ		1000L
> +#define KHZ_PER_MHZ		1000L
> +#define HZ_PER_MHZ		1000000L
> +
>  #define MILLIWATT_PER_WATT	1000L
>  #define MICROWATT_PER_MILLIWATT	1000L
>  #define MICROWATT_PER_WATT	1000000L
> -- 
> 2.17.1
> 

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 3/9] devfreq: Use HZ macros
  2021-02-24 14:42 ` [PATCH v2 3/9] devfreq: " Daniel Lezcano
@ 2021-02-26  6:08   ` Chanwoo Choi
  2021-02-26  8:51     ` Daniel Lezcano
  0 siblings, 1 reply; 37+ messages in thread
From: Chanwoo Choi @ 2021-02-26  6:08 UTC (permalink / raw)
  To: Daniel Lezcano, rafael, andriy.shevchenko
  Cc: linux-kernel, Chanwoo Choi, MyungJoo Ham, Kyungmin Park,
	open list:DEVICE FREQUENCY (DEVFREQ)



On 21. 2. 24. 오후 11:42, Daniel Lezcano wrote:
> HZ unit conversion macros are available in units.h, use them and
> remove the duplicate definition.
> 
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> Reviewed-by: Christian Eggers <ceggers@arri.de>
> Acked-by: Chanwoo Choi <cw00.choi@samsung.com>
> ---
>   drivers/devfreq/devfreq.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/devfreq/devfreq.c b/drivers/devfreq/devfreq.c
> index bf3047896e41..b6d63f02d293 100644
> --- a/drivers/devfreq/devfreq.c
> +++ b/drivers/devfreq/devfreq.c
> @@ -26,6 +26,7 @@
>   #include <linux/hrtimer.h>
>   #include <linux/of.h>
>   #include <linux/pm_qos.h>
> +#include <linux/units.h>
>   #include "governor.h"
>   
>   #define CREATE_TRACE_POINTS
> @@ -33,7 +34,6 @@
>   
>   #define IS_SUPPORTED_FLAG(f, name) ((f & DEVFREQ_GOV_FLAG_##name) ? true : false)
>   #define IS_SUPPORTED_ATTR(f, name) ((f & DEVFREQ_GOV_ATTR_##name) ? true : false)
> -#define HZ_PER_KHZ	1000
>   
>   static struct class *devfreq_class;
>   static struct dentry *devfreq_debugfs;
> 

I changed the patch title with 'PM /' prefix as following
in order to keep the consistent patch style if there are no any special 
objection.
- PM / devfreq: Use HZ macros

Applied it.

Thanks,
Chanwoo Choi

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 3/9] devfreq: Use HZ macros
  2021-02-26  6:08   ` Chanwoo Choi
@ 2021-02-26  8:51     ` Daniel Lezcano
  2021-02-26  9:14       ` Chanwoo Choi
  0 siblings, 1 reply; 37+ messages in thread
From: Daniel Lezcano @ 2021-02-26  8:51 UTC (permalink / raw)
  To: Chanwoo Choi, rafael, andriy.shevchenko
  Cc: linux-kernel, Chanwoo Choi, MyungJoo Ham, Kyungmin Park,
	open list:DEVICE FREQUENCY (DEVFREQ)

On 26/02/2021 07:08, Chanwoo Choi wrote:
> 
> 
> On 21. 2. 24. 오후 11:42, Daniel Lezcano wrote:
>> HZ unit conversion macros are available in units.h, use them and
>> remove the duplicate definition.
>>
>> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
>> Reviewed-by: Christian Eggers <ceggers@arri.de>
>> Acked-by: Chanwoo Choi <cw00.choi@samsung.com>
>> ---
>>   drivers/devfreq/devfreq.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/devfreq/devfreq.c b/drivers/devfreq/devfreq.c
>> index bf3047896e41..b6d63f02d293 100644
>> --- a/drivers/devfreq/devfreq.c
>> +++ b/drivers/devfreq/devfreq.c
>> @@ -26,6 +26,7 @@
>>   #include <linux/hrtimer.h>
>>   #include <linux/of.h>
>>   #include <linux/pm_qos.h>
>> +#include <linux/units.h>
>>   #include "governor.h"
>>     #define CREATE_TRACE_POINTS
>> @@ -33,7 +34,6 @@
>>     #define IS_SUPPORTED_FLAG(f, name) ((f & DEVFREQ_GOV_FLAG_##name)
>> ? true : false)
>>   #define IS_SUPPORTED_ATTR(f, name) ((f & DEVFREQ_GOV_ATTR_##name) ?
>> true : false)
>> -#define HZ_PER_KHZ    1000
>>     static struct class *devfreq_class;
>>   static struct dentry *devfreq_debugfs;
>>
> 
> I changed the patch title with 'PM /' prefix as following
> in order to keep the consistent patch style if there are no any special
> objection.
> - PM / devfreq: Use HZ macros
> 
> Applied it.

It should not compile. This patch and others depend on 1/9.

It would make sense to merge all of them through linux-pm.


-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 3/9] devfreq: Use HZ macros
  2021-02-26  8:51     ` Daniel Lezcano
@ 2021-02-26  9:14       ` Chanwoo Choi
  0 siblings, 0 replies; 37+ messages in thread
From: Chanwoo Choi @ 2021-02-26  9:14 UTC (permalink / raw)
  To: Daniel Lezcano, rafael, andriy.shevchenko
  Cc: linux-kernel, Chanwoo Choi, MyungJoo Ham, Kyungmin Park,
	open list:DEVICE FREQUENCY (DEVFREQ)



On 21. 2. 26. 오후 5:51, Daniel Lezcano wrote:
> On 26/02/2021 07:08, Chanwoo Choi wrote:
>>
>>
>> On 21. 2. 24. 오후 11:42, Daniel Lezcano wrote:
>>> HZ unit conversion macros are available in units.h, use them and
>>> remove the duplicate definition.
>>>
>>> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
>>> Reviewed-by: Christian Eggers <ceggers@arri.de>
>>> Acked-by: Chanwoo Choi <cw00.choi@samsung.com>
>>> ---
>>>    drivers/devfreq/devfreq.c | 2 +-
>>>    1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/devfreq/devfreq.c b/drivers/devfreq/devfreq.c
>>> index bf3047896e41..b6d63f02d293 100644
>>> --- a/drivers/devfreq/devfreq.c
>>> +++ b/drivers/devfreq/devfreq.c
>>> @@ -26,6 +26,7 @@
>>>    #include <linux/hrtimer.h>
>>>    #include <linux/of.h>
>>>    #include <linux/pm_qos.h>
>>> +#include <linux/units.h>
>>>    #include "governor.h"
>>>      #define CREATE_TRACE_POINTS
>>> @@ -33,7 +34,6 @@
>>>      #define IS_SUPPORTED_FLAG(f, name) ((f & DEVFREQ_GOV_FLAG_##name)
>>> ? true : false)
>>>    #define IS_SUPPORTED_ATTR(f, name) ((f & DEVFREQ_GOV_ATTR_##name) ?
>>> true : false)
>>> -#define HZ_PER_KHZ    1000
>>>      static struct class *devfreq_class;
>>>    static struct dentry *devfreq_debugfs;
>>>
>>
>> I changed the patch title with 'PM /' prefix as following
>> in order to keep the consistent patch style if there are no any special
>> objection.
>> - PM / devfreq: Use HZ macros
>>
>> Applied it.
> 
> It should not compile. This patch and others depend on 1/9.
> 
> It would make sense to merge all of them through linux-pm.

OK. I'm OK to apply it to linux-pm.

Thanks,
Chanwoo Choi

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 8/9] mtd/drivers/nand: Use HZ macros
  2021-02-24 14:42   ` Daniel Lezcano
@ 2021-03-02 16:31     ` Miquel Raynal
  -1 siblings, 0 replies; 37+ messages in thread
From: Miquel Raynal @ 2021-03-02 16:31 UTC (permalink / raw)
  To: Daniel Lezcano, rafael, andriy.shevchenko
  Cc: Miquel Raynal, Vignesh Raghavendra, Martin Blumenstingl,
	Richard Weinberger, linux-kernel, Ramuthevar Vadivel Murugan,
	open list:NAND FLASH SUBSYSTEM

On Wed, 2021-02-24 at 14:42:18 UTC, Daniel Lezcano wrote:
> HZ unit conversion macros are available in units.h, use them and
> remove the duplicate definition.
> 
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>

Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git nand/next, thanks.

Miquel

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 8/9] mtd/drivers/nand: Use HZ macros
@ 2021-03-02 16:31     ` Miquel Raynal
  0 siblings, 0 replies; 37+ messages in thread
From: Miquel Raynal @ 2021-03-02 16:31 UTC (permalink / raw)
  To: Daniel Lezcano, rafael, andriy.shevchenko
  Cc: Miquel Raynal, Vignesh Raghavendra, Martin Blumenstingl,
	Richard Weinberger, linux-kernel, Ramuthevar Vadivel Murugan,
	open list:NAND FLASH SUBSYSTEM

On Wed, 2021-02-24 at 14:42:18 UTC, Daniel Lezcano wrote:
> HZ unit conversion macros are available in units.h, use them and
> remove the duplicate definition.
> 
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>

Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git nand/next, thanks.

Miquel

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 9/9] phy/drivers/stm32: Use HZ macros
  2021-02-24 14:42   ` Daniel Lezcano
@ 2021-03-02 16:32     ` Vinod Koul
  -1 siblings, 0 replies; 37+ messages in thread
From: Vinod Koul @ 2021-03-02 16:32 UTC (permalink / raw)
  To: Daniel Lezcano
  Cc: rafael, andriy.shevchenko, linux-kernel, Kishon Vijay Abraham I,
	Maxime Coquelin, Alexandre Torgue, Amelie Delaunay,
	Etienne Carriere, Chunfeng Yun,
	moderated list:ARM/STM32 ARCHITECTURE,
	moderated list:ARM/STM32 ARCHITECTURE

On 24-02-21, 15:42, Daniel Lezcano wrote:
> HZ unit conversion macros are available in units.h, use them and
> remove the duplicate definition.
> 
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> ---
>  drivers/phy/st/phy-stm32-usbphyc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/phy/st/phy-stm32-usbphyc.c b/drivers/phy/st/phy-stm32-usbphyc.c
> index a54317e96c41..02dd12bb4692 100644
> --- a/drivers/phy/st/phy-stm32-usbphyc.c
> +++ b/drivers/phy/st/phy-stm32-usbphyc.c
> @@ -14,6 +14,7 @@
>  #include <linux/of_platform.h>
>  #include <linux/phy/phy.h>
>  #include <linux/reset.h>
> +#include <linux/units.h>
>  
>  #define STM32_USBPHYC_PLL	0x0
>  #define STM32_USBPHYC_MISC	0x8
> @@ -48,7 +49,6 @@ static const char * const supplies_names[] = {
>  #define PLL_FVCO_MHZ		2880
>  #define PLL_INFF_MIN_RATE_HZ	19200000
>  #define PLL_INFF_MAX_RATE_HZ	38400000
> -#define HZ_PER_MHZ		1000000L

I dont see this in units.h, can you send this once it is merged upstream

-- 
~Vinod

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 9/9] phy/drivers/stm32: Use HZ macros
@ 2021-03-02 16:32     ` Vinod Koul
  0 siblings, 0 replies; 37+ messages in thread
From: Vinod Koul @ 2021-03-02 16:32 UTC (permalink / raw)
  To: Daniel Lezcano
  Cc: rafael, andriy.shevchenko, linux-kernel, Kishon Vijay Abraham I,
	Maxime Coquelin, Alexandre Torgue, Amelie Delaunay,
	Etienne Carriere, Chunfeng Yun,
	moderated list:ARM/STM32 ARCHITECTURE,
	moderated list:ARM/STM32 ARCHITECTURE

On 24-02-21, 15:42, Daniel Lezcano wrote:
> HZ unit conversion macros are available in units.h, use them and
> remove the duplicate definition.
> 
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> ---
>  drivers/phy/st/phy-stm32-usbphyc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/phy/st/phy-stm32-usbphyc.c b/drivers/phy/st/phy-stm32-usbphyc.c
> index a54317e96c41..02dd12bb4692 100644
> --- a/drivers/phy/st/phy-stm32-usbphyc.c
> +++ b/drivers/phy/st/phy-stm32-usbphyc.c
> @@ -14,6 +14,7 @@
>  #include <linux/of_platform.h>
>  #include <linux/phy/phy.h>
>  #include <linux/reset.h>
> +#include <linux/units.h>
>  
>  #define STM32_USBPHYC_PLL	0x0
>  #define STM32_USBPHYC_MISC	0x8
> @@ -48,7 +49,6 @@ static const char * const supplies_names[] = {
>  #define PLL_FVCO_MHZ		2880
>  #define PLL_INFF_MIN_RATE_HZ	19200000
>  #define PLL_INFF_MAX_RATE_HZ	38400000
> -#define HZ_PER_MHZ		1000000L

I dont see this in units.h, can you send this once it is merged upstream

-- 
~Vinod

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 8/9] mtd/drivers/nand: Use HZ macros
  2021-03-02 16:31     ` Miquel Raynal
@ 2021-03-02 17:03       ` Daniel Lezcano
  -1 siblings, 0 replies; 37+ messages in thread
From: Daniel Lezcano @ 2021-03-02 17:03 UTC (permalink / raw)
  To: Miquel Raynal, rafael, andriy.shevchenko
  Cc: Vignesh Raghavendra, Martin Blumenstingl, Richard Weinberger,
	linux-kernel, Ramuthevar Vadivel Murugan,
	open list:NAND FLASH SUBSYSTEM

On 02/03/2021 17:31, Miquel Raynal wrote:
> On Wed, 2021-02-24 at 14:42:18 UTC, Daniel Lezcano wrote:
>> HZ unit conversion macros are available in units.h, use them and
>> remove the duplicate definition.
>>
>> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> 
> Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git nand/next, thanks.

Actually, I was expecting to have it merged through linux-pm as this
patch depends on 1/9



-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 8/9] mtd/drivers/nand: Use HZ macros
@ 2021-03-02 17:03       ` Daniel Lezcano
  0 siblings, 0 replies; 37+ messages in thread
From: Daniel Lezcano @ 2021-03-02 17:03 UTC (permalink / raw)
  To: Miquel Raynal, rafael, andriy.shevchenko
  Cc: Vignesh Raghavendra, Martin Blumenstingl, Richard Weinberger,
	linux-kernel, Ramuthevar Vadivel Murugan,
	open list:NAND FLASH SUBSYSTEM

On 02/03/2021 17:31, Miquel Raynal wrote:
> On Wed, 2021-02-24 at 14:42:18 UTC, Daniel Lezcano wrote:
>> HZ unit conversion macros are available in units.h, use them and
>> remove the duplicate definition.
>>
>> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> 
> Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git nand/next, thanks.

Actually, I was expecting to have it merged through linux-pm as this
patch depends on 1/9



-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 9/9] phy/drivers/stm32: Use HZ macros
  2021-03-02 16:32     ` Vinod Koul
@ 2021-03-02 17:03       ` Daniel Lezcano
  -1 siblings, 0 replies; 37+ messages in thread
From: Daniel Lezcano @ 2021-03-02 17:03 UTC (permalink / raw)
  To: Vinod Koul
  Cc: rafael, andriy.shevchenko, linux-kernel, Kishon Vijay Abraham I,
	Maxime Coquelin, Alexandre Torgue, Amelie Delaunay,
	Etienne Carriere, Chunfeng Yun,
	moderated list:ARM/STM32 ARCHITECTURE,
	moderated list:ARM/STM32 ARCHITECTURE

On 02/03/2021 17:32, Vinod Koul wrote:
> On 24-02-21, 15:42, Daniel Lezcano wrote:
>> HZ unit conversion macros are available in units.h, use them and
>> remove the duplicate definition.
>>
>> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
>> ---
>>  drivers/phy/st/phy-stm32-usbphyc.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/phy/st/phy-stm32-usbphyc.c b/drivers/phy/st/phy-stm32-usbphyc.c
>> index a54317e96c41..02dd12bb4692 100644
>> --- a/drivers/phy/st/phy-stm32-usbphyc.c
>> +++ b/drivers/phy/st/phy-stm32-usbphyc.c
>> @@ -14,6 +14,7 @@
>>  #include <linux/of_platform.h>
>>  #include <linux/phy/phy.h>
>>  #include <linux/reset.h>
>> +#include <linux/units.h>
>>  
>>  #define STM32_USBPHYC_PLL	0x0
>>  #define STM32_USBPHYC_MISC	0x8
>> @@ -48,7 +49,6 @@ static const char * const supplies_names[] = {
>>  #define PLL_FVCO_MHZ		2880
>>  #define PLL_INFF_MIN_RATE_HZ	19200000
>>  #define PLL_INFF_MAX_RATE_HZ	38400000
>> -#define HZ_PER_MHZ		1000000L
> 
> I dont see this in units.h, can you send this once it is merged upstream

Actually, it is the first patch of the series.

I asked Rafael if he can merge the entire series.




-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 9/9] phy/drivers/stm32: Use HZ macros
@ 2021-03-02 17:03       ` Daniel Lezcano
  0 siblings, 0 replies; 37+ messages in thread
From: Daniel Lezcano @ 2021-03-02 17:03 UTC (permalink / raw)
  To: Vinod Koul
  Cc: rafael, andriy.shevchenko, linux-kernel, Kishon Vijay Abraham I,
	Maxime Coquelin, Alexandre Torgue, Amelie Delaunay,
	Etienne Carriere, Chunfeng Yun,
	moderated list:ARM/STM32 ARCHITECTURE,
	moderated list:ARM/STM32 ARCHITECTURE

On 02/03/2021 17:32, Vinod Koul wrote:
> On 24-02-21, 15:42, Daniel Lezcano wrote:
>> HZ unit conversion macros are available in units.h, use them and
>> remove the duplicate definition.
>>
>> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
>> ---
>>  drivers/phy/st/phy-stm32-usbphyc.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/phy/st/phy-stm32-usbphyc.c b/drivers/phy/st/phy-stm32-usbphyc.c
>> index a54317e96c41..02dd12bb4692 100644
>> --- a/drivers/phy/st/phy-stm32-usbphyc.c
>> +++ b/drivers/phy/st/phy-stm32-usbphyc.c
>> @@ -14,6 +14,7 @@
>>  #include <linux/of_platform.h>
>>  #include <linux/phy/phy.h>
>>  #include <linux/reset.h>
>> +#include <linux/units.h>
>>  
>>  #define STM32_USBPHYC_PLL	0x0
>>  #define STM32_USBPHYC_MISC	0x8
>> @@ -48,7 +49,6 @@ static const char * const supplies_names[] = {
>>  #define PLL_FVCO_MHZ		2880
>>  #define PLL_INFF_MIN_RATE_HZ	19200000
>>  #define PLL_INFF_MAX_RATE_HZ	38400000
>> -#define HZ_PER_MHZ		1000000L
> 
> I dont see this in units.h, can you send this once it is merged upstream

Actually, it is the first patch of the series.

I asked Rafael if he can merge the entire series.




-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 8/9] mtd/drivers/nand: Use HZ macros
  2021-03-02 17:03       ` Daniel Lezcano
@ 2021-03-02 17:12         ` Miquel Raynal
  -1 siblings, 0 replies; 37+ messages in thread
From: Miquel Raynal @ 2021-03-02 17:12 UTC (permalink / raw)
  To: Daniel Lezcano
  Cc: rafael, andriy.shevchenko, Vignesh Raghavendra,
	Martin Blumenstingl, Richard Weinberger, linux-kernel,
	Ramuthevar Vadivel Murugan, open list:NAND FLASH SUBSYSTEM

Hi Daniel,

Daniel Lezcano <daniel.lezcano@linaro.org> wrote on Tue, 2 Mar 2021
18:03:12 +0100:

> On 02/03/2021 17:31, Miquel Raynal wrote:
> > On Wed, 2021-02-24 at 14:42:18 UTC, Daniel Lezcano wrote:  
> >> HZ unit conversion macros are available in units.h, use them and
> >> remove the duplicate definition.
> >>
> >> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>  
> > 
> > Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git nand/next, thanks.  
> 
> Actually, I was expecting to have it merged through linux-pm as this
> patch depends on 1/9
> 
> 
> 

No problem, I just removed it from my tree. However in this case
please fix the subject prefix to "mtd: rawnand: intel:". With this nit
fixed,

Acked-by: Miquel Raynal <miquel.raynal@bootlin.com>

Thanks,
Miquèl

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 8/9] mtd/drivers/nand: Use HZ macros
@ 2021-03-02 17:12         ` Miquel Raynal
  0 siblings, 0 replies; 37+ messages in thread
From: Miquel Raynal @ 2021-03-02 17:12 UTC (permalink / raw)
  To: Daniel Lezcano
  Cc: rafael, andriy.shevchenko, Vignesh Raghavendra,
	Martin Blumenstingl, Richard Weinberger, linux-kernel,
	Ramuthevar Vadivel Murugan, open list:NAND FLASH SUBSYSTEM

Hi Daniel,

Daniel Lezcano <daniel.lezcano@linaro.org> wrote on Tue, 2 Mar 2021
18:03:12 +0100:

> On 02/03/2021 17:31, Miquel Raynal wrote:
> > On Wed, 2021-02-24 at 14:42:18 UTC, Daniel Lezcano wrote:  
> >> HZ unit conversion macros are available in units.h, use them and
> >> remove the duplicate definition.
> >>
> >> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>  
> > 
> > Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git nand/next, thanks.  
> 
> Actually, I was expecting to have it merged through linux-pm as this
> patch depends on 1/9
> 
> 
> 

No problem, I just removed it from my tree. However in this case
please fix the subject prefix to "mtd: rawnand: intel:". With this nit
fixed,

Acked-by: Miquel Raynal <miquel.raynal@bootlin.com>

Thanks,
Miquèl

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 9/9] phy/drivers/stm32: Use HZ macros
  2021-03-02 17:03       ` Daniel Lezcano
@ 2021-03-03  5:40         ` Vinod Koul
  -1 siblings, 0 replies; 37+ messages in thread
From: Vinod Koul @ 2021-03-03  5:40 UTC (permalink / raw)
  To: Daniel Lezcano
  Cc: rafael, andriy.shevchenko, linux-kernel, Kishon Vijay Abraham I,
	Maxime Coquelin, Alexandre Torgue, Amelie Delaunay,
	Etienne Carriere, Chunfeng Yun,
	moderated list:ARM/STM32 ARCHITECTURE,
	moderated list:ARM/STM32 ARCHITECTURE

On 02-03-21, 18:03, Daniel Lezcano wrote:
> On 02/03/2021 17:32, Vinod Koul wrote:
> > On 24-02-21, 15:42, Daniel Lezcano wrote:
> >> HZ unit conversion macros are available in units.h, use them and
> >> remove the duplicate definition.
> >>
> >> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> >> ---
> >>  drivers/phy/st/phy-stm32-usbphyc.c | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/drivers/phy/st/phy-stm32-usbphyc.c b/drivers/phy/st/phy-stm32-usbphyc.c
> >> index a54317e96c41..02dd12bb4692 100644
> >> --- a/drivers/phy/st/phy-stm32-usbphyc.c
> >> +++ b/drivers/phy/st/phy-stm32-usbphyc.c
> >> @@ -14,6 +14,7 @@
> >>  #include <linux/of_platform.h>
> >>  #include <linux/phy/phy.h>
> >>  #include <linux/reset.h>
> >> +#include <linux/units.h>
> >>  
> >>  #define STM32_USBPHYC_PLL	0x0
> >>  #define STM32_USBPHYC_MISC	0x8
> >> @@ -48,7 +49,6 @@ static const char * const supplies_names[] = {
> >>  #define PLL_FVCO_MHZ		2880
> >>  #define PLL_INFF_MIN_RATE_HZ	19200000
> >>  #define PLL_INFF_MAX_RATE_HZ	38400000
> >> -#define HZ_PER_MHZ		1000000L
> > 
> > I dont see this in units.h, can you send this once it is merged upstream
> 
> Actually, it is the first patch of the series.
> 
> I asked Rafael if he can merge the entire series.

Pls cc folks on cover at least so that they are aware.

Thanks
-- 
~Vinod

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 9/9] phy/drivers/stm32: Use HZ macros
@ 2021-03-03  5:40         ` Vinod Koul
  0 siblings, 0 replies; 37+ messages in thread
From: Vinod Koul @ 2021-03-03  5:40 UTC (permalink / raw)
  To: Daniel Lezcano
  Cc: rafael, andriy.shevchenko, linux-kernel, Kishon Vijay Abraham I,
	Maxime Coquelin, Alexandre Torgue, Amelie Delaunay,
	Etienne Carriere, Chunfeng Yun,
	moderated list:ARM/STM32 ARCHITECTURE,
	moderated list:ARM/STM32 ARCHITECTURE

On 02-03-21, 18:03, Daniel Lezcano wrote:
> On 02/03/2021 17:32, Vinod Koul wrote:
> > On 24-02-21, 15:42, Daniel Lezcano wrote:
> >> HZ unit conversion macros are available in units.h, use them and
> >> remove the duplicate definition.
> >>
> >> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> >> ---
> >>  drivers/phy/st/phy-stm32-usbphyc.c | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/drivers/phy/st/phy-stm32-usbphyc.c b/drivers/phy/st/phy-stm32-usbphyc.c
> >> index a54317e96c41..02dd12bb4692 100644
> >> --- a/drivers/phy/st/phy-stm32-usbphyc.c
> >> +++ b/drivers/phy/st/phy-stm32-usbphyc.c
> >> @@ -14,6 +14,7 @@
> >>  #include <linux/of_platform.h>
> >>  #include <linux/phy/phy.h>
> >>  #include <linux/reset.h>
> >> +#include <linux/units.h>
> >>  
> >>  #define STM32_USBPHYC_PLL	0x0
> >>  #define STM32_USBPHYC_MISC	0x8
> >> @@ -48,7 +49,6 @@ static const char * const supplies_names[] = {
> >>  #define PLL_FVCO_MHZ		2880
> >>  #define PLL_INFF_MIN_RATE_HZ	19200000
> >>  #define PLL_INFF_MAX_RATE_HZ	38400000
> >> -#define HZ_PER_MHZ		1000000L
> > 
> > I dont see this in units.h, can you send this once it is merged upstream
> 
> Actually, it is the first patch of the series.
> 
> I asked Rafael if he can merge the entire series.

Pls cc folks on cover at least so that they are aware.

Thanks
-- 
~Vinod

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 9/9] phy/drivers/stm32: Use HZ macros
  2021-03-03  5:40         ` Vinod Koul
@ 2021-03-03  6:09           ` Daniel Lezcano
  -1 siblings, 0 replies; 37+ messages in thread
From: Daniel Lezcano @ 2021-03-03  6:09 UTC (permalink / raw)
  To: Vinod Koul
  Cc: rafael, andriy.shevchenko, linux-kernel, Kishon Vijay Abraham I,
	Maxime Coquelin, Alexandre Torgue, Amelie Delaunay,
	Etienne Carriere, Chunfeng Yun,
	moderated list:ARM/STM32 ARCHITECTURE,
	moderated list:ARM/STM32 ARCHITECTURE

On 03/03/2021 06:40, Vinod Koul wrote:
> On 02-03-21, 18:03, Daniel Lezcano wrote:
>> On 02/03/2021 17:32, Vinod Koul wrote:
>>> On 24-02-21, 15:42, Daniel Lezcano wrote:
>>>> HZ unit conversion macros are available in units.h, use them and
>>>> remove the duplicate definition.
>>>>
>>>> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
>>>> ---
>>>>  drivers/phy/st/phy-stm32-usbphyc.c | 2 +-
>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/drivers/phy/st/phy-stm32-usbphyc.c b/drivers/phy/st/phy-stm32-usbphyc.c
>>>> index a54317e96c41..02dd12bb4692 100644
>>>> --- a/drivers/phy/st/phy-stm32-usbphyc.c
>>>> +++ b/drivers/phy/st/phy-stm32-usbphyc.c
>>>> @@ -14,6 +14,7 @@
>>>>  #include <linux/of_platform.h>
>>>>  #include <linux/phy/phy.h>
>>>>  #include <linux/reset.h>
>>>> +#include <linux/units.h>
>>>>  
>>>>  #define STM32_USBPHYC_PLL	0x0
>>>>  #define STM32_USBPHYC_MISC	0x8
>>>> @@ -48,7 +49,6 @@ static const char * const supplies_names[] = {
>>>>  #define PLL_FVCO_MHZ		2880
>>>>  #define PLL_INFF_MIN_RATE_HZ	19200000
>>>>  #define PLL_INFF_MAX_RATE_HZ	38400000
>>>> -#define HZ_PER_MHZ		1000000L
>>>
>>> I dont see this in units.h, can you send this once it is merged upstream
>>
>> Actually, it is the first patch of the series.
>>
>> I asked Rafael if he can merge the entire series.
> 
> Pls cc folks on cover at least so that they are aware.

Yeah, sorry for that. I tend to rely too much on the default 'cccmd'.



-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 9/9] phy/drivers/stm32: Use HZ macros
@ 2021-03-03  6:09           ` Daniel Lezcano
  0 siblings, 0 replies; 37+ messages in thread
From: Daniel Lezcano @ 2021-03-03  6:09 UTC (permalink / raw)
  To: Vinod Koul
  Cc: rafael, andriy.shevchenko, linux-kernel, Kishon Vijay Abraham I,
	Maxime Coquelin, Alexandre Torgue, Amelie Delaunay,
	Etienne Carriere, Chunfeng Yun,
	moderated list:ARM/STM32 ARCHITECTURE,
	moderated list:ARM/STM32 ARCHITECTURE

On 03/03/2021 06:40, Vinod Koul wrote:
> On 02-03-21, 18:03, Daniel Lezcano wrote:
>> On 02/03/2021 17:32, Vinod Koul wrote:
>>> On 24-02-21, 15:42, Daniel Lezcano wrote:
>>>> HZ unit conversion macros are available in units.h, use them and
>>>> remove the duplicate definition.
>>>>
>>>> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
>>>> ---
>>>>  drivers/phy/st/phy-stm32-usbphyc.c | 2 +-
>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/drivers/phy/st/phy-stm32-usbphyc.c b/drivers/phy/st/phy-stm32-usbphyc.c
>>>> index a54317e96c41..02dd12bb4692 100644
>>>> --- a/drivers/phy/st/phy-stm32-usbphyc.c
>>>> +++ b/drivers/phy/st/phy-stm32-usbphyc.c
>>>> @@ -14,6 +14,7 @@
>>>>  #include <linux/of_platform.h>
>>>>  #include <linux/phy/phy.h>
>>>>  #include <linux/reset.h>
>>>> +#include <linux/units.h>
>>>>  
>>>>  #define STM32_USBPHYC_PLL	0x0
>>>>  #define STM32_USBPHYC_MISC	0x8
>>>> @@ -48,7 +49,6 @@ static const char * const supplies_names[] = {
>>>>  #define PLL_FVCO_MHZ		2880
>>>>  #define PLL_INFF_MIN_RATE_HZ	19200000
>>>>  #define PLL_INFF_MAX_RATE_HZ	38400000
>>>> -#define HZ_PER_MHZ		1000000L
>>>
>>> I dont see this in units.h, can you send this once it is merged upstream
>>
>> Actually, it is the first patch of the series.
>>
>> I asked Rafael if he can merge the entire series.
> 
> Pls cc folks on cover at least so that they are aware.

Yeah, sorry for that. I tend to rely too much on the default 'cccmd'.



-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 5/9] hwmon/drivers/mr75203: Use HZ macros
  2021-02-24 14:42 ` [PATCH v2 5/9] hwmon/drivers/mr75203: " Daniel Lezcano
@ 2021-03-11 21:44   ` Guenter Roeck
  0 siblings, 0 replies; 37+ messages in thread
From: Guenter Roeck @ 2021-03-11 21:44 UTC (permalink / raw)
  To: Daniel Lezcano
  Cc: rafael, andriy.shevchenko, linux-kernel, Jean Delvare,
	open list:HARDWARE MONITORING

On Wed, Feb 24, 2021 at 03:42:15PM +0100, Daniel Lezcano wrote:
> HZ unit conversion macros are available in units.h, use them and
> remove the duplicate definition.

I assume the idea is to submit the series together, so

Acked-by: Guenter Roeck <linux@roeck-us.net>

Guenter

> 
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> Reviewed-by: Christian Eggers <ceggers@arri.de>
> ---
>  drivers/hwmon/mr75203.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/hwmon/mr75203.c b/drivers/hwmon/mr75203.c
> index 18da5a25e89a..868243dba1ee 100644
> --- a/drivers/hwmon/mr75203.c
> +++ b/drivers/hwmon/mr75203.c
> @@ -17,6 +17,7 @@
>  #include <linux/property.h>
>  #include <linux/regmap.h>
>  #include <linux/reset.h>
> +#include <linux/units.h>
>  
>  /* PVT Common register */
>  #define PVT_IP_CONFIG	0x04
> @@ -37,7 +38,6 @@
>  #define CLK_SYNTH_EN		BIT(24)
>  #define CLK_SYS_CYCLES_MAX	514
>  #define CLK_SYS_CYCLES_MIN	2
> -#define HZ_PER_MHZ		1000000L
>  
>  #define SDIF_DISABLE	0x04
>  

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 4/9] iio/drivers/as73211: Use HZ macros
  2021-02-24 14:42 ` [PATCH v2 4/9] iio/drivers/as73211: " Daniel Lezcano
@ 2021-03-14 16:59   ` Jonathan Cameron
  0 siblings, 0 replies; 37+ messages in thread
From: Jonathan Cameron @ 2021-03-14 16:59 UTC (permalink / raw)
  To: Daniel Lezcano
  Cc: rafael, andriy.shevchenko, linux-kernel, Christian Eggers,
	Lars-Peter Clausen, Peter Meerwald-Stadler,
	open list:AMS AS73211 DRIVER

On Wed, 24 Feb 2021 15:42:14 +0100
Daniel Lezcano <daniel.lezcano@linaro.org> wrote:

> HZ unit conversion macros are available in units.h, use them and
> remove the duplicate definition.
> 
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> Reviewed-by: Christian Eggers <ceggers@arri.de>
Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

> ---
>  drivers/iio/light/as73211.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/drivers/iio/light/as73211.c b/drivers/iio/light/as73211.c
> index 7b32dfaee9b3..3ba2378df3dd 100644
> --- a/drivers/iio/light/as73211.c
> +++ b/drivers/iio/light/as73211.c
> @@ -24,8 +24,7 @@
>  #include <linux/module.h>
>  #include <linux/mutex.h>
>  #include <linux/pm.h>
> -
> -#define HZ_PER_KHZ 1000
> +#include <linux/units.h>
>  
>  #define AS73211_DRV_NAME "as73211"
>  


^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 6/9] iio/drivers/hid-sensor: Use HZ macros
  2021-02-24 14:42 ` [PATCH v2 6/9] iio/drivers/hid-sensor: " Daniel Lezcano
@ 2021-03-14 17:00   ` Jonathan Cameron
  0 siblings, 0 replies; 37+ messages in thread
From: Jonathan Cameron @ 2021-03-14 17:00 UTC (permalink / raw)
  To: Daniel Lezcano
  Cc: rafael, andriy.shevchenko, linux-kernel, Lars-Peter Clausen,
	Peter Meerwald-Stadler, open list:IIO SUBSYSTEM AND DRIVERS

On Wed, 24 Feb 2021 15:42:16 +0100
Daniel Lezcano <daniel.lezcano@linaro.org> wrote:

> HZ unit conversion macros are available in units.h, use them and
> remove the duplicate definition.
> 
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

> ---
>  drivers/iio/common/hid-sensors/hid-sensor-attributes.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/drivers/iio/common/hid-sensors/hid-sensor-attributes.c b/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
> index 442ff787f7af..dda3b67e494f 100644
> --- a/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
> +++ b/drivers/iio/common/hid-sensors/hid-sensor-attributes.c
> @@ -11,13 +11,12 @@
>  #include <linux/kernel.h>
>  #include <linux/slab.h>
>  #include <linux/time.h>
> +#include <linux/units.h>
>  
>  #include <linux/hid-sensor-hub.h>
>  #include <linux/iio/iio.h>
>  #include <linux/iio/sysfs.h>
>  
> -#define HZ_PER_MHZ	1000000L
> -
>  static struct {
>  	u32 usage_id;
>  	int unit; /* 0 for default others from HID sensor spec */


^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 1/9] units: Add the HZ macros
  2021-02-25 11:22 ` Andy Shevchenko
@ 2021-03-19  8:08   ` Daniel Lezcano
  0 siblings, 0 replies; 37+ messages in thread
From: Daniel Lezcano @ 2021-03-19  8:08 UTC (permalink / raw)
  To: rafael
  Cc: Andy Shevchenko, linux-kernel, Christian Eggers,
	Rafael J. Wysocki, Lukasz Luba


Hi Rafael,

is it possible to merge this series through linux-pm ?


On 25/02/2021 12:22, Andy Shevchenko wrote:
> On Wed, Feb 24, 2021 at 03:42:11PM +0100, Daniel Lezcano wrote:
>> The macros for the unit conversion for frequency are duplicated in
>> different places.
>>
>> Provide these macros in the 'units' header, so they can be reused.
> 
> For the all that have not been tagged:
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> 
> Thanks!
> 
>> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
>> Reviewed-by: Christian Eggers <ceggers@arri.de>
>> Reviewed-by: Andy Shevchenko <Andy.Shevchenko@gmail.com>
>> ---
>>  include/linux/units.h | 4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/include/linux/units.h b/include/linux/units.h
>> index dcc30a53fa93..218ec0d314b6 100644
>> --- a/include/linux/units.h
>> +++ b/include/linux/units.h
>> @@ -4,6 +4,10 @@
>>  
>>  #include <linux/math.h>
>>  
>> +#define HZ_PER_KHZ		1000L
>> +#define KHZ_PER_MHZ		1000L
>> +#define HZ_PER_MHZ		1000000L
>> +
>>  #define MILLIWATT_PER_WATT	1000L
>>  #define MICROWATT_PER_MILLIWATT	1000L
>>  #define MICROWATT_PER_WATT	1000000L
>> -- 
>> 2.17.1
>>
> 


-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 1/9] units: Add the HZ macros
  2021-02-24 14:42 [PATCH v2 1/9] units: Add the HZ macros Daniel Lezcano
                   ` (9 preceding siblings ...)
  2021-02-25 11:22 ` Andy Shevchenko
@ 2021-05-10 12:59 ` Andy Shevchenko
  2021-07-12 16:34 ` Andy Shevchenko
  11 siblings, 0 replies; 37+ messages in thread
From: Andy Shevchenko @ 2021-05-10 12:59 UTC (permalink / raw)
  To: Daniel Lezcano
  Cc: rafael, linux-kernel, Christian Eggers, Rafael J. Wysocki, Lukasz Luba

On Wed, Feb 24, 2021 at 03:42:11PM +0100, Daniel Lezcano wrote:
> The macros for the unit conversion for frequency are duplicated in
> different places.
> 
> Provide these macros in the 'units' header, so they can be reused.

Hmm... I think you need to rebase, add cover letter and resend.

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 1/9] units: Add the HZ macros
  2021-02-24 14:42 [PATCH v2 1/9] units: Add the HZ macros Daniel Lezcano
                   ` (10 preceding siblings ...)
  2021-05-10 12:59 ` Andy Shevchenko
@ 2021-07-12 16:34 ` Andy Shevchenko
  2021-07-15 14:03   ` Daniel Lezcano
  11 siblings, 1 reply; 37+ messages in thread
From: Andy Shevchenko @ 2021-07-12 16:34 UTC (permalink / raw)
  To: Daniel Lezcano
  Cc: rafael, linux-kernel, Christian Eggers, Rafael J. Wysocki, Lukasz Luba

On Wed, Feb 24, 2021 at 03:42:11PM +0100, Daniel Lezcano wrote:
> The macros for the unit conversion for frequency are duplicated in
> different places.
> 
> Provide these macros in the 'units' header, so they can be reused.

Any progress on this? Are you planning to resubmit?

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 37+ messages in thread

* Re: [PATCH v2 1/9] units: Add the HZ macros
  2021-07-12 16:34 ` Andy Shevchenko
@ 2021-07-15 14:03   ` Daniel Lezcano
  0 siblings, 0 replies; 37+ messages in thread
From: Daniel Lezcano @ 2021-07-15 14:03 UTC (permalink / raw)
  To: Andy Shevchenko
  Cc: rafael, linux-kernel, Christian Eggers, Rafael J. Wysocki, Lukasz Luba

On 12/07/2021 18:34, Andy Shevchenko wrote:
> On Wed, Feb 24, 2021 at 03:42:11PM +0100, Daniel Lezcano wrote:
>> The macros for the unit conversion for frequency are duplicated in
>> different places.
>>
>> Provide these macros in the 'units' header, so they can be reused.
> 
> Any progress on this? Are you planning to resubmit?

Yes, I'll resubmit a new series.




-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

^ permalink raw reply	[flat|nested] 37+ messages in thread

end of thread, other threads:[~2021-07-15 14:03 UTC | newest]

Thread overview: 37+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-24 14:42 [PATCH v2 1/9] units: Add the HZ macros Daniel Lezcano
2021-02-24 14:42 ` [PATCH v2 2/9] thermal/drivers/devfreq_cooling: Use " Daniel Lezcano
2021-02-24 14:42 ` [PATCH v2 3/9] devfreq: " Daniel Lezcano
2021-02-26  6:08   ` Chanwoo Choi
2021-02-26  8:51     ` Daniel Lezcano
2021-02-26  9:14       ` Chanwoo Choi
2021-02-24 14:42 ` [PATCH v2 4/9] iio/drivers/as73211: " Daniel Lezcano
2021-03-14 16:59   ` Jonathan Cameron
2021-02-24 14:42 ` [PATCH v2 5/9] hwmon/drivers/mr75203: " Daniel Lezcano
2021-03-11 21:44   ` Guenter Roeck
2021-02-24 14:42 ` [PATCH v2 6/9] iio/drivers/hid-sensor: " Daniel Lezcano
2021-03-14 17:00   ` Jonathan Cameron
2021-02-24 14:42 ` [PATCH v2 7/9] i2c/drivers/ov02q10: " Daniel Lezcano
2021-02-24 14:42 ` [PATCH v2 8/9] mtd/drivers/nand: " Daniel Lezcano
2021-02-24 14:42   ` Daniel Lezcano
2021-03-02 16:31   ` Miquel Raynal
2021-03-02 16:31     ` Miquel Raynal
2021-03-02 17:03     ` Daniel Lezcano
2021-03-02 17:03       ` Daniel Lezcano
2021-03-02 17:12       ` Miquel Raynal
2021-03-02 17:12         ` Miquel Raynal
2021-02-24 14:42 ` [PATCH v2 9/9] phy/drivers/stm32: " Daniel Lezcano
2021-02-24 14:42   ` Daniel Lezcano
2021-03-02 16:32   ` Vinod Koul
2021-03-02 16:32     ` Vinod Koul
2021-03-02 17:03     ` Daniel Lezcano
2021-03-02 17:03       ` Daniel Lezcano
2021-03-03  5:40       ` Vinod Koul
2021-03-03  5:40         ` Vinod Koul
2021-03-03  6:09         ` Daniel Lezcano
2021-03-03  6:09           ` Daniel Lezcano
2021-02-24 14:52 ` [PATCH v2 1/9] units: Add the " Daniel Lezcano
2021-02-25 11:22 ` Andy Shevchenko
2021-03-19  8:08   ` Daniel Lezcano
2021-05-10 12:59 ` Andy Shevchenko
2021-07-12 16:34 ` Andy Shevchenko
2021-07-15 14:03   ` Daniel Lezcano

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.