Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933286AbeAKKQ4 (ORCPT + 1 other); Thu, 11 Jan 2018 05:16:56 -0500 Received: from out3-smtp.messagingengine.com ([66.111.4.27]:55259 "EHLO out3-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753778AbeAKKQy (ORCPT ); Thu, 11 Jan 2018 05:16:54 -0500 X-ME-Sender: Subject: Re: [PATCH v2 0/2] staging: rtl8723bs: Fix multiple trailing semicolons To: Joe Perches , linux-kernel@vger.kernel.org Cc: devel@driverdev.osuosl.org, Greg Kroah-Hartman , Larry Finger , Hans de Goede , Bastien Nocera , Arnd Bergmann , Dan Carpenter References: <20180110165129.12604-1-luisbg@kernel.org> <1515633543.9619.159.camel@perches.com> From: Luis de Bethencourt Message-ID: Date: Thu, 11 Jan 2018 10:16:50 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 MIME-Version: 1.0 In-Reply-To: <1515633543.9619.159.camel@perches.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: On 01/11/2018 01:19 AM, Joe Perches wrote: > On Wed, 2018-01-10 at 17:01 +0000, Luis de Bethencourt wrote: >> On 01/10/2018 04:51 PM, Luis de Bethencourt wrote: >>> Hi, >>> >>> Removing the last patch from the original series after Dan's comment. >>> >>> Tomorrow, I will send a new patch that removes the commented out code. This will >>> replace that last patch. >>> >>> Thanks for the review, >>> Luis >>> >>> >>> Luis de Bethencourt (2): >>> staging: rtl8723bs: Fix trailing semicolon >>> staging: rtl8723bs: Fix trailing semicolon >>> >>> drivers/staging/rtl8723bs/hal/HalPhyRf_8723B.c | 2 +- >>> drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c | 2 +- >>> 2 files changed, 2 insertions(+), 2 deletions(-) >>> >> >> A little bit too late but I just realized that instead of LKML (which is down), >> I can link to the archives of the driverdev mailing list instead. >> >> Here is Dan's email which I reference above: >> http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/2018-January/115392.html >> >> Thanks, >> Luis > > If you really want to remove these, perhaps do it treewide. > > $ git grep -P -n ";\s*;\s*$" -- "*.[ch]"|grep -vw for|grep -vP ":\s+;\s*;\s*$" > arch/arc/kernel/setup.c:376: char *opt_nm = NULL;; > arch/arc/kernel/unwind.c:369: panic("Attention !!! Dwarf FDE parsing errors\n");; > arch/arm/include/asm/kvm_emulate.h:134: return cpsr_mode > USR_MODE;; > arch/arm/kernel/time.c:86:static clock_access_fn __read_boot_clock = dummy_clock_access;; > arch/arm64/kernel/ptrace.c:1422: int err, idx = compat_ptrace_hbp_num_to_idx(num);; > arch/mips/include/asm/checksum.h:113: u32 sum = (__force u32)csum;; > arch/powerpc/kvm/book3s_xive.c:160: return -ENOMEM;; > arch/powerpc/platforms/powernv/pci-ioda.c:1843: return -ENODEV;; > arch/x86/boot/compressed/eboot.c:442: u32 *handles = (u32 *)uga_handle;; > arch/x86/boot/compressed/eboot.c:487: u64 *handles = (u64 *)uga_handle;; > block/sed-opal.c:493: return -EOPNOTSUPP;; > drivers/clocksource/mips-gic-timer.c:208: return -EINVAL;; > drivers/clocksource/mips-gic-timer.c:213: return -EINVAL;; > drivers/clocksource/timer-sun5i.c:337: return PTR_ERR(timer_base);; > drivers/dax/device.c:136: return NULL;; > drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c:721: union lane_status dpcd_lane_status[LANE_COUNT_DP_MAX] = {{{0}}};; > drivers/gpu/drm/amd/powerplay/amd_powerplay.c:165: return -EINVAL;; > drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c:683: aspace = NULL;; > drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmm.c:645: block = (size >> page[i].shift) << page[i].shift;; > drivers/gpu/drm/scheduler/gpu_scheduler.c:464: int i;; > drivers/hwmon/lm75.c:103: break;; > drivers/iommu/intel-svm.c:196: unsigned long mask = __rounddown_pow_of_two(address ^ last);; > drivers/md/dm-thin.c:1725: struct dm_thin_endio_hook *h = dm_per_bio_data(bio, sizeof(struct dm_thin_endio_hook));; > drivers/md/raid1.c:1111: r1_bio->behind_master_bio = behind_bio;; > drivers/net/can/rx-offload.c:259: return can_rx_offload_init_queue(dev, offload, weight);; > drivers/net/ethernet/mellanox/mlx5/core/ipoib/ipoib.c:488: mlx5e_close_channels(&epriv->channels);; > drivers/net/wireless/realtek/rtl8xxxu/rtl8xxxu.h:1175: u8 dummy4_1:1;; > drivers/pci/pci-driver.c:971: return pm_generic_freeze_late(dev);; > drivers/powercap/intel_rapl.c:1212: int nr_pl, ret;; > drivers/soc/imx/gpc.c:351: return PTR_ERR(domain->supply);; > drivers/staging/rtl8723bs/hal/HalPhyRf_8723B.c:1055: regEA4 = PHY_QueryBBReg(pDM_Odm->Adapter, rRx_Power_Before_IQK_A_2, bMaskDWord);; > drivers/staging/rtl8723bs/hal/rtl8723b_phycfg.c:156: tmplong2 = PHY_QueryBBReg(Adapter, rFPGA0_XA_HSSIParameter2|MaskforPhySet, bMaskDWord);; > drivers/staging/rtl8723bs/os_dep/ioctl_linux.c:970: ret = -EINVAL;; > drivers/target/target_core_user.c:1409: return ret;; > include/net/vxlan.h:304: return features;; > net/netfilter/nft_dynset.c:167: err = nft_validate_register_load(priv->sreg_key, set->klen);; > samples/bpf/xdp_monitor_kern.c:107: u64 *cnt;; > scripts/mod/modpost.c:1688: const char *tosec = sec_name(elf, get_secindex(elf, sym));; > sound/core/pcm_native.c:3449: struct snd_pcm_runtime *runtime = substream->runtime;; > sound/soc/codecs/cs42l73.c:1358: return ret;; > tools/perf/util/evlist.c:1763: goto state_err;; > tools/testing/selftests/nsfs/pidns.c:73: return pr_err("Don't get EPERM");; > Hi Joe, I was following the good-old premise of reading/fixing the code for hardware you have. Since you think there is value on fixing the trailing semicolons treewide. I will slowly do that over the next few days. Thanks for the feedback, Luis