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");;

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to