fixes the problem by casting the signed registers to s16.
Tested on an INA219.
Signed-off-by: Fabio Baltieri
---
drivers/hwmon/ina2xx.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/drivers/hwmon/ina2xx.c b/drivers/hwmon/ina2xx.c
index 93d26e8..bfd3f3e 100644
On Sun, Jun 8, 2014 at 9:30 PM, Guenter Roeck wrote:
> On Sun, Jun 08, 2014 at 01:16:00PM -0700, Guenter Roeck wrote:
>> On Sat, Jun 07, 2014 at 09:47:01PM +0100, Fabio Baltieri wrote:
>> > All devices supported by the ina2xx driver are bidirectional and reports
>&
On Sun, Jun 8, 2014 at 9:30 PM, Guenter Roeck li...@roeck-us.net wrote:
On Sun, Jun 08, 2014 at 01:16:00PM -0700, Guenter Roeck wrote:
On Sat, Jun 07, 2014 at 09:47:01PM +0100, Fabio Baltieri wrote:
All devices supported by the ina2xx driver are bidirectional and reports
the measured value
fixes the problem by casting the signed registers to s16.
Tested on an INA219.
Signed-off-by: Fabio Baltieri fabio.balti...@gmail.com
---
drivers/hwmon/ina2xx.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/drivers/hwmon/ina2xx.c b/drivers/hwmon/ina2xx.c
index 93d26e8
the problem by using a s16 instead, and has been tested
on an INA219.
Signed-off-by: Fabio Baltieri
---
drivers/hwmon/ina2xx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/hwmon/ina2xx.c b/drivers/hwmon/ina2xx.c
index 93d26e8..d994280 100644
--- a/drivers/hwmon/ina2xx.c
the problem by using a s16 instead, and has been tested
on an INA219.
Signed-off-by: Fabio Baltieri fabio.balti...@gmail.com
---
drivers/hwmon/ina2xx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/hwmon/ina2xx.c b/drivers/hwmon/ina2xx.c
index 93d26e8..d994280 100644
at constant rate.
Signed-off-by: Fabio Baltieri
---
Changes from v1:
- fixed some wrong indentations
net/mac80211/ieee80211_i.h | 1 -
net/mac80211/led.c | 19 +++
net/mac80211/led.h | 2 +-
net/mac80211/status.c | 2 +-
net/mac80211/tx.c | 1 -
5
e code is simpler, so ... :)
Good enough... I'll send the v2 then. :-)
Thanks,
Fabio
--
Fabio Baltieri
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordom
... :)
Good enough... I'll send the v2 then. :-)
Thanks,
Fabio
--
Fabio Baltieri
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http
at constant rate.
Signed-off-by: Fabio Baltieri fabio.balti...@gmail.com
---
Changes from v1:
- fixed some wrong indentations
net/mac80211/ieee80211_i.h | 1 -
net/mac80211/led.c | 19 +++
net/mac80211/led.h | 2 +-
net/mac80211/status.c | 2 +-
net/mac80211
On Wed, Jul 24, 2013 at 10:14:25AM +0200, Johannes Berg wrote:
> On Wed, 2013-07-24 at 02:09 +0200, Fabio Baltieri wrote:
> > Changes mac80211 LED trigger code to use the generic
> > led_trigger_blink_oneshot() API for transmit and receive activity
> > indication.
> &g
On Wed, Jul 24, 2013 at 10:14:25AM +0200, Johannes Berg wrote:
On Wed, 2013-07-24 at 02:09 +0200, Fabio Baltieri wrote:
Changes mac80211 LED trigger code to use the generic
led_trigger_blink_oneshot() API for transmit and receive activity
indication.
This gives a better feedback
at constant rate.
Signed-off-by: Fabio Baltieri
---
net/mac80211/ieee80211_i.h | 1 -
net/mac80211/led.c | 21 +
net/mac80211/led.h | 2 +-
net/mac80211/status.c | 2 +-
net/mac80211/tx.c | 1 -
5 files changed, 11 insertions(+), 16 deletions
result for an activity
indicator: one visible blink for sporadic events and a steady on-off
blink sequence on constant traffic.
Would you consider this patch for wireless-next?
Thanks,
Fabio
Fabio Baltieri (1):
mac80211: use oneshot blink API for LED triggers
net/mac80211/ieee80211_i.h | 1 -
net
result for an activity
indicator: one visible blink for sporadic events and a steady on-off
blink sequence on constant traffic.
Would you consider this patch for wireless-next?
Thanks,
Fabio
Fabio Baltieri (1):
mac80211: use oneshot blink API for LED triggers
net/mac80211/ieee80211_i.h | 1 -
net
at constant rate.
Signed-off-by: Fabio Baltieri fabio.balti...@gmail.com
---
net/mac80211/ieee80211_i.h | 1 -
net/mac80211/led.c | 21 +
net/mac80211/led.h | 2 +-
net/mac80211/status.c | 2 +-
net/mac80211/tx.c | 1 -
5 files changed, 11
Add missing static qualifiers to hash_process_data and hash_hw_final.
Signed-off-by: Fabio Baltieri
---
drivers/crypto/ux500/hash/hash_core.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/crypto/ux500/hash/hash_core.c
b/drivers/crypto/ux500/hash/hash_core.c
Add missing __iomem to struct cryp_register pointers, this solve some
"incorrect type in initializer (different address spaces)" sparse
warnings.
Signed-off-by: Fabio Baltieri
---
drivers/crypto/ux500/cryp/cryp.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
Use dmaengine_prep_slave_sg inline function instead of going through the
structures manually.
Signed-off-by: Fabio Baltieri
---
drivers/crypto/ux500/cryp/cryp_core.c | 20 ++--
drivers/crypto/ux500/hash/hash_core.c | 4 ++--
2 files changed, 12 insertions(+), 12 deletions
Use dmaengine_submit instead of calling desc->tx_submit manually.
Signed-off-by: Fabio Baltieri
---
drivers/crypto/ux500/cryp/cryp_core.c | 2 +-
drivers/crypto/ux500/hash/hash_core.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/crypto/ux500/cryp/cryp_core.c
Use dmaengine_device_control inline function instead of going through the
structures manually.
Signed-off-by: Fabio Baltieri
---
drivers/crypto/ux500/cryp/cryp_core.c | 4 ++--
drivers/crypto/ux500/hash/hash_core.c | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers
Always use readl when reading memory mapped registers.
Signed-off-by: Fabio Baltieri
---
drivers/crypto/ux500/hash/hash_core.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/drivers/crypto/ux500/hash/hash_core.c
b/drivers/crypto/ux500/hash
.
Thanks,
Fabio
Fabio Baltieri (6):
crypto: ux500/hash: use readl on iomem addresses
crypto: ux500/hash: add missing static qualifiers
crypto: ux500/crypt: add missing __iomem qualifiers
crypto: ux500: use dmaengine_device_control API
crypto: ux500: use dmaengine_prep_slave_sg API
.
Thanks,
Fabio
Fabio Baltieri (6):
crypto: ux500/hash: use readl on iomem addresses
crypto: ux500/hash: add missing static qualifiers
crypto: ux500/crypt: add missing __iomem qualifiers
crypto: ux500: use dmaengine_device_control API
crypto: ux500: use dmaengine_prep_slave_sg API
Always use readl when reading memory mapped registers.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
drivers/crypto/ux500/hash/hash_core.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/drivers/crypto/ux500/hash/hash_core.c
b/drivers
Use dmaengine_device_control inline function instead of going through the
structures manually.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
drivers/crypto/ux500/cryp/cryp_core.c | 4 ++--
drivers/crypto/ux500/hash/hash_core.c | 2 +-
2 files changed, 3 insertions(+), 3 deletions
Use dmaengine_prep_slave_sg inline function instead of going through the
structures manually.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
drivers/crypto/ux500/cryp/cryp_core.c | 20 ++--
drivers/crypto/ux500/hash/hash_core.c | 4 ++--
2 files changed, 12
Use dmaengine_submit instead of calling desc-tx_submit manually.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
drivers/crypto/ux500/cryp/cryp_core.c | 2 +-
drivers/crypto/ux500/hash/hash_core.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/crypto
Add missing __iomem to struct cryp_register pointers, this solve some
incorrect type in initializer (different address spaces) sparse
warnings.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
drivers/crypto/ux500/cryp/cryp.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions
Add missing static qualifiers to hash_process_data and hash_hw_final.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
drivers/crypto/ux500/hash/hash_core.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/crypto/ux500/hash/hash_core.c
b/drivers/crypto
On Fri, Jun 21, 2013 at 08:40:00AM +0530, Vinod Koul wrote:
> On Thu, Jun 20, 2013 at 11:17:39AM +0200, Fabio Baltieri wrote:
> > Fix sparse warnings:
> >
> > drivers/dma/ste_dma40.c:81:26: warning: symbol 'dma40_memcpy_conf_phy' was
> > not declared. Should it
On Fri, Jun 21, 2013 at 08:40:00AM +0530, Vinod Koul wrote:
On Thu, Jun 20, 2013 at 11:17:39AM +0200, Fabio Baltieri wrote:
Fix sparse warnings:
drivers/dma/ste_dma40.c:81:26: warning: symbol 'dma40_memcpy_conf_phy' was
not declared. Should it be static?
drivers/dma/ste_dma40.c:95:26
Fix sparse warnings:
drivers/dma/ste_dma40.c:81:26: warning: symbol 'dma40_memcpy_conf_phy' was not
declared. Should it be static?
drivers/dma/ste_dma40.c:95:26: warning: symbol 'dma40_memcpy_conf_log' was not
declared. Should it be static?
Signed-off-by: Fabio Baltieri
---
Hi Linus,
just
o... Is there a specific reasion to use the output register instead?
Thanks,
Fabio
> if (ret < 0) {
> dev_err(pct->dev, "%s failed\n", __func__);
> return ret;
> --
> 1.7.10
>
--
Fabio Baltieri
--
To unsubscribe from this list
other conditions in the same 'switch', it may make
sense to have a patch just for that.
Thanks,
Fabio
> /*
>* if argument = 1 set the pull up
>* else clear the pull up
> @@ -1030,8 +1051,6 @@ static int abx500_pin_config_set(struct pinctrl_dev
> *
(ret >= 0)
> + reserve = 1;
> +
> + ret = pinconf_generic_parse_dt_config(np, , );
> + if (nconfigs)
> + has_config = 1;
> +
> + np_config = of_parse_phandle(np, "ste,config", 0);
> + if (np_config) {
> +
,
.get_group_pins = abx500_get_group_pins,
.pin_dbg_show = abx500_pin_dbg_show,
+ .dt_node_to_map = abx500_dt_node_to_map,
+ .dt_free_map = abx500_dt_free_map,
};
static int abx500_pin_config_get(struct pinctrl_dev *pctldev,
--
1.7.10
--
Fabio Baltieri
configuration requested\n);
-
- return -EINVAL;
}
return ret;
--
1.7.10
--
Fabio Baltieri
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org
(pct-dev, %s failed\n, __func__);
return ret;
--
1.7.10
--
Fabio Baltieri
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please
Fix sparse warnings:
drivers/dma/ste_dma40.c:81:26: warning: symbol 'dma40_memcpy_conf_phy' was not
declared. Should it be static?
drivers/dma/ste_dma40.c:95:26: warning: symbol 'dma40_memcpy_conf_log' was not
declared. Should it be static?
Signed-off-by: Fabio Baltieri fabio.balti
On Mon, Jun 17, 2013 at 05:42:48PM +0200, Linus Walleij wrote:
> On Thu, Jun 13, 2013 at 3:56 PM, Fabio Baltieri
> wrote:
>
> > Set coherent_dma_mask to DMA_BIT_MASK(32) for dma40 platform_device, as
> > without this DMA allocations were failing with the error:
> >
On Mon, Jun 17, 2013 at 05:42:48PM +0200, Linus Walleij wrote:
On Thu, Jun 13, 2013 at 3:56 PM, Fabio Baltieri
fabio.balti...@linaro.org wrote:
Set coherent_dma_mask to DMA_BIT_MASK(32) for dma40 platform_device, as
without this DMA allocations were failing with the error:
dma40
: In function 'pl022_resume_resources':
drivers/spi/spi-pl022.c:2334:6: warning: unused variable 'ret'
[-Wunused-variable]
introduced in:
f1c9cf0 spi: pl022: use pinctrl PM helpers
Signed-off-by: Fabio Baltieri
---
Hi Linus,
this is just a quick build warning fixup patch found while playing with
today's
: In function 'pl022_resume_resources':
drivers/spi/spi-pl022.c:2334:6: warning: unused variable 'ret'
[-Wunused-variable]
introduced in:
f1c9cf0 spi: pl022: use pinctrl PM helpers
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
Hi Linus,
this is just a quick build warning fixup patch found
Add necessary code to restart ux500 based machines using
prcmu_system_reset().
Signed-off-by: Fabio Baltieri
---
Hi Linus,
this is something I had in my tree for some time. This adds basic
soft-reboot support for all ux500 machines using prcmu (the actual
reboot code was already
Add necessary code to restart ux500 based machines using
prcmu_system_reset().
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
Hi Linus,
this is something I had in my tree for some time. This adds basic
soft-reboot support for all ux500 machines using prcmu (the actual
reboot code
Set coherent_dma_mask to DMA_BIT_MASK(32) for dma40 platform_device, as
without this DMA allocations were failing with the error:
dma40 dma40.0: coherent DMA mask is unset
when booting without device-tree.
Signed-off-by: Fabio Baltieri
---
Hi Linus, Lee,
I found this while removing the last
Set coherent_dma_mask to DMA_BIT_MASK(32) for dma40 platform_device, as
without this DMA allocations were failing with the error:
dma40 dma40.0: coherent DMA mask is unset
when booting without device-tree.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
Hi Linus, Lee,
I found
Add a field with the tx/rx register address to the DMA parameters
structure, and set it to the correct address during device
initialization.
This address used to be hardcoded in the DMA controller driver, it now
needs to be explicitly figured out by the device driver.
Signed-off-by: Fabio
Implement a DMA slave config prepare routine, as until now the MSP
driver depended on the DMA controller completing the channel
configuration on its own, but this is not the case anymore since the
recent DMA driver updates.
Signed-off-by: Fabio Baltieri
---
sound/soc/ux500/ux500_pcm.c | 30
Move struct ux500_msp_dma_params declaration from ux500_msp_i2s_drvdata
to ux500_msp, this saves some confusing pointer passing and allows to
access all DMA configuration fields from ux500_msp_i2s.
Signed-off-by: Fabio Baltieri
---
sound/soc/ux500/ux500_msp_dai.c | 11 ---
sound/soc
er to make it work with the current
dma40 implementation, and works fine even when booting with device-tree.
Would you consider merging these for asoc/topic/ux500?
Thanks,
Fabio
Fabio Baltieri (3):
ASoC: ux500: Move DMA parameters into ux500_msp
ASoC: ux500: Set DMA address during device i
it work with the current
dma40 implementation, and works fine even when booting with device-tree.
Would you consider merging these for asoc/topic/ux500?
Thanks,
Fabio
Fabio Baltieri (3):
ASoC: ux500: Move DMA parameters into ux500_msp
ASoC: ux500: Set DMA address during device init
ASoC: ux500
Implement a DMA slave config prepare routine, as until now the MSP
driver depended on the DMA controller completing the channel
configuration on its own, but this is not the case anymore since the
recent DMA driver updates.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
sound/soc
Move struct ux500_msp_dma_params declaration from ux500_msp_i2s_drvdata
to ux500_msp, this saves some confusing pointer passing and allows to
access all DMA configuration fields from ux500_msp_i2s.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
sound/soc/ux500/ux500_msp_dai.c | 11
Add a field with the tx/rx register address to the DMA parameters
structure, and set it to the correct address during device
initialization.
This address used to be hardcoded in the DMA controller driver, it now
needs to be explicitly figured out by the device driver.
Signed-off-by: Fabio
On Tue, Jun 04, 2013 at 11:03:41AM +0200, Linus Walleij wrote:
> On Thu, May 30, 2013 at 3:27 PM, Fabio Baltieri
> wrote:
>
> > Ux500 ASoC driver is expected to have a specific device name to get
> > clock resources correctly. This patch provides the necessary
> &
On Tue, Jun 04, 2013 at 11:03:41AM +0200, Linus Walleij wrote:
On Thu, May 30, 2013 at 3:27 PM, Fabio Baltieri
fabio.balti...@linaro.org wrote:
Ux500 ASoC driver is expected to have a specific device name to get
clock resources correctly. This patch provides the necessary
OF_DEV_AUXDATA
Fix typo of VAMIC2 LDO regulator name in some DT-related files. This
patch replaces all occurrences with the right name.
Signed-off-by: Fabio Baltieri
---
arch/arm/boot/dts/dbx5x0.dtsi | 4 ++--
arch/arm/boot/dts/href.dtsi | 2 +-
arch/arm/boot/dts/hrefv60plus.dts | 2 +-
arch/arm
Ux500 ASoC driver is expected to have a specific device name to get
clock resources correctly. This patch provides the necessary
OF_DEV_AUXDATA to match the name in DT and non-DT cases.
Signed-off-by: Fabio Baltieri
---
arch/arm/mach-ux500/cpu-db8500.c | 2 ++
1 file changed, 2 insertions
Add of_compatible string to the ab8500-codec cell to allow the driver to
grab handlers such as regulators from device-tree when available.
Signed-off-by: Fabio Baltieri
---
drivers/mfd/ab8500-core.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/mfd/ab8500-core.c b/drivers/mfd
Add regulator DT bindings for the ab8500-codec driver.
Signed-off-by: Fabio Baltieri
---
arch/arm/boot/dts/dbx5x0.dtsi | 5 +
1 file changed, 5 insertions(+)
diff --git a/arch/arm/boot/dts/dbx5x0.dtsi b/arch/arm/boot/dts/dbx5x0.dtsi
index d0b6e32..8e14042 100644
--- a/arch/arm/boot/dts
Fix VAMIC1 LDO comment in DT files to be make it coherent with the
others.
Signed-off-by: Fabio Baltieri
---
arch/arm/boot/dts/dbx5x0.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/dbx5x0.dtsi b/arch/arm/boot/dts/dbx5x0.dtsi
index b6bc4ff..e6ea7ba
safely go
through arm-soc, as there are no build dependencies or regression issues
in general.
Thanks,
Fabio
Fabio Baltieri (5):
ARM: ux500: Fix trivial typo in v-anamic1 comment
ARM: ux500: Correct anamic2 typo in DT files
ARM: ux500: Add DT regulators for ab8500-codec
ARM: ux500: Provide
safely go
through arm-soc, as there are no build dependencies or regression issues
in general.
Thanks,
Fabio
Fabio Baltieri (5):
ARM: ux500: Fix trivial typo in v-anamic1 comment
ARM: ux500: Correct anamic2 typo in DT files
ARM: ux500: Add DT regulators for ab8500-codec
ARM: ux500: Provide
Fix VAMIC1 LDO comment in DT files to be make it coherent with the
others.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
arch/arm/boot/dts/dbx5x0.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/dbx5x0.dtsi b/arch/arm/boot/dts/dbx5x0.dtsi
Add regulator DT bindings for the ab8500-codec driver.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
arch/arm/boot/dts/dbx5x0.dtsi | 5 +
1 file changed, 5 insertions(+)
diff --git a/arch/arm/boot/dts/dbx5x0.dtsi b/arch/arm/boot/dts/dbx5x0.dtsi
index d0b6e32..8e14042 100644
Add of_compatible string to the ab8500-codec cell to allow the driver to
grab handlers such as regulators from device-tree when available.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
drivers/mfd/ab8500-core.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/mfd/ab8500
Ux500 ASoC driver is expected to have a specific device name to get
clock resources correctly. This patch provides the necessary
OF_DEV_AUXDATA to match the name in DT and non-DT cases.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
arch/arm/mach-ux500/cpu-db8500.c | 2 ++
1 file
Fix typo of VAMIC2 LDO regulator name in some DT-related files. This
patch replaces all occurrences with the right name.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
arch/arm/boot/dts/dbx5x0.dtsi | 4 ++--
arch/arm/boot/dts/href.dtsi | 2 +-
arch/arm/boot/dts
that the two device are
configure consistently. The check is added at machine driver level, as
how to lock DAI configuration depend of the actual hardware
implementation.
Signed-off-by: Fabio Baltieri
---
sound/soc/ux500/mop500_ab8500.c | 35 +++
1 file changed, 35
that the two device are
configure consistently. The check is added at machine driver level, as
how to lock DAI configuration depend of the actual hardware
implementation.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
sound/soc/ux500/mop500_ab8500.c | 35
On Mon, May 27, 2013 at 01:44:38PM +0200, Linus Walleij wrote:
> On Fri, May 24, 2013 at 3:28 PM, Fabio Baltieri
> wrote:
>
> > Enable ux500 specific ALSA SoC drivers by default on u8500_defconfig.
> >
> > Signed-off-by: Fabio Baltieri
>
> Thanks, patch applied
On Mon, May 27, 2013 at 01:44:38PM +0200, Linus Walleij wrote:
On Fri, May 24, 2013 at 3:28 PM, Fabio Baltieri
fabio.balti...@linaro.org wrote:
Enable ux500 specific ALSA SoC drivers by default on u8500_defconfig.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
Thanks, patch
Enable ux500 specific ALSA SoC drivers by default on u8500_defconfig.
Signed-off-by: Fabio Baltieri
---
Hi Linus,
As I noticed that you just updated your ux500-defconfig branch, and the ux500
sound driver should be working fine in the next kernel, I'm sending a patch to
enable it by default
Drop struct i2s_controller from the ux500 ASoC driver as right now it is
instantiated but not used anywhere. Also drop a mismatched
device_unregister in the process.
Signed-off-by: Fabio Baltieri
---
sound/soc/ux500/ux500_msp_i2s.c | 19 ---
sound/soc/ux500/ux500_msp_i2s.h | 12
Define ab8500 codec operations structure on its own rather than inline
with snd_soc_dai_drivers to clean up the code and make the style
coherent with other codec drivers.
Signed-off-by: Fabio Baltieri
---
sound/soc/codecs/ab8500-codec.c | 19 +++
1 file changed, 7 insertions
-off-by: Fabio Baltieri
---
sound/soc/ux500/ux500_msp_i2s.c | 56 ++---
sound/soc/ux500/ux500_msp_i2s.h | 6 -
2 files changed, 2 insertions(+), 60 deletions(-)
diff --git a/sound/soc/ux500/ux500_msp_i2s.c b/sound/soc/ux500/ux500_msp_i2s.c
index f2db6c9
Ux500 has two equivalent enum for device id, one in platform_data and
one in a local header. Fix this by dropping the local one.
Signed-off-by: Fabio Baltieri
---
sound/soc/ux500/ux500_msp_i2s.h | 10 ++
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/sound/soc/ux500
Add a missing include that was resulting in some sparse warning for
non-static structure without forward declaration.
Signed-off-by: Fabio Baltieri
---
sound/soc/ux500/mop500_ab8500.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/sound/soc/ux500/mop500_ab8500.c b/sound/soc/ux500
Drop unused fields and structures from ux500_msp_i2s header file, as
those looks like leftover from a previous implementation of the driver.
Signed-off-by: Fabio Baltieri
---
sound/soc/ux500/ux500_msp_dai.h | 2 --
sound/soc/ux500/ux500_msp_i2s.h | 31 ---
2 files
a bunch of sparse fixes and generic code cleanup and
dead code removal.
Thanks,
Fabio
Fabio Baltieri (6):
ASoC: ux500: Drop pinctrl sleep support
ASoC: ab8500-codec: Move codec ops on a separate structure
ASoC: ux500: Drop dangling struct i2s_controller
ASoC: ux500: Drop unused code from msp
a bunch of sparse fixes and generic code cleanup and
dead code removal.
Thanks,
Fabio
Fabio Baltieri (6):
ASoC: ux500: Drop pinctrl sleep support
ASoC: ab8500-codec: Move codec ops on a separate structure
ASoC: ux500: Drop dangling struct i2s_controller
ASoC: ux500: Drop unused code from msp
Drop unused fields and structures from ux500_msp_i2s header file, as
those looks like leftover from a previous implementation of the driver.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
sound/soc/ux500/ux500_msp_dai.h | 2 --
sound/soc/ux500/ux500_msp_i2s.h | 31
Ux500 has two equivalent enum for device id, one in platform_data and
one in a local header. Fix this by dropping the local one.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
sound/soc/ux500/ux500_msp_i2s.h | 10 ++
1 file changed, 2 insertions(+), 8 deletions(-)
diff
Add a missing include that was resulting in some sparse warning for
non-static structure without forward declaration.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
sound/soc/ux500/mop500_ab8500.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/sound/soc/ux500/mop500_ab8500.c b
-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
sound/soc/ux500/ux500_msp_i2s.c | 56 ++---
sound/soc/ux500/ux500_msp_i2s.h | 6 -
2 files changed, 2 insertions(+), 60 deletions(-)
diff --git a/sound/soc/ux500/ux500_msp_i2s.c b/sound/soc/ux500
Define ab8500 codec operations structure on its own rather than inline
with snd_soc_dai_drivers to clean up the code and make the style
coherent with other codec drivers.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
sound/soc/codecs/ab8500-codec.c | 19 +++
1 file
Drop struct i2s_controller from the ux500 ASoC driver as right now it is
instantiated but not used anywhere. Also drop a mismatched
device_unregister in the process.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
sound/soc/ux500/ux500_msp_i2s.c | 19 ---
sound/soc
Enable ux500 specific ALSA SoC drivers by default on u8500_defconfig.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
Hi Linus,
As I noticed that you just updated your ux500-defconfig branch, and the ux500
sound driver should be working fine in the next kernel, I'm sending a patch
Replace hard coded rx slot numbers from ab8500_codec_set_dai_tdm_slot
using the ones requested by the machine driver in rx_mask instead.
Signed-off-by: Fabio Baltieri
---
sound/soc/codecs/ab8500-codec.c | 29 -
sound/soc/codecs/ab8500-codec.h | 35
Replace hard-coded tx slot numbers from ab8500_codec_set_dai_tdm_slot
using the ones requested by the machine driver in tx_mask instead.
Signed-off-by: Fabio Baltieri
---
sound/soc/codecs/ab8500-codec.c | 31 +++
sound/soc/codecs/ab8500-codec.h | 7 +++
2 files
for helping me figuring out the weird ab8500 slot
mapping logic.
Fabio
Fabio Baltieri (2):
ASoC: ab8500-codec: Set tx dai slots from tx_mask
ASoC: ab8500-codec: Set rx dai slots from rx_mask
sound/soc/codecs/ab8500-codec.c | 60 ++---
sound/soc/codecs/ab8500
Hi Samuel,
On Tue, May 21, 2013 at 10:56:50AM +0200, Samuel Ortiz wrote:
> Hi Fabio,
>
> On Fri, May 17, 2013 at 09:51:40AM +0200, Fabio Baltieri wrote:
> > Hello Samuel,
> >
> > On Fri, May 17, 2013 at 12:43:37AM +0200, Samuel Ortiz wrote:
> > > > Fabio
Hi Samuel,
On Tue, May 21, 2013 at 10:56:50AM +0200, Samuel Ortiz wrote:
Hi Fabio,
On Fri, May 17, 2013 at 09:51:40AM +0200, Fabio Baltieri wrote:
Hello Samuel,
On Fri, May 17, 2013 at 12:43:37AM +0200, Samuel Ortiz wrote:
Fabio Baltieri (5):
mfd: abx500-core: Fix sparse
for helping me figuring out the weird ab8500 slot
mapping logic.
Fabio
Fabio Baltieri (2):
ASoC: ab8500-codec: Set tx dai slots from tx_mask
ASoC: ab8500-codec: Set rx dai slots from rx_mask
sound/soc/codecs/ab8500-codec.c | 60 ++---
sound/soc/codecs/ab8500
Replace hard-coded tx slot numbers from ab8500_codec_set_dai_tdm_slot
using the ones requested by the machine driver in tx_mask instead.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
sound/soc/codecs/ab8500-codec.c | 31 +++
sound/soc/codecs/ab8500
Replace hard coded rx slot numbers from ab8500_codec_set_dai_tdm_slot
using the ones requested by the machine driver in rx_mask instead.
Signed-off-by: Fabio Baltieri fabio.balti...@linaro.org
---
sound/soc/codecs/ab8500-codec.c | 29 -
sound/soc/codecs/ab8500-codec.h
On Fri, May 17, 2013 at 10:25:10AM +0200, Linus Walleij wrote:
> On Fri, May 17, 2013 at 9:51 AM, Fabio Baltieri
> wrote:
> > Hello Samuel,
> (...)
> > The last two:
> >
> > mfd: ab8500-sysctrl: Set sysctrl_dev during probe
> > mfd: ab8500-sysctrl
On Wed, May 15, 2013 at 09:35:17PM +0200, Linus Walleij wrote:
> On Wed, May 8, 2013 at 11:09 AM, Fabio Baltieri
> wrote:
>
> > Update MSP1 pinctrl definitions in mop500_family_pinmap by removing
> > sleep state and setting default ones as pin hogs, as those are used by
>
1 - 100 of 664 matches
Mail list logo
