From: Lee Jones <lee.jones@linaro.org> To: Jonathan Cameron <jic23@kernel.org> Cc: knaack.h@gmx.de, lars@metafoo.de, pmeerw@pmeerw.net, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org Subject: Re: [PATCH 00/30] First batch of W=1 fixes for IIO Date: Mon, 20 Jul 2020 08:51:34 +0100 [thread overview] Message-ID: <20200720075134.GM3165313@dell> (raw) In-Reply-To: <20200718154204.280fcb83@archlinux> On Sat, 18 Jul 2020, Jonathan Cameron wrote: > On Thu, 16 Jul 2020 14:58:58 +0100 > Lee Jones <lee.jones@linaro.org> wrote: > > > This set is part of a larger effort attempting to clean-up W=1 > > kernel builds, which are currently overwhelmingly riddled with > > niggly little warnings. > Thanks for these. > > One general comment is I'd have appreciated a spot of grouping to bring > a particular group of identical problems together in the series. > Example being the @lock(:) ones in this set. There are a few reasons for me not doing this. Firstly, some of the files experiencing 'lock:' issues had other problems which needed to be dealt with anyway. I also do not know what other issues exist before I start drafting patches - until now I have been going through the list of issues, file-by-file, from top (first complained about) to bottom. I have no mechanism (or desire) to pre-read 100's of lines of warnings to see which ones could be easily grouped. I also wanted to keep changes encapsulated into 1 patch per file, for bisectability and revertability proposes. I'm certainly not disagreeing or arguing with you, just explaining that it is something that I've thought about. > It doesn't matter that much though. I'm just being fussy :) > > Jonathan > > > > > Lee Jones (30): > > iio: adc: ad_sigma_delta: Remove unused variable 'ret' > > iio: accel: bma220_spi: Do not define 'struct acpi_device_id' when > > !CONFIG_ACPI > > iio: common: ms_sensors: ms_sensors_i2c: Fix misspelling of parameter > > 'client' > > iio: adc: ad7298: Demote obvious misuse of kerneldoc to standard > > comment blocks > > iio: chemical: sgp30: Add description for sgp_read_cmd()'s > > 'duration_us' > > iio: adc: ad7923: Demote obvious misuse of kerneldoc to standard > > comment blocks > > iio: dac: ad5360: Fix kerneldoc attribute formatting for 'lock' > > iio: dac: ad5380: Fix kerneldoc attribute formatting for 'lock' > > iio: dummy: iio_simple_dummy: Demote unworthy kerneldocs and correct > > misspelling > > iio: dummy: iio_simple_dummy: Add newline after function-end > > iio: dac: ad5421: Fix kerneldoc attribute formatting for 'lock' > > iio: gyro: adis16080: Fix formatting issue and compiler attribute > > ordering > > iio: dummy: iio_simple_dummy_events: Demote file header and supply > > descriptions for val2 params > > iio: dac: ad5064: Value returned by ad5064_vref_name may not be 'const > > * const' > > iio: dummy: iio_dummy_evgen: Demote file header and supply description > > for 'irq_sim_domain' > > iio: adc: ad7887: Demote seemingly unintentional kerneldoc header > > iio: adc: ad7949: Fix misspelling issue and compiler attribute > > ordering > > iio: dummy: iio_simple_dummy_buffer: Demote file header and correct > > misspelling > > iio: dac: ad5064: Fix a few kerneldoc misdemeanours > > iio: dac: ad5446: Complete 'struct ad5446_state' doc and demote > > unworthy kerneldocs > > iio: dac: ad5449: Fix kerneldoc attribute formatting for 'lock' > > iio: dac: ad5755: Fix kerneldoc attribute formatting for 'lock' > > iio: dac: ad5758: Move and fix-up kerneldoc header and demote unworthy > > kerneldoc > > iio: dac: ad5761: Fix kerneldoc attribute formatting for 'lock' > > iio: dac: ad5764: Fix misdocumenting and formatting error > > iio: dac: ad5791: Complete 'struct ad5791_chip_info' documentation > > iio: light: cm32181: Fix formatting and docrot issues in > > cm32181_acpi_get_cpm() > > iio: magnetometer: mmc35240: Fix function header formatting > > iio: imu: kmx61: Fix formatting in kerneldoc function headers > > iio: dac: ad7303: Complete 'struct ad7303_state' doc and reorder > > compiler attribute > > > > drivers/iio/accel/bma220_spi.c | 2 ++ > > drivers/iio/adc/ad7298.c | 8 ++++---- > > drivers/iio/adc/ad7887.c | 2 +- > > drivers/iio/adc/ad7923.c | 8 ++++---- > > drivers/iio/adc/ad7949.c | 4 ++-- > > drivers/iio/adc/ad_sigma_delta.c | 7 ++----- > > drivers/iio/chemical/sgp30.c | 1 + > > drivers/iio/common/ms_sensors/ms_sensors_i2c.c | 2 +- > > drivers/iio/dac/ad5064.c | 10 +++++----- > > drivers/iio/dac/ad5360.c | 2 +- > > drivers/iio/dac/ad5380.c | 2 +- > > drivers/iio/dac/ad5421.c | 2 +- > > drivers/iio/dac/ad5446.c | 11 +++++++---- > > drivers/iio/dac/ad5449.c | 2 +- > > drivers/iio/dac/ad5755.c | 2 +- > > drivers/iio/dac/ad5758.c | 18 +++++++++--------- > > drivers/iio/dac/ad5761.c | 2 +- > > drivers/iio/dac/ad5764.c | 5 ++--- > > drivers/iio/dac/ad5791.c | 10 ++++------ > > drivers/iio/dac/ad7303.c | 5 ++++- > > drivers/iio/dummy/iio_dummy_evgen.c | 4 +++- > > drivers/iio/dummy/iio_simple_dummy.c | 7 ++++--- > > drivers/iio/dummy/iio_simple_dummy_buffer.c | 4 ++-- > > drivers/iio/dummy/iio_simple_dummy_events.c | 4 +++- > > drivers/iio/gyro/adis16080.c | 4 ++-- > > drivers/iio/imu/kmx61.c | 14 +++++++------- > > drivers/iio/light/cm32181.c | 8 ++++---- > > drivers/iio/magnetometer/mmc35240.c | 4 ++-- > > 28 files changed, 81 insertions(+), 73 deletions(-) > > > -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog
WARNING: multiple messages have this Message-ID (diff)
From: Lee Jones <lee.jones@linaro.org> To: Jonathan Cameron <jic23@kernel.org> Cc: lars@metafoo.de, linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, pmeerw@pmeerw.net, knaack.h@gmx.de, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 00/30] First batch of W=1 fixes for IIO Date: Mon, 20 Jul 2020 08:51:34 +0100 [thread overview] Message-ID: <20200720075134.GM3165313@dell> (raw) In-Reply-To: <20200718154204.280fcb83@archlinux> On Sat, 18 Jul 2020, Jonathan Cameron wrote: > On Thu, 16 Jul 2020 14:58:58 +0100 > Lee Jones <lee.jones@linaro.org> wrote: > > > This set is part of a larger effort attempting to clean-up W=1 > > kernel builds, which are currently overwhelmingly riddled with > > niggly little warnings. > Thanks for these. > > One general comment is I'd have appreciated a spot of grouping to bring > a particular group of identical problems together in the series. > Example being the @lock(:) ones in this set. There are a few reasons for me not doing this. Firstly, some of the files experiencing 'lock:' issues had other problems which needed to be dealt with anyway. I also do not know what other issues exist before I start drafting patches - until now I have been going through the list of issues, file-by-file, from top (first complained about) to bottom. I have no mechanism (or desire) to pre-read 100's of lines of warnings to see which ones could be easily grouped. I also wanted to keep changes encapsulated into 1 patch per file, for bisectability and revertability proposes. I'm certainly not disagreeing or arguing with you, just explaining that it is something that I've thought about. > It doesn't matter that much though. I'm just being fussy :) > > Jonathan > > > > > Lee Jones (30): > > iio: adc: ad_sigma_delta: Remove unused variable 'ret' > > iio: accel: bma220_spi: Do not define 'struct acpi_device_id' when > > !CONFIG_ACPI > > iio: common: ms_sensors: ms_sensors_i2c: Fix misspelling of parameter > > 'client' > > iio: adc: ad7298: Demote obvious misuse of kerneldoc to standard > > comment blocks > > iio: chemical: sgp30: Add description for sgp_read_cmd()'s > > 'duration_us' > > iio: adc: ad7923: Demote obvious misuse of kerneldoc to standard > > comment blocks > > iio: dac: ad5360: Fix kerneldoc attribute formatting for 'lock' > > iio: dac: ad5380: Fix kerneldoc attribute formatting for 'lock' > > iio: dummy: iio_simple_dummy: Demote unworthy kerneldocs and correct > > misspelling > > iio: dummy: iio_simple_dummy: Add newline after function-end > > iio: dac: ad5421: Fix kerneldoc attribute formatting for 'lock' > > iio: gyro: adis16080: Fix formatting issue and compiler attribute > > ordering > > iio: dummy: iio_simple_dummy_events: Demote file header and supply > > descriptions for val2 params > > iio: dac: ad5064: Value returned by ad5064_vref_name may not be 'const > > * const' > > iio: dummy: iio_dummy_evgen: Demote file header and supply description > > for 'irq_sim_domain' > > iio: adc: ad7887: Demote seemingly unintentional kerneldoc header > > iio: adc: ad7949: Fix misspelling issue and compiler attribute > > ordering > > iio: dummy: iio_simple_dummy_buffer: Demote file header and correct > > misspelling > > iio: dac: ad5064: Fix a few kerneldoc misdemeanours > > iio: dac: ad5446: Complete 'struct ad5446_state' doc and demote > > unworthy kerneldocs > > iio: dac: ad5449: Fix kerneldoc attribute formatting for 'lock' > > iio: dac: ad5755: Fix kerneldoc attribute formatting for 'lock' > > iio: dac: ad5758: Move and fix-up kerneldoc header and demote unworthy > > kerneldoc > > iio: dac: ad5761: Fix kerneldoc attribute formatting for 'lock' > > iio: dac: ad5764: Fix misdocumenting and formatting error > > iio: dac: ad5791: Complete 'struct ad5791_chip_info' documentation > > iio: light: cm32181: Fix formatting and docrot issues in > > cm32181_acpi_get_cpm() > > iio: magnetometer: mmc35240: Fix function header formatting > > iio: imu: kmx61: Fix formatting in kerneldoc function headers > > iio: dac: ad7303: Complete 'struct ad7303_state' doc and reorder > > compiler attribute > > > > drivers/iio/accel/bma220_spi.c | 2 ++ > > drivers/iio/adc/ad7298.c | 8 ++++---- > > drivers/iio/adc/ad7887.c | 2 +- > > drivers/iio/adc/ad7923.c | 8 ++++---- > > drivers/iio/adc/ad7949.c | 4 ++-- > > drivers/iio/adc/ad_sigma_delta.c | 7 ++----- > > drivers/iio/chemical/sgp30.c | 1 + > > drivers/iio/common/ms_sensors/ms_sensors_i2c.c | 2 +- > > drivers/iio/dac/ad5064.c | 10 +++++----- > > drivers/iio/dac/ad5360.c | 2 +- > > drivers/iio/dac/ad5380.c | 2 +- > > drivers/iio/dac/ad5421.c | 2 +- > > drivers/iio/dac/ad5446.c | 11 +++++++---- > > drivers/iio/dac/ad5449.c | 2 +- > > drivers/iio/dac/ad5755.c | 2 +- > > drivers/iio/dac/ad5758.c | 18 +++++++++--------- > > drivers/iio/dac/ad5761.c | 2 +- > > drivers/iio/dac/ad5764.c | 5 ++--- > > drivers/iio/dac/ad5791.c | 10 ++++------ > > drivers/iio/dac/ad7303.c | 5 ++++- > > drivers/iio/dummy/iio_dummy_evgen.c | 4 +++- > > drivers/iio/dummy/iio_simple_dummy.c | 7 ++++--- > > drivers/iio/dummy/iio_simple_dummy_buffer.c | 4 ++-- > > drivers/iio/dummy/iio_simple_dummy_events.c | 4 +++- > > drivers/iio/gyro/adis16080.c | 4 ++-- > > drivers/iio/imu/kmx61.c | 14 +++++++------- > > drivers/iio/light/cm32181.c | 8 ++++---- > > drivers/iio/magnetometer/mmc35240.c | 4 ++-- > > 28 files changed, 81 insertions(+), 73 deletions(-) > > > -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-07-20 7:51 UTC|newest] Thread overview: 142+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-07-16 13:58 [PATCH 00/30] First batch of W=1 fixes for IIO Lee Jones 2020-07-16 13:58 ` Lee Jones 2020-07-16 13:58 ` [PATCH 01/30] iio: adc: ad_sigma_delta: Remove unused variable 'ret' Lee Jones 2020-07-16 13:58 ` Lee Jones 2020-07-18 14:24 ` Jonathan Cameron 2020-07-18 14:24 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 02/30] iio: accel: bma220_spi: Do not define 'struct acpi_device_id' when !CONFIG_ACPI Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 14:26 ` Jonathan Cameron 2020-07-18 14:26 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 03/30] iio: common: ms_sensors: ms_sensors_i2c: Fix misspelling of parameter 'client' Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 14:27 ` Jonathan Cameron 2020-07-18 14:27 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 04/30] iio: adc: ad7298: Demote obvious misuse of kerneldoc to standard comment blocks Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 14:30 ` Jonathan Cameron 2020-07-18 14:30 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 05/30] iio: chemical: sgp30: Add description for sgp_read_cmd()'s 'duration_us' Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 14:33 ` Jonathan Cameron 2020-07-18 14:33 ` Jonathan Cameron 2020-09-17 18:33 ` Jonathan Cameron 2020-09-17 18:33 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 06/30] iio: adc: ad7923: Demote obvious misuse of kerneldoc to standard comment blocks Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 14:35 ` Jonathan Cameron 2020-07-18 14:35 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 07/30] iio: dac: ad5360: Fix kerneldoc attribute formatting for 'lock' Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 14:36 ` Jonathan Cameron 2020-07-18 14:36 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 08/30] iio: dac: ad5380: " Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 14:36 ` Jonathan Cameron 2020-07-18 14:36 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 09/30] iio: dummy: iio_simple_dummy: Demote unworthy kerneldocs and correct misspelling Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 14:39 ` Jonathan Cameron 2020-07-18 14:39 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 10/30] iio: dummy: iio_simple_dummy: Add newline after function-end Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 14:40 ` Jonathan Cameron 2020-07-18 14:40 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 11/30] iio: dac: ad5421: Fix kerneldoc attribute formatting for 'lock' Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 14:42 ` Jonathan Cameron 2020-07-18 14:42 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 12/30] iio: gyro: adis16080: Fix formatting issue and compiler attribute ordering Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 14:54 ` Jonathan Cameron 2020-07-18 14:54 ` Jonathan Cameron 2020-09-17 18:33 ` Jonathan Cameron 2020-09-17 18:33 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 13/30] iio: dummy: iio_simple_dummy_events: Demote file header and supply descriptions for val2 params Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 14:55 ` Jonathan Cameron 2020-07-18 14:55 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 14/30] iio: dac: ad5064: Value returned by ad5064_vref_name may not be 'const * const' Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 14:57 ` Jonathan Cameron 2020-07-18 14:57 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 15/30] iio: dummy: iio_dummy_evgen: Demote file header and supply description for 'irq_sim_domain' Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 9:24 ` Bartosz Golaszewski 2020-07-18 9:24 ` Bartosz Golaszewski 2020-07-18 15:00 ` Jonathan Cameron 2020-07-18 15:00 ` Jonathan Cameron 2020-09-17 18:32 ` Jonathan Cameron 2020-09-17 18:32 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 16/30] iio: adc: ad7887: Demote seemingly unintentional kerneldoc header Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 15:01 ` Jonathan Cameron 2020-07-18 15:01 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 17/30] iio: adc: ad7949: Fix misspelling issue and compiler attribute ordering Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 15:03 ` Jonathan Cameron 2020-07-18 15:03 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 18/30] iio: dummy: iio_simple_dummy_buffer: Demote file header and correct misspelling Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 15:02 ` Jonathan Cameron 2020-07-18 15:02 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 19/30] iio: dac: ad5064: Fix a few kerneldoc misdemeanours Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 15:04 ` Jonathan Cameron 2020-07-18 15:04 ` Jonathan Cameron 2020-09-17 18:31 ` Jonathan Cameron 2020-09-17 18:31 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 20/30] iio: dac: ad5446: Complete 'struct ad5446_state' doc and demote unworthy kerneldocs Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 15:05 ` Jonathan Cameron 2020-07-18 15:05 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 21/30] iio: dac: ad5449: Fix kerneldoc attribute formatting for 'lock' Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 15:06 ` Jonathan Cameron 2020-07-18 15:06 ` Jonathan Cameron 2020-07-18 15:07 ` Jonathan Cameron 2020-07-18 15:07 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 22/30] iio: dac: ad5755: " Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 15:08 ` Jonathan Cameron 2020-07-18 15:08 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 23/30] iio: dac: ad5758: Move and fix-up kerneldoc header and demote unworthy kerneldoc Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 15:09 ` Jonathan Cameron 2020-07-18 15:09 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 24/30] iio: dac: ad5761: Fix kerneldoc attribute formatting for 'lock' Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 15:10 ` Jonathan Cameron 2020-07-18 15:10 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 25/30] iio: dac: ad5764: Fix misdocumenting and formatting error Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 15:10 ` Jonathan Cameron 2020-07-18 15:10 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 26/30] iio: dac: ad5791: Complete 'struct ad5791_chip_info' documentation Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 15:12 ` Jonathan Cameron 2020-07-18 15:12 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 27/30] iio: light: cm32181: Fix formatting and docrot issues in cm32181_acpi_get_cpm() Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 15:14 ` Jonathan Cameron 2020-07-18 15:14 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 28/30] iio: magnetometer: mmc35240: Fix function header formatting Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 15:16 ` Jonathan Cameron 2020-07-18 15:16 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 29/30] iio: imu: kmx61: Fix formatting in kerneldoc function headers Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 15:17 ` Jonathan Cameron 2020-07-18 15:17 ` Jonathan Cameron 2020-07-16 13:59 ` [PATCH 30/30] iio: dac: ad7303: Complete 'struct ad7303_state' doc and reorder compiler attribute Lee Jones 2020-07-16 13:59 ` Lee Jones 2020-07-18 15:18 ` Jonathan Cameron 2020-07-18 15:18 ` Jonathan Cameron 2020-09-17 18:29 ` Jonathan Cameron 2020-09-17 18:29 ` Jonathan Cameron 2020-09-17 18:30 ` Jonathan Cameron 2020-09-17 18:30 ` Jonathan Cameron 2020-07-18 14:42 ` [PATCH 00/30] First batch of W=1 fixes for IIO Jonathan Cameron 2020-07-18 14:42 ` Jonathan Cameron 2020-07-20 7:51 ` Lee Jones [this message] 2020-07-20 7:51 ` Lee Jones
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20200720075134.GM3165313@dell \ --to=lee.jones@linaro.org \ --cc=jic23@kernel.org \ --cc=knaack.h@gmx.de \ --cc=lars@metafoo.de \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-iio@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=pmeerw@pmeerw.net \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.