Re: [PATCH v2 1/2] staging: iio: ad7780: check if ad778x before gain update

2018-11-11 Thread Ardelean, Alexandru
On Sun, 2018-11-11 at 12:58 +, Jonathan Cameron wrote: > On Thu, 8 Nov 2018 13:44:17 + > "Ardelean, Alexandru" wrote: > > > On Thu, 2018-11-08 at 11:03 -0200, Giuliano Belinassi wrote: > > > Only the ad778x have the 'gain' status bit. Check it before updating > > > through a new variable

Re: [PATCH v4 1/2] staging: iio: ad7780: update voltage on read

2018-11-11 Thread Ardelean, Alexandru
On Sun, 2018-11-11 at 14:30 +, Jonathan Cameron wrote: > On Tue, 6 Nov 2018 09:24:44 + > "Ardelean, Alexandru" wrote: > > > On Mon, 2018-11-05 at 17:14 -0200, Renato Lui Geh wrote: > > > The ad7780 driver previously did not read the correct device output, > > > as > > > it read an

[PATCH 10/12] staging: wilc1000: added 'WILC_' prefix for constants to have clear namespace

2018-11-11 Thread Ajay.Kathat
From: Ajay Singh For better namespace added 'WILC_' prefix for driver specific constants. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/host_interface.c | 162 +++--- drivers/staging/wilc1000/host_interface.h | 59

[PATCH 05/12] staging: wilc1000: use enum contants for mac status & added 'WILC_' prefix

2018-11-11 Thread Ajay.Kathat
From: Ajay Singh Clubbed mac status constants inside the enum constant and also added 'WILC_' prefix to have better namespace. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/host_interface.c | 15 --- drivers/staging/wilc1000/linux_wlan.c | 4 ++--

[PATCH 12/12] staging: wilc1000: rename tsf_hi element of network_info struct

2018-11-11 Thread Ajay.Kathat
From: Ajay Singh Rename 'tsf_hi' element in network_info struct as it's not used to store only the higher 32-bit value but the complete 64-bit tsf value. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/host_interface.c | 8 drivers/staging/wilc1000/host_interface.h

[PATCH 09/12] staging: wilc1000: rename firmware related constants to follow better namespace

2018-11-11 Thread Ajay.Kathat
From: Ajay Singh Rename the firmware related macro by prefix with 'WILC_FW_' prefix for a better namespace. Also, move them to wilc_wlan_if.h file along with other FW related macros. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/host_interface.c | 4 ++--

[PATCH 11/12] staging: wilc1000: use ENOBUFS error code instead of WILC_TX_ERR_NO_BUF macro

2018-11-11 Thread Ajay.Kathat
From: Ajay Singh Make use of 'ENOBUFS' instead of WILC_TX_ERR_NO_BUF macro. The value of WILC_TX_ERR_NO_BUF is -2, which is confusing with ENOENT error code. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/linux_wlan.c | 2 +- drivers/staging/wilc1000/wilc_wlan.c| 2 +-

[PATCH 06/12] staging: wilc1000: restructure macros used to handle security type

2018-11-11 Thread Ajay.Kathat
From: Ajay Singh Restructure the code to have a proper namespace for macro defined to handle the security types. Move them as part of wilc_wlan_if.h header as along with other macro defined for firmware. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/linux_wlan.c | 2 +-

[PATCH 03/12] staging: wilc1000: remove unused macros and struct members in host_interface.h

2018-11-11 Thread Ajay.Kathat
From: Ajay Singh Cleanup patch to remove the unused macros and struct members in host_interface.h header file. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/host_interface.h | 11 --- 1 file changed, 11 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.h

[PATCH 04/12] staging: wilc1000: move sdio_cmd52 & sdio_cmd53 struct to source file

2018-11-11 Thread Ajay.Kathat
From: Ajay Singh Moved sdio_cmd52 & sdio_cmd53 struct from wilc_wlan_if.h, as its used in wilc_sdio.cfile. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/wilc_sdio.c| 19 +++ drivers/staging/wilc1000/wilc_wlan_if.h | 19 --- 2 files changed, 19

[PATCH 07/12] staging: wilc1000: use proper namespace for macros in wilc_wlan_if.h header

2018-11-11 Thread Ajay.Kathat
From: Ajay Singh Rename the macros defined in wilc_wlan_if.h header to have clear namespace. As convention used 'WILC_FW_' prefix for constants defined for firmware and 'WILC_' prefix for driver constants. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/host_interface.c | 6

[PATCH 01/12] staging: wilc1000: use macro from ieee80211.h in register frame api's

2018-11-11 Thread Ajay.Kathat
From: Ajay Singh Make use of ieee80211.h provided macros and removed the extra macro defined for the same purpose. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/host_interface.c | 4 ++-- drivers/staging/wilc1000/host_interface.h | 3 ---

[PATCH 08/12] staging: wilc1000: move current_tx_rate enum to wilc_wlan_if.h file

2018-11-11 Thread Ajay.Kathat
From: Ajay Singh Move 'current_tx_rate' enum to wilc_wlan_if.h, to have it along with the other FW related to constants. Also added prefix to have a better namespace. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/host_interface.h | 16

[PATCH 02/12] staging: wilc1000: use WLAN_PMKID_LEN macro from ieee80211.h header

2018-11-11 Thread Ajay.Kathat
From: Ajay Singh Make use of WLAN_PMKID_LEN macro provided by ieee80211.h header instead of PMKID_LEN. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/host_interface.c | 4 ++-- drivers/staging/wilc1000/host_interface.h | 3 +--

[PATCH 00/12] staging: wilc1000: better namespace for constants and cleanup

2018-11-11 Thread Ajay.Kathat
From: Ajay Singh This patch series contains changes to address the review feedback comments [1]. It mainly contains the changes to have clear names for the constants. Most of the constants names are taken care, few are left unchanged in hope with another cleanup those macros might be deleted.

Re: [PATCH v6 00/33] staging: mt7621-pci: Parse ports info from DT and other minor cleanups

2018-11-11 Thread Sergio Paracuellos
On Mon, Nov 12, 2018 at 08:40:10AM +1100, NeilBrown wrote: > On Sun, Nov 11 2018, Greg KH wrote: > > > On Sun, Nov 04, 2018 at 11:49:26AM +0100, Sergio Paracuellos wrote: > >> This patch series parse remaining port info from device tree storing > >> it in mt7621_pcie_port struct created for this.

Re: [kbuild-all] [staging:staging-testing 42/93] drivers/staging//media/tegra-vde/tegra-vde.c:265:5: warning: format '%d' expects argument of type 'int', but argument 6 has type 'long unsigned int'

2018-11-11 Thread Rong Chen
On 11/07/2018 09:11 PM, Greg Kroah-Hartman wrote: On Wed, Nov 07, 2018 at 08:45:40PM +0800, kbuild test robot wrote: tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing head: 64ce301c971bb323ab63f59014498d7e975586ee commit:

Re: How can I get user space tools of erofs?

2018-11-11 Thread cgxu519
Hi Chengguang, Good question! It's in the final stage of preparation to open source erofs-mkfs (actually they are now struggle at how to properly spilt into reasonable patches this week), hopefully the implementation could be released at the next week. (sorry I didn't mean to delay, I have

Price Inquiry

2018-11-11 Thread Daniel Murray
Hi,friend, This is Daniel Murray and i am from Sinara Group Co.Ltd Group Co.,LTD in Russia. We are glad to know about your company from the web and we are interested in your products. Could you kindly send us your Latest catalog and price list for our trial order. Best Regards, Daniel

Re: [PATCH v6 00/33] staging: mt7621-pci: Parse ports info from DT and other minor cleanups

2018-11-11 Thread NeilBrown
On Sun, Nov 11 2018, Greg KH wrote: > On Sun, Nov 04, 2018 at 11:49:26AM +0100, Sergio Paracuellos wrote: >> This patch series parse remaining port info from device tree storing >> it in mt7621_pcie_port struct created for this. It also performs a lot >> of cleanups to get the driver in a good

Re: [PATCH v2] ARM: staging: bcm2835-audio: interpolate audio delay

2018-11-11 Thread Stefan Wahren
Hi Mike, > Mike Brady hat am 11. November 2018 um 19:21 > geschrieben: > > > When the BCM2835 audio output is used, userspace sees a jitter up to 10ms > in the audio position, aka "delay" -- the number of frames that must > be output before a new frame would be played. > Make this a bit nicer

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

2018-11-11 Thread Greg Kroah-Hartman
On Sun, Nov 11, 2018 at 11:14:14AM +0530, Nishad Kamdar wrote: > Use the gpiod interface instead of the deprecated old non-descriptor > interface. > > Signed-off-by: Nishad Kamdar > --- > drivers/staging/greybus/arche-apb-ctrl.c | 158 ++- > 1 file changed, 65 insertions(+),

Re: [PATCH v6 00/33] staging: mt7621-pci: Parse ports info from DT and other minor cleanups

2018-11-11 Thread Greg KH
On Sun, Nov 04, 2018 at 11:49:26AM +0100, Sergio Paracuellos wrote: > This patch series parse remaining port info from device tree storing > it in mt7621_pcie_port struct created for this. It also performs a lot > of cleanups to get the driver in a good shape to give it a try to get > mainlined.

[PATCH] staging: android: ion: Add chunk heap initialization

2018-11-11 Thread Alexey Skidanov
Create chunk heap of specified size and base address by adding "ion_chunk_heap=size@start" kernel boot parameter. Signed-off-by: Alexey Skidanov --- drivers/staging/android/ion/ion_chunk_heap.c | 40 1 file changed, 40 insertions(+) diff --git

Re: [alsa-devel] [PATCH v2] staging: bcm2835-audio: interpolate audio delay

2018-11-11 Thread Mike Brady
> On 6 Nov 2018, at 21:31, Takashi Iwai wrote: > > On Tue, 06 Nov 2018 22:05:11 +0100, > Mike Brady wrote: >> >> >>> On 5 Nov 2018, at 16:11, Takashi Iwai wrote: >>> >>> On Mon, 05 Nov 2018 16:57:07 +0100, >>> Mike Brady wrote: > One another thing I'd like to point out is that

Re: [PATCH V2 3/5] Drivers: hv: kvp: Fix the recent regression caused by incorrect clean-up

2018-11-11 Thread gre...@linuxfoundation.org
On Sun, Nov 11, 2018 at 04:18:03AM +, Dexuan Cui wrote: > > From: gre...@linuxfoundation.org > > Sent: Thursday, November 1, 2018 21:54 > > To: Dexuan Cui > > Cc: Michael Kelley ; KY Srinivasan > > ; linux-ker...@vger.kernel.org; > > de...@linuxdriverproject.org; o...@aepfle.de;

Re: [PATCH v4 2/2] staging: iio: ad7780: remove unnecessary stashed voltage value

2018-11-11 Thread Jonathan Cameron
On Tue, 6 Nov 2018 09:25:27 + "Ardelean, Alexandru" wrote: > On Mon, 2018-11-05 at 17:16 -0200, Renato Lui Geh wrote: > > This patch removes the unnecessary field int_vref_mv in ad7780_state > > referring to the device's voltage. > > > > Looks good from my side. Applied to the togreg

Re: [PATCH v4 1/2] staging: iio: ad7780: update voltage on read

2018-11-11 Thread Jonathan Cameron
On Tue, 6 Nov 2018 09:24:44 + "Ardelean, Alexandru" wrote: > On Mon, 2018-11-05 at 17:14 -0200, Renato Lui Geh wrote: > > The ad7780 driver previously did not read the correct device output, as > > it read an outdated value set at initialization. It now updates its > > voltage on read. > >

Re: [PATCH v2 2/2] staging: iio: ad7780: generates pattern_mask from PAT bits

2018-11-11 Thread Jonathan Cameron
On Fri, 9 Nov 2018 20:18:58 -0200 Giuliano Augusto Faulin Belinassi wrote: > Hi > > >While I agree that it looks nicer to indent all these to the same level, > >you also need to think about the fact that the kernel git repo is already > >pretty big as-is, so it's a good idea if a patch adds as

Re: [PATCH v2 1/2] staging: iio: ad7780: check if ad778x before gain update

2018-11-11 Thread Jonathan Cameron
On Fri, 9 Nov 2018 20:15:45 -0200 Giuliano Augusto Faulin Belinassi wrote: > > Just some random though. Instead of introducing extra level of indentation > > you > > can simply check whether is_ad778x is asserted and simply return. > > I agree that the patch would be smaller if I do that,

Re: [PATCH v2 1/2] staging: iio: ad7780: check if ad778x before gain update

2018-11-11 Thread Jonathan Cameron
On Thu, 8 Nov 2018 13:44:17 + "Ardelean, Alexandru" wrote: > On Thu, 2018-11-08 at 11:03 -0200, Giuliano Belinassi wrote: > > Only the ad778x have the 'gain' status bit. Check it before updating > > through a new variable is_ad778x in chip_info. > > > > Looks good. Alex, formal tags

Re: [PATCH v3 4/4] staging: iio: ad7816: Add device tree table.

2018-11-11 Thread Jonathan Cameron
On Fri, 9 Nov 2018 08:11:57 + "Ardelean, Alexandru" wrote: > On Fri, 2018-11-09 at 13:08 +0530, Nishad Kamdar wrote: > > Add device tree table for matching vendor ID. > > One comment inline for this. > > Thanks > Alex > > > > > Signed-off-by: Nishad Kamdar > > --- > >

Re: [PATCH v3 3/4] staging: iio: ad7816: Set RD/WR pin and CONVST pin as outputs.

2018-11-11 Thread Jonathan Cameron
On Fri, 9 Nov 2018 13:07:18 +0530 Nishad Kamdar wrote: > The RD/WR pin and CONVST pin are logical inputs to the AD78xx > chip as per the datasheet. Hence convert them to outputs. > > Signed-off-by: Nishad Kamdar Hi Nishad, Applied to the togreg branch of iio.git and pushed out as testing for

Re: [PATCH v3 2/4] staging: iio: ad7816: Do not use busy_pin in case of AD7818

2018-11-11 Thread Jonathan Cameron
On Fri, 9 Nov 2018 13:06:24 +0530 Nishad Kamdar wrote: > AD7818 does not support busy_pin functionality as per datasheet. > Hence drop busy_pin when AD7818 is used. > > Signed-off-by: Nishad Kamdar Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play

Re: [PATCH v3 1/4] staging: iio: ad7816: Switch to the gpio descriptor interface

2018-11-11 Thread Jonathan Cameron
On Sun, 11 Nov 2018 12:24:05 + Jonathan Cameron wrote: > On Fri, 9 Nov 2018 13:05:17 +0530 > Nishad Kamdar wrote: > > > Use the gpiod interface for rdwr_pin, convert_pin and busy_pin > > instead of the deprecated old non-descriptor interface. > > > > Signed-off-by: Nishad Kamdar >

Re: [PATCH v3 1/4] staging: iio: ad7816: Switch to the gpio descriptor interface

2018-11-11 Thread Jonathan Cameron
On Fri, 9 Nov 2018 13:05:17 +0530 Nishad Kamdar wrote: > Use the gpiod interface for rdwr_pin, convert_pin and busy_pin > instead of the deprecated old non-descriptor interface. > > Signed-off-by: Nishad Kamdar Applied to the togreg branch of iio.git and pushed out as testing for the

Re: [PATCH 4/6] dt-bindings:iio:resolver: Add docs for ad2s90

2018-11-11 Thread Jonathan Cameron
On Fri, 9 Nov 2018 20:00:42 -0200 Matheus Tavares wrote: > This patch adds the device tree binding documentation for the ad2s90 > resolver-to-digital converter. > > Signed-off-by: Matheus Tavares > --- > .../bindings/iio/resolver/ad2s90.txt | 26 +++ > 1 file

Re: [PATCH 3/6] staging:iio:ad2s90: Add max frequency check at probe

2018-11-11 Thread Jonathan Cameron
On Fri, 9 Nov 2018 20:00:41 -0200 Matheus Tavares wrote: > This patch adds a max frequency check at the beginning of ad2s90_probe > function so that when it is set to a value above 0.83Mhz, dev_err is > called with an appropriate message and -EINVAL is returned. > > The defined limit is

Re: [PATCH 2/6] staging:iio:ad2s90: Remove spi setup that should be done via dt

2018-11-11 Thread Jonathan Cameron
On Fri, 9 Nov 2018 20:00:40 -0200 Matheus Tavares wrote: > The ad2s90 driver currently sets some spi settings (max_speed_hz and > mode) at ad2s90_probe. This should, instead, be handled via device tree. > This patch removes these configurations from the probe function. > > Note: The way in

Re: [PATCH 0/6] staging:iio:ad2s90: Add dt support and move out of staging

2018-11-11 Thread Jonathan Cameron
On Fri, 9 Nov 2018 20:00:38 -0200 Matheus Tavares wrote: > This patch set adds device tree support to ad2s90, with standard > device tree id table, adds the respective dt-binding documentation, > solves a codestyle warning and move the driver out of staging. > > This patch set completes all

Re: [PATCH 6/6] staging:iio:ad2s90: Move out of staging

2018-11-11 Thread Jonathan Cameron
On Fri, 9 Nov 2018 20:00:44 -0200 Matheus Tavares wrote: > Move ad2s90 resolver driver out of staging to the main tree. > > Signed-off-by: Matheus Tavares > Signed-off-by: Victor Colombo For a move out of staging patch, please disable move detection. It let's us see the whole driver and