Linux-RTC Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed
@ 2020-01-12 17:13 Geert Uytterhoeven
  2020-01-12 21:45 ` kbuild test robot
                   ` (3 more replies)
  0 siblings, 4 replies; 13+ messages in thread
From: Geert Uytterhoeven @ 2020-01-12 17:13 UTC (permalink / raw)
  To: Alessandro Zummo, Alexandre Belloni, Akinobu Mita, Joe Perches
  Cc: linux-rtc, linux-kernel, Geert Uytterhoeven

Merely enabling I2C and RTC selects REGMAP_I2C and REGMAP_SPI, even when
no driver needs it.  While the former can be moduler, the latter cannot,
and thus becomes built-in.

Fix this by moving the select statements for REGMAP_I2C and REGMAP_SPI
from the RTC_I2C_AND_SPI helper to the individual drivers that depend on
it.

Note that the comment for RTC_I2C_AND_SPI refers to SND_SOC_I2C_AND_SPI
for more information, but the latter does not select REGMAP_{I2C,SPI}
itself, and defers that to the individual drivers, too.

Fixes: 080481f54ef62121 ("rtc: merge ds3232 and ds3234")
Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
---
Joe: When merging addresses, scripts/get_maintainer.pl replaces
     Alexandre's authoritative email address from MAINTAINERS by the
     obsolete address in the SoB-line of the commit referred to by the
     Fixes-line.

 drivers/rtc/Kconfig | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig
index d77515d8382c7a72..738fa071884094d1 100644
--- a/drivers/rtc/Kconfig
+++ b/drivers/rtc/Kconfig
@@ -848,14 +848,14 @@ config RTC_I2C_AND_SPI
 	default m if I2C=m
 	default y if I2C=y
 	default y if SPI_MASTER=y
-	select REGMAP_I2C if I2C
-	select REGMAP_SPI if SPI_MASTER
 
 comment "SPI and I2C RTC drivers"
 
 config RTC_DRV_DS3232
 	tristate "Dallas/Maxim DS3232/DS3234"
 	depends on RTC_I2C_AND_SPI
+	select REGMAP_I2C if I2C
+	select REGMAP_SPI if SPI_MASTER
 	help
 	  If you say yes here you get support for Dallas Semiconductor
 	  DS3232 and DS3234 real-time clock chips. If an interrupt is associated
@@ -875,6 +875,8 @@ config RTC_DRV_DS3232_HWMON
 config RTC_DRV_PCF2127
 	tristate "NXP PCF2127"
 	depends on RTC_I2C_AND_SPI
+	select REGMAP_I2C if I2C
+	select REGMAP_SPI if SPI_MASTER
 	select WATCHDOG_CORE if WATCHDOG
 	help
 	  If you say yes here you get support for the NXP PCF2127/29 RTC
@@ -891,6 +893,8 @@ config RTC_DRV_PCF2127
 config RTC_DRV_RV3029C2
 	tristate "Micro Crystal RV3029/3049"
 	depends on RTC_I2C_AND_SPI
+	select REGMAP_I2C if I2C
+	select REGMAP_SPI if SPI_MASTER
 	help
 	  If you say yes here you get support for the Micro Crystal
 	  RV3029 and RV3049 RTC chips.
-- 
2.17.1


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

* Re: [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed
  2020-01-12 17:13 [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed Geert Uytterhoeven
@ 2020-01-12 21:45 ` kbuild test robot
  2020-01-13  1:29 ` kbuild test robot
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 13+ messages in thread
From: kbuild test robot @ 2020-01-12 21:45 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: kbuild-all, Alessandro Zummo, Alexandre Belloni, Akinobu Mita,
	Joe Perches, linux-rtc, linux-kernel, Geert Uytterhoeven


[-- Attachment #1: Type: text/plain, Size: 9269 bytes --]

Hi Geert,

I love your patch! Yet something to improve:

[auto build test ERROR on abelloni/rtc-next]
[also build test ERROR on v5.5-rc5 next-20200110]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:    https://github.com/0day-ci/linux/commits/Geert-Uytterhoeven/rtc-i2c-spi-Avoid-inclusion-of-REGMAP-support-when-not-needed/20200113-013332
base:   https://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git rtc-next
config: powerpc-defconfig (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 7.5.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        GCC_VERSION=7.5.0 make.cross ARCH=powerpc 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All error/warnings (new ones prefixed by >>):

>> drivers/rtc/rtc-ds1307.c:1570:21: error: variable 'regmap_config' has initializer but incomplete type
    static const struct regmap_config regmap_config = {
                        ^~~~~~~~~~~~~
>> drivers/rtc/rtc-ds1307.c:1571:3: error: 'const struct regmap_config' has no member named 'reg_bits'
     .reg_bits = 8,
      ^~~~~~~~
>> drivers/rtc/rtc-ds1307.c:1571:14: warning: excess elements in struct initializer
     .reg_bits = 8,
                 ^
   drivers/rtc/rtc-ds1307.c:1571:14: note: (near initialization for 'regmap_config')
>> drivers/rtc/rtc-ds1307.c:1572:3: error: 'const struct regmap_config' has no member named 'val_bits'
     .val_bits = 8,
      ^~~~~~~~
   drivers/rtc/rtc-ds1307.c:1572:14: warning: excess elements in struct initializer
     .val_bits = 8,
                 ^
   drivers/rtc/rtc-ds1307.c:1572:14: note: (near initialization for 'regmap_config')
   drivers/rtc/rtc-ds1307.c: In function 'ds1307_probe':
>> drivers/rtc/rtc-ds1307.c:1596:19: error: implicit declaration of function 'devm_regmap_init_i2c'; did you mean 'devm_request_irq'? [-Werror=implicit-function-declaration]
     ds1307->regmap = devm_regmap_init_i2c(client, &regmap_config);
                      ^~~~~~~~~~~~~~~~~~~~
                      devm_request_irq
>> drivers/rtc/rtc-ds1307.c:1596:17: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
     ds1307->regmap = devm_regmap_init_i2c(client, &regmap_config);
                    ^
   drivers/rtc/rtc-ds1307.c: At top level:
>> drivers/rtc/rtc-ds1307.c:1570:35: error: storage size of 'regmap_config' isn't known
    static const struct regmap_config regmap_config = {
                                      ^~~~~~~~~~~~~
   cc1: some warnings being treated as errors

vim +/regmap_config +1570 drivers/rtc/rtc-ds1307.c

445c02076f1e60 Akinobu Mita             2016-01-25  1569  
11e5890b5342c8 Heiner Kallweit          2017-03-10 @1570  static const struct regmap_config regmap_config = {
11e5890b5342c8 Heiner Kallweit          2017-03-10 @1571  	.reg_bits = 8,
11e5890b5342c8 Heiner Kallweit          2017-03-10 @1572  	.val_bits = 8,
11e5890b5342c8 Heiner Kallweit          2017-03-10  1573  };
11e5890b5342c8 Heiner Kallweit          2017-03-10  1574  
5a167f4543e45d Greg Kroah-Hartman       2012-12-21  1575  static int ds1307_probe(struct i2c_client *client,
d2653e92732bd3 Jean Delvare             2008-04-29  1576  			const struct i2c_device_id *id)
1abb0dc92d706e David Brownell           2006-06-25  1577  {
1abb0dc92d706e David Brownell           2006-06-25  1578  	struct ds1307		*ds1307;
1abb0dc92d706e David Brownell           2006-06-25  1579  	int			err = -ENODEV;
584ce30c72954e Heiner Kallweit          2017-08-29  1580  	int			tmp;
7624df482d7aba Heiner Kallweit          2017-07-12  1581  	const struct chip_desc	*chip;
82e2d43f6315d7 Heiner Kallweit          2017-07-12  1582  	bool			want_irq;
8bc2a40730ec74 Michael Lange            2016-01-21  1583  	bool			ds1307_can_wakeup_device = false;
042fa8c7c04dd4 Alexandre Belloni        2017-09-04  1584  	unsigned char		regs[8];
01ce893d37f053 Jingoo Han               2013-11-12  1585  	struct ds1307_platform_data *pdata = dev_get_platdata(&client->dev);
d8490fd55ab90e Heiner Kallweit          2017-07-12  1586  	u8			trickle_charger_setup = 0;
1abb0dc92d706e David Brownell           2006-06-25  1587  
edca66d2ceae6b Jingoo Han               2013-07-03  1588  	ds1307 = devm_kzalloc(&client->dev, sizeof(struct ds1307), GFP_KERNEL);
40ce972d59fcfd David Anders             2012-03-23  1589  	if (!ds1307)
c065f35c179290 David Brownell           2007-07-17  1590  		return -ENOMEM;
045e0e85f2f6ee David Brownell           2007-07-17  1591  
11e5890b5342c8 Heiner Kallweit          2017-03-10  1592  	dev_set_drvdata(&client->dev, ds1307);
11e5890b5342c8 Heiner Kallweit          2017-03-10  1593  	ds1307->dev = &client->dev;
11e5890b5342c8 Heiner Kallweit          2017-03-10  1594  	ds1307->name = client->name;
33df2ee1bb59b8 Joakim Tjernlund         2009-06-17  1595  
11e5890b5342c8 Heiner Kallweit          2017-03-10 @1596  	ds1307->regmap = devm_regmap_init_i2c(client, &regmap_config);
11e5890b5342c8 Heiner Kallweit          2017-03-10  1597  	if (IS_ERR(ds1307->regmap)) {
11e5890b5342c8 Heiner Kallweit          2017-03-10  1598  		dev_err(ds1307->dev, "regmap allocation failed\n");
11e5890b5342c8 Heiner Kallweit          2017-03-10  1599  		return PTR_ERR(ds1307->regmap);
11e5890b5342c8 Heiner Kallweit          2017-03-10  1600  	}
11e5890b5342c8 Heiner Kallweit          2017-03-10  1601  
11e5890b5342c8 Heiner Kallweit          2017-03-10  1602  	i2c_set_clientdata(client, ds1307);
7ef6d2c266e5b0 Javier Martinez Canillas 2017-03-03  1603  
7ef6d2c266e5b0 Javier Martinez Canillas 2017-03-03  1604  	if (client->dev.of_node) {
7ef6d2c266e5b0 Javier Martinez Canillas 2017-03-03  1605  		ds1307->type = (enum ds_type)
7ef6d2c266e5b0 Javier Martinez Canillas 2017-03-03  1606  			of_device_get_match_data(&client->dev);
7ef6d2c266e5b0 Javier Martinez Canillas 2017-03-03  1607  		chip = &chips[ds1307->type];
7ef6d2c266e5b0 Javier Martinez Canillas 2017-03-03  1608  	} else if (id) {
9c19b8930d2cf9 Tin Huynh                2016-11-30  1609  		chip = &chips[id->driver_data];
3760f736716f74 Jean Delvare             2008-04-29  1610  		ds1307->type = id->driver_data;
9c19b8930d2cf9 Tin Huynh                2016-11-30  1611  	} else {
9c19b8930d2cf9 Tin Huynh                2016-11-30  1612  		const struct acpi_device_id *acpi_id;
9c19b8930d2cf9 Tin Huynh                2016-11-30  1613  
9c19b8930d2cf9 Tin Huynh                2016-11-30  1614  		acpi_id = acpi_match_device(ACPI_PTR(ds1307_acpi_ids),
11e5890b5342c8 Heiner Kallweit          2017-03-10  1615  					    ds1307->dev);
9c19b8930d2cf9 Tin Huynh                2016-11-30  1616  		if (!acpi_id)
9c19b8930d2cf9 Tin Huynh                2016-11-30  1617  			return -ENODEV;
9c19b8930d2cf9 Tin Huynh                2016-11-30  1618  		chip = &chips[acpi_id->driver_data];
9c19b8930d2cf9 Tin Huynh                2016-11-30  1619  		ds1307->type = acpi_id->driver_data;
9c19b8930d2cf9 Tin Huynh                2016-11-30  1620  	}
33df2ee1bb59b8 Joakim Tjernlund         2009-06-17  1621  
82e2d43f6315d7 Heiner Kallweit          2017-07-12  1622  	want_irq = client->irq > 0 && chip->alarm;
82e2d43f6315d7 Heiner Kallweit          2017-07-12  1623  
9c19b8930d2cf9 Tin Huynh                2016-11-30  1624  	if (!pdata)
d8490fd55ab90e Heiner Kallweit          2017-07-12  1625  		trickle_charger_setup = ds1307_trickle_init(ds1307, chip);
9c19b8930d2cf9 Tin Huynh                2016-11-30  1626  	else if (pdata->trickle_charger_setup)
d8490fd55ab90e Heiner Kallweit          2017-07-12  1627  		trickle_charger_setup = pdata->trickle_charger_setup;
33b04b7b7c03d0 Matti Vaittinen          2014-10-13  1628  
d8490fd55ab90e Heiner Kallweit          2017-07-12  1629  	if (trickle_charger_setup && chip->trickle_charger_reg) {
d8490fd55ab90e Heiner Kallweit          2017-07-12  1630  		trickle_charger_setup |= DS13XX_TRICKLE_CHARGER_MAGIC;
11e5890b5342c8 Heiner Kallweit          2017-03-10  1631  		dev_dbg(ds1307->dev,
11e5890b5342c8 Heiner Kallweit          2017-03-10  1632  			"writing trickle charger info 0x%x to 0x%x\n",
d8490fd55ab90e Heiner Kallweit          2017-07-12  1633  			trickle_charger_setup, chip->trickle_charger_reg);
11e5890b5342c8 Heiner Kallweit          2017-03-10  1634  		regmap_write(ds1307->regmap, chip->trickle_charger_reg,
d8490fd55ab90e Heiner Kallweit          2017-07-12  1635  			     trickle_charger_setup);
33b04b7b7c03d0 Matti Vaittinen          2014-10-13  1636  	}
eb86c3064b3c53 Wolfram Sang             2012-05-29  1637  

:::::: The code at line 1570 was first introduced by commit
:::::: 11e5890b5342c82eefbaa39aec4767ae21ae8803 rtc: ds1307: convert driver to regmap

:::::: TO: Heiner Kallweit <hkallweit1@gmail.com>
:::::: CC: Alexandre Belloni <alexandre.belloni@free-electrons.com>

---
0-DAY kernel test infrastructure                 Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 25647 bytes --]

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

* Re: [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed
  2020-01-12 17:13 [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed Geert Uytterhoeven
  2020-01-12 21:45 ` kbuild test robot
@ 2020-01-13  1:29 ` kbuild test robot
  2020-01-13  6:36 ` Joe Perches
  2020-01-27 22:45 ` Alexandre Belloni
  3 siblings, 0 replies; 13+ messages in thread
From: kbuild test robot @ 2020-01-13  1:29 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: kbuild-all, Alessandro Zummo, Alexandre Belloni, Akinobu Mita,
	Joe Perches, linux-rtc, linux-kernel, Geert Uytterhoeven


[-- Attachment #1: Type: text/plain, Size: 1406 bytes --]

Hi Geert,

I love your patch! Yet something to improve:

[auto build test ERROR on abelloni/rtc-next]
[also build test ERROR on v5.5-rc5 next-20200110]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:    https://github.com/0day-ci/linux/commits/Geert-Uytterhoeven/rtc-i2c-spi-Avoid-inclusion-of-REGMAP-support-when-not-needed/20200113-013332
base:   https://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git rtc-next
config: arm-mv78xx0_defconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 7.5.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        GCC_VERSION=7.5.0 make.cross ARCH=arm 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   drivers/rtc/rtc-ds1307.o: In function `ds1307_probe':
>> rtc-ds1307.c:(.text+0x14f4): undefined reference to `__devm_regmap_init_i2c'

---
0-DAY kernel test infrastructure                 Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 20000 bytes --]

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

* Re: [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed
  2020-01-12 17:13 [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed Geert Uytterhoeven
  2020-01-12 21:45 ` kbuild test robot
  2020-01-13  1:29 ` kbuild test robot
@ 2020-01-13  6:36 ` Joe Perches
  2020-01-13  7:25   ` Geert Uytterhoeven
  2020-01-27 22:45 ` Alexandre Belloni
  3 siblings, 1 reply; 13+ messages in thread
From: Joe Perches @ 2020-01-13  6:36 UTC (permalink / raw)
  To: Geert Uytterhoeven, Alessandro Zummo, Alexandre Belloni, Akinobu Mita
  Cc: linux-rtc, linux-kernel

On Sun, 2020-01-12 at 18:13 +0100, Geert Uytterhoeven wrote:
> Merely enabling I2C and RTC selects REGMAP_I2C and REGMAP_SPI, even when
> no driver needs it.  While the former can be moduler, the latter cannot,
> and thus becomes built-in.
> 
> Fix this by moving the select statements for REGMAP_I2C and REGMAP_SPI
> from the RTC_I2C_AND_SPI helper to the individual drivers that depend on
> it.
> 
> Note that the comment for RTC_I2C_AND_SPI refers to SND_SOC_I2C_AND_SPI
> for more information, but the latter does not select REGMAP_{I2C,SPI}
> itself, and defers that to the individual drivers, too.
> 
> Fixes: 080481f54ef62121 ("rtc: merge ds3232 and ds3234")
> Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
> ---
> Joe: When merging addresses, scripts/get_maintainer.pl replaces
>      Alexandre's authoritative email address from MAINTAINERS by the
>      obsolete address in the SoB-line of the commit referred to by the
>      Fixes-line.

Hi Geert

What are you doing to get this changed output?

I get the same get_maintainer address output either way
with only with the 'blamed_fixes:' content added.

Your email with 'Fixes:' line:

$ ./scripts/get_maintainer.pl ~/geert_1.mbox 
Alessandro Zummo <a.zummo@towertech.it> (maintainer:REAL TIME CLOCK (RTC) SUBSYSTEM)
Alexandre Belloni <alexandre.belloni@bootlin.com> (maintainer:REAL TIME CLOCK (RTC) SUBSYSTEM,blamed_fixes:1/1=100%)
Akinobu Mita <akinobu.mita@gmail.com> (blamed_fixes:1/1=100%)
linux-rtc@vger.kernel.org (open list:REAL TIME CLOCK (RTC) SUBSYSTEM)
linux-kernel@vger.kernel.org (open list)

Your email without 'Fixes:' line:

$ ./scripts/get_maintainer.pl ~/geert_2.mbox 
Alessandro Zummo <a.zummo@towertech.it> (maintainer:REAL TIME CLOCK (RTC) SUBSYSTEM)
Alexandre Belloni <alexandre.belloni@bootlin.com> (maintainer:REAL TIME CLOCK (RTC) SUBSYSTEM)
linux-rtc@vger.kernel.org (open list:REAL TIME CLOCK (RTC) SUBSYSTEM)
linux-kernel@vger.kernel.org (open list)




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

* Re: [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed
  2020-01-13  6:36 ` Joe Perches
@ 2020-01-13  7:25   ` Geert Uytterhoeven
  2020-01-13  7:40     ` Joe Perches
  0 siblings, 1 reply; 13+ messages in thread
From: Geert Uytterhoeven @ 2020-01-13  7:25 UTC (permalink / raw)
  To: Joe Perches
  Cc: Alessandro Zummo, Alexandre Belloni, Akinobu Mita, linux-rtc,
	Linux Kernel Mailing List

Hi Joe,

On Mon, Jan 13, 2020 at 7:36 AM Joe Perches <joe@perches.com> wrote:
> On Sun, 2020-01-12 at 18:13 +0100, Geert Uytterhoeven wrote:
> > Merely enabling I2C and RTC selects REGMAP_I2C and REGMAP_SPI, even when
> > no driver needs it.  While the former can be moduler, the latter cannot,
> > and thus becomes built-in.
> >
> > Fix this by moving the select statements for REGMAP_I2C and REGMAP_SPI
> > from the RTC_I2C_AND_SPI helper to the individual drivers that depend on
> > it.
> >
> > Note that the comment for RTC_I2C_AND_SPI refers to SND_SOC_I2C_AND_SPI
> > for more information, but the latter does not select REGMAP_{I2C,SPI}
> > itself, and defers that to the individual drivers, too.
> >
> > Fixes: 080481f54ef62121 ("rtc: merge ds3232 and ds3234")
> > Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
> > ---
> > Joe: When merging addresses, scripts/get_maintainer.pl replaces
> >      Alexandre's authoritative email address from MAINTAINERS by the
> >      obsolete address in the SoB-line of the commit referred to by the
> >      Fixes-line.
>
> Hi Geert
>
> What are you doing to get this changed output?

scripts/get_maintainer.pl
0001-rtc-i2c-spi-Avoid-inclusion-of-REGMAP-support-when-n.patch

> I get the same get_maintainer address output either way
> with only with the 'blamed_fixes:' content added.

Thanks, I can confirm it's fixed in next-20200110.
With v5.5-rc6, it still gives the old addresss.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

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

* Re: [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed
  2020-01-13  7:25   ` Geert Uytterhoeven
@ 2020-01-13  7:40     ` Joe Perches
  2020-01-13  7:57       ` Geert Uytterhoeven
  0 siblings, 1 reply; 13+ messages in thread
From: Joe Perches @ 2020-01-13  7:40 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Alessandro Zummo, Alexandre Belloni, Akinobu Mita, linux-rtc,
	Linux Kernel Mailing List

On Mon, 2020-01-13 at 08:25 +0100, Geert Uytterhoeven wrote:
> Hi Joe,
> 
> On Mon, Jan 13, 2020 at 7:36 AM Joe Perches <joe@perches.com> wrote:
> > On Sun, 2020-01-12 at 18:13 +0100, Geert Uytterhoeven wrote:
> > > Merely enabling I2C and RTC selects REGMAP_I2C and REGMAP_SPI, even when
> > > no driver needs it.  While the former can be moduler, the latter cannot,
> > > and thus becomes built-in.
> > > 
> > > Fix this by moving the select statements for REGMAP_I2C and REGMAP_SPI
> > > from the RTC_I2C_AND_SPI helper to the individual drivers that depend on
> > > it.
> > > 
> > > Note that the comment for RTC_I2C_AND_SPI refers to SND_SOC_I2C_AND_SPI
> > > for more information, but the latter does not select REGMAP_{I2C,SPI}
> > > itself, and defers that to the individual drivers, too.
> > > 
> > > Fixes: 080481f54ef62121 ("rtc: merge ds3232 and ds3234")
> > > Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
> > > ---
> > > Joe: When merging addresses, scripts/get_maintainer.pl replaces
> > >      Alexandre's authoritative email address from MAINTAINERS by the
> > >      obsolete address in the SoB-line of the commit referred to by the
> > >      Fixes-line.
> > 
> > Hi Geert
> > 
> > What are you doing to get this changed output?
> 
> scripts/get_maintainer.pl
> 0001-rtc-i2c-spi-Avoid-inclusion-of-REGMAP-support-when-n.patch
> 
> > I get the same get_maintainer address output either way
> > with only with the 'blamed_fixes:' content added.
> 
> Thanks, I can confirm it's fixed in next-20200110.
> With v5.5-rc6, it still gives the old addresss.

Well, OK, get_maintainer is the same but there is a
different .mailmap in -next

$ git log --stat -p -1 94a250713
commit 94a25071301a898d8c603db2f05a0016eb7b7d28
Author: Alexandre Belloni <alexandre.belloni@bootlin.com>
Date:   Tue Dec 10 14:46:53 2019 +0100

    mailmap: Update email address for Alexandre Belloni
    
    Free Electrons is now Bootlin.
    
    Link: https://lore.kernel.org/r/20191210134653.2995661-1-alexandre.belloni@bootlin.com
    Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>



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

* Re: [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed
  2020-01-13  7:40     ` Joe Perches
@ 2020-01-13  7:57       ` Geert Uytterhoeven
  2020-01-13  8:12         ` Joe Perches
  0 siblings, 1 reply; 13+ messages in thread
From: Geert Uytterhoeven @ 2020-01-13  7:57 UTC (permalink / raw)
  To: Joe Perches
  Cc: Alessandro Zummo, Alexandre Belloni, Akinobu Mita, linux-rtc,
	Linux Kernel Mailing List

Hi Joe,

On Mon, Jan 13, 2020 at 8:41 AM Joe Perches <joe@perches.com> wrote:
> On Mon, 2020-01-13 at 08:25 +0100, Geert Uytterhoeven wrote:
> > On Mon, Jan 13, 2020 at 7:36 AM Joe Perches <joe@perches.com> wrote:
> > > On Sun, 2020-01-12 at 18:13 +0100, Geert Uytterhoeven wrote:
> > > > Merely enabling I2C and RTC selects REGMAP_I2C and REGMAP_SPI, even when
> > > > no driver needs it.  While the former can be moduler, the latter cannot,
> > > > and thus becomes built-in.
> > > >
> > > > Fix this by moving the select statements for REGMAP_I2C and REGMAP_SPI
> > > > from the RTC_I2C_AND_SPI helper to the individual drivers that depend on
> > > > it.
> > > >
> > > > Note that the comment for RTC_I2C_AND_SPI refers to SND_SOC_I2C_AND_SPI
> > > > for more information, but the latter does not select REGMAP_{I2C,SPI}
> > > > itself, and defers that to the individual drivers, too.
> > > >
> > > > Fixes: 080481f54ef62121 ("rtc: merge ds3232 and ds3234")
> > > > Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
> > > > ---
> > > > Joe: When merging addresses, scripts/get_maintainer.pl replaces
> > > >      Alexandre's authoritative email address from MAINTAINERS by the
> > > >      obsolete address in the SoB-line of the commit referred to by the
> > > >      Fixes-line.
> > >
> > > Hi Geert
> > >
> > > What are you doing to get this changed output?
> >
> > scripts/get_maintainer.pl
> > 0001-rtc-i2c-spi-Avoid-inclusion-of-REGMAP-support-when-n.patch
> >
> > > I get the same get_maintainer address output either way
> > > with only with the 'blamed_fixes:' content added.
> >
> > Thanks, I can confirm it's fixed in next-20200110.
> > With v5.5-rc6, it still gives the old addresss.
>
> Well, OK, get_maintainer is the same but there is a
> different .mailmap in -next

Indeed.

However, I think the precedence should be

    MAINTAINERS > .mailmap > SoB in referenced commit

instead of

    .mailmap > SoB in referenced commit > MAINTAINERS

Do you agree?
Thanks!

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

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

* Re: [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed
  2020-01-13  7:57       ` Geert Uytterhoeven
@ 2020-01-13  8:12         ` Joe Perches
  2020-01-14 10:13           ` Geert Uytterhoeven
  0 siblings, 1 reply; 13+ messages in thread
From: Joe Perches @ 2020-01-13  8:12 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Alessandro Zummo, Alexandre Belloni, Akinobu Mita, linux-rtc,
	Linux Kernel Mailing List

On Mon, 2020-01-13 at 08:57 +0100, Geert Uytterhoeven wrote:
> On Mon, Jan 13, 2020 at 8:41 AM Joe Perches <joe@perches.com> wrote:
> > On Mon, 2020-01-13 at 08:25 +0100, Geert Uytterhoeven wrote:
> > > On Mon, Jan 13, 2020 at 7:36 AM Joe Perches <joe@perches.com> wrote:
> > > > On Sun, 2020-01-12 at 18:13 +0100, Geert Uytterhoeven wrote:
> > > > > Merely enabling I2C and RTC selects REGMAP_I2C and REGMAP_SPI, even when
> > > > > no driver needs it.  While the former can be moduler, the latter cannot,
> > > > > and thus becomes built-in.
> > > > > 
> > > > > Fix this by moving the select statements for REGMAP_I2C and REGMAP_SPI
> > > > > from the RTC_I2C_AND_SPI helper to the individual drivers that depend on
> > > > > it.
> > > > > 
> > > > > Note that the comment for RTC_I2C_AND_SPI refers to SND_SOC_I2C_AND_SPI
> > > > > for more information, but the latter does not select REGMAP_{I2C,SPI}
> > > > > itself, and defers that to the individual drivers, too.
> > > > > 
> > > > > Fixes: 080481f54ef62121 ("rtc: merge ds3232 and ds3234")
> > > > > Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
> > > > > ---
> > > > > Joe: When merging addresses, scripts/get_maintainer.pl replaces
> > > > >      Alexandre's authoritative email address from MAINTAINERS by the
> > > > >      obsolete address in the SoB-line of the commit referred to by the
> > > > >      Fixes-line.
> > > > What are you doing to get this changed output?
> > > 
> > > scripts/get_maintainer.pl
> > > 0001-rtc-i2c-spi-Avoid-inclusion-of-REGMAP-support-when-n.patch
> > > 
> > > > I get the same get_maintainer address output either way
> > > > with only with the 'blamed_fixes:' content added.
> > > 
> > > Thanks, I can confirm it's fixed in next-20200110.
> > > With v5.5-rc6, it still gives the old addresss.
> > 
> > Well, OK, get_maintainer is the same but there is a
> > different .mailmap in -next
> 
> Indeed.
> 
> However, I think the precedence should be
> 
>     MAINTAINERS > .mailmap > SoB in referenced commit
> 
> instead of
> 
>     .mailmap > SoB in referenced commit > MAINTAINERS
> 
> Do you agree?

Well, not really.  Priority is:

	1 .mailmap address
	2 SoB address in commit message
	2 SoB
address in referenced Fixes: commits
	3 MAINTAINERS address

MAINTAINER entries are sometimes stale and .mailmap
is generally more current so perhaps it should be

	1 .mailmap address
	2 SoB address in commit message
	3 MAINTAINERS address
	4 SoB address in referenced Fixes: commits

But it seems a restructuring of get_maintainer
would be required to do that as the Fixes: line is
parsed before any filenames in a patch and I'm not
too bothered by the precedence and output right
now to muck around in get_maintainer's internals.

cheers, Joe


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

* Re: [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed
  2020-01-13  8:12         ` Joe Perches
@ 2020-01-14 10:13           ` Geert Uytterhoeven
  0 siblings, 0 replies; 13+ messages in thread
From: Geert Uytterhoeven @ 2020-01-14 10:13 UTC (permalink / raw)
  To: Joe Perches
  Cc: Alessandro Zummo, Alexandre Belloni, Akinobu Mita, linux-rtc,
	Linux Kernel Mailing List

Hi Joe,

On Mon, Jan 13, 2020 at 9:13 AM Joe Perches <joe@perches.com> wrote:
> On Mon, 2020-01-13 at 08:57 +0100, Geert Uytterhoeven wrote:
> > On Mon, Jan 13, 2020 at 8:41 AM Joe Perches <joe@perches.com> wrote:
> > > On Mon, 2020-01-13 at 08:25 +0100, Geert Uytterhoeven wrote:
> > > > On Mon, Jan 13, 2020 at 7:36 AM Joe Perches <joe@perches.com> wrote:
> > > > > On Sun, 2020-01-12 at 18:13 +0100, Geert Uytterhoeven wrote:
> > > > > > Merely enabling I2C and RTC selects REGMAP_I2C and REGMAP_SPI, even when
> > > > > > no driver needs it.  While the former can be moduler, the latter cannot,
> > > > > > and thus becomes built-in.
> > > > > >
> > > > > > Fix this by moving the select statements for REGMAP_I2C and REGMAP_SPI
> > > > > > from the RTC_I2C_AND_SPI helper to the individual drivers that depend on
> > > > > > it.
> > > > > >
> > > > > > Note that the comment for RTC_I2C_AND_SPI refers to SND_SOC_I2C_AND_SPI
> > > > > > for more information, but the latter does not select REGMAP_{I2C,SPI}
> > > > > > itself, and defers that to the individual drivers, too.
> > > > > >
> > > > > > Fixes: 080481f54ef62121 ("rtc: merge ds3232 and ds3234")
> > > > > > Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
> > > > > > ---
> > > > > > Joe: When merging addresses, scripts/get_maintainer.pl replaces
> > > > > >      Alexandre's authoritative email address from MAINTAINERS by the
> > > > > >      obsolete address in the SoB-line of the commit referred to by the
> > > > > >      Fixes-line.
> > > > > What are you doing to get this changed output?
> > > >
> > > > scripts/get_maintainer.pl
> > > > 0001-rtc-i2c-spi-Avoid-inclusion-of-REGMAP-support-when-n.patch
> > > >
> > > > > I get the same get_maintainer address output either way
> > > > > with only with the 'blamed_fixes:' content added.
> > > >
> > > > Thanks, I can confirm it's fixed in next-20200110.
> > > > With v5.5-rc6, it still gives the old addresss.
> > >
> > > Well, OK, get_maintainer is the same but there is a
> > > different .mailmap in -next
> >
> > Indeed.
> >
> > However, I think the precedence should be
> >
> >     MAINTAINERS > .mailmap > SoB in referenced commit
> >
> > instead of
> >
> >     .mailmap > SoB in referenced commit > MAINTAINERS
> >
> > Do you agree?
>
> Well, not really.  Priority is:
>
>         1 .mailmap address
>         2 SoB address in commit message
>         2 SoB
> address in referenced Fixes: commits
>         3 MAINTAINERS address
>
> MAINTAINER entries are sometimes stale and .mailmap
> is generally more current so perhaps it should be
>
>         1 .mailmap address
>         2 SoB address in commit message
>         3 MAINTAINERS address
>         4 SoB address in referenced Fixes: commits
>
> But it seems a restructuring of get_maintainer
> would be required to do that as the Fixes: line is
> parsed before any filenames in a patch and I'm not
> too bothered by the precedence and output right
> now to muck around in get_maintainer's internals.

Exactly. There's a reason I just reported the issue, instead of sending a
patch ;-)

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

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

* Re: [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed
  2020-01-12 17:13 [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed Geert Uytterhoeven
                   ` (2 preceding siblings ...)
  2020-01-13  6:36 ` Joe Perches
@ 2020-01-27 22:45 ` Alexandre Belloni
  2020-01-28  8:33   ` Geert Uytterhoeven
  3 siblings, 1 reply; 13+ messages in thread
From: Alexandre Belloni @ 2020-01-27 22:45 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Alessandro Zummo, Akinobu Mita, Joe Perches, linux-rtc, linux-kernel

On 12/01/2020 18:13:49+0100, Geert Uytterhoeven wrote:
> Merely enabling I2C and RTC selects REGMAP_I2C and REGMAP_SPI, even when
> no driver needs it.  While the former can be moduler, the latter cannot,
> and thus becomes built-in.
> 
> Fix this by moving the select statements for REGMAP_I2C and REGMAP_SPI
> from the RTC_I2C_AND_SPI helper to the individual drivers that depend on
> it.
> 
> Note that the comment for RTC_I2C_AND_SPI refers to SND_SOC_I2C_AND_SPI
> for more information, but the latter does not select REGMAP_{I2C,SPI}
> itself, and defers that to the individual drivers, too.
> 
> Fixes: 080481f54ef62121 ("rtc: merge ds3232 and ds3234")
> Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
> ---
> Joe: When merging addresses, scripts/get_maintainer.pl replaces
>      Alexandre's authoritative email address from MAINTAINERS by the
>      obsolete address in the SoB-line of the commit referred to by the
>      Fixes-line.
> 
>  drivers/rtc/Kconfig | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
> 
Applied, thanks.

-- 
Alexandre Belloni, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

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

* Re: [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed
  2020-01-27 22:45 ` Alexandre Belloni
@ 2020-01-28  8:33   ` Geert Uytterhoeven
  2020-01-28  8:50     ` Alexandre Belloni
  0 siblings, 1 reply; 13+ messages in thread
From: Geert Uytterhoeven @ 2020-01-28  8:33 UTC (permalink / raw)
  To: Alexandre Belloni
  Cc: Alessandro Zummo, Akinobu Mita, Joe Perches, linux-rtc,
	Linux Kernel Mailing List

Hi Alexandre,

On Mon, Jan 27, 2020 at 11:45 PM Alexandre Belloni
<alexandre.belloni@bootlin.com> wrote:
> On 12/01/2020 18:13:49+0100, Geert Uytterhoeven wrote:
> > Merely enabling I2C and RTC selects REGMAP_I2C and REGMAP_SPI, even when
> > no driver needs it.  While the former can be moduler, the latter cannot,
> > and thus becomes built-in.
> >
> > Fix this by moving the select statements for REGMAP_I2C and REGMAP_SPI
> > from the RTC_I2C_AND_SPI helper to the individual drivers that depend on
> > it.
> >
> > Note that the comment for RTC_I2C_AND_SPI refers to SND_SOC_I2C_AND_SPI
> > for more information, but the latter does not select REGMAP_{I2C,SPI}
> > itself, and defers that to the individual drivers, too.
> >
> > Fixes: 080481f54ef62121 ("rtc: merge ds3232 and ds3234")
> > Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
> > ---
> > Joe: When merging addresses, scripts/get_maintainer.pl replaces
> >      Alexandre's authoritative email address from MAINTAINERS by the
> >      obsolete address in the SoB-line of the commit referred to by the
> >      Fixes-line.
> >
> >  drivers/rtc/Kconfig | 8 ++++++--
> >  1 file changed, 6 insertions(+), 2 deletions(-)
> >
> Applied, thanks.

According to the reports from kbuild test report, some drivers are still
missing some selects, which is exposed by this patch.
But perhaps you have already fixed those in your tree?

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

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

* Re: [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed
  2020-01-28  8:33   ` Geert Uytterhoeven
@ 2020-01-28  8:50     ` Alexandre Belloni
  2020-01-28  8:59       ` Geert Uytterhoeven
  0 siblings, 1 reply; 13+ messages in thread
From: Alexandre Belloni @ 2020-01-28  8:50 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Alessandro Zummo, Akinobu Mita, Joe Perches, linux-rtc,
	Linux Kernel Mailing List

Hi,

On 28/01/2020 09:33:28+0100, Geert Uytterhoeven wrote:
> Hi Alexandre,
> 
> On Mon, Jan 27, 2020 at 11:45 PM Alexandre Belloni
> <alexandre.belloni@bootlin.com> wrote:
> > On 12/01/2020 18:13:49+0100, Geert Uytterhoeven wrote:
> > > Merely enabling I2C and RTC selects REGMAP_I2C and REGMAP_SPI, even when
> > > no driver needs it.  While the former can be moduler, the latter cannot,
> > > and thus becomes built-in.
> > >
> > > Fix this by moving the select statements for REGMAP_I2C and REGMAP_SPI
> > > from the RTC_I2C_AND_SPI helper to the individual drivers that depend on
> > > it.
> > >
> > > Note that the comment for RTC_I2C_AND_SPI refers to SND_SOC_I2C_AND_SPI
> > > for more information, but the latter does not select REGMAP_{I2C,SPI}
> > > itself, and defers that to the individual drivers, too.
> > >
> > > Fixes: 080481f54ef62121 ("rtc: merge ds3232 and ds3234")
> > > Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
> > > ---
> > > Joe: When merging addresses, scripts/get_maintainer.pl replaces
> > >      Alexandre's authoritative email address from MAINTAINERS by the
> > >      obsolete address in the SoB-line of the commit referred to by the
> > >      Fixes-line.
> > >
> > >  drivers/rtc/Kconfig | 8 ++++++--
> > >  1 file changed, 6 insertions(+), 2 deletions(-)
> > >
> > Applied, thanks.
> 
> According to the reports from kbuild test report, some drivers are still
> missing some selects, which is exposed by this patch.
> But perhaps you have already fixed those in your tree?
> 

I did fix a bunch of those in
https://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git/commit/?h=rtc-next&id=578c2b661e2b1b474ea3571a3c3c6d57bae89e8d

kbuild reported that it successfully built a few configuration with that
patch applied so I'm hoping it is enough.

-- 
Alexandre Belloni, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

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

* Re: [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed
  2020-01-28  8:50     ` Alexandre Belloni
@ 2020-01-28  8:59       ` Geert Uytterhoeven
  0 siblings, 0 replies; 13+ messages in thread
From: Geert Uytterhoeven @ 2020-01-28  8:59 UTC (permalink / raw)
  To: Alexandre Belloni
  Cc: Alessandro Zummo, Akinobu Mita, Joe Perches, linux-rtc,
	Linux Kernel Mailing List

Hi Alexandre,

On Tue, Jan 28, 2020 at 9:50 AM Alexandre Belloni
<alexandre.belloni@bootlin.com> wrote:
> On 28/01/2020 09:33:28+0100, Geert Uytterhoeven wrote:
> > On Mon, Jan 27, 2020 at 11:45 PM Alexandre Belloni
> > <alexandre.belloni@bootlin.com> wrote:
> > > On 12/01/2020 18:13:49+0100, Geert Uytterhoeven wrote:
> > > > Merely enabling I2C and RTC selects REGMAP_I2C and REGMAP_SPI, even when
> > > > no driver needs it.  While the former can be moduler, the latter cannot,
> > > > and thus becomes built-in.
> > > >
> > > > Fix this by moving the select statements for REGMAP_I2C and REGMAP_SPI
> > > > from the RTC_I2C_AND_SPI helper to the individual drivers that depend on
> > > > it.
> > > >
> > > > Note that the comment for RTC_I2C_AND_SPI refers to SND_SOC_I2C_AND_SPI
> > > > for more information, but the latter does not select REGMAP_{I2C,SPI}
> > > > itself, and defers that to the individual drivers, too.
> > > >
> > > > Fixes: 080481f54ef62121 ("rtc: merge ds3232 and ds3234")
> > > > Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
> > > > ---
> > > > Joe: When merging addresses, scripts/get_maintainer.pl replaces
> > > >      Alexandre's authoritative email address from MAINTAINERS by the
> > > >      obsolete address in the SoB-line of the commit referred to by the
> > > >      Fixes-line.
> > > >
> > > >  drivers/rtc/Kconfig | 8 ++++++--
> > > >  1 file changed, 6 insertions(+), 2 deletions(-)
> > > >
> > > Applied, thanks.
> >
> > According to the reports from kbuild test report, some drivers are still
> > missing some selects, which is exposed by this patch.
> > But perhaps you have already fixed those in your tree?
> >
>
> I did fix a bunch of those in
> https://git.kernel.org/pub/scm/linux/kernel/git/abelloni/linux.git/commit/?h=rtc-next&id=578c2b661e2b1b474ea3571a3c3c6d57bae89e8d
>
> kbuild reported that it successfully built a few configuration with that
> patch applied so I'm hoping it is enough.

Great, thanks a lot!

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

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

end of thread, back to index

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-12 17:13 [PATCH] rtc: i2c/spi: Avoid inclusion of REGMAP support when not needed Geert Uytterhoeven
2020-01-12 21:45 ` kbuild test robot
2020-01-13  1:29 ` kbuild test robot
2020-01-13  6:36 ` Joe Perches
2020-01-13  7:25   ` Geert Uytterhoeven
2020-01-13  7:40     ` Joe Perches
2020-01-13  7:57       ` Geert Uytterhoeven
2020-01-13  8:12         ` Joe Perches
2020-01-14 10:13           ` Geert Uytterhoeven
2020-01-27 22:45 ` Alexandre Belloni
2020-01-28  8:33   ` Geert Uytterhoeven
2020-01-28  8:50     ` Alexandre Belloni
2020-01-28  8:59       ` Geert Uytterhoeven

Linux-RTC Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-rtc/0 linux-rtc/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-rtc linux-rtc/ https://lore.kernel.org/linux-rtc \
		linux-rtc@vger.kernel.org
	public-inbox-index linux-rtc

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-rtc


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git