[PATCH] KVM: PPC: drop duplicate header asm/iommu.h

2016-11-18 Thread Geliang Tang
Drop duplicate header asm/iommu.h from book3s_64_vio_hv.c. Signed-off-by: Geliang Tang --- arch/powerpc/kvm/book3s_64_vio_hv.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/powerpc/kvm/book3s_64_vio_hv.c b/arch/powerpc/kvm/book3s_64_vio_hv.c index d461c44..e4c4ea9 100644 --- a/arch

[PATCH] skd_main: drop duplicate header scatterlist.h

2016-11-18 Thread Geliang Tang
Drop duplicate header scatterlist.h from skd_main.c. Signed-off-by: Geliang Tang --- drivers/block/skd_main.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/block/skd_main.c b/drivers/block/skd_main.c index 1e536b9..abf805e 100644 --- a/drivers/block/skd_main.c +++ b/drivers/block

[PATCH] f2fs: drop duplicate header timer.h

2016-11-18 Thread Geliang Tang
Drop duplicate header timer.h from segment.c. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- fs/f2fs/segment.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c index 82c3ac4..98e5be4 100644 --- a/fs/f2fs/segment.c +++ b/fs/f2fs/segment.c @@

[PATCH] f2fs: drop duplicate header timer.h

2016-11-18 Thread Geliang Tang
Drop duplicate header timer.h from segment.c. Signed-off-by: Geliang Tang --- fs/f2fs/segment.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c index 82c3ac4..98e5be4 100644 --- a/fs/f2fs/segment.c +++ b/fs/f2fs/segment.c @@ -16,7 +16,6 @@ #include

[PATCH] ARM: lpc32xx: drop duplicate header device.h

2016-11-18 Thread Geliang Tang
Drop duplicate header device.h from phy3250.c. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- arch/arm/mach-lpc32xx/phy3250.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-lpc32xx/phy3250.c b/arch/arm/mach-lpc32xx/phy3250.c index 0e4cbbe..6c52bd3 100644 --- a/ar

[PATCH] ARM: lpc32xx: drop duplicate header device.h

2016-11-18 Thread Geliang Tang
Drop duplicate header device.h from phy3250.c. Signed-off-by: Geliang Tang --- arch/arm/mach-lpc32xx/phy3250.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-lpc32xx/phy3250.c b/arch/arm/mach-lpc32xx/phy3250.c index 0e4cbbe..6c52bd3 100644 --- a/arch/arm/mach-lpc32xx/phy3250.c

[PATCH] Blackfin arch: drop duplicate header seq_file.h

2016-11-18 Thread Geliang Tang
Drop duplicate header seq_file.h from irqchip.c. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- arch/blackfin/kernel/irqchip.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/blackfin/kernel/irqchip.c b/arch/blackfin/kernel/irqchip.c index 052cde5..39a5a7d 100644 ---

[PATCH] Blackfin arch: drop duplicate header seq_file.h

2016-11-18 Thread Geliang Tang
Drop duplicate header seq_file.h from irqchip.c. Signed-off-by: Geliang Tang --- arch/blackfin/kernel/irqchip.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/blackfin/kernel/irqchip.c b/arch/blackfin/kernel/irqchip.c index 052cde5..39a5a7d 100644 --- a/arch/blackfin/kernel/irqchip.c

[PATCH] ARM: ixp4xx: drop duplicate header gpio.h

2016-11-18 Thread Geliang Tang
Drop duplicate header gpio.h from dsmg600-setup.c. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- arch/arm/mach-ixp4xx/dsmg600-setup.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-ixp4xx/dsmg600-setup.c b/arch/arm/mach-ixp4xx/dsmg600-setup.c index 43ee06d..b

[PATCH] ARM: ixp4xx: drop duplicate header gpio.h

2016-11-18 Thread Geliang Tang
Drop duplicate header gpio.h from dsmg600-setup.c. Signed-off-by: Geliang Tang --- arch/arm/mach-ixp4xx/dsmg600-setup.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/mach-ixp4xx/dsmg600-setup.c b/arch/arm/mach-ixp4xx/dsmg600-setup.c index 43ee06d..b3bd0e1 100644 --- a/arch/arm

[PATCH] gpio: mb86s7x: use builtin_platform_driver

2016-11-18 Thread Geliang Tang
Use builtin_platform_driver() helper to simplify the code. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/gpio/gpio-mb86s7x.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/gpio/gpio-mb86s7x.c b/drivers/gpio/gpio-mb86s7x.c index d55af50..f

[PATCH] pinctrl: stm32: use builtin_platform_driver

2016-11-18 Thread Geliang Tang
Use builtin_platform_driver() helper to simplify the code. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/pinctrl/stm32/pinctrl-stm32f429.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/pinctrl/stm32/pinctrl-stm32f429.c b/drivers/pinctrl

[PATCH] pinctrl: mediatek: use builtin_platform_driver

2016-11-18 Thread Geliang Tang
Use builtin_platform_driver() helper to simplify the code. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/pinctrl/mediatek/pinctrl-mt6397.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/pinctrl/mediatek/pinctrl-mt6397.c b/drivers/p

[PATCH] dmaengine: mv_xor: use builtin_platform_driver

2016-11-18 Thread Geliang Tang
Use builtin_platform_driver() helper to simplify the code. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/dma/mv_xor.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/dma/mv_xor.c b/drivers/dma/mv_xor.c index 23f7528..1865e3f

[PATCH] pinctrl: mediatek: use builtin_platform_driver

2016-11-18 Thread Geliang Tang
Use builtin_platform_driver() helper to simplify the code. Signed-off-by: Geliang Tang --- drivers/pinctrl/mediatek/pinctrl-mt6397.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/pinctrl/mediatek/pinctrl-mt6397.c b/drivers/pinctrl/mediatek/pinctrl-mt6397.c

[PATCH] dmaengine: mv_xor: use builtin_platform_driver

2016-11-18 Thread Geliang Tang
Use builtin_platform_driver() helper to simplify the code. Signed-off-by: Geliang Tang --- drivers/dma/mv_xor.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/dma/mv_xor.c b/drivers/dma/mv_xor.c index 23f7528..1865e3f 100644 --- a/drivers/dma/mv_xor.c +++ b

[PATCH] gpio: mb86s7x: use builtin_platform_driver

2016-11-18 Thread Geliang Tang
Use builtin_platform_driver() helper to simplify the code. Signed-off-by: Geliang Tang --- drivers/gpio/gpio-mb86s7x.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/gpio/gpio-mb86s7x.c b/drivers/gpio/gpio-mb86s7x.c index d55af50..ffb73f6 100644 --- a/drivers

[PATCH] pinctrl: stm32: use builtin_platform_driver

2016-11-18 Thread Geliang Tang
Use builtin_platform_driver() helper to simplify the code. Signed-off-by: Geliang Tang --- drivers/pinctrl/stm32/pinctrl-stm32f429.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/pinctrl/stm32/pinctrl-stm32f429.c b/drivers/pinctrl/stm32/pinctrl-stm32f429.c

[PATCH] gpio: etraxfs: use builtin_platform_driver

2016-11-18 Thread Geliang Tang
Use builtin_platform_driver() helper to simplify the code. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/gpio/gpio-etraxfs.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/gpio/gpio-etraxfs.c b/drivers/gpio/gpio-etraxfs.c index 0

[PATCH] gpio: etraxfs: use builtin_platform_driver

2016-11-18 Thread Geliang Tang
Use builtin_platform_driver() helper to simplify the code. Signed-off-by: Geliang Tang --- drivers/gpio/gpio-etraxfs.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/gpio/gpio-etraxfs.c b/drivers/gpio/gpio-etraxfs.c index 00b022c..a254d5b 100644 --- a/drivers

[PATCH] ACPI / PMIC: AXP288: use builtin_platform_driver

2016-11-18 Thread Geliang Tang
Use builtin_platform_driver() helper to simplify the code. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/acpi/pmic/intel_pmic_xpower.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/acpi/pmic/intel_pmic_xpower.c b/drivers/acp

[PATCH] ACPI / PMIC: AXP288: use builtin_platform_driver

2016-11-18 Thread Geliang Tang
Use builtin_platform_driver() helper to simplify the code. Signed-off-by: Geliang Tang --- drivers/acpi/pmic/intel_pmic_xpower.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/acpi/pmic/intel_pmic_xpower.c b/drivers/acpi/pmic/intel_pmic_xpower.c index e6e991a

[PATCH] drm: mediatek: use ERR_CAST inlined function

2016-11-18 Thread Geliang Tang
Use ERR_CAST inlined function instead of ERR_PTR(PTR_ERR(...)). Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/gpu/drm/mediatek/mtk_drm_gem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/mediatek/mtk_drm_gem.c b/drivers/gpu/drm/me

[PATCH] drm: mediatek: use ERR_CAST inlined function

2016-11-18 Thread Geliang Tang
Use ERR_CAST inlined function instead of ERR_PTR(PTR_ERR(...)). Signed-off-by: Geliang Tang --- drivers/gpu/drm/mediatek/mtk_drm_gem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/mediatek/mtk_drm_gem.c b/drivers/gpu/drm/mediatek/mtk_drm_gem.c index

[PATCH] scsi: a100u2w: use module_pci_driver

2016-11-16 Thread Geliang Tang
Use module_pci_driver() helper to simplify the code. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/scsi/a100u2w.c | 13 + 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/drivers/scsi/a100u2w.c b/drivers/scsi/a100u2w.c index 8086bd0..b2942ec

[PATCH] scsi: dmx3191d: use module_pci_driver

2016-11-16 Thread Geliang Tang
Use module_pci_driver() helper to simplify the code. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/scsi/dmx3191d.c | 13 + 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/drivers/scsi/dmx3191d.c b/drivers/scsi/dmx3191d.c index 3aa4657..6af3394

[PATCH] scsi: dmx3191d: use module_pci_driver

2016-11-16 Thread Geliang Tang
Use module_pci_driver() helper to simplify the code. Signed-off-by: Geliang Tang --- drivers/scsi/dmx3191d.c | 13 + 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/drivers/scsi/dmx3191d.c b/drivers/scsi/dmx3191d.c index 3aa4657..6af3394 100644 --- a/drivers/scsi

[PATCH] scsi: a100u2w: use module_pci_driver

2016-11-16 Thread Geliang Tang
Use module_pci_driver() helper to simplify the code. Signed-off-by: Geliang Tang --- drivers/scsi/a100u2w.c | 13 + 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/drivers/scsi/a100u2w.c b/drivers/scsi/a100u2w.c index 8086bd0..b2942ec 100644 --- a/drivers/scsi

[PATCH] [media] netup_unidvb: use module_pci_driver

2016-11-16 Thread Geliang Tang
Use module_pci_driver() helper to simplify the code. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/media/pci/netup_unidvb/netup_unidvb_core.c | 13 + 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/drivers/media/pci/netup_unidvb/netup_unidvb_cor

[PATCH] [media] netup_unidvb: use module_pci_driver

2016-11-16 Thread Geliang Tang
Use module_pci_driver() helper to simplify the code. Signed-off-by: Geliang Tang --- drivers/media/pci/netup_unidvb/netup_unidvb_core.c | 13 + 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/drivers/media/pci/netup_unidvb/netup_unidvb_core.c b/drivers/media/pci

[PATCH] [media] sh_mobile_ceu_camera: use module_platform_driver

2016-11-15 Thread Geliang Tang
Use module_platform_driver() helper to simplify the code. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/media/platform/soc_camera/sh_mobile_ceu_camera.c | 13 + 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/drivers/media/platform/soc_

[PATCH] [media] sh_mobile_ceu_camera: use module_platform_driver

2016-11-15 Thread Geliang Tang
Use module_platform_driver() helper to simplify the code. Signed-off-by: Geliang Tang --- drivers/media/platform/soc_camera/sh_mobile_ceu_camera.c | 13 + 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/drivers/media/platform/soc_camera/sh_mobile_ceu_camera.c b

[PATCH] xen-platform: use builtin_pci_driver

2016-11-14 Thread Geliang Tang
Use builtin_pci_driver() helper to simplify the code. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/xen/platform-pci.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/xen/platform-pci.c b/drivers/xen/platform-pci.c index b59c9455..112ce42

[PATCH] gpio: intel-mid: use builtin_pci_driver

2016-11-14 Thread Geliang Tang
Use builtin_pci_driver() helper to simplify the code. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/gpio/gpio-intel-mid.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/gpio/gpio-intel-mid.c b/drivers/gpio/gpio-intel-mid.c index 1

[PATCH] xen-platform: use builtin_pci_driver

2016-11-14 Thread Geliang Tang
Use builtin_pci_driver() helper to simplify the code. Signed-off-by: Geliang Tang --- drivers/xen/platform-pci.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/xen/platform-pci.c b/drivers/xen/platform-pci.c index b59c9455..112ce42 100644 --- a/drivers/xen

[PATCH] gpio: intel-mid: use builtin_pci_driver

2016-11-14 Thread Geliang Tang
Use builtin_pci_driver() helper to simplify the code. Signed-off-by: Geliang Tang --- drivers/gpio/gpio-intel-mid.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/gpio/gpio-intel-mid.c b/drivers/gpio/gpio-intel-mid.c index 164de64..a1e44c2 100644 --- a/drivers

[PATCH] crypto: nx - drop duplicate header types.h

2016-11-11 Thread Geliang Tang
Drop duplicate header types.h from nx.c. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/crypto/nx/nx.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/crypto/nx/nx.c b/drivers/crypto/nx/nx.c index 42f0f22..036057a 100644 --- a/drivers/crypto/nx/nx.c +++ b/d

[PATCH] crypto: nx - drop duplicate header types.h

2016-11-11 Thread Geliang Tang
Drop duplicate header types.h from nx.c. Signed-off-by: Geliang Tang --- drivers/crypto/nx/nx.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/crypto/nx/nx.c b/drivers/crypto/nx/nx.c index 42f0f22..036057a 100644 --- a/drivers/crypto/nx/nx.c +++ b/drivers/crypto/nx/nx.c @@ -32,7

[PATCH] crypto: jitterentropy - drop duplicate header module.h

2016-11-11 Thread Geliang Tang
Drop duplicate header module.h from jitterentropy-kcapi.c. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- crypto/jitterentropy-kcapi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/crypto/jitterentropy-kcapi.c b/crypto/jitterentropy-kcapi.c index c4938497..787dccc

[PATCH] crypto: jitterentropy - drop duplicate header module.h

2016-11-11 Thread Geliang Tang
Drop duplicate header module.h from jitterentropy-kcapi.c. Signed-off-by: Geliang Tang --- crypto/jitterentropy-kcapi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/crypto/jitterentropy-kcapi.c b/crypto/jitterentropy-kcapi.c index c4938497..787dccc 100644 --- a/crypto/jitterentropy

[PATCH] HID: intel_ish-hid: use list_move_tail in ishtp_cl_read_start

2016-11-11 Thread Geliang Tang
Use list_move() instead of list_del() + list_add_tail() to simplify the code. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/hid/intel-ish-hid/ishtp/client.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/hid/intel-ish-hid/ishtp/clien

[PATCH] drm/radeon: use list_move in radeon_vm_bo_update

2016-11-11 Thread Geliang Tang
Use list_move() instead of list_del() + list_add() to simplify the code. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/gpu/drm/radeon/radeon_vm.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_vm.c b/drivers/gpu/drm/

[PATCH] HID: intel_ish-hid: use list_move_tail in ishtp_cl_read_start

2016-11-11 Thread Geliang Tang
Use list_move() instead of list_del() + list_add_tail() to simplify the code. Signed-off-by: Geliang Tang --- drivers/hid/intel-ish-hid/ishtp/client.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/hid/intel-ish-hid/ishtp/client.c b/drivers/hid/intel-ish-hid

[PATCH] drm/radeon: use list_move in radeon_vm_bo_update

2016-11-11 Thread Geliang Tang
Use list_move() instead of list_del() + list_add() to simplify the code. Signed-off-by: Geliang Tang --- drivers/gpu/drm/radeon/radeon_vm.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_vm.c b/drivers/gpu/drm/radeon/radeon_vm.c index

Re: [PATCH] ramoops: add pdata NULL check to ramoops_probe

2016-09-29 Thread Geliang Tang
On Wed, Sep 28, 2016 at 12:45:13PM -0700, Kees Cook wrote: > On Wed, Sep 28, 2016 at 3:32 AM, Geliang Tang <geliangt...@gmail.com> wrote: > > When the pdata is NULL, ramoops_probe() segfaults. So this patch adds > > a NULL check to it. > > While I don't mind the c

Re: [PATCH] ramoops: add pdata NULL check to ramoops_probe

2016-09-29 Thread Geliang Tang
On Wed, Sep 28, 2016 at 12:45:13PM -0700, Kees Cook wrote: > On Wed, Sep 28, 2016 at 3:32 AM, Geliang Tang wrote: > > When the pdata is NULL, ramoops_probe() segfaults. So this patch adds > > a NULL check to it. > > While I don't mind the check, is this even possible?

[PATCH] ramoops: add pdata NULL check to ramoops_probe

2016-09-28 Thread Geliang Tang
When the pdata is NULL, ramoops_probe() segfaults. So this patch adds a NULL check to it. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- fs/pstore/ram.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/fs/pstore/ram.c b/fs/pstore/ram.c index 6ad831b..dd9832d 100644 --- a/fs/

[PATCH] ramoops: add pdata NULL check to ramoops_probe

2016-09-28 Thread Geliang Tang
When the pdata is NULL, ramoops_probe() segfaults. So this patch adds a NULL check to it. Signed-off-by: Geliang Tang --- fs/pstore/ram.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/fs/pstore/ram.c b/fs/pstore/ram.c index 6ad831b..dd9832d 100644 --- a/fs/pstore/ram.c +++ b/fs/pstore

Re: [PATCH] jbd2: move more common code into journal_init_common()

2016-09-17 Thread Geliang Tang
On Thu, Sep 15, 2016 at 03:58:18PM -0400, Theodore Ts'o wrote: > On Thu, Sep 15, 2016 at 12:03:09PM -0400, Theodore Ts'o wrote: > > On Wed, Sep 07, 2016 at 03:16:24PM +0200, Jan Kara wrote: > > > On Wed 07-09-16 20:41:13, Geliang Tang wrote: > > > >

Re: [PATCH] jbd2: move more common code into journal_init_common()

2016-09-17 Thread Geliang Tang
On Thu, Sep 15, 2016 at 03:58:18PM -0400, Theodore Ts'o wrote: > On Thu, Sep 15, 2016 at 12:03:09PM -0400, Theodore Ts'o wrote: > > On Wed, Sep 07, 2016 at 03:16:24PM +0200, Jan Kara wrote: > > > On Wed 07-09-16 20:41:13, Geliang Tang wrote: > > > >

[PATCH] jbd2: move more common code into journal_init_common()

2016-09-07 Thread Geliang Tang
There are some repetitive code in jbd2_journal_init_dev() and jbd2_journal_init_inode(). So this patch moves the common code into journal_init_common() helper to simplify the code. And fix the coding style warnings reported by checkpatch.pl by the way. Signed-off-by: Geliang Tang <gelia

[PATCH] jbd2: move more common code into journal_init_common()

2016-09-07 Thread Geliang Tang
There are some repetitive code in jbd2_journal_init_dev() and jbd2_journal_init_inode(). So this patch moves the common code into journal_init_common() helper to simplify the code. And fix the coding style warnings reported by checkpatch.pl by the way. Signed-off-by: Geliang Tang --- fs/jbd2

[PATCH] Re: [PATCH] jbd2: add jbd2_journal_init() helper

2016-09-07 Thread Geliang Tang
On Tue, Sep 06, 2016 at 05:15:54PM +0200, Jan Kara wrote: > On Sat 03-09-16 17:33:39, Eric Ren wrote: > > Hi Geliang, > > > > On 08/31/2016 08:23 PM, Geliang Tang wrote: > > >There are some repetitive code in jbd2_journal_init_dev() and > > >jbd2_journ

[PATCH] Re: [PATCH] jbd2: add jbd2_journal_init() helper

2016-09-07 Thread Geliang Tang
On Tue, Sep 06, 2016 at 05:15:54PM +0200, Jan Kara wrote: > On Sat 03-09-16 17:33:39, Eric Ren wrote: > > Hi Geliang, > > > > On 08/31/2016 08:23 PM, Geliang Tang wrote: > > >There are some repetitive code in jbd2_journal_init_dev() and > > >jbd2_journ

[PATCH] jbd2: add jbd2_journal_init() helper

2016-08-31 Thread Geliang Tang
There are some repetitive code in jbd2_journal_init_dev() and jbd2_journal_init_inode(). So this patch extracts the common code into jbd2_journal_init() helper to simplify the code. And fix the coding style warnings reported by checkpatch.pl by the way. Signed-off-by: Geliang Tang <gelia

[PATCH] jbd2: add jbd2_journal_init() helper

2016-08-31 Thread Geliang Tang
There are some repetitive code in jbd2_journal_init_dev() and jbd2_journal_init_inode(). So this patch extracts the common code into jbd2_journal_init() helper to simplify the code. And fix the coding style warnings reported by checkpatch.pl by the way. Signed-off-by: Geliang Tang --- fs/jbd2

[PATCH 1/2] ramoops: move spin_lock_init after kmalloc error checking

2016-08-30 Thread Geliang Tang
If cxt->pstore.buf allocated failed, no need to initialize cxt->pstore.buf_lock. So this patch moves spin_lock_init() after the error checking. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- fs/pstore/ram.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 1/2] ramoops: move spin_lock_init after kmalloc error checking

2016-08-30 Thread Geliang Tang
If cxt->pstore.buf allocated failed, no need to initialize cxt->pstore.buf_lock. So this patch moves spin_lock_init() after the error checking. Signed-off-by: Geliang Tang --- fs/pstore/ram.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/pstore/ram.c b/fs/pstore

[PATCH 2/2] ramoops: use buffer_size() and buffer_start()

2016-08-30 Thread Geliang Tang
Since buffer_size() and buffer_start() have been defined in ram_core.c, use them instead of open-coding. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- fs/pstore/ram_core.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/fs/pstore/ram_core.c b/fs/

[PATCH 2/2] ramoops: use buffer_size() and buffer_start()

2016-08-30 Thread Geliang Tang
Since buffer_size() and buffer_start() have been defined in ram_core.c, use them instead of open-coding. Signed-off-by: Geliang Tang --- fs/pstore/ram_core.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/fs/pstore/ram_core.c b/fs/pstore/ram_core.c index 76c3f80

[PATCH] zram: use writer semaphores in device attributes store

2016-06-16 Thread Geliang Tang
Since the device attributes store provides write access. This patch uses down_write()/up_write() instead of down_read()/up_read() in mem_used_max_store() and compact_store(). Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/block/zram/zram_drv.c | 10 +- 1 file chan

[PATCH] zram: use writer semaphores in device attributes store

2016-06-16 Thread Geliang Tang
Since the device attributes store provides write access. This patch uses down_write()/up_write() instead of down_read()/up_read() in mem_used_max_store() and compact_store(). Signed-off-by: Geliang Tang --- drivers/block/zram/zram_drv.c | 10 +- 1 file changed, 5 insertions(+), 5

[PATCH] zram: update zram to use zpool

2016-06-15 Thread Geliang Tang
) The 'pages_compacted' attribute is showed in zram by calling zs_pool_stats(). So in order not to call zs_pool_state() I move the attribute to zsmalloc. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/block/zram/Kconfig| 3 ++- drivers/block/zram/zram_drv.

[PATCH] zram: add zpool support v2

2016-06-15 Thread Geliang Tang
On Mon, Jun 13, 2016 at 11:11:00AM +0200, Vitaly Wool wrote: > Den 8 juni 2016 6:33 em skrev "Dan Streetman" <ddstr...@ieee.org>: > > > > On Wed, Jun 8, 2016 at 5:39 AM, Geliang Tang <geliangt...@gmail.com> > wrote: > > > This patch adds zpo

[PATCH] zram: update zram to use zpool

2016-06-15 Thread Geliang Tang
) The 'pages_compacted' attribute is showed in zram by calling zs_pool_stats(). So in order not to call zs_pool_state() I move the attribute to zsmalloc. Signed-off-by: Geliang Tang --- drivers/block/zram/Kconfig| 3 ++- drivers/block/zram/zram_drv.c | 59

[PATCH] zram: add zpool support v2

2016-06-15 Thread Geliang Tang
On Mon, Jun 13, 2016 at 11:11:00AM +0200, Vitaly Wool wrote: > Den 8 juni 2016 6:33 em skrev "Dan Streetman" : > > > > On Wed, Jun 8, 2016 at 5:39 AM, Geliang Tang > wrote: > > > This patch adds zpool support for zram, it will allow us to use both > >

[PATCH 2/2] fuse: use list_first_entry

2016-06-08 Thread Geliang Tang
To make the intention clearer, use list_first_entry() instead of list_entry(). Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- fs/fuse/dev.c | 6 +++--- fs/fuse/file.c | 3 ++- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/fs/fuse/dev.c b/fs/fuse/dev.c index 1

[PATCH 1/2] fuse: use list_first_entry_or_null

2016-06-08 Thread Geliang Tang
Use list_first_entry_or_null() instead of list_empty() and list_entry() to simplify the code. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- fs/fuse/dev.c | 7 +++ fs/fuse/file.c | 9 - 2 files changed, 7 insertions(+), 9 deletions(-) diff --git a/fs/fuse/dev.c b/f

[PATCH 2/2] fuse: use list_first_entry

2016-06-08 Thread Geliang Tang
To make the intention clearer, use list_first_entry() instead of list_entry(). Signed-off-by: Geliang Tang --- fs/fuse/dev.c | 6 +++--- fs/fuse/file.c | 3 ++- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/fs/fuse/dev.c b/fs/fuse/dev.c index 1432cf7..a766983 100644 --- a/fs

[PATCH 1/2] fuse: use list_first_entry_or_null

2016-06-08 Thread Geliang Tang
Use list_first_entry_or_null() instead of list_empty() and list_entry() to simplify the code. Signed-off-by: Geliang Tang --- fs/fuse/dev.c | 7 +++ fs/fuse/file.c | 9 - 2 files changed, 7 insertions(+), 9 deletions(-) diff --git a/fs/fuse/dev.c b/fs/fuse/dev.c index cbece12

[PATCH] cpumask: fix code comment

2016-06-08 Thread Geliang Tang
Fix code comment for cpumask_parse(). Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- include/linux/cpumask.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/cpumask.h b/include/linux/cpumask.h index e828cf6..da7fbf1 100644 --- a/include/linux/cpu

[PATCH] cpumask: fix code comment

2016-06-08 Thread Geliang Tang
Fix code comment for cpumask_parse(). Signed-off-by: Geliang Tang --- include/linux/cpumask.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/cpumask.h b/include/linux/cpumask.h index e828cf6..da7fbf1 100644 --- a/include/linux/cpumask.h +++ b/include/linux

[PATCH] powerpc/nvram: remove unused pstore headers

2016-06-08 Thread Geliang Tang
Since the pstore code has moved away from nvram.c, remove unused pstore headers pstore.h and kmsg_dump.h. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- arch/powerpc/platforms/pseries/nvram.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/powerpc/platforms/pseries/nvra

[PATCH] powerpc/nvram: remove unused pstore headers

2016-06-08 Thread Geliang Tang
Since the pstore code has moved away from nvram.c, remove unused pstore headers pstore.h and kmsg_dump.h. Signed-off-by: Geliang Tang --- arch/powerpc/platforms/pseries/nvram.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/arch/powerpc/platforms/pseries/nvram.c b/arch/powerpc/platforms

[PATCH] zram: add zpool support

2016-06-08 Thread Geliang Tang
This patch adds zpool support for zram, it will allow us to use both the zpool api and directly zsmalloc api in zram. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/block/zram/zram_drv.c | 97 +++ drivers/block/zram/zram_drv.h | 5

[PATCH] zram: add zpool support

2016-06-08 Thread Geliang Tang
This patch adds zpool support for zram, it will allow us to use both the zpool api and directly zsmalloc api in zram. Signed-off-by: Geliang Tang --- drivers/block/zram/zram_drv.c | 97 +++ drivers/block/zram/zram_drv.h | 5 +++ 2 files changed, 102

Re: [PATCH 2/4] mm: kmemleak: remove unused header cpumask.h

2016-06-02 Thread Geliang Tang
On Thu, Jun 02, 2016 at 10:32:41AM +0100, Catalin Marinas wrote: > On Thu, Jun 02, 2016 at 02:15:34PM +0800, Geliang Tang wrote: > > Remove unused header cpumask.h from mm/kmemleak.c. > > > > Signed-off-by: Geliang Tang <geliangt...@gmail.com> > > Acked-

Re: [PATCH 2/4] mm: kmemleak: remove unused header cpumask.h

2016-06-02 Thread Geliang Tang
On Thu, Jun 02, 2016 at 10:32:41AM +0100, Catalin Marinas wrote: > On Thu, Jun 02, 2016 at 02:15:34PM +0800, Geliang Tang wrote: > > Remove unused header cpumask.h from mm/kmemleak.c. > > > > Signed-off-by: Geliang Tang > > Acked-by: Catalin Marinas This

Re: [PATCH v2 2/3] efi-pstore: implement efivars_pstore_exit()

2016-06-02 Thread Geliang Tang
On Wed, Nov 11, 2015 at 03:24:33PM -0800, Kees Cook wrote: > On Wed, Nov 11, 2015 at 3:23 PM, Luck, Tony wrote: > module_init(efivars_pstore_init); > >>> > >>> Looks OK to me. Kees, are you picking this up? > >> > >> I can, though usually it goes through Tony. > > > >

Re: [PATCH v2 2/3] efi-pstore: implement efivars_pstore_exit()

2016-06-02 Thread Geliang Tang
On Wed, Nov 11, 2015 at 03:24:33PM -0800, Kees Cook wrote: > On Wed, Nov 11, 2015 at 3:23 PM, Luck, Tony wrote: > module_init(efivars_pstore_init); > >>> > >>> Looks OK to me. Kees, are you picking this up? > >> > >> I can, though usually it goes through Tony. > > > > Can I count that as

[PATCH 3/4] mm/vmstat: remove unused header cpumask.h

2016-06-02 Thread Geliang Tang
Remove unused header cpumask.h from mm/vmstat.c. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- mm/vmstat.c | 1 - 1 file changed, 1 deletion(-) diff --git a/mm/vmstat.c b/mm/vmstat.c index 1b585f8..3653449 100644 --- a/mm/vmstat.c +++ b/mm/vmstat.c @@ -15,7 +15,6 @@ #i

[PATCH 3/4] mm/vmstat: remove unused header cpumask.h

2016-06-02 Thread Geliang Tang
Remove unused header cpumask.h from mm/vmstat.c. Signed-off-by: Geliang Tang --- mm/vmstat.c | 1 - 1 file changed, 1 deletion(-) diff --git a/mm/vmstat.c b/mm/vmstat.c index 1b585f8..3653449 100644 --- a/mm/vmstat.c +++ b/mm/vmstat.c @@ -15,7 +15,6 @@ #include #include #include

[PATCH 4/4] mm/zsmalloc: remove unused header cpumask.h

2016-06-02 Thread Geliang Tang
Remove unused header cpumask.h from mm/zsmalloc.c. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- mm/zsmalloc.c | 1 - 1 file changed, 1 deletion(-) diff --git a/mm/zsmalloc.c b/mm/zsmalloc.c index b6d4f25..a93327e 100644 --- a/mm/zsmalloc.c +++ b/mm/zsmalloc.c @@ -57,7

[PATCH 1/4] mm/init-mm: remove unused header cpumask.h

2016-06-02 Thread Geliang Tang
Remove unused header cpumask.h from mm/init-mm.c. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- mm/init-mm.c | 1 - 1 file changed, 1 deletion(-) diff --git a/mm/init-mm.c b/mm/init-mm.c index a56a851..2acae89 100644 --- a/mm/init-mm.c +++ b/mm/init-mm.c @@ -3,7 +3,6 @@ #i

[PATCH 4/4] mm/zsmalloc: remove unused header cpumask.h

2016-06-02 Thread Geliang Tang
Remove unused header cpumask.h from mm/zsmalloc.c. Signed-off-by: Geliang Tang --- mm/zsmalloc.c | 1 - 1 file changed, 1 deletion(-) diff --git a/mm/zsmalloc.c b/mm/zsmalloc.c index b6d4f25..a93327e 100644 --- a/mm/zsmalloc.c +++ b/mm/zsmalloc.c @@ -57,7 +57,6 @@ #include #include

[PATCH 1/4] mm/init-mm: remove unused header cpumask.h

2016-06-02 Thread Geliang Tang
Remove unused header cpumask.h from mm/init-mm.c. Signed-off-by: Geliang Tang --- mm/init-mm.c | 1 - 1 file changed, 1 deletion(-) diff --git a/mm/init-mm.c b/mm/init-mm.c index a56a851..2acae89 100644 --- a/mm/init-mm.c +++ b/mm/init-mm.c @@ -3,7 +3,6 @@ #include #include #include

[PATCH 2/4] mm: kmemleak: remove unused header cpumask.h

2016-06-02 Thread Geliang Tang
Remove unused header cpumask.h from mm/kmemleak.c. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- mm/kmemleak.c | 1 - 1 file changed, 1 deletion(-) diff --git a/mm/kmemleak.c b/mm/kmemleak.c index e642992..2617309 100644 --- a/mm/kmemleak.c +++ b/mm/kmemleak.c @@ -82,7

[PATCH 2/4] mm: kmemleak: remove unused header cpumask.h

2016-06-02 Thread Geliang Tang
Remove unused header cpumask.h from mm/kmemleak.c. Signed-off-by: Geliang Tang --- mm/kmemleak.c | 1 - 1 file changed, 1 deletion(-) diff --git a/mm/kmemleak.c b/mm/kmemleak.c index e642992..2617309 100644 --- a/mm/kmemleak.c +++ b/mm/kmemleak.c @@ -82,7 +82,6 @@ #include #include

[PATCH] crypto: skcipher - remove unused header cpumask.h

2016-06-01 Thread Geliang Tang
Remove unused header cpumask.h from crypto/ablkcipher.c. Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- crypto/ablkcipher.c | 1 - 1 file changed, 1 deletion(-) diff --git a/crypto/ablkcipher.c b/crypto/ablkcipher.c index e5b5721..6b80516 100644 --- a/crypto/ablkcipher.c +++ b/

[PATCH] crypto: skcipher - remove unused header cpumask.h

2016-06-01 Thread Geliang Tang
Remove unused header cpumask.h from crypto/ablkcipher.c. Signed-off-by: Geliang Tang --- crypto/ablkcipher.c | 1 - 1 file changed, 1 deletion(-) diff --git a/crypto/ablkcipher.c b/crypto/ablkcipher.c index e5b5721..6b80516 100644 --- a/crypto/ablkcipher.c +++ b/crypto/ablkcipher.c @@ -14,7

[PATCH] pstore: set ecc_notice_size to zero

2016-06-01 Thread Geliang Tang
Since a @ecc_notice_size parameter has been added in pstore_info.read(), this patch sets this parameter to zero in efi_pstore_read() and erst_reader(). Signed-off-by: Geliang Tang <geliangt...@gmail.com> --- drivers/acpi/apei/erst.c | 1 + drivers/firmware/efi/efi-pstore.c | 5 ++

[PATCH] pstore: set ecc_notice_size to zero

2016-06-01 Thread Geliang Tang
Since a @ecc_notice_size parameter has been added in pstore_info.read(), this patch sets this parameter to zero in efi_pstore_read() and erst_reader(). Signed-off-by: Geliang Tang --- drivers/acpi/apei/erst.c | 1 + drivers/firmware/efi/efi-pstore.c | 5 + 2 files changed, 6

Re: [PATCH v2] pstore: add lzo/lz4 compression support

2016-04-29 Thread Geliang Tang
On Thu, Feb 18, 2016 at 09:37:26AM -0800, Kees Cook wrote: > On Thu, Feb 18, 2016 at 6:04 AM, Geliang Tang <geliangt...@163.com> wrote: > > Like zlib compression in pstore, this patch added lzo and lz4 > > compression support so that users can have more options and better

Re: [PATCH v2] pstore: add lzo/lz4 compression support

2016-04-29 Thread Geliang Tang
On Thu, Feb 18, 2016 at 09:37:26AM -0800, Kees Cook wrote: > On Thu, Feb 18, 2016 at 6:04 AM, Geliang Tang wrote: > > Like zlib compression in pstore, this patch added lzo and lz4 > > compression support so that users can have more options and better > > compression ratio.

[PATCH] cpufreq: mt8173: use list_for_each_entry*()

2016-04-04 Thread Geliang Tang
Use list_for_each_entry*() instead of list_for_each*() to simplify the code. Signed-off-by: Geliang Tang <geliangt...@163.com> --- drivers/cpufreq/mt8173-cpufreq.c | 14 -- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/drivers/cpufreq/mt8173-cpufreq.c b/d

[PATCH] cpufreq: mt8173: use list_for_each_entry*()

2016-04-04 Thread Geliang Tang
Use list_for_each_entry*() instead of list_for_each*() to simplify the code. Signed-off-by: Geliang Tang --- drivers/cpufreq/mt8173-cpufreq.c | 14 -- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/drivers/cpufreq/mt8173-cpufreq.c b/drivers/cpufreq/mt8173-cpufreq.c

[PATCH] staging: rtl8712: use container_of() instead of LIST_CONTAINOR()

2016-04-04 Thread Geliang Tang
This patch drops the local definition of LIST_CONTAINOR(), and uses container_of() instead of it. Signed-off-by: Geliang Tang <geliangt...@163.com> --- drivers/staging/rtl8712/osdep_service.h | 3 --- drivers/staging/rtl8712/rtl8712_recv.c| 10 +- drivers/staging/r

[PATCH] staging: rtl8712: use container_of() instead of LIST_CONTAINOR()

2016-04-04 Thread Geliang Tang
This patch drops the local definition of LIST_CONTAINOR(), and uses container_of() instead of it. Signed-off-by: Geliang Tang --- drivers/staging/rtl8712/osdep_service.h | 3 --- drivers/staging/rtl8712/rtl8712_recv.c| 10 +- drivers/staging/rtl8712/rtl8712_xmit.c

[PATCH] ftrace: use kasprintf() in ftrace_profile_tracefs()

2016-03-15 Thread Geliang Tang
Use kasprintf() instead of kmalloc() and snprintf(). Signed-off-by: Geliang Tang <geliangt...@163.com> --- kernel/trace/ftrace.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c index 2ece9f1..69aceae 100644 --- a/kernel

[PATCH] ftrace: use kasprintf() in ftrace_profile_tracefs()

2016-03-15 Thread Geliang Tang
Use kasprintf() instead of kmalloc() and snprintf(). Signed-off-by: Geliang Tang --- kernel/trace/ftrace.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c index 2ece9f1..69aceae 100644 --- a/kernel/trace/ftrace.c +++ b/kernel

<    1   2   3   4   5   6   7   8   9   10   >