Re: [PATCH RFC 00/11] staging: vc04_services: Improve driver load/unload

2019-01-10 Thread Peter Robinson
Hi Stefan, > > > > I get difference results with 5.0-rc1 but neither of the above apps > > > > work either, will follow up based on the rest of the thread there. > > > > > > > > > > My first step with Raspbian is to enable the Camera interface which > > > results into an appending of the

Re: [PATCH] staging: vboxvideo: vbox_main: Remove unnecessary local variable

2019-01-10 Thread Sidong Yang
On Thu, Jan 10, 2019 at 10:44:08PM +0300, Dan Carpenter wrote: > On Thu, Jan 10, 2019 at 05:00:24PM +, Sidong Yang wrote: > > I think you just point out that my code isn't obvious because the > > function returns negative error codes. I agree with you. But what if > > change my code like

Re: [PATCH] binder: create node flag to request sender's security context

2019-01-10 Thread kbuild test robot
Hi Todd, I love your patch! Perhaps something to improve: [auto build test WARNING on linus/master] [also build test WARNING on v5.0-rc1 next-20190110] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux

Re: [PATCH v2 1/2] Move ralink-gdma to its own directory

2019-01-10 Thread George Hilliard
On Thu, Jan 10, 2019, 6:21 PM NeilBrown > On Thu, Jan 10 2019, thirtythreefo...@gmail.com wrote: > > > From: George Hilliard > > > > This is in preparation to allow it and the mt7621-dma drivers to be > > built separately. They are completely independent pieces of software, > > and the Kconfig

Re: [PATCH v2 1/2] Move ralink-gdma to its own directory

2019-01-10 Thread NeilBrown
On Thu, Jan 10 2019, thirtythreefo...@gmail.com wrote: > From: George Hilliard > > This is in preparation to allow it and the mt7621-dma drivers to be > built separately. They are completely independent pieces of software, > and the Kconfig specifies very different requirements. > > Cc:

[PATCH v2 1/2] Move ralink-gdma to its own directory

2019-01-10 Thread thirtythreeforty
From: George Hilliard This is in preparation to allow it and the mt7621-dma drivers to be built separately. They are completely independent pieces of software, and the Kconfig specifies very different requirements. Cc: linux-ker...@vger.kernel.org Cc: de...@driverdev.osuosl.org Cc: Neil Brown

[PATCH v2 2/2] Mediatek staging: Use individual config flags in Makefile

2019-01-10 Thread thirtythreeforty
From: George Hilliard These drivers are useful on other MT76xx SoCs, which have compatible peripherals. The drivers are selectable in Kconfig, but they were quietly excluded from the build because the SOC_MT7621 chip was not selected. So, make the main staging Makefile use the same flags as

[PATCH] binderfs: rename header to binderfs.h

2019-01-10 Thread Christian Brauner
It doesn't make sense to call the header binder_ctl.h when its sole existence is tied to binderfs. So give it a sensible name. Users will far more easily remember binderfs.h than binder_ctl.h. Signed-off-by: Christian Brauner --- drivers/android/binderfs.c | 2 +-

Re: [PATCH] staging: rtl8723bs: Mark ACPI table declaration as used

2019-01-10 Thread Nick Desaulniers
On Wed, Jan 9, 2019 at 11:01 PM Greg KH wrote: > > On Wed, Jan 09, 2019 at 04:19:30PM -0800, Nick Desaulniers wrote: > > Digging up an old thread to point out I was wrong: > > > > On Thu, Sep 27, 2018 at 1:08 PM Nick Desaulniers > > wrote: > > > > This is not a million-little-commits problem;

Re: [PATCH v3] staging: comedi: ni_pcidio: make defines uppercase

2019-01-10 Thread Ian Abbott
On 10/01/2019 14:57, Alexander Schroth wrote: According to the Linux coding guidelines, defines should be written in uppercase. This patch converts all define-statements in the ni_pcidio.c file to uppercase, thus matching the coding style of the kernel. Signed-off-by: Alexander Schroth

Re: [PATCH] staging: vboxvideo: vbox_main: Remove unnecessary local variable

2019-01-10 Thread Dan Carpenter
On Thu, Jan 10, 2019 at 05:00:24PM +, Sidong Yang wrote: > I think you just point out that my code isn't obvious because the > function returns negative error codes. I agree with you. But what if > change my code like if(hgsmi_query_conf() != 0). > That's even worse! :P You should do

Re: [PATCH RFC 00/11] staging: vc04_services: Improve driver load/unload

2019-01-10 Thread Stefan Wahren
Hi Peter, > Peter Robinson hat am 10. Januar 2019 um 07:34 > geschrieben: > > > > > Hi Stefan, > > > > > ... > > > > > > I get difference results with 5.0-rc1 but neither of the above apps > > > work either, will follow up based on the rest of the thread there. > > > > > > > My first step

[PATCH] binder: create node flag to request sender's security context

2019-01-10 Thread Todd Kjos
To allow servers to verify client identity, allow a node flag to be set that causes the sender's security context to be delivered with the transaction. The BR_TRANSACTION command is extended in BR_TRANSACTION_SEC_CTX to contain a pointer to the security context string. Signed-off-by: Todd Kjos

[PATCH v5 3/3] staging: greybus: arche-platform: Switch to the gpio descriptor interface

2019-01-10 Thread Nishad Kamdar
Use the gpiod interface instead of the deprecated old non-descriptor interface while continuing to ignore gpio flags from device tree in "svc_reset_onoff()" for now. Signed-off-by: Nishad Kamdar --- Changes in v5: - Change the commit message. - Restore the names of the gpio device-tree

[PATCH v5 2/3] staging: greybus: arche-apb-ctrl.c: Switch to the gpio descriptor interface

2019-01-10 Thread Nishad Kamdar
Use the gpiod interface instead of the deprecated old non-descriptor interface while continuing to ignore gpio flags from device tree in functions "deassert_reset()" and "assert_reset()" for now. Signed-off-by: Nishad Kamdar --- Changes in v5: - Change the commit message. - Restore the names

[PATCH v5 1/3] staging: greybus: gpio: switch GPIO portions to use GPIOLIB_IRQCHIP

2019-01-10 Thread Nishad Kamdar
Convert the GPIO driver to use the GPIO irqchip library GPIOLIB_IRQCHIP instead of reimplementing the same. Signed-off-by: Nishad Kamdar --- Changes in v5: - Restore "struct irq_chip irqc" in "struct gb_gpio_controller" This is because we cannot use the gpio-chip irqchip, as that will

[PATCH v5 0/3] greybus: gpio: Switch to the gpio descriptor interface

2019-01-10 Thread Nishad Kamdar
This patch series converts uses of the old GPIO API to the GPIO descriptor API. It also converts the GPIO driver to use the GPIO irqchip library GPIOLIB_IRQCHIP instead of repimplementing the same. Changes in v5: - Drop the patch which uses gpiochip_get_data() in place of

Re: [PATCH v4 4/4] staging: greybus: arche-platform: Switch to the gpio descriptor interface

2019-01-10 Thread Nishad Kamdar
On Wed, Jan 09, 2019 at 12:35:47PM +0100, Johan Hovold wrote: > On Sat, Dec 22, 2018 at 08:23:02PM +0530, Nishad Kamdar wrote: > > Use the gpiod interface instead of the deprecated > > old non-descriptor interface. > > > > Signed-off-by: Nishad Kamdar > > --- > > Changes in v4: > > - Move

Re: [PATCH v4 3/4] staging: greybus: arche-apb-ctrl.c: Switch to the gpio descriptor interface

2019-01-10 Thread Nishad Kamdar
On Wed, Jan 09, 2019 at 12:30:43PM +0100, Johan Hovold wrote: > On Sat, Dec 22, 2018 at 08:22:09PM +0530, Nishad Kamdar wrote: > > Use the gpiod interface instead of the deprecated old non-descriptor > > interface. > > > > Signed-off-by: Nishad Kamdar > > --- > > Changes in v4: > > - Use

Re: [PATCH v4 2/4] staging: greybus: gpio: Use gpiochip_get_data() in place of gpio_chip_to_gb_gpio_controller()

2019-01-10 Thread Nishad Kamdar
On Wed, Jan 09, 2019 at 11:58:24AM +0100, Johan Hovold wrote: > On Sat, Dec 22, 2018 at 08:21:00PM +0530, Nishad Kamdar wrote: > > This patch drops gpio_chip_to_gb_gpio_controller(), > > and uses gpiochip_get_data() to retrieve the container > > of struct gpio_chip. > > So this will break the

Re: [PATCH v4 1/4] staging: greybus: gpio: switch GPIO portions to use GPIOLIB_IRQCHIP

2019-01-10 Thread Nishad Kamdar
On Wed, Jan 09, 2019 at 11:44:14AM +0100, Johan Hovold wrote: > On Sat, Dec 22, 2018 at 08:17:02PM +0530, Nishad Kamdar wrote: > > Convert the GPIO driver to use the GPIO irqchip library > > GPIOLIB_IRQCHIP instead of reimplementing the same. > > > > Signed-off-by: Nishad Kamdar > > --- > >

Re: [PATCH] staging: vboxvideo: vbox_main: Remove unnecessary local variable

2019-01-10 Thread Sidong Yang
On Thu, Jan 10, 2019 at 03:23:58PM +0300, Dan Carpenter wrote: > On Thu, Jan 10, 2019 at 06:13:47AM +, Sidong Yang wrote: > > Removed unnecessary local variable in have_hgsmi_mode_hints. > > The result of hgsmi_query_conf should be directly compared without > > assigning to local variable. > >

[PATCH v3] staging: comedi: ni_pcidio: make defines uppercase

2019-01-10 Thread Alexander Schroth
According to the Linux coding guidelines, defines should be written in uppercase. This patch converts all define-statements in the ni_pcidio.c file to uppercase, thus matching the coding style of the kernel. Signed-off-by: Alexander Schroth Signed-off-by: Marco Ammon --- v2: After feedback,

[PATCH v2] vmbus: Switch to use new generic UUID API

2019-01-10 Thread Andy Shevchenko
There are new types and helpers that are supposed to be used in new code. As a preparation to get rid of legacy types and API functions do the conversion here. Cc: "K. Y. Srinivasan" Cc: Haiyang Zhang Cc: Stephen Hemminger Cc: de...@linuxdriverproject.org Signed-off-by: Andy Shevchenko ---

Re: [linux-sunxi] [PATCH v2 1/2] media: v4l: Add definitions for the HEVC slice format and controls

2019-01-10 Thread ayaka
I forget a important thing, for the rkvdec and rk hevc decoder, it would requests cabac table, scaling list, picture parameter set and reference picture storing in one or various of DMA buffers. I am not talking about the data been parsed, the decoder would requests a raw data. For the pps

Re: [PATCH v2] comedi/ni_pcidio: make all defines uppercase

2019-01-10 Thread Dan Carpenter
On Thu, Jan 10, 2019 at 12:05:41PM +0100, Marco Ammon wrote: > Hi, > > thank you for your feedback. > > On 18/12/18 12:18 Ian Abbott wrote: > > Minor quibble: There should be a description of the 'v2' changes here > > (after the '---' line). > > > > Also, the patch "Subject:" line is missing the

Re: [PATCH v2] comedi/ni_pcidio: make all defines uppercase

2019-01-10 Thread Greg Kroah-Hartman
On Thu, Jan 10, 2019 at 12:05:41PM +0100, Marco Ammon wrote: > Hi, > > thank you for your feedback. > > On 18/12/18 12:18 Ian Abbott wrote: > > Minor quibble: There should be a description of the 'v2' changes here > > (after the '---' line). > > > > Also, the patch "Subject:" line is missing the

Re: [PATCH] staging: vboxvideo: vbox_main: Remove unnecessary local variable

2019-01-10 Thread Dan Carpenter
On Thu, Jan 10, 2019 at 06:13:47AM +, Sidong Yang wrote: > Removed unnecessary local variable in have_hgsmi_mode_hints. > The result of hgsmi_query_conf should be directly compared without > assigning to local variable. > > Signed-off-by: Sidong Yang > --- I sort of prefer the original...

Re: [PATCH v2] comedi/ni_pcidio: make all defines uppercase

2019-01-10 Thread Marco Ammon
Hi, thank you for your feedback. On 18/12/18 12:18 Ian Abbott wrote: > Minor quibble: There should be a description of the 'v2' changes here > (after the '---' line). > > Also, the patch "Subject:" line is missing the "staging:" tag.  The > usual set of tags for patches to this driver is

Re: [PATCH] media: imx-csi: Input connections to CSI should be optional

2019-01-10 Thread Philipp Zabel
On Wed, 2019-01-09 at 10:34 -0800, Steve Longerbeam wrote: > Some imx platforms do not have fwnode connections to all CSI input > ports, and should not be treated as an error. This includes the > imx6q SabreAuto, which has no connections to ipu1_csi1 and ipu2_csi0. > Return -ENOTCONN in

Attention:

2019-01-10 Thread Mr. Godwin Emefiele
Attention: This is to inform you that we have received the directives to release your payment, but we are have contradicting release instruction from one Mr Grant Wong who submitted an account as stated below where the money should be transferred into: Beneficiary: