* [PATCH 0/3] media: radio-bcm2048: multiple small cleanups
@ 2016-10-19 20:47 Jean-Baptiste Abbadie
2016-10-19 20:47 ` [PATCH 1/3] Staging: media: radio-bcm2048: Fix symbolic permissions Jean-Baptiste Abbadie
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Jean-Baptiste Abbadie @ 2016-10-19 20:47 UTC (permalink / raw)
To: Mauro Carvalho Chehab, Greg Kroah-Hartman, linux-media
Cc: Jean-Baptiste Abbadie
Hello,
This is a series of minor patches to fix checkpatch.pl issues.
Regards,
Jean-Baptiste
Jean-Baptiste Abbadie (3):
Staging: media: radio-bcm2048: Fix symbolic permissions
Staging: media: radio-bcm2048: Fix indentation
Staging: media: radio-bcm2048: Remove FSF address from GPL notice
drivers/staging/media/bcm2048/radio-bcm2048.c | 66 +++++++++++++--------------
1 file changed, 31 insertions(+), 35 deletions(-)
--
2.10.0
^ permalink raw reply [flat|nested] 7+ messages in thread* [PATCH 1/3] Staging: media: radio-bcm2048: Fix symbolic permissions 2016-10-19 20:47 [PATCH 0/3] media: radio-bcm2048: multiple small cleanups Jean-Baptiste Abbadie @ 2016-10-19 20:47 ` Jean-Baptiste Abbadie 2016-10-19 21:06 ` Mauro Carvalho Chehab 2016-10-19 20:47 ` [PATCH 2/3] Staging: media: radio-bcm2048: Fix indentation Jean-Baptiste Abbadie 2016-10-19 20:47 ` [PATCH 3/3] Staging: media: radio-bcm2048: Remove FSF address from GPL notice Jean-Baptiste Abbadie 2 siblings, 1 reply; 7+ messages in thread From: Jean-Baptiste Abbadie @ 2016-10-19 20:47 UTC (permalink / raw) To: Mauro Carvalho Chehab, Greg Kroah-Hartman, linux-media Cc: Jean-Baptiste Abbadie This replaces the S_* style permissions by numbers for the __ATTR macros Signed-off-by: Jean-Baptiste Abbadie <jb@abbadie.fr> --- drivers/staging/media/bcm2048/radio-bcm2048.c | 58 +++++++++++++-------------- 1 file changed, 29 insertions(+), 29 deletions(-) diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers/staging/media/bcm2048/radio-bcm2048.c index fe637ce8f4e7..188d045d44ad 100644 --- a/drivers/staging/media/bcm2048/radio-bcm2048.c +++ b/drivers/staging/media/bcm2048/radio-bcm2048.c @@ -2057,67 +2057,67 @@ property_signed_read(fm_rssi, int, "%d") DEFINE_SYSFS_PROPERTY(region, unsigned, int, "%u", 0) static struct device_attribute attrs[] = { - __ATTR(power_state, S_IRUGO | S_IWUSR, bcm2048_power_state_read, + __ATTR(power_state, 0644, bcm2048_power_state_read, bcm2048_power_state_write), - __ATTR(mute, S_IRUGO | S_IWUSR, bcm2048_mute_read, + __ATTR(mute, 0644, bcm2048_mute_read, bcm2048_mute_write), - __ATTR(audio_route, S_IRUGO | S_IWUSR, bcm2048_audio_route_read, + __ATTR(audio_route, 0644, bcm2048_audio_route_read, bcm2048_audio_route_write), - __ATTR(dac_output, S_IRUGO | S_IWUSR, bcm2048_dac_output_read, + __ATTR(dac_output, 0644, bcm2048_dac_output_read, bcm2048_dac_output_write), - __ATTR(fm_hi_lo_injection, S_IRUGO | S_IWUSR, + __ATTR(fm_hi_lo_injection, 0644, bcm2048_fm_hi_lo_injection_read, bcm2048_fm_hi_lo_injection_write), - __ATTR(fm_frequency, S_IRUGO | S_IWUSR, bcm2048_fm_frequency_read, + __ATTR(fm_frequency, 0644, bcm2048_fm_frequency_read, bcm2048_fm_frequency_write), - __ATTR(fm_af_frequency, S_IRUGO | S_IWUSR, + __ATTR(fm_af_frequency, 0644, bcm2048_fm_af_frequency_read, bcm2048_fm_af_frequency_write), - __ATTR(fm_deemphasis, S_IRUGO | S_IWUSR, bcm2048_fm_deemphasis_read, + __ATTR(fm_deemphasis, 0644, bcm2048_fm_deemphasis_read, bcm2048_fm_deemphasis_write), - __ATTR(fm_rds_mask, S_IRUGO | S_IWUSR, bcm2048_fm_rds_mask_read, + __ATTR(fm_rds_mask, 0644, bcm2048_fm_rds_mask_read, bcm2048_fm_rds_mask_write), - __ATTR(fm_best_tune_mode, S_IRUGO | S_IWUSR, + __ATTR(fm_best_tune_mode, 0644, bcm2048_fm_best_tune_mode_read, bcm2048_fm_best_tune_mode_write), - __ATTR(fm_search_rssi_threshold, S_IRUGO | S_IWUSR, + __ATTR(fm_search_rssi_threshold, 0644, bcm2048_fm_search_rssi_threshold_read, bcm2048_fm_search_rssi_threshold_write), - __ATTR(fm_search_mode_direction, S_IRUGO | S_IWUSR, + __ATTR(fm_search_mode_direction, 0644, bcm2048_fm_search_mode_direction_read, bcm2048_fm_search_mode_direction_write), - __ATTR(fm_search_tune_mode, S_IRUGO | S_IWUSR, + __ATTR(fm_search_tune_mode, 0644, bcm2048_fm_search_tune_mode_read, bcm2048_fm_search_tune_mode_write), - __ATTR(rds, S_IRUGO | S_IWUSR, bcm2048_rds_read, + __ATTR(rds, 0644, bcm2048_rds_read, bcm2048_rds_write), - __ATTR(rds_b_block_mask, S_IRUGO | S_IWUSR, + __ATTR(rds_b_block_mask, 0644, bcm2048_rds_b_block_mask_read, bcm2048_rds_b_block_mask_write), - __ATTR(rds_b_block_match, S_IRUGO | S_IWUSR, + __ATTR(rds_b_block_match, 0644, bcm2048_rds_b_block_match_read, bcm2048_rds_b_block_match_write), - __ATTR(rds_pi_mask, S_IRUGO | S_IWUSR, bcm2048_rds_pi_mask_read, + __ATTR(rds_pi_mask, 0644, bcm2048_rds_pi_mask_read, bcm2048_rds_pi_mask_write), - __ATTR(rds_pi_match, S_IRUGO | S_IWUSR, bcm2048_rds_pi_match_read, + __ATTR(rds_pi_match, 0644, bcm2048_rds_pi_match_read, bcm2048_rds_pi_match_write), - __ATTR(rds_wline, S_IRUGO | S_IWUSR, bcm2048_rds_wline_read, + __ATTR(rds_wline, 0644, bcm2048_rds_wline_read, bcm2048_rds_wline_write), - __ATTR(rds_pi, S_IRUGO, bcm2048_rds_pi_read, NULL), - __ATTR(rds_rt, S_IRUGO, bcm2048_rds_rt_read, NULL), - __ATTR(rds_ps, S_IRUGO, bcm2048_rds_ps_read, NULL), - __ATTR(fm_rds_flags, S_IRUGO, bcm2048_fm_rds_flags_read, NULL), - __ATTR(region_bottom_frequency, S_IRUGO, + __ATTR(rds_pi, 0444, bcm2048_rds_pi_read, NULL), + __ATTR(rds_rt, 0444, bcm2048_rds_rt_read, NULL), + __ATTR(rds_ps, 0444, bcm2048_rds_ps_read, NULL), + __ATTR(fm_rds_flags, 0444, bcm2048_fm_rds_flags_read, NULL), + __ATTR(region_bottom_frequency, 0444, bcm2048_region_bottom_frequency_read, NULL), - __ATTR(region_top_frequency, S_IRUGO, + __ATTR(region_top_frequency, 0444, bcm2048_region_top_frequency_read, NULL), - __ATTR(fm_carrier_error, S_IRUGO, + __ATTR(fm_carrier_error, 0444, bcm2048_fm_carrier_error_read, NULL), - __ATTR(fm_rssi, S_IRUGO, + __ATTR(fm_rssi, 0444, bcm2048_fm_rssi_read, NULL), - __ATTR(region, S_IRUGO | S_IWUSR, bcm2048_region_read, + __ATTR(region, 0644, bcm2048_region_read, bcm2048_region_write), - __ATTR(rds_data, S_IRUGO, bcm2048_rds_data_read, NULL), + __ATTR(rds_data, 0444, bcm2048_rds_data_read, NULL), }; static int bcm2048_sysfs_unregister_properties(struct bcm2048_device *bdev, -- 2.10.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 1/3] Staging: media: radio-bcm2048: Fix symbolic permissions 2016-10-19 20:47 ` [PATCH 1/3] Staging: media: radio-bcm2048: Fix symbolic permissions Jean-Baptiste Abbadie @ 2016-10-19 21:06 ` Mauro Carvalho Chehab 0 siblings, 0 replies; 7+ messages in thread From: Mauro Carvalho Chehab @ 2016-10-19 21:06 UTC (permalink / raw) To: Jean-Baptiste Abbadie Cc: Mauro Carvalho Chehab, Greg Kroah-Hartman, linux-media, Hans Verkuil Em Wed, 19 Oct 2016 22:47:12 +0200 Jean-Baptiste Abbadie <jb@abbadie.fr> escreveu: You should run get_maintainers.pl and check whomever submitted the driver, in order to get review. In particular, this driver looks to be submitted by Hans, although I guess he didn't authored: commit 899127b67df098e6d878f27be05dc91401cc6685 Author: Hans Verkuil <hans.verkuil@cisco.com> Date: Mon Nov 4 08:34:42 2013 -0300 [media] This adds support for the BCM2048 radio module found in Nokia N900 Add suport for Nokia N900 radio. This driver is far from being ready to be added at the main tree, as it creates its own sysfs interface, and violates lots of Coding Style rules, doing even evil things like returning from a function inside a macro. So, it is being added at staging with the condition that it will be soon be fixed. > This replaces the S_* style permissions by numbers for the __ATTR macros I really prefer to see permissions like 0644, instead of those weird S_* macros, as I can understand right away what is permitted. > > Signed-off-by: Jean-Baptiste Abbadie <jb@abbadie.fr> > --- > drivers/staging/media/bcm2048/radio-bcm2048.c | 58 +++++++++++++-------------- > 1 file changed, 29 insertions(+), 29 deletions(-) > > diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers/staging/media/bcm2048/radio-bcm2048.c > index fe637ce8f4e7..188d045d44ad 100644 > --- a/drivers/staging/media/bcm2048/radio-bcm2048.c > +++ b/drivers/staging/media/bcm2048/radio-bcm2048.c > @@ -2057,67 +2057,67 @@ property_signed_read(fm_rssi, int, "%d") > DEFINE_SYSFS_PROPERTY(region, unsigned, int, "%u", 0) > > static struct device_attribute attrs[] = { > - __ATTR(power_state, S_IRUGO | S_IWUSR, bcm2048_power_state_read, > + __ATTR(power_state, 0644, bcm2048_power_state_read, > bcm2048_power_state_write), > - __ATTR(mute, S_IRUGO | S_IWUSR, bcm2048_mute_read, > + __ATTR(mute, 0644, bcm2048_mute_read, > bcm2048_mute_write), > - __ATTR(audio_route, S_IRUGO | S_IWUSR, bcm2048_audio_route_read, > + __ATTR(audio_route, 0644, bcm2048_audio_route_read, > bcm2048_audio_route_write), > - __ATTR(dac_output, S_IRUGO | S_IWUSR, bcm2048_dac_output_read, > + __ATTR(dac_output, 0644, bcm2048_dac_output_read, > bcm2048_dac_output_write), > - __ATTR(fm_hi_lo_injection, S_IRUGO | S_IWUSR, > + __ATTR(fm_hi_lo_injection, 0644, > bcm2048_fm_hi_lo_injection_read, > bcm2048_fm_hi_lo_injection_write), > - __ATTR(fm_frequency, S_IRUGO | S_IWUSR, bcm2048_fm_frequency_read, > + __ATTR(fm_frequency, 0644, bcm2048_fm_frequency_read, > bcm2048_fm_frequency_write), > - __ATTR(fm_af_frequency, S_IRUGO | S_IWUSR, > + __ATTR(fm_af_frequency, 0644, > bcm2048_fm_af_frequency_read, > bcm2048_fm_af_frequency_write), > - __ATTR(fm_deemphasis, S_IRUGO | S_IWUSR, bcm2048_fm_deemphasis_read, > + __ATTR(fm_deemphasis, 0644, bcm2048_fm_deemphasis_read, > bcm2048_fm_deemphasis_write), > - __ATTR(fm_rds_mask, S_IRUGO | S_IWUSR, bcm2048_fm_rds_mask_read, > + __ATTR(fm_rds_mask, 0644, bcm2048_fm_rds_mask_read, > bcm2048_fm_rds_mask_write), > - __ATTR(fm_best_tune_mode, S_IRUGO | S_IWUSR, > + __ATTR(fm_best_tune_mode, 0644, > bcm2048_fm_best_tune_mode_read, > bcm2048_fm_best_tune_mode_write), > - __ATTR(fm_search_rssi_threshold, S_IRUGO | S_IWUSR, > + __ATTR(fm_search_rssi_threshold, 0644, > bcm2048_fm_search_rssi_threshold_read, > bcm2048_fm_search_rssi_threshold_write), > - __ATTR(fm_search_mode_direction, S_IRUGO | S_IWUSR, > + __ATTR(fm_search_mode_direction, 0644, > bcm2048_fm_search_mode_direction_read, > bcm2048_fm_search_mode_direction_write), > - __ATTR(fm_search_tune_mode, S_IRUGO | S_IWUSR, > + __ATTR(fm_search_tune_mode, 0644, > bcm2048_fm_search_tune_mode_read, > bcm2048_fm_search_tune_mode_write), > - __ATTR(rds, S_IRUGO | S_IWUSR, bcm2048_rds_read, > + __ATTR(rds, 0644, bcm2048_rds_read, > bcm2048_rds_write), > - __ATTR(rds_b_block_mask, S_IRUGO | S_IWUSR, > + __ATTR(rds_b_block_mask, 0644, > bcm2048_rds_b_block_mask_read, > bcm2048_rds_b_block_mask_write), > - __ATTR(rds_b_block_match, S_IRUGO | S_IWUSR, > + __ATTR(rds_b_block_match, 0644, > bcm2048_rds_b_block_match_read, > bcm2048_rds_b_block_match_write), > - __ATTR(rds_pi_mask, S_IRUGO | S_IWUSR, bcm2048_rds_pi_mask_read, > + __ATTR(rds_pi_mask, 0644, bcm2048_rds_pi_mask_read, > bcm2048_rds_pi_mask_write), > - __ATTR(rds_pi_match, S_IRUGO | S_IWUSR, bcm2048_rds_pi_match_read, > + __ATTR(rds_pi_match, 0644, bcm2048_rds_pi_match_read, > bcm2048_rds_pi_match_write), > - __ATTR(rds_wline, S_IRUGO | S_IWUSR, bcm2048_rds_wline_read, > + __ATTR(rds_wline, 0644, bcm2048_rds_wline_read, > bcm2048_rds_wline_write), > - __ATTR(rds_pi, S_IRUGO, bcm2048_rds_pi_read, NULL), > - __ATTR(rds_rt, S_IRUGO, bcm2048_rds_rt_read, NULL), > - __ATTR(rds_ps, S_IRUGO, bcm2048_rds_ps_read, NULL), > - __ATTR(fm_rds_flags, S_IRUGO, bcm2048_fm_rds_flags_read, NULL), > - __ATTR(region_bottom_frequency, S_IRUGO, > + __ATTR(rds_pi, 0444, bcm2048_rds_pi_read, NULL), > + __ATTR(rds_rt, 0444, bcm2048_rds_rt_read, NULL), > + __ATTR(rds_ps, 0444, bcm2048_rds_ps_read, NULL), > + __ATTR(fm_rds_flags, 0444, bcm2048_fm_rds_flags_read, NULL), > + __ATTR(region_bottom_frequency, 0444, > bcm2048_region_bottom_frequency_read, NULL), > - __ATTR(region_top_frequency, S_IRUGO, > + __ATTR(region_top_frequency, 0444, > bcm2048_region_top_frequency_read, NULL), > - __ATTR(fm_carrier_error, S_IRUGO, > + __ATTR(fm_carrier_error, 0444, > bcm2048_fm_carrier_error_read, NULL), > - __ATTR(fm_rssi, S_IRUGO, > + __ATTR(fm_rssi, 0444, > bcm2048_fm_rssi_read, NULL), > - __ATTR(region, S_IRUGO | S_IWUSR, bcm2048_region_read, > + __ATTR(region, 0644, bcm2048_region_read, > bcm2048_region_write), > - __ATTR(rds_data, S_IRUGO, bcm2048_rds_data_read, NULL), > + __ATTR(rds_data, 0444, bcm2048_rds_data_read, NULL), > }; > > static int bcm2048_sysfs_unregister_properties(struct bcm2048_device *bdev, Thanks, Mauro ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 2/3] Staging: media: radio-bcm2048: Fix indentation 2016-10-19 20:47 [PATCH 0/3] media: radio-bcm2048: multiple small cleanups Jean-Baptiste Abbadie 2016-10-19 20:47 ` [PATCH 1/3] Staging: media: radio-bcm2048: Fix symbolic permissions Jean-Baptiste Abbadie @ 2016-10-19 20:47 ` Jean-Baptiste Abbadie 2016-10-19 21:09 ` Mauro Carvalho Chehab 2016-10-19 20:47 ` [PATCH 3/3] Staging: media: radio-bcm2048: Remove FSF address from GPL notice Jean-Baptiste Abbadie 2 siblings, 1 reply; 7+ messages in thread From: Jean-Baptiste Abbadie @ 2016-10-19 20:47 UTC (permalink / raw) To: Mauro Carvalho Chehab, Greg Kroah-Hartman, linux-media Cc: Jean-Baptiste Abbadie Align multiple lines statement with parentheses Signed-off-by: Jean-Baptiste Abbadie <jb@abbadie.fr> --- drivers/staging/media/bcm2048/radio-bcm2048.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers/staging/media/bcm2048/radio-bcm2048.c index 188d045d44ad..f66bea631e8e 100644 --- a/drivers/staging/media/bcm2048/radio-bcm2048.c +++ b/drivers/staging/media/bcm2048/radio-bcm2048.c @@ -997,7 +997,7 @@ static int bcm2048_set_fm_search_tune_mode(struct bcm2048_device *bdev, timeout = BCM2048_AUTO_SEARCH_TIMEOUT; if (!wait_for_completion_timeout(&bdev->compl, - msecs_to_jiffies(timeout))) + msecs_to_jiffies(timeout))) dev_err(&bdev->client->dev, "IRQ timeout.\n"); if (value) @@ -2202,7 +2202,7 @@ static ssize_t bcm2048_fops_read(struct file *file, char __user *buf, } /* interruptible_sleep_on(&bdev->read_queue); */ if (wait_event_interruptible(bdev->read_queue, - bdev->rds_data_available) < 0) { + bdev->rds_data_available) < 0) { retval = -EINTR; goto done; } -- 2.10.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 2/3] Staging: media: radio-bcm2048: Fix indentation 2016-10-19 20:47 ` [PATCH 2/3] Staging: media: radio-bcm2048: Fix indentation Jean-Baptiste Abbadie @ 2016-10-19 21:09 ` Mauro Carvalho Chehab 0 siblings, 0 replies; 7+ messages in thread From: Mauro Carvalho Chehab @ 2016-10-19 21:09 UTC (permalink / raw) To: Jean-Baptiste Abbadie Cc: Mauro Carvalho Chehab, Greg Kroah-Hartman, linux-media Em Wed, 19 Oct 2016 22:47:13 +0200 Jean-Baptiste Abbadie <jb@abbadie.fr> escreveu: > Align multiple lines statement with parentheses Looks OK to me. Greg, do you want to pick it on your tree or do you prefer if I pick myself? If you prefer to pick it: Acked-by: Mauro Carvalho Chehab <mchehab@s-opensource.com> > > Signed-off-by: Jean-Baptiste Abbadie <jb@abbadie.fr> > --- > drivers/staging/media/bcm2048/radio-bcm2048.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers/staging/media/bcm2048/radio-bcm2048.c > index 188d045d44ad..f66bea631e8e 100644 > --- a/drivers/staging/media/bcm2048/radio-bcm2048.c > +++ b/drivers/staging/media/bcm2048/radio-bcm2048.c > @@ -997,7 +997,7 @@ static int bcm2048_set_fm_search_tune_mode(struct bcm2048_device *bdev, > timeout = BCM2048_AUTO_SEARCH_TIMEOUT; > > if (!wait_for_completion_timeout(&bdev->compl, > - msecs_to_jiffies(timeout))) > + msecs_to_jiffies(timeout))) > dev_err(&bdev->client->dev, "IRQ timeout.\n"); > > if (value) > @@ -2202,7 +2202,7 @@ static ssize_t bcm2048_fops_read(struct file *file, char __user *buf, > } > /* interruptible_sleep_on(&bdev->read_queue); */ > if (wait_event_interruptible(bdev->read_queue, > - bdev->rds_data_available) < 0) { > + bdev->rds_data_available) < 0) { > retval = -EINTR; > goto done; > } Thanks, Mauro ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 3/3] Staging: media: radio-bcm2048: Remove FSF address from GPL notice 2016-10-19 20:47 [PATCH 0/3] media: radio-bcm2048: multiple small cleanups Jean-Baptiste Abbadie 2016-10-19 20:47 ` [PATCH 1/3] Staging: media: radio-bcm2048: Fix symbolic permissions Jean-Baptiste Abbadie 2016-10-19 20:47 ` [PATCH 2/3] Staging: media: radio-bcm2048: Fix indentation Jean-Baptiste Abbadie @ 2016-10-19 20:47 ` Jean-Baptiste Abbadie 2016-10-19 21:08 ` Mauro Carvalho Chehab 2 siblings, 1 reply; 7+ messages in thread From: Jean-Baptiste Abbadie @ 2016-10-19 20:47 UTC (permalink / raw) To: Mauro Carvalho Chehab, Greg Kroah-Hartman, linux-media Cc: Jean-Baptiste Abbadie Removes the superfluous statement about writing to the FSF in the GPL notice Signed-off-by: Jean-Baptiste Abbadie <jb@abbadie.fr> --- drivers/staging/media/bcm2048/radio-bcm2048.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers/staging/media/bcm2048/radio-bcm2048.c index f66bea631e8e..607dd5285149 100644 --- a/drivers/staging/media/bcm2048/radio-bcm2048.c +++ b/drivers/staging/media/bcm2048/radio-bcm2048.c @@ -17,10 +17,6 @@ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU * General Public License for more details. * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software - * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA - * 02110-1301 USA */ /* -- 2.10.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 3/3] Staging: media: radio-bcm2048: Remove FSF address from GPL notice 2016-10-19 20:47 ` [PATCH 3/3] Staging: media: radio-bcm2048: Remove FSF address from GPL notice Jean-Baptiste Abbadie @ 2016-10-19 21:08 ` Mauro Carvalho Chehab 0 siblings, 0 replies; 7+ messages in thread From: Mauro Carvalho Chehab @ 2016-10-19 21:08 UTC (permalink / raw) To: Jean-Baptiste Abbadie Cc: Mauro Carvalho Chehab, Greg Kroah-Hartman, linux-media Em Wed, 19 Oct 2016 22:47:14 +0200 Jean-Baptiste Abbadie <jb@abbadie.fr> escreveu: > Removes the superfluous statement about writing to the FSF in the GPL > notice Looks OK to me. Greg, do you want to pick it on your tree or do you prefer if I pick myself? If you prefer to pick it: Acked-by: Mauro Carvalho Chehab <mchehab@s-opensource.com> > > Signed-off-by: Jean-Baptiste Abbadie <jb@abbadie.fr> > --- > drivers/staging/media/bcm2048/radio-bcm2048.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/drivers/staging/media/bcm2048/radio-bcm2048.c b/drivers/staging/media/bcm2048/radio-bcm2048.c > index f66bea631e8e..607dd5285149 100644 > --- a/drivers/staging/media/bcm2048/radio-bcm2048.c > +++ b/drivers/staging/media/bcm2048/radio-bcm2048.c > @@ -17,10 +17,6 @@ > * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > * General Public License for more details. > * > - * You should have received a copy of the GNU General Public License > - * along with this program; if not, write to the Free Software > - * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA > - * 02110-1301 USA > */ > > /* Thanks, Mauro ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-10-19 21:09 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-10-19 20:47 [PATCH 0/3] media: radio-bcm2048: multiple small cleanups Jean-Baptiste Abbadie 2016-10-19 20:47 ` [PATCH 1/3] Staging: media: radio-bcm2048: Fix symbolic permissions Jean-Baptiste Abbadie 2016-10-19 21:06 ` Mauro Carvalho Chehab 2016-10-19 20:47 ` [PATCH 2/3] Staging: media: radio-bcm2048: Fix indentation Jean-Baptiste Abbadie 2016-10-19 21:09 ` Mauro Carvalho Chehab 2016-10-19 20:47 ` [PATCH 3/3] Staging: media: radio-bcm2048: Remove FSF address from GPL notice Jean-Baptiste Abbadie 2016-10-19 21:08 ` Mauro Carvalho Chehab
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.