[PATCH 2/2] ext4: remove unused variable

Signed-off-by: Eric Engestrom --- fs/ext4/page-io.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/fs/ext4/page-io.c b/fs/ext4/page-io.c index a6132a7..b4cbee9 100644 --- a/fs/ext4/page-io.c +++ b/fs/ext4/page-io.c @@ -405,14 +405,12 @@ int ext4_bio_write_page(struct

[PATCH 1/2] ext4: remove unused variable

Signed-off-by: Eric Engestrom --- fs/ext4/dir.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/fs/ext4/dir.c b/fs/ext4/dir.c index 67415e0..acd76f1 100644 --- a/fs/ext4/dir.c +++ b/fs/ext4/dir.c @@ -627,7 +627,7 @@ int ext4_check_all_de(struct inode *dir, struct

[PATCH] debugfs: remove unused variable

Signed-off-by: Eric Engestrom --- fs/debugfs/file.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/fs/debugfs/file.c b/fs/debugfs/file.c index 592059f..04eca0b 100644 --- a/fs/debugfs/file.c +++ b/fs/debugfs/file.c @@ -195,7 +195,6 @@ static int full_proxy_release(struct

[PATCH] rcu: add missing prototype

`make W=1` warns: kernel/rcu/tree.c:226:15: warning: no previous prototype for ‘rcu_rnp_online_cpus’ [-Wmissing-prototypes] unsigned long rcu_rnp_online_cpus(struct rcu_node *rnp) ^~~ Signed-off-by: Eric Engestrom --- I couldn't figure out how this li

Re: [PATCH] debugfs: remove unused variable

On Wed, Sep 21, 2016 at 10:01:11AM +0200, Greg Kroah-Hartman wrote: > On Tue, Sep 20, 2016 at 05:17:15PM +0100, Eric Engestrom wrote: > > Signed-off-by: Eric Engestrom > > --- > > fs/debugfs/file.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) >

[PATCH] debugfs: propagate release() call result

Signed-off-by: Eric Engestrom --- fs/debugfs/file.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/debugfs/file.c b/fs/debugfs/file.c index 592059f..40e586f 100644 --- a/fs/debugfs/file.c +++ b/fs/debugfs/file.c @@ -209,7 +209,7 @@ static int full_proxy_release(struct

Re: [PATCH] debugfs: remove unused variable

On Wed, Sep 21, 2016 at 10:50:38AM +0200, Nicolai Stange wrote: > Greg Kroah-Hartman writes: > > > On Tue, Sep 20, 2016 at 05:17:15PM +0100, Eric Engestrom wrote: > >> Signed-off-by: Eric Engestrom > >> --- > >> fs/debugfs/file.c | 3 +-- > >&

[PATCH v2] debugfs: propagate release() call result

The result was being ignored and 0 was always returned. Return the actual result instead. Signed-off-by: Eric Engestrom --- v2: add a commit message --- fs/debugfs/file.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/debugfs/file.c b/fs/debugfs/file.c index 592059f

Re: [PATCH] video: fbdev: via_aux_vt1636: remove VLA usage

On Wednesday, 2018-03-07 13:47:03 -0600, Gustavo A. R. Silva wrote: > In preparation to enabling -Wvla, remove VLA and replace it > with a fixed-length array instead. Also, remove variable 'len'. > > Notice that no new IDs have been added in seven years. > > Signed-off-by: Gustavo A. R. Silva >

Re: [PATCH] video: fbdev: via: remove VLA usage

Sorry for my reply on v1, I missed that Emil already replied to another post of the same patch (side note on that, making use of `-vX` and `--in-reply-to` helps track the evolution of patches) This looks good to me: Reviewed-by: Eric Engestrom > --- > drivers/video/fbdev/via/via_aux_sii164.c

[trivial PATCH] Documentation/sparse: fix typo

Signed-off-by: Eric Engestrom --- Documentation/dev-tools/sparse.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/dev-tools/sparse.rst b/Documentation/dev-tools/sparse.rst index 78aa00a604a009071361..c401c952a340a50fa769 100644 --- a/Documentation/dev-tools

Re: [PATCH] drm/msm: Move call to PTR_ERR_OR_ZERO after reassignment

+ pr_err("Could not get lvl-vdd reg, %ld", ret); > ctrl->lvl_vreg = NULL; > - return PTR_ERR(ctrl->lvl_vreg); > + return ret; > } > > return 0; > -- > 2.1.4 Reviewed-by: Eric Engestrom

[PATCH] Fix CONFIG_PM_OPP without CONFIG_OF build failure

From: Rufus Hamade A few `#ifdef CONFIG_OF` were missing or misplaced, resulting in a few "unused function" warnings in core.c, and preventing the build of cpu.c because of the redefinition of `dev_pm_opp_set_sharing_cpus()`. Signed-off-by: Rufus Hamade Reviewed-by: Eric Engestrom

[PATCH v2] Fix CONFIG_PM_OPP without CONFIG_OF build failure

From: Rufus Hamade A few `#ifdef CONFIG_OF` were missing, resulting in "unused function" warnings in core.c. Signed-off-by: Rufus Hamade Reviewed-by: Eric Engestrom Tested-by: Eric Engestrom --- v2: drop incorrect change in cpu.c (Viresh Kumar) On Thu, May 05, 2016 at 08:31:

Re: [PATCH] drm/amd/powerplay: use ARRAY_SIZE() for size of array

On Wed, May 11, 2016 at 11:18:43PM +0530, Muhammad Falak R Wani wrote: > Use ARRAY_SIZE() for the size calculation of the array. Also move the > condition evaulation function out of the for loop. > Although, any respectable c-compiler would optimize this and evaluate > the function only once outsid

Re: [PATCH] drm/mm: avoid possible null pointer dereference

->start + node->size; > + > /* Find the relevant hole to add our node to */ > drm_mm_for_each_hole(hole, mm, hole_start, hole_end) { > if (hole_start > node->start || hole_end < end) > -- > 2.1.4 Reviewed-by: Eric Engestrom

Re: [PATCH 1/1] drm/mediatek: do not dereference NULL

(struct mtk_dpi *dpi, > unsigned int factor; > > if (!dpi) { > - dev_err(dpi->dev, "invalid argument\n"); > + dev_err(NULL, "invalid argument\n"); > return -EINVAL; > } > > -- > 2.1.4 Seems obvious ^^ Reviewed-by: Eric Engestrom

Re: [PATCH] dma-buf: use vma_pages().

With that fixed: Reviewed-by: Eric Engestrom

Re: [PATCH v2] dma-buf: use vma_pages().

On Mon, May 23, 2016 at 05:08:42PM +0530, Muhammad Falak R Wani wrote: > Replace explicit computation of vma page count by a call to > vma_pages(). > Also, include All good! Reviewed-by: Eric Engestrom

Re: [PATCH] drm/mm: avoid possible null pointer dereference

On Mon, May 23, 2016 at 02:02:26PM +0100, Chris Wilson wrote: > On Mon, May 23, 2016 at 02:56:45PM +0200, Daniel Vetter wrote: > > On Mon, May 23, 2016 at 11:27:14AM +0100, Eric Engestrom wrote: > > > On Wed, May 18, 2016 at 10:17:19PM +0200, Heinrich Schuchardt wrote: > >

Re: [PATCH] drm/mm: avoid possible null pointer dereference

On Mon, May 23, 2016 at 02:46:59PM +0100, Chris Wilson wrote: > On Mon, May 23, 2016 at 02:38:29PM +0100, Eric Engestrom wrote: > > On Mon, May 23, 2016 at 02:02:26PM +0100, Chris Wilson wrote: > > > On Mon, May 23, 2016 at 02:56:45PM +0200, Daniel Vetter wrote: > > > &

Re: [PATCH v2] drm/amd/powerplay: use ARRAY_SIZE() to calculate array size.

On Fri, May 13, 2016 at 11:06:40PM +0530, Muhammad Falak R Wani wrote: > It is preferred to use ARRAY_SIZE() for size calculation, instead > using sizeof(array)/sizeof(*array). It makes the code more readable. > > Signed-off-by: Muhammad Falak R Wani Reviewed-by: Eric Engestrom

[PATCH] drm: Remove duplicate `const` qualifier

Commit 1eb8345 added a bunch of legitimate `const`, but added two on that line. Signed-off-by: Eric Engestrom --- drivers/gpu/drm/drm_fb_cma_helper.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/drm_fb_cma_helper.c b/drivers/gpu/drm/drm_fb_cma_helper.c

[PATCH 2/2] drm/i915: remove dead code

79e539453b34e35f39299a899d263b0a1f1670bd ("DRM: i915: add mode setting support") added those variables but never used them. Signed-off-by: Eric Engestrom --- drivers/gpu/drm/i915/intel_tv.c | 12 1 file changed, 12 deletions(-) diff --git a/drivers/gpu/drm/i915/in

[PATCH 1/2] drm/i915: remove left over dead code

ae80152ddad252f33893b92dd69f00cc53c5949f ("drm/i915: Rewrite VLV/CHV watermark code") removed everything that would have used those vars. Signed-off-by: Eric Engestrom --- drivers/gpu/drm/i915/intel_pm.c | 14 -- 1 file changed, 14 deletions(-) diff --git a/drivers/gp

[PATCH] acpi: remove left over dead code

39fe394d05be43481ceac8b3db19dfd5189097f6 ("ACPI video: remove deprecated procfs I/F") removed everything that would've used this variable. Signed-off-by: Eric Engestrom --- drivers/acpi/acpi_video.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/drivers/acpi/acpi_vi

[PATCH] procfs: add conditional compilation check

`proc_timers_operations` is only used when CONFIG_CHECKPOINT_RESTORE is enabled. Signed-off-by: Eric Engestrom --- fs/proc/base.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/fs/proc/base.c b/fs/proc/base.c index 4f764c2..28fc7a3 100644 --- a/fs/proc/base.c +++ b/fs/proc/base.c

[PATCH] net-sysfs: remove left over dead code

This format hasn't been used since 04ed3e741d0f133e02bed7fa5c98edba128f90e7 ("net: change netdev->features to u32") Signed-off-by: Eric Engestrom --- net/core/net-sysfs.c | 1 - 1 file changed, 1 deletion(-) diff --git a/net/core/net-sysfs.c b/net/core/net-sysfs.c index b6c8a

[PATCH] net/rtnetlink: remove dead code

3b766cd832328fcb87db3507e7b98cf42f21689d ("net/core: Add reading VF statistics through the PF netdevice") added that variable but it's never been used. Signed-off-by: Eric Engestrom --- net/core/rtnetlink.c | 9 - 1 file changed, 9 deletions(-) diff --git a/net/core/rt

[PATCH] net/ipv4: remove left over dead code

8cc785f6f429c2a3fb81745dc142cbd72a462c4a ("net: ipv4: make the ping /proc code AF-independent") removed the code using it, but renamed this variable instead of removing it. Signed-off-by: Eric Engestrom --- net/ipv4/ping.c | 7 --- 1 file changed, 7 deletions(-) diff --git

[PATCH] ethernet/atl1c: remove left over dead code

Left over from c24588afc536a35c924d014f13b669b20ccf8553 ("atl1c: using fixed TXQ configuration for l2cb and l1c") Signed-off-by: Eric Engestrom --- drivers/net/ethernet/atheros/atl1c/atl1c_main.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/net/ethernet/ath

[PATCH 00/41] Documentation: fix spelling mistakes

as well? Those patches can be applied independently (obviously), so there is no need to wait for the whole series to be ack'ed. Cheers, Eric [1] https://github.com/lucasdemarchi/codespell Eric Engestrom (41): Documentation: debugfs: fix spelling mistake Documentation: sysfs: fix spell

[PATCH 01/41] Documentation: debugfs: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/ABI/testing/debugfs-driver-genwqe | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/ABI/testing/debugfs-driver-genwqe b/Documentation/ABI/testing/debugfs-driver-genwqe index 1c2f256..b45b016 100644 --- a

[PATCH 02/41] Documentation: sysfs: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/ABI/testing/sysfs-bus-coresight-devices-etm4x | 2 +- Documentation/ABI/testing/sysfs-bus-event_source-devices-events | 4 ++-- Documentation/ABI/testing/sysfs-bus-event_source-devices-hv_24x7 | 2 +- Documentation/ABI/testing/sysfs-driver

[PATCH 08/41] Documentation: dt: arm: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/devicetree/bindings/arm/cci.txt| 2 +- Documentation/devicetree/bindings/arm/spear-misc.txt | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/arm/cci.txt b/Documentation/devicetree

[PATCH 07/41] Documentation: dt: arc: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/devicetree/bindings/arc/archs-pct.txt | 2 +- Documentation/devicetree/bindings/arc/pct.txt | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/arc/archs-pct.txt b/Documentation/devicetree

[PATCH 03/41] Documentation: arm: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/arm/Atmel/README | 2 +- Documentation/arm/keystone/knav-qmss.txt | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/arm/Atmel/README b/Documentation/arm/Atmel/README index 0931cf7..adffac7 100644 --- a

[PATCH 05/41] Documentation: cpu-hotplug: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/cpu-hotplug.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/cpu-hotplug.txt b/Documentation/cpu-hotplug.txt index dd68821..12e0b8f 100644 --- a/Documentation/cpu-hotplug.txt +++ b/Documentation/cpu-hotplug.txt

[PATCH 06/41] Documentation: device-mapper: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/device-mapper/cache-policies.txt | 4 ++-- Documentation/device-mapper/statistics.txt | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Documentation/device-mapper/cache-policies.txt b/Documentation/device-mapper/cache

[PATCH 04/41] Documentation: cgroup: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/cgroup-v2.txt | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/Documentation/cgroup-v2.txt b/Documentation/cgroup-v2.txt index 4cc07ce..c836b69 100644 --- a/Documentation/cgroup-v2.txt +++ b/Documentation/cgroup

[PATCH 10/41] Documentation: dt: display: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/devicetree/bindings/display/exynos/exynos_dsim.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/display/exynos/exynos_dsim.txt b/Documentation/devicetree/bindings/display/exynos

[PATCH 09/41] Documentation: dt: clock: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/devicetree/bindings/clock/rockchip,rk3188-cru.txt | 2 +- Documentation/devicetree/bindings/clock/rockchip,rk3288-cru.txt | 2 +- Documentation/devicetree/bindings/clock/st/st,clkgen.txt| 2 +- 3 files changed, 3 insertions(+), 3 deletions

[PATCH 12/41] Documentation: dt: input: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/devicetree/bindings/input/ads7846.txt | 2 +- Documentation/devicetree/bindings/input/touchscreen/fsl-mx25-tcq.txt | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/input

[PATCH 11/41] Documentation: dt: dma: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/devicetree/bindings/dma/xilinx/xilinx_dma.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/dma/xilinx/xilinx_dma.txt b/Documentation/devicetree/bindings/dma/xilinx/xilinx_dma.txt index

[PATCH 13/41] Documentation: dt: interrupt-controller: fix spelling mistakes

Signed-off-by: Eric Engestrom --- .../devicetree/bindings/interrupt-controller/ti,omap4-wugen-mpu | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/interrupt-controller/ti,omap4-wugen-mpu b/Documentation/devicetree/bindings

[PATCH 18/41] Documentation: dt: net: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/devicetree/bindings/net/hisilicon-hns-nic.txt | 2 +- Documentation/devicetree/bindings/net/stmmac.txt| 4 ++-- Documentation/devicetree/bindings/net/ti,dp83867.txt| 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff

[PATCH 17/41] Documentation: dt: mtd: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/devicetree/bindings/mtd/brcm,brcmnand.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/mtd/brcm,brcmnand.txt b/Documentation/devicetree/bindings/mtd/brcm,brcmnand.txt index c2546ce..0f6985b

[PATCH 14/41] Documentation: dt: media: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/devicetree/bindings/media/xilinx/video.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/media/xilinx/video.txt b/Documentation/devicetree/bindings/media/xilinx/video.txt index cbd46fa

[PATCH 15/41] Documentation: dt: mfd: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/devicetree/bindings/mfd/qcom-rpm.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/mfd/qcom-rpm.txt b/Documentation/devicetree/bindings/mfd/qcom-rpm.txt index 5e97a95..b98b291 100644

[PATCH 16/41] Documentation: dt: mmc: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.txt b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.txt index 0cb827b

[PATCH 19/41] Documentation: dt: opp: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/devicetree/bindings/opp/opp.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/opp/opp.txt b/Documentation/devicetree/bindings/opp/opp.txt index 601256f..ee91cbd 100644 --- a/Documentation

[PATCH 22/41] Documentation: dt: soc: fix spelling mistakes

Signed-off-by: Eric Engestrom --- .../devicetree/bindings/soc/ti/keystone-navigator-qmss.txt | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Documentation/devicetree/bindings/soc/ti/keystone-navigator-qmss.txt b/Documentation/devicetree/bindings/soc/ti

[PATCH 21/41] Documentation: dt: power: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/devicetree/bindings/power/qcom,coincell-charger.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/power/qcom,coincell-charger.txt b/Documentation/devicetree/bindings/power/qcom,coincell

[PATCH 20/41] Documentation: dt: pinctrl: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.txt b/Documentation/devicetree/bindings/pinctrl/qcom,pmic-gpio.txt index

[PATCH 24/41] Documentation: fb: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/fb/udlfb.txt | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Documentation/fb/udlfb.txt b/Documentation/fb/udlfb.txt index 57d2f29..c985cb6 100644 --- a/Documentation/fb/udlfb.txt +++ b/Documentation/fb/udlfb.txt @@ -9,7

[PATCH 26/41] Documentation: kasan: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/kasan.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/kasan.txt b/Documentation/kasan.txt index 7dd95b3..9c696e4 100644 --- a/Documentation/kasan.txt +++ b/Documentation/kasan.txt @@ -116,7 +116,7 @@ Memory

[PATCH 25/41] Documentation: filesystems: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/filesystems/autofs4.txt | 6 +++--- Documentation/filesystems/cifs/CHANGES | 2 +- Documentation/filesystems/proc.txt | 4 ++-- Documentation/filesystems/vfs.txt | 2 +- 4 files changed, 7 insertions(+), 7 deletions(-) diff --git a

[PATCH 23/41] Documentation: DocBook: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/DocBook/media/dvb/net.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/DocBook/media/dvb/net.xml b/Documentation/DocBook/media/dvb/net.xml index d2e44b7..da095ed 100644 --- a/Documentation/DocBook/media/dvb

[PATCH 28/41] Documentation: locking: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/locking/lockdep-design.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/locking/lockdep-design.txt b/Documentation/locking/lockdep-design.txt index 5001280..9de1c15 100644 --- a/Documentation/locking

[PATCH 37/41] Documentation: video4linux: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/video4linux/vivid.txt | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Documentation/video4linux/vivid.txt b/Documentation/video4linux/vivid.txt index e35d376..8da5d2a 100644 --- a/Documentation/video4linux/vivid.txt +++ b

[PATCH 41/41] Documentation: xillybus: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/xillybus.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/xillybus.txt b/Documentation/xillybus.txt index 81d111b..1660145 100644 --- a/Documentation/xillybus.txt +++ b/Documentation/xillybus.txt @@ -215,7

[PATCH 38/41] Documentation: virtual: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/virtual/kvm/devices/s390_flic.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/virtual/kvm/devices/s390_flic.txt b/Documentation/virtual/kvm/devices/s390_flic.txt index e3e314c..13a1261 100644 --- a

[PATCH 39/41] Documentation: vm: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/vm/transhuge.txt | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Documentation/vm/transhuge.txt b/Documentation/vm/transhuge.txt index d9cb65c..2f383f6 100644 --- a/Documentation/vm/transhuge.txt +++ b/Documentation/vm

[PATCH 40/41] Documentation: x86: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/x86/intel_mpx.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/x86/intel_mpx.txt b/Documentation/x86/intel_mpx.txt index 818518a..1a5a121 100644 --- a/Documentation/x86/intel_mpx.txt +++ b/Documentation/x86

[PATCH 33/41] Documentation: RCU: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/RCU/stallwarn.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/RCU/stallwarn.txt b/Documentation/RCU/stallwarn.txt index 0f7fb42..e93d041 100644 --- a/Documentation/RCU/stallwarn.txt +++ b/Documentation/RCU

[PATCH 36/41] Documentation: sound: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/sound/alsa/compress_offload.txt | 2 +- Documentation/sound/alsa/soc/dapm.txt | 2 +- Documentation/sound/alsa/soc/overview.txt | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Documentation/sound/alsa

[PATCH 34/41] Documentation: robust-futexes: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/robust-futexes.txt | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Documentation/robust-futexes.txt b/Documentation/robust-futexes.txt index af6fce2..61c22d6 100644 --- a/Documentation/robust-futexes.txt +++ b

[PATCH 35/41] Documentation: scsi: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/scsi/ChangeLog.megaraid_sas | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/Documentation/scsi/ChangeLog.megaraid_sas b/Documentation/scsi/ChangeLog.megaraid_sas index 18b5709..20dcf20 100644 --- a

[PATCH 32/41] Documentation: pps: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/pps/pps.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/pps/pps.txt b/Documentation/pps/pps.txt index 7cb7264..50022b3 100644 --- a/Documentation/pps/pps.txt +++ b/Documentation/pps/pps.txt @@ -98,7 +98,7

[PATCH 30/41] Documentation: networking: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/networking/altera_tse.txt | 6 +++--- Documentation/networking/ipvlan.txt | 6 +++--- Documentation/networking/pktgen.txt | 6 +++--- Documentation/networking/vrf.txt| 2 +- Documentation/networking/xfrm_sync.txt | 6 +++--- 5

[PATCH 27/41] Documentation: laptops: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/laptops/toshiba_haps.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/laptops/toshiba_haps.txt b/Documentation/laptops/toshiba_haps.txt index 11dbcfd..0c1d88d 100644 --- a/Documentation/laptops/toshiba_haps.txt

[PATCH 31/41] Documentation: power: fix spelling mistake

Signed-off-by: Eric Engestrom --- Documentation/power/opp.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/power/opp.txt b/Documentation/power/opp.txt index c6279c2..1721a86 100644 --- a/Documentation/power/opp.txt +++ b/Documentation/power/opp.txt @@ -146,7

[PATCH 29/41] Documentation: lzo: fix spelling mistakes

Signed-off-by: Eric Engestrom --- Documentation/lzo.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Documentation/lzo.txt b/Documentation/lzo.txt index ea45dd3..285c54f 100644 --- a/Documentation/lzo.txt +++ b/Documentation/lzo.txt @@ -69,9 +69,9 @@ Description

Re: [patch] drm: atmel-hlcdc: fix a NULL check

On Mon, Apr 25, 2016 at 12:04:54PM +0300, Dan Carpenter wrote: > If kmalloc() returned NULL we would end up dereferencing "state" a > couple lines later. > > Signed-off-by: Dan Carpenter Reviewed-by: Eric Engestrom

[PATCH 1/4] perf tools: remove duplicate const qualifier

Signed-off-by: Eric Engestrom --- tools/perf/util/thread.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/perf/util/thread.c b/tools/perf/util/thread.c index dfd00c6..de2036d 100644 --- a/tools/perf/util/thread.c +++ b/tools/perf/util/thread.c @@ -233,7 +233,7 @@ void

[PATCH 4/4] ARM: remove duplicate const qualifier

Signed-off-by: Eric Engestrom --- arch/arm/mach-at91/pm.c | 2 +- arch/arm/mach-bcm/bcm_kona_smc.c| 2 +- arch/arm/mach-clps711x/board-autcpu12.c | 2 +- arch/arm/mach-cns3xxx/core.c| 2 +- arch/arm/mach-davinci/da8xx-dt.c| 2 +- arch/arm/mach-lpc32xx

[PATCH 3/4] dmaengine: pxa_dma: remove duplicate const qualifier

Signed-off-by: Eric Engestrom --- drivers/dma/pxa_dma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dma/pxa_dma.c b/drivers/dma/pxa_dma.c index 77c1c44..315901d 100644 --- a/drivers/dma/pxa_dma.c +++ b/drivers/dma/pxa_dma.c @@ -1321,7 +1321,7 @@ static int

[PATCH 2/4] drivers: firmware: psci: remove duplicate const qualifier

Signed-off-by: Eric Engestrom --- drivers/firmware/psci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/firmware/psci.c b/drivers/firmware/psci.c index 11bfee8..0ab477ba 100644 --- a/drivers/firmware/psci.c +++ b/drivers/firmware/psci.c @@ -563,7 +563,7 @@ static

[PATCH] MAINTAINERS: remove Rajendra Nayak

I'm guessing he doesn't work there anymore: : host cluster5.us.messagelabs.com[216.82.242.147] said: 550 Invalid recipient (#5.1.1) (in reply to RCPT TO command) Signed-off-by: Eric Engestrom --- This needs a confirmation from someone at TI --- MAINTAINERS | 1 - 1 file changed,

Re: [PATCH 4/4] ARM: remove duplicate const qualifier

On Mon, Apr 25, 2016 at 11:57:15AM +0200, Alexandre Belloni wrote: > Hi, > > this requires a commit message. OK? This seems rather pointless in this case (and even more so for the typo fix I sent yesterday), but I guess you have some general rule to enforce. Would this do then? "The second `const

Re: [PATCH 4/4] ARM: remove duplicate const qualifier

On Mon, Apr 25, 2016 at 01:38:18PM +0200, Arnd Bergmann wrote: > Maybe say you you found it (llvm, sparse, coccinelle?), and why this > is causing a problem for anyone. If it's just unnecessary but not > harmful, I'd probably ignore the patch. $ grep -rE '(^|\W)const(\s+\w+)+\s+const\s' I just ha

Re: [PATCH 4/4] ARM: remove duplicate const qualifier

On Mon, Apr 25, 2016 at 10:12:16AM -0400, Nicolas Pitre wrote: > Beware. > > I added many of those exactly because gcc did not ignore them when > compiling with LTO where the lack of a const qualifier to qualify the > actual array content, and not only the reference to that content, > generated

[PATCH v2] MAINTAINERS: fix Rajendra Nayak's address

Signed-off-by: Eric Engestrom --- v2: replace with new address instead of just removing the old one --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 1d5b4be..81ebdab 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -7978,7 +7978,7

[PATCH] MAINTAINERS: remove defunct spear mailing list

It looks like the email address for this mailing list doesn't exist anymore: : host mxb-00178001.gslb.pphosted.com[91.207.212.93] said: 550 5.1.1 User Unknown (in reply to RCPT TO command) Signed-off-by: Eric Engestrom --- Maybe it should be replaced with an updated address? In the mean

[RFC] Add missing #include

ke and was forgotten, I can send an actual patchset to fix this, but it feels like it was left out on purpose. Cheers, Eric Engestrom PS: If this is a cleanup that needs to be done, I might to it for other headers as well. --- include/linux/compiler-gcc.h | 2 ++ 1 file changed, 2 insertions(+) di

Re: [PATCH v4] drm: add fourcc codes for 16bit R and RG

On Wednesday, 2017-01-04 14:50:02 +0100, Rainer Hochecker wrote: > From: Rainer Hochecker > > Signed-off-by: Rainer Hochecker > --- > include/uapi/drm/drm_fourcc.h | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/include/uapi/drm/drm_fourcc.h b/include/uapi/drm/drm_fourcc.h > in

Re: [PATCH v2 1/7] lib: string: add functions to case-convert strings

On Tue, Jul 05, 2016 at 01:47:05PM -0700, Markus Mayer wrote: > All functions return a pointer to the terminating '\0' character in the > modified string ("dst" or "s", respectively). I think this is going to be confusing. From the man: The strcpy() and strncpy() functions return a pointe

[PATCH 1/2] libnvdimm: add missing macros

Signed-off-by: Eric Engestrom --- This can't compile without these macros… Is this header really used by anyone? Should it be removed, to avoid bit-rot? --- include/uapi/linux/ndctl.h | 8 1 file changed, 8 insertions(+) diff --git a/include/uapi/linux/ndctl.h b/include/uapi/

[PATCH 2/2] netfilter: add missing macro

Signed-off-by: Eric Engestrom --- This can't compile without this macro… Is this header really used by anyone? Should it be removed, to avoid bit-rot? --- include/uapi/linux/netfilter/xt_sctp.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/uapi/linux/netfilter/xt_sctp

Re: [PATCH v5 3/3] MAINTAINERS: Add entry for Mali-DP driver

On Wed, Jun 15, 2016 at 03:51:35PM +0100, Liviu Dudau wrote: > Add MAINTAINERS entry for ARM Mali-DP driver and update the > HDLCD file matching pattern to cover only HDLCD rather than > the whole drivers/gpu/drm/arm directory. > > Signed-off-by: Liviu Dudau > --- > MAINTAINERS | 10 +- >

Re: [PATCH v2] drm: move allocation out of drm_get_format_name()

On Monday, 2016-11-07 10:10:13 +0200, Jani Nikula wrote: > On Mon, 07 Nov 2016, Eric Engestrom wrote: > > Fixes: 90844f00049e9f42573fd31d7c32e8fd31d3fd07 > > > > drm: make drm_get_format_name thread-safe > > > > Signed-off-by: Eric Engestrom > >

[PATCH v3] drm: move allocation out of drm_get_format_name()

le Syrjälä Signed-off-by: Eric Engestrom --- v3 - fix "Fixes" tag, replace it with an actual commit message - collect ack & r-b v2 - use single-field struct instead of typedef to let the compiler enforce the type (Christian König) --- include/drm/drm_fourcc.h

Re: [Intel-gfx] [PATCH v3] drm: move allocation out of drm_get_format_name()

On Wednesday, 2016-11-09 02:13:25 +0100, Daniel Vetter wrote: > On Wed, Nov 09, 2016 at 02:09:16AM +0100, Daniel Vetter wrote: > > On Wed, Nov 09, 2016 at 12:17:52AM +0000, Eric Engestrom wrote: > > > The function's behaviour was changed in 90844f00049e, without chang

Re: [Intel-gfx] [PATCH v3] drm: move allocation out of drm_get_format_name()

On Wednesday, 2016-11-09 14:13:40 +0100, Daniel Vetter wrote: > On Wed, Nov 9, 2016 at 12:42 PM, Eric Engestrom > wrote: > >> Well, had to drop it again since it didn't compile: > >> > >> > >> CC [M] drivers/gpu/drm/drm_blend.o

Re: [patch] drm/i915: fix a read size argument

fe5a66f91c88 ("drm/i915: Read PSR caps/intermediate freqs/etc. only > once on eDP") > Signed-off-by: Dan Carpenter Good catch! What tool did you use to find it, or did you find it by inspection? Reviewed-by: Eric Engestrom (btw, there's a missing `---` here, between the

Re: [PATCH] drm/atomic: fix memory leak when fetching format name

On Friday, 2016-11-11 16:26:22 +, Colin King wrote: > From: Colin Ian King > > drm_get_format_name allocates memory that is not currently free'd > when printing the state. Fix this by kfree'ing the memory after > use. You are correct, but there are more cases of this, and another fix has bee

[PATCH v4] drm: move allocation out of drm_get_format_name()

le Syrjälä Signed-off-by: Eric Engestrom --- v4 - rebase on drm-next (d8c1abd968f1c880ad8c) - add missed `const` to the return value v3 - fix "Fixes" tag, replace it with an actual commit message - collect ack & r-b v2 - use single-field struct instead of typedef to let the co

Re: [PATCH] drm: add fourcc codes for 16bit R and GR

On Tuesday, 2017-01-03 17:56:10 +0100, Rainer Hochecker wrote: > On Mon, Jan 2, 2017 at 3:31 PM, Rainer Hochecker wrote: > > > > I chose GR16 because that matches with Mesa texture formats. Unfortunately > > RG16 is already taken by DRM_FORMAT_RGB565 > > So GR32 / RG32 might be better. All other c

Re: [Intel-gfx] [PATCH v2] drm: add fourcc codes for 16bit R and GR

On Wednesday, 2017-01-04 11:06:09 +0200, Jani Nikula wrote: > On Wed, 04 Jan 2017, Daniel Vetter wrote: > > On Tue, Jan 03, 2017 at 08:02:07PM +0100, Rainer Hochecker wrote: > >> From: Rainer Hochecker > >> > >> Now sent with git send-email: > >> > >> Signed-off-by: Rainer Hochecker > >> --- >

[PATCH] drm/i915/gvt: fix deadlock in dispatch_workload()'s error path

90d27a1 moved the lock before this error path but forgot to add an unlock here. Fixes: 90d27a1b180e51ef0713 ("drm/i915/gvt: fix deadlock in workload_thread") Cc: Pei Zhang Cc: Zhenyu Wang Signed-off-by: Eric Engestrom --- drivers/gpu/drm/i915/gvt/scheduler.c | 1 + 1 file changed, 1

  1   2   >