[PATCH] staging: greybus: arche-apb-ctrl: fix unused warnings on resume/suspend

of these functions removes the warnings. Signed-off-by: Jérémy Lefaure <jeremy.lefa...@lse.epita.fr> --- drivers/staging/greybus/arche-apb-ctrl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/greybus/arche-apb-ctrl.c b/drivers/staging/greybus/arche-apb-

[PATCH v2 3/6] staging: media: atomisp: select REGMAP_I2C needed by ap1302.c

REGMAP_I2C should be enabled to build the driver ap1302 because it uses regmap functions. Signed-off-by: Jérémy Lefaure <jeremy.lefa...@lse.epita.fr> --- v2: rebase on staging-testing drivers/staging/media/atomisp/i2c/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/s

Re: [PATCH 4/6] staging: media: atomisp: fix build when PM is disabled

On Thu, 16 Mar 2017 17:50:10 + Alan Cox wrote: > > + if (IS_ENABLED(CONFI_PM)) { > > I think not. > > Please actually test build both ways at the very least. > > Alan > I did test and it did compile both ways. I didn't think about checking the output of the

[PATCH v2 4/6] staging: media: atomisp: fix build errors when PM is disabled

-by: Jérémy Lefaure <jeremy.lefa...@lse.epita.fr> --- drivers/staging/media/atomisp/pci/atomisp2/atomisp_v4l2.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/media/atomisp/pci/atomisp2/atomisp_v4l2.c b/drivers/staging/media/atomisp/pci/atomisp2/atomisp_

Re: [PATCH 3/6] staging: media: atomisp: select REGMAP_I2C needed by ap1302.c

On Thu, 16 Mar 2017 11:28:28 +0900 Greg Kroah-Hartman <gre...@linuxfoundation.org> wrote: > On Wed, Mar 15, 2017 at 02:09:18PM -0400, Jérémy Lefaure wrote: > > REGMAP_I2C should be enabled to build the driver ap1302 because it uses > > regmap functions. > > > &

Re: [PATCH 1/6] staging: media: atomisp: add missing include in vlv2_plat_clock.c

On Thu, 16 Mar 2017 11:25:37 +0900 Greg Kroah-Hartman <gre...@linuxfoundation.org> wrote: > On Wed, Mar 15, 2017 at 02:09:16PM -0400, Jérémy Lefaure wrote: > > To use IO functions like writel, readl, ioremap_nocache and iounmap, > > asm/io.h should be included. > >

[PATCH v2 1/6] staging: media: atomisp: add missing include in vlv2_plat_clock.c

To use IO functions like writel, readl, ioremap_nocache and iounmap, linux/io.h should be included. Signed-off-by: Jérémy Lefaure <jeremy.lefa...@lse.epita.fr> --- drivers/staging/media/atomisp/platform/clock/vlv2_plat_clock.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/s

[PATCH] Revert "staging: media: atomisp: fill properly hmm_bo_type_strings when ION is disabled"

This reverts commit fde469701c7efabebf885e785edf367bfb1a8f3f. Adding a preprocessor condition is not the best solution to fix this issue. Let's revert this commit before fixing this problem in a more appropriate way. Signed-off-by: Jérémy Lefaure <jeremy.lefa...@lse.epita.fr> --- d

[PATCH 6/6] staging: media: atomisp: remove duplicate includes

These duplicate includes have been found with scripts/checkincludes.pl but they have been removed manually to avoid removing false positives. Signed-off-by: Jérémy Lefaure <jeremy.lefa...@lse.epita.fr> --- drivers/staging/media/atomisp/i2c/ap1302.c | 1 - d

[PATCH 4/6] staging: media: atomisp: fix build when PM is disabled

be called only if PM is enabled. Same thing apply in atomisp_pci_probe: atomis_mrfld_power_down should be called only if PM is enabled. Signed-off-by: Jérémy Lefaure <jeremy.lefa...@lse.epita.fr> --- .../media/atomisp/pci/atomisp2/atomisp_cmd.c | 30 +- .../media/atomi

[PATCH 0/6] staging: media: atomisp: various fixes

Hello, Here are 6 patches for drivers/staging/media/atomisp. Patches 1 to 5 fix the build and patch 6 removes duplicate includes. These patches are base on git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-next. I am not sure if I should split patch 2 (one patch per

[PATCH 3/6] staging: media: atomisp: select REGMAP_I2C needed by ap1302.c

REGMAP_I2C should be enabled to build the driver ap1302 because it uses regmap functions. Signed-off-by: Jérémy Lefaure <jeremy.lefa...@lse.epita.fr> --- drivers/staging/media/atomisp/i2c/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/media/atomisp/i2c/Kcon

[PATCH 2/6] staging: media: atomisp: add missing dependencies in Kconfig

Signed-off-by: Jérémy Lefaure <jeremy.lefa...@lse.epita.fr> --- drivers/staging/media/atomisp/Kconfig| 2 +- drivers/staging/media/atomisp/i2c/Kconfig| 16 drivers/staging/media/atomisp/i2c/imx/Kconfig| 1 + drivers/staging/media/atomisp/i2c/ov5693/K

[PATCH 5/6] staging: media: atomisp: fill properly hmm_bo_type_strings when ION is disabled

When CONFIG_ION is disabled, HMM_BO_LAST is 3. In this case, "i" should not be added in hmm_bo_type_strings. Signed-off-by: Jérémy Lefaure <jeremy.lefa...@lse.epita.fr> --- drivers/staging/media/atomisp/pci/atomisp2/hmm/hmm.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a

[PATCH 1/6] staging: media: atomisp: add missing include in vlv2_plat_clock.c

To use IO functions like writel, readl, ioremap_nocache and iounmap, asm/io.h should be included. Signed-off-by: Jérémy Lefaure <jeremy.lefa...@lse.epita.fr> --- drivers/staging/media/atomisp/platform/clock/vlv2_plat_clock.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/s

Re: [PATCH 5/6] staging: media: atomisp: fill properly hmm_bo_type_strings when ION is disabled

On Thu, 16 Mar 2017 17:51:06 + Alan Cox <a...@linux.intel.com> wrote: > On Wed, 2017-03-15 at 14:09 -0400, Jérémy Lefaure wrote: > > When CONFIG_ION is disabled, HMM_BO_LAST is 3. In this case, "i" > > should > > not be added in hmm_bo_type_strings.

Re: [PATCH 00/18] use ARRAY_SIZE macro

On Mon, 2 Oct 2017 15:22:24 -0400 bfie...@fieldses.org (J. Bruce Fields) wrote: > Mainly I'd just like to know which you're asking for. Do you want me to > apply this, or to ACK it so someone else can? If it's sent as a series > I tend to assume the latter. > > But in this case I'm assuming

[PATCH 17/18] staging: rtl8723bs: use ARRAY_SIZE

/sizeof(*E)) | (sizeof(E)@p /sizeof(E[...])) | (sizeof(E)@p /sizeof(T)) ) Signed-off-by: Jérémy Lefaure <jeremy.lefa...@lse.epita.fr> --- drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 5 +- drivers/staging/rtl8723bs/core/rtw_rf.c | 7 +-- drivers/staging/rtl8723

[PATCH 18/18] staging: rtlwifi: use ARRAY_SIZE

(*E)) | (sizeof(E)@p /sizeof(E[...])) | (sizeof(E)@p /sizeof(T)) ) Signed-off-by: Jérémy Lefaure <jeremy.lefa...@lse.epita.fr> --- drivers/staging/rtlwifi/phydm/phydm_debug.c| 4 ++-- drivers/staging/rtlwifi/phydm/rtl8822b/halhwimg8822b_bb.c | 10 -- drivers/s

[PATCH 00/18] use ARRAY_SIZE macro

Hi everyone, Using ARRAY_SIZE improves the code readability. I used coccinelle (I made a change to the array_size.cocci file [1]) to find several places where ARRAY_SIZE could be used instead of other macros or sizeof division. I tried to divide the changes into a patch per subsystem (excepted

[PATCH 16/18] media: staging: atomisp: use ARRAY_SIZE

(*E)) | (sizeof(E)@p /sizeof(E[...])) | (sizeof(E)@p /sizeof(T)) ) Signed-off-by: Jérémy Lefaure <jeremy.lefa...@lse.epita.fr> --- .../pci/atomisp2/css2400/camera/pipe/src/pipe_binarydesc.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/staging

Re: [PATCH 00/18] use ARRAY_SIZE macro

On Mon, 2 Oct 2017 09:01:31 +1100 "Tobin C. Harding" wrote: > > In order to reduce the size of the To: and Cc: lines, each patch of the > > series is sent only to the maintainers and lists concerned by the patch. > > This cover letter is sent to every list concerned by this