-by: Ben Gardiner ben.l.gardi...@gmail.com
---
drivers/spi/spi-davinci.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/spi/spi-davinci.c b/drivers/spi/spi-davinci.c
index 707966b..f0f4dbd 100644
--- a/drivers/spi/spi-davinci.c
+++ b/drivers/spi/spi-davinci.c
@@ -404,6 +404,7 @@ static
Hi Trent,
Thanks for the quick review.
On Tue, Sep 10, 2013 at 10:44 PM, Trent Piepho tpie...@gmail.com wrote:
It is supposed to be possible to call setup() on one slave while
transfers on another slave attached to the same master are in
progress.
A cursory look at the code makes it seem
but I have no
hands-on experience yet with CMA.
Is there any other means by which the hole's location and size can be
specified which does not require patching this file? I imagine KConfig
would work, but is this an acceptable use of KConfig?
Best Regards,
Ben Gardiner
---
Nanometrics Inc.
http
On Tue, Oct 2, 2012 at 7:13 AM, Sekhar Nori nsek...@ti.com wrote:
On 10/1/2012 7:20 PM, Ben Gardiner wrote:
On Mon, Oct 1, 2012 at 8:32 AM, Matt Porter mpor...@ti.com wrote:
On Mon, Oct 01, 2012 at 05:34:02PM +0530, Sekhar Nori wrote:
Hi Matt,
On 9/29/2012 1:07 AM, Matt Porter wrote:
L3RAM
and not because of
any particular technical merits.
Best Regards,
Ben Gardiner
[1] https://patchwork.kernel.org/patch/840552/
[2] https://patchwork.kernel.org/patch/549351/
---
Nanometrics Inc.
http://www.nanometrics.ca
___
Davinci-linux-open-source mailing list
and hence
applicable to devices other than mtd.
Again, I apologize for requesting features for boards that are not yet
in mainline.
Best Regards,
Ben Gardiner
[1] http://article.gmane.org/gmane.linux.drivers.mtd/36876/
---
Nanometrics Inc.
http://www.nanometrics.ca
Hi Christophe,
On Tue, Oct 25, 2011 at 8:58 AM, Christophe Aeschlimann
c.aeschlim...@acn-group.ch wrote:
Hi Ben,
Sorry for bumping that old thread out of the ash :)
On 11.02.2011 15:34, Ben Gardiner wrote:
Hi Sudhakar,
On Fri, Feb 11, 2011 at 6:03 AM, Rajashekhara, Sudhakar
sudhakar
in the application. Returning _POS_XRUN should
be sufficient here in ASoC.
Best Regards,
Ben Gardiner
---
Nanometrics Inc.
http://www.nanometrics.ca
___
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com
wrong -- please help me out here.
If there is an established method of bubbling HW errors up to userspace that I
completely missed: I am sorry for the noise and would be glad to hear about how
to
use it with davinci-pcm.
Best Regards,
Ben Gardiner
Ben Gardiner (3):
[RFC] ASoC: davinci-pcm
to the cpudai instance from a substream (sorry, please help!)
[not ready yet, please comment]
Not-Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
---
sound/soc/davinci/davinci-pcm.c |8 +++-
sound/soc/davinci/davinci-pcm.h |3 +++
2 files changed, 10 insertions(+), 1 deletions
The McASP has RERR and XERR bits in its RSTAT and XSTAT registers which
report the OR'd state of several potential errors.
Register a function to check the status of these bits and report HW errors
back up the stack.
[not ready yet, please comment]
Not-Signed-off-by: Ben Gardiner bengardi
by a function -- in these cases, rename to last_link.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
---
sound/soc/davinci/davinci-pcm.c | 123 +++
1 files changed, 59 insertions(+), 64 deletions(-)
diff --git a/sound/soc/davinci/davinci-pcm.c b/sound/soc
/sprufm1/sprufm1.pdf
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
---
Tested with a logic analyzer on da850evm where an unsigned 16bit codec was
registered with the mcasp and a 16bit signed file was played to it. ALSA's
plug pcm mapped to unsigned and the samples serialized as unsigned 16bit
this series on-top of branch master @
a0cb2d282a57fd33ae57af2f2092974c82111b77 in
git://gitorious.org/linux-davinci/linux-davinci.git on da850evm and heliosv1
hardware.
Ben Gardiner (5):
ARM: davinci: da850, omapl138: introduce shared da850 reference
components
ARM: davinci: da850, omapl138
Extract the value of .boot_params used by the da850/omapl138 machines to
a common DA850_REFERENCE_BOOT_PARAMS defined in the da850-reference.h
header introduced in a previous commit.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: James Nuss jamesn...@nanometrics.ca
---
arch
to sharing a definition
here.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: James Nuss jamesn...@nanometrics.ca
---
arch/arm/mach-davinci/Makefile |3 ++-
arch/arm/mach-davinci/board-da850-evm.c|8 ++--
arch/arm/mach-davinci/board
Add support for the Nanometrics Helios Platform V1 Hardware. Initially
support includes only serial console and watchdog support.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: James Nuss jamesn...@nanometrics.ca
---
arch/arm/mach-davinci/Kconfig |9
Extract the common davinci_uart_config.enabled_uarts=0x7 passed to
davinci_serial_init() from all of the da850/omapl138 boards to the
da850-reference object introduced in a previous commit.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: James Nuss jamesn...@nanometrics.ca
Extract the common add_preferred_console(ttyS, 2, 115200) found both of
the da850evm and hawk boards into a shared function in the da850-reference
object introduced in a previous commit.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: James Nuss jamesn...@nanometrics.ca
);
spin_lock_irq(davinci_nand_lock);
--
1.7.1
___
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
--
Best Regards,
Ben Gardiner
Hi Sekhar,
On Tue, Aug 2, 2011 at 12:00 PM, Nori, Sekhar nsek...@ti.com wrote:
Hi Ben,
On Tue, Aug 02, 2011 at 18:32:39, Ben Gardiner wrote:
Hi Sudhakar,
On Thu, Jul 21, 2011 at 5:39 AM, Rajashekhara, Sudhakar
sudhakar@ti.com wrote:
This patch adds cpufreq support for NAND driver
for EDMA CC1 on DA850 for
which there are no current users in mainline.
So, not sending for v3.1.
If it's not too late:
Tested this patch and it fixes SD/MMC1 support on da850.
Tested-by: Ben Gardiner bengardi...@nanometrics.ca
Thanks, Ido!
Best Regards,
Ben Gardiner
---
Nanometrics Inc.
http
before registering with gen_pool_add_virt().
This commit breaks runtime of davinci boards since the regions that
the sram init is now trying to ioremap have been iomapped by their
iotable entries. The iotable entries will be removed in the patches
to come.
Signed-off-by: Ben Gardiner bengardi
sram region is changed from
the ARM local RAM region to the Shared RAM region. This change is needed
to support mcasp ping-pong buffers on da850. Suspend was tested with rtcwake
and was found to work.
Ben Gardiner (2):
[v3] davinci: sram: ioremap the davinci_soc_info specified sram
regions
From: Subhasish Ghosh subhas...@mistralsolutions.com
This patch modifies the sram allocator to allocate memory
from the DA8XX shared RAM.
Signed-off-by: Subhasish Ghosh subhas...@mistralsolutions.com
[rebased onto consolidated SRAM patches]
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
of SRAM_VIRT, also remove
the SRAM_VIRT definition.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: Sergei Shtylyov sshtyl...@mvista.com
Acked-by: Jean-Christophe PLAGNIOL-VILLARD plagn...@jcrosoft.com
---
Changes since v2:
* Added Jean-Christophe PLAGNIOL-VILLARD's Ack
buffers on da850. Suspend was tested with rtcwake
and was found to work.
Ben Gardiner (2):
[v2] davinci: sram: ioremap the davinci_soc_info specified sram
regions
[v2] davinci: da850-dm646x: remove the SRAM_VIRT iotable entry
Subhasish Ghosh (1):
[v2] davinci: da850: changed SRAM allocator
of SRAM_VIRT, also remove
the SRAM_VIRT definition.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: Sergei Shtylyov sshtyl...@mvista.com
---
Changes since v1:
* squashed patches 3-8 in v1 series into this patch (Sergei Shtylyov)
---
arch/arm/mach-davinci/da850.c
before registering with pv_pool_create().
This commit breaks runtime of davinci boards since the regions that
the sram init is now trying to ioremap have been iomapped by their
iotable entries. The iotable entries will be removed in the patches
to come.
Signed-off-by: Ben Gardiner bengardi
From: Subhasish Ghosh subhas...@mistralsolutions.com
This patch modifies the sram allocator to allocate memory
from the DA8XX shared RAM.
Signed-off-by: Subhasish Ghosh subhas...@mistralsolutions.com
[rebased onto consolidated SRAM patches]
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Hi Liam,
On Wed, May 25, 2011 at 5:52 AM, Liam Girdwood l...@ti.com wrote:
On 24/05/11 19:50, Ben Gardiner wrote:
[...]
@@ -603,15 +605,19 @@ static int davinci_pcm_prepare(struct
snd_pcm_substream *substream)
print_buf_info(prtd-asp_link[0], asp_link[0
In the previous commit 'ASoC: davinci-pcm: convert to BATCH mode', the phase
offset of 2 was mentioned in the commit message but not well commented in the
source.
Add descriptive comments of the phase offset with and without ping-pong
buffers enabled.
Signed-off-by: Ben Gardiner bengardi
there appear to be fewer
syscalls per execution of aplay.
[1] http://article.gmane.org/gmane.linux.alsa.devel/85122
Ben Gardiner (6):
ASoC: davinci-pcm: trivial: make ping-pong params setup symmetrical
ASoC: davinci-pcm: expand the .formats
ASoC: davinci-pcm: increase the maximum channels
ASoC
. It was determined through trial and
error that the following two changes fix this problem:
1) Move the edma_start calls from prepare to trigger and 2) reverse the order
of starting the asp and ram channels.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: Steven Faludi stevenfal
The setup of the pong channel uses EDMA_CHAN_SLOT instead of 0x3f as the
setup of the ping channel does.
Make the setup of ping and pong symmetric. There is no functional change
introduced by this patch.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: Steven Faludi stevenfal
;
}
Update the .format member of the snd_pcm_hardware instances it registers to
reflect this capability.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: Steven Faludi stevenfal...@nanometrics.ca
---
sound/soc/davinci/davinci-pcm.c | 20 ++--
1 files changed, 18
for this
offset -- I opted for the latter since it makes the first-time setup clearer.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: Steven Faludi stevenfal...@nanometrics.ca
---
sound/soc/davinci/davinci-pcm.c | 67 +++
1 files changed, 19
Extract functions that modify the prtd-period member in preparation for
conversion to BATCH mode playback.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: Steven Faludi stevenfal...@nanometrics.ca
---
sound/soc/davinci/davinci-pcm.c | 22 ++
1 files
Based on the registration of davinci-mcasp.1 in the davinci-evm platform
setup for da830 and dm6467, davinci-pcm can handle more than the currently
reported maximum channels of 2.
Increase the maximum channels to 384 to match the maximum reported by
davinci-mcasp.1.
Signed-off-by: Ben Gardiner
keep this list sorted.
Will do so in V2. I am currently imagining that the v2 series will be
based on Russell's next SRAM consolidation patch based on
Jean-Christophe's genpool patch.
Best Regards,
Ben Gardiner
---
Nanometrics Inc.
http://www.nanometrics.ca
The davinci-i2s driver copies the platform data for playback and capture
sram sizes which is in turn used by davinci-pcm to allocate ping-pong
buffers.
Copy also the platform data in davinci-mcasp probe.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
---
sound/soc/davinci/davinci
SRAM allocator to the Shared RAM region are
merged.
[1] http://article.gmane.org/gmane.linux.kernel/1098928
[2] http://article.gmane.org/gmane.linux.ports.arm.kernel/117261
Ben Gardiner (2):
ASoC: davinci-mcasp: enable ping-pong SRAM buffers
davinci: da850evm: enable mcasp ping-pong buffers
to match those
specified by Troy Kisky in his introduction of ping-pong buffers for dm644x.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
---
arch/arm/mach-davinci/board-da850-evm.c |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-davinci/board
it).
Best Regards,
Ben Gardiner
---
Nanometrics Inc.
http://www.nanometrics.ca
___
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
From: Nori, Sekhar nsek...@ti.com
Hi Russell,
On Thu, May 12, 2011 at 23:15:46, Russell King - ARM Linux wrote:
diff --git a/arch/arm/mach-davinci/pm.c b/arch/arm/mach-davinci/pm.c
index 1bd73a0..c2f9767 100644
--- a/arch/arm/mach-davinci/pm.c
+++ b/arch/arm/mach-davinci/pm.c
[...]
@@
RAM region. This change is needed
to support mcasp ping-pong buffers on da850. Suspend was tested with rtcwake
and was found to work.
Ben Gardiner (7):
davinci: sram: ioremap the davinci_soc_info specified sram regions
davinci: da850: remove the SRAM_VIRT iotable entry
davinci: dm355: remove
before registering with pv_pool_create().
This commit breaks runtime of davinci boards since the regions that
the sram init is now trying to ioremap have been iomapped by their
iotable entries. The iotable entries will be removed in the patches
to come.
Signed-off-by: Ben Gardiner bengardi
The sram region defined for da850 in the iotable entry removed is also
defined in its davinci_soc_info, davinci_soc_info_da850 in da850.c.
Remove this duplicate information which is now uneccessary since sram
init will ioremap the region defined by davinci_soc_info_da850.
Signed-off-by: Ben
The sram region defined for dm644x in the iotable entry removed is also
defined in its davinci_soc_info, davinci_soc_info_dm644x in da644x.c.
Remove this duplicate information which is now uneccessary since sram
init will ioremap the region defined by davinci_soc_info_dm644x.
Signed-off-by: Ben
The sram region defined for dm365 in the iotable entry removed is also
defined in its davinci_soc_info, davinci_soc_info_dm365 in dm365.c.
Remove this duplicate information which is now uneccessary since sram
init will ioremap the region defined by davinci_soc_info_dm365.
Signed-off-by: Ben
The sram region defined for dm355 in the iotable entry removed is also
defined in its davinci_soc_info, davinci_soc_info_dm355 in dm355.c.
Remove this duplicate information which is now uneccessary since sram
init will ioremap the region defined by davinci_soc_info_dm355.
Signed-off-by: Ben
The sram region defined for dm646x in the iotable entry removed is also
defined in its davinci_soc_info, davinci_soc_info_dm646x in dm646x.c.
Remove this duplicate information which is now uneccessary since sram
init will ioremap the region defined by davinci_soc_info_dm646x.
Signed-off-by: Ben
Previously all platforms were iomapping their SRAM using iotable entries with
SRAM_VIRT as the target virtual address.
Remove the SRAM_VIRT definition now that all the iotables are removed and
sram init is doing ioremaps before regsitering with pv_pool_create.
Signed-off-by: Ben Gardiner
From: Subhasish Ghosh subhas...@mistralsolutions.com
This patch modifies the sram allocator to allocate memory
from the DA8XX shared RAM.
Signed-off-by: Subhasish Ghosh subhas...@mistralsolutions.com
[rebased onto consolidated SRAM patches]
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
present in the
Documentation/ folder.
Anyway, the easiest way for you to move quickly is
to use ramdisk so you can bypass all driver specific
issues.
I tested suspend here with the patch applied on top of 2.6.39-rc7
using rtcwake -d /dev/rtc0 -s 20 -m mem and it works.
Best Regards,
Ben Gardiner
Hi All,
On Fri, May 13, 2011 at 5:26 PM, Ben Gardiner
bengardi...@nanometrics.ca wrote:
[...]
Does anyone have any clues why ping-pong buffers on da850 to the McASP
would work? Or any clues why the patches are broken?
To answer my own question: the reason ping-pong buffering was not
working
be plenty to spare)
Not-signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
---
arch/arm/configs/da8xx_omapl_defconfig |1 +
arch/arm/mach-davinci/board-da850-evm.c |4 +++-
2 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/arch/arm/configs/da8xx_omapl_defconfig
b/arch/arm
The davinci-i2s driver copies the platform data for playback and capture
sram sizes which is in turn used by davinci-pcm to allocate ping-pong
buffers.
Copy also the platform data in the davinci-mcasp probe().
Not-signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
---
sound/soc/davinci
added pointer debug and observed that the
period_update and hwptr_update events progress; but that's as far as I
have probed.
Does anyone have any clues why ping-pong buffers on da850 to the McASP
would work? Or any clues why the patches are broken?
Best Regards,
Ben Gardiner
Ben Gardiner (2
On Tue, Apr 26, 2011 at 6:46 AM, Mark Brown
broo...@opensource.wolfsonmicro.com wrote:
On Thu, Apr 21, 2011 at 02:19:01PM -0400, Ben Gardiner wrote:
The current check for the number of tdm-slots specified by platform data is
always true (x = 2 || x = 32); therefore the else branch that warns
The current driver creates value for set/clr of PDIR using (x26) instead
of the #defines that are convieniently made available.
Update the driver to use the bitfield definitions of PDIR. There is no
functional change introduced by this patch.
Signed-off-by: Ben Gardiner bengardi
in this series, fix _CBM_CFS hw_params --
For codec clock-master/frame-slave mode (_CMB_CFS), clear bits ACLKX and ACLKR
in the PDIR register to set the pins as inputs and hence allow externally
sourced bit-clocks.
[1] http://www.ti.com/litv/pdf/sprufm1
Signed-off-by: Ben Gardiner bengardi
/pdf/sprufm1
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: James Nuss jamesn...@nanometrics.ca
---
sound/soc/davinci/davinci-mcasp.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/soc/davinci/davinci-mcasp.c
b/sound/soc/davinci/davinci-mcasp.c
-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: James Nuss jamesn...@nanometrics.ca
---
sound/soc/davinci/davinci-mcasp.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/soc/davinci/davinci-mcasp.c
b/sound/soc/davinci/davinci-mcasp.c
index fb55d2c
Hi Jon,
On Mon, Apr 18, 2011 at 9:47 PM, Jon Povey jon.po...@racelogic.co.uk wrote:
Ben Gardiner wrote:
When creating this series I noticed that there are obvious
similarities between the existing recovery routine implemented by
Philby John and John Povey
I'm not sure how the 20us
Hi Sekhar,
Thank you for reviewing the series.
On Wed, Apr 13, 2011 at 12:25 PM, Nori, Sekhar nsek...@ti.com wrote:
Hi Ben,
On Wed, Apr 06, 2011 at 03:08:03, Ben Gardiner wrote:
This series for the i2c-davinci driver implements both a register dump and
a pulsed-SCL recovery of the bus
Hi Sekhar,
On Wed, Apr 13, 2011 at 11:10 AM, Nori, Sekhar nsek...@ti.com wrote:
Hi Ben,
On Wed, Apr 06, 2011 at 03:08:09, Ben Gardiner wrote:
Both the da850 and da830 have an I2C controller which has the ICPFUNC
registers. Indicate this for all da830 and da850 boards by setting
On Wed, Apr 13, 2011 at 11:42 AM, Nori, Sekhar nsek...@ti.com wrote:
On Wed, Apr 13, 2011 at 20:52:05, Ben Gardiner wrote:
I understand check for pdata to be NULL. If you think it is the
better option I'd be happy to implement it but I don't understand how
to implement let each board choose
Produce a dump of the register contents of the I2C controller (at the debug
level) whenever an i2c recovery is initiated.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Cc: Bastian Ruppert bastian.rupp...@sewerin.de
Cc: Brad Griffis bgrif...@ti.com
Cc: Sekhar Nori nsek...@ti.com
Cc: Ben
/omap_applications_processors/f/42/p/99895/350610.aspx
Ben Gardiner (6):
i2c-davinci: register dump before attempted bus recovery
i2c-davinci: convert davinci_i2c_platform_data to kerneldoc
i2c-davinci: introduce a dev- function pointer for scl pulsing
i2c-davinci: use the DA8xx's ICPFUNC to toggle I2C as gpio
/gmane.linux.davinci/22291
[2]
http://e2e.ti.com/support/dsp/omap_applications_processors/f/42/p/99895/350610.aspx
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Cc: Bastian Ruppert bastian.rupp...@sewerin.de
Cc: Brad Griffis bgrif...@ti.com
Cc: Jon Povey jon.po...@racelogic.co.uk
Cc: Philby John pj
Both the da850 and da830 have an I2C controller which has the ICPFUNC
registers. Indicate this for all da830 and da850 boards by setting the
has_pfunc flag true in the da8xx utility setup routine for registering the
I2C controller
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Cc: Sekhar
If the platform has indicated that this I2c controller has the ICPFUNC
regsiters then also dump their contents when producing a debug register dump.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Cc: Bastian Ruppert bastian.rupp...@sewerin.de
Cc: Brad Griffis bgrif...@ti.com
Cc: Sekhar
The davinci_i2c_platform_data struct is mach-specific but still deserves
to be documented using kernel doc.
Signed-off-by: Ben Gardinerbengardi...@nanometrics.ca
Cc: Sekhar Nori nsek...@ti.com
Cc: Ben Dooks ben-li...@fluff.org
---
arch/arm/mach-davinci/include/mach/i2c.h | 17
The current implementation of the I2C recovery routine checks whether
platform data -scl_pin has been assigned and if so, the generic_i2c_clock_pulse
routine is executed.
In preparation for an alternative recovery routine; introduce a pulse_scl
function pointer to the driver private structure and
) on the baseboard.
The first time it appeared to work -- the repeating messages on the
console stopped. Sometime the messages started up again; I repeated
the procedure but this time the messages did not stop.
I tried your patch:
Ben Gardiner bengardi...@nanometrics.ca
15.03.2011 22:08
]
--
To unsubscribe from this list: send the line unsubscribe linux-i2c in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Best Regards,
Ben Gardiner
---
Nanometrics Inc.
http://www.nanometrics.ca
Buytenhek as part
of the ARM device tree support [1].
To reproduce Chritophe's problem, try using the L138's GPIOs directly
in a gpio-keys driver instance.
Best Regards,
Ben Gardiner
[1] http://article.gmane.org/gmane.linux.drivers.devicetree/4261
---
Nanometrics Inc.
http://www.nanometrics.ca
should not
be included in the 'defaults.'
Again, I am sorry for jumping in at the last minute -- I would like to
see the da850's get the best mainline support possible and I hoped
that this suggestion could help accomplish that.
Best Regards,
Ben Gardiner
---
Nanometrics Inc.
http
was
c63d0f1df0284f2fcb28c0943b17848500d01515. Everything is working as
expected.
Thanks again for taking up this series along with the complications
introduced by the cherry pick.
Best Regards,
Ben Gardiner
---
Nanometrics Inc.
http://www.nanometrics.ca
___
Davinci
On Mon, Dec 13, 2010 at 4:53 PM, Kevin Hilman
khil...@deeprootsystems.com wrote:
Ben Gardiner bengardi...@nanometrics.ca writes:
[...]
Everything seems to be in order there; I tested the resulting kernel
with evtest and the expected output was observed. Note that
davinci-next still contains
Hi Kevin,
On Fri, Dec 10, 2010 at 11:33 AM, Ben Gardiner
bengardi...@nanometrics.ca wrote:
On Fri, Dec 10, 2010 at 11:16 AM, Kevin Hilman
[...]
This series looks good to me, so I'll be queuing it in davinci-next for
2.6.38. It should show up in davinci git shortly. [...]
Thank you very
On Mon, Dec 13, 2010 at 4:53 PM, Kevin Hilman
khil...@deeprootsystems.com wrote:
Ben Gardiner bengardi...@nanometrics.ca writes:
[...]
Yes, there will be double commits in master, because of the way I manage
master using 'git merge -ours'. But there shouldn't be double commits
between my
Hi Kevin,
On Fri, Dec 10, 2010 at 11:16 AM, Kevin Hilman
khil...@deeprootsystems.com wrote:
Hi Ben,
Ben Gardiner bengardi...@nanometrics.ca writes:
The da850-evm baseboard (BB) and its UI board both have tca6416 IO expanders.
They are bootstrapped to different I2C addresses so they can
think) be suitable for mainline.
Disclaimer:
I'm not an expert in irq's or gpio-keys; this is, in fact, my first proposed
feature. Please feel free to correct me -- I welcome the chance to learn from
your expertise.
Ben Gardiner (4):
da850-evm: add UI Expander pushbuttons
da850-evm: extract
and the input-polldev
to support such buttons.
[Ben Gardiner bengardi...@nanometrics.ca: fold code to use more
of the original gpio_keys infrastructure; cleanups and other
improvements.]
Signed-off-by: Gabor Juhos juh...@openwrt.org
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Tested-by: Ben
Use the mach-davinci/Kconfig to enable gpio-keys-polled as default when
da850-evm machine is enabled.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
CC: Kevin Hilman khil...@deeprootsystems.com
CC: Nori, Sekhar nsek...@ti.com
CC: Gabor Juhos juh...@openwrt.org
---
Changes since v5
The setup and teardown methods of the UI expander reference the SEL_{A,B,C}
pins by 'magic number' in each function. This uses the common enum for their
offsets
in the expander setup and teardown functions.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: Chris Cordahi
polling interval that did not affect the feel of the
responsiveness of the buttons. It is very subjective but 200ms seems to be a
good value that accepts firm pushes but rejects very light ones. The key values
assigned to the buttons were arbitrarily chosen to be F1-F8.
Signed-off-by: Ben Gardiner
there was no
apararent way to register the LEDs connected to gpio's on the tca6416 while
simultaneously registering the tca6416-keypad instance.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: Chris Cordahi christophercord...@nanometrics.ca
CC: Govindarajan, Sriramakrishnan s...@ti.com
in
da850_evm_ui_expander_setup() which means that the 'exp' in
'exp_setup_' stands-for IO-expander.
Based on the function names where the your exp_setup_* labels were
introduced I think that the label could be renamed 'mmc_setup_*' and
'usb11_setup_*'.
Best Regards,
Ben Gardiner
---
Nanometrics Inc.
http
:57, Ben Gardiner wrote:
arch/arm/mach-davinci/board-da850-evm.c | 98
++-
1 files changed, 97 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-davinci/board-da850-evm.c
b/arch/arm/mach-davinci/board-da850-evm.c
+static struct gpio_keys_button
for Kconfig entries for both
INPUT_POLLDEV and KEYBOARD_GPIO in arch/arm/mach-davinci, just one to
enable the gpio_buttons driver.
--
Best Regards,
Ben Gardiner
[1] http://article.gmane.org/gmane.linux.kernel.input/14044
[2] http://article.gmane.org/gmane.linux.kernel.input/16478
[3] http
The setup and teardown methods of the UI expander reference the SEL_{A,B,C}
pins by 'magic number' in each function. This uses the common enum for their
offsets
in the expander setup and teardown functions.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: Chris Cordahi
picks
up the task when the time is right -- whether or not that is me.
My apologies for the patchbomb. I will do my best to make time for reviews
and testing of any future versions of the series.
Best Regards,
Ben Gardiner
---
The da850-evm baseboard (BB) and its UI board both have tca6416 IO
the time is right. I don't mean to imply that
this version of the patch should be integrated in the final series)
Signed-off-by: Gabor Juhos juh...@openwrt.org
Cc: Dmitry Torokhov dmitry.torok...@gmail.com
Cc: Mike Frysinger vap...@gentoo.org
Cc: linux-in...@vger.kernel.org
Signed-off-by: Ben Gardiner
there was no
apararent way to register the LEDs connected to gpio's on the tca6416 while
simultaneously registering the tca6416-keypad instance.
Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
Reviewed-by: Chris Cordahi christophercord...@nanometrics.ca
CC: Govindarajan, Sriramakrishnan s...@ti.com
Hi Sekhar,
On Tue, Nov 23, 2010 at 7:38 AM, Nori, Sekhar nsek...@ti.com wrote:
On Mon, Nov 22, 2010 at 19:20:48, Ben Gardiner wrote:
[...]
config KEYBOARD_GPIO
default MACH_DAVINCI_DA850_EVM
select INPUT_POLLDEV
So 1) gpio-keys functionality is default for the da850evm
Hi Sekhar,
On Tue, Nov 23, 2010 at 7:42 AM, Nori, Sekhar nsek...@ti.com wrote:
On Mon, Nov 22, 2010 at 19:45:46, Ben Gardiner wrote:
[...]
I was assuming that those pins were not exported as gpio pins on
purpose; I was taking the prudent approach to prevent haphazard
toggling of sw_rst
).
config KEYBOARD_GPIO
default MACH_DAVINCI_DA850_EVM
select INPUT_POLLDEV if MACH_DAVINCI_DA850_EVM
I know it is always better to show the code: I will extract these
contentious Kconfig changes into their own patch in the series when I
post a new version.
Best Regards,
Ben
1 - 100 of 144 matches
Mail list logo