== Series Details == Series: series starting with [01/12] kernel.h: Add for_each_if() URL : https://patchwork.freedesktop.org/series/46158/ State : warning == Summary == $ dim checkpatch origin/drm-tip c4210eba974e kernel.h: Add for_each_if() -:6: WARNING:TYPO_SPELLING: 'ambigious' may be misspelled - perhaps 'ambiguous'? #6: To avoid compilers complainig about ambigious else blocks when putting -:54: ERROR:MULTISTATEMENT_MACRO_USE_DO_WHILE: Macros starting with if should be enclosed by a do - while loop to avoid possible if/else logic defects #54: FILE: include/linux/kernel.h:75: +#define for_each_if(condition) if (!(condition)) {} else total: 1 errors, 1 warnings, 0 checks, 16 lines checked 2c84afb2e4ab blk: use for_each_if -:28: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #28: FILE: include/linux/blk-cgroup.h:385: + for_each_if (((d_blkg) = __blkg_lookup(css_to_blkcg(pos_css), \ -:37: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #37: FILE: include/linux/blk-cgroup.h:400: + for_each_if (((d_blkg) = __blkg_lookup(css_to_blkcg(pos_css), \ total: 0 errors, 2 warnings, 0 checks, 16 lines checked b11893dc7547 cgroup: use for_each_if -:26: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #26: FILE: include/linux/cgroup.h:290: + for_each_if ((leader) == (leader)->group_leader) total: 0 errors, 1 warnings, 0 checks, 10 lines checked 5f741b77ec38 cpufreq: use for_each_if -:24: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #24: FILE: include/linux/cpufreq.h:652: + for_each_if (pos->frequency != CPUFREQ_ENTRY_INVALID) -:35: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #35: FILE: include/linux/cpufreq.h:664: + for_each_if (pos->frequency == CPUFREQ_ENTRY_INVALID) total: 0 errors, 2 warnings, 0 checks, 20 lines checked 897d7cd7d949 dmar: Use for_each_If -:20: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #20: FILE: include/linux/dmar.h:88: + for_each_if (!drhd->ignored) -:25: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #25: FILE: include/linux/dmar.h:92: + for_each_if ((i=drhd->iommu, !drhd->ignored)) -:25: ERROR:SPACING: spaces required around that '=' (ctx:VxV) #25: FILE: include/linux/dmar.h:92: + for_each_if ((i=drhd->iommu, !drhd->ignored)) ^ -:30: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #30: FILE: include/linux/dmar.h:96: + for_each_if ((i=drhd->iommu, true)) -:30: ERROR:SPACING: spaces required around that '=' (ctx:VxV) #30: FILE: include/linux/dmar.h:96: + for_each_if ((i=drhd->iommu, true)) ^ -:40: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #40: FILE: include/linux/dmar.h:112: + for_each_if (d) total: 2 errors, 4 warnings, 0 checks, 27 lines checked a4033b0182e7 mm: use for_each_if -:33: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #33: FILE: include/linux/mmzone.h:943: + for_each_if (populated_zone(zone)) total: 0 errors, 1 warnings, 0 checks, 10 lines checked 064dc6caec67 ide: use for_each_if -:6: WARNING:TYPO_SPELLING: 'ambigious' may be misspelled - perhaps 'ambiguous'? #6: Avoids complaints from gcc about ambigious else clauses. Not that any -:22: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #22: FILE: include/linux/ide.h:1604: + for_each_if ((dev)->dev_flags & IDE_DFLAG_PRESENT) total: 0 errors, 2 warnings, 0 checks, 8 lines checked 3c045b076644 netdev: use for_each_if -:21: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #21: FILE: include/linux/netdevice.h:2479: + for_each_if (netdev_master_upper_dev_get_rcu(slave) == (bond)) total: 0 errors, 1 warnings, 0 checks, 8 lines checked 5fe76dc59d48 nubus: use for_each_if -:21: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #21: FILE: include/linux/nubus.h:130: + for_each_func_rsrc(f) for_each_if (f->board == b) total: 0 errors, 1 warnings, 0 checks, 8 lines checked 25bfbbe13776 pci: use for_each_if -:21: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #21: FILE: include/linux/pci.h:604: + for_each_if (pci_is_bridge(dev)) total: 0 errors, 1 warnings, 0 checks, 8 lines checked a139628ea314 sched: use for_each_if in topology.h -:21: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #21: FILE: include/linux/topology.h:43: + for_each_if (nr_cpus_node(node)) total: 0 errors, 1 warnings, 0 checks, 8 lines checked 6d05c728cf9f usb: use for_each_if -:21: WARNING:SPACING: space prohibited between function name and open parenthesis '(' #21: FILE: include/linux/usb.h:735: + for_each_if (child) total: 0 errors, 1 warnings, 0 checks, 8 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
