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 it'
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 series.
>
> Why don
/sizeof(*E))
|
(sizeof(E)@p /sizeof(E[...]))
|
(sizeof(E)@p /sizeof(T))
)
Signed-off-by: Jérémy Lefaure
---
drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 5 +-
drivers/staging/rtl8723bs/core/rtw_rf.c | 7 +--
drivers/staging/rtl8723bs/hal/HalHWImg8723B_BB.c | 9 ++-
drivers
(*E))
|
(sizeof(E)@p /sizeof(E[...]))
|
(sizeof(E)@p /sizeof(T))
)
Signed-off-by: Jérémy Lefaure
---
drivers/staging/rtlwifi/phydm/phydm_debug.c| 4 ++--
drivers/staging/rtlwifi/phydm/rtl8822b/halhwimg8822b_bb.c | 10 --
drivers/staging/rtlwifi/phydm/rtl8822b
(*E))
|
(sizeof(E)@p /sizeof(E[...]))
|
(sizeof(E)@p /sizeof(T))
)
Signed-off-by: Jérémy Lefaure
---
.../pci/atomisp2/css2400/camera/pipe/src/pipe_binarydesc.c | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git
a/drivers/staging/media/atomisp/pci/atomisp2/css2400
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 for
: Jérémy Lefaure
---
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_v4l2.c
index 20b409789de2
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 compilation and I don't ha
ction `gmin_get_config_var':
(.text+0xe062b): undefined reference to `efivar_entry_get'
Signed-off-by: Jérémy Lefaure
---
v2:
* rebased on staging-testing
* removed ACPI and VIDEO_V4L2_SUBDEV_API dependencies as I am not sure that
it's the best solution to fix the building issue.
drivers/s
REGMAP_I2C should be enabled to build the driver ap1302 because it uses
regmap functions.
Signed-off-by: Jérémy Lefaure
---
v2: rebase on staging-testing
drivers/staging/media/atomisp/i2c/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/media/atomisp/i2c/Kconfig
b
To use IO functions like writel, readl, ioremap_nocache and iounmap,
linux/io.h should be included.
Signed-off-by: Jérémy Lefaure
---
drivers/staging/media/atomisp/platform/clock/vlv2_plat_clock.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/media/atomisp/platform/clock
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
---
drivers/staging/media/atomisp/pci/ato
On Thu, 16 Mar 2017 17:51:06 +
Alan Cox 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.
>
> No the goal is to rem
On Thu, 16 Mar 2017 11:28:28 +0900
Greg Kroah-Hartman 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.
> >
> > Signed-off-by: Jérémy Lefaure
&
On Thu, 16 Mar 2017 11:25:37 +0900
Greg Kroah-Hartman 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.
>
> Ok, but:
>
> >
>
To use IO functions like writel, readl, ioremap_nocache and iounmap,
asm/io.h should be included.
Signed-off-by: Jérémy Lefaure
---
drivers/staging/media/atomisp/platform/clock/vlv2_plat_clock.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/media/atomisp/platform/clock
Signed-off-by: Jérémy Lefaure
---
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/Kconfig | 2 +-
drivers/staging
REGMAP_I2C should be enabled to build the driver ap1302 because it uses
regmap functions.
Signed-off-by: Jérémy Lefaure
---
drivers/staging/media/atomisp/i2c/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/media/atomisp/i2c/Kconfig
b/drivers/staging/media/atomisp
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 missing
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
---
drivers/staging/media/atomisp/pci/atomisp2/hmm/hmm.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/staging/media/atomisp/pc
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
---
.../media/atomisp/pci/atomisp2/atomisp_cmd.c | 30 +-
.../media/atomisp/pci/atomisp2/atomisp_v4l2.c
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
---
drivers/staging/media/atomisp/i2c/ap1302.c | 1 -
drivers/staging/media/atomisp/i2c
these functions removes the
warnings.
Signed-off-by: Jérémy Lefaure
---
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-ctrl.c
index 3fda0cd6bb42
23 matches
Mail list logo