Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Russell King - ARM Linux
On Tue, Sep 02, 2008 at 03:15:10PM -0700, Tony Lindgren wrote: > Back online now. These changes look ggood to me in general, except > I suggest that we use the following standard: > > - Keep OMAP1_IO_ADDRESS() and OMAP2_IO_ADDRESS() as I have some > experimental patches to compile in both omap1

Re: [PATCH 12/12] CORE OFF macro added, disabled by default

2008-09-03 Thread Högander Jouni
"ext Rajendra Nayak" <[EMAIL PROTECTED]> writes: > > WARNING: Enabling this option may cause your device to hang! > > +config OMAP_CPUIDLE_CORE_OFF > + bool "CORE OFF support in CPUIdle" > + depends on ARCH_OMAP34XX > + default n > + help > + Say Y if you want to enabl

Reset unused clocks during boot causing problem on 5912 OSK

2008-09-03 Thread Arun KS
Hi all, I am trying to write an ASOC driver for tlv320aic23 on 5912osk. When i do an aplay, the system is hanging. Later i found that it is hanging when mcbsp1 is requested in funcion omap_mcbsp_dai_startup in file sound/soc/omap/omap-mcbsp.c The reason is that the unused clocks are switched off

Re: Reset unused clocks during boot causing problem on 5912 OSK

2008-09-03 Thread Felipe Balbi
On Wed, Sep 03, 2008 at 07:32:51PM +0530, ext Arun KS wrote: > Hi all, > > I am trying to write an ASOC driver for tlv320aic23 on 5912osk. > When i do an aplay, the system is hanging. > Later i found that it is hanging when mcbsp1 is requested in funcion > omap_mcbsp_dai_startup in file sound/soc/

Re: [PATCH 12/15] OMAP3 camera driver: Add Sensor and Lens Driver.

2008-09-03 Thread Sakari Ailus
ext Aguirre Rodriguez, Sergio Alberto wrote: From: Sergio Aguirre <[EMAIL PROTECTED]> OMAP: CAM: Add Sensor and Lens Driver This adds the following sensor drivers: * Micron MT9P012 sensor * DW9710 Lens driver Signed-off-by: Sergio Aguirre <[EMAIL PROTECTED]> --- drivers/media/video/Kconfig

Re: [PATCH 17/33] add tea5761 radio driver

2008-09-03 Thread Eduardo Valentin
Hi guys, Sorry for the late answer. On Mon, Sep 01, 2008 at 10:59:38AM +0300, ext Felipe Balbi wrote: > On Mon, Sep 01, 2008 at 10:24:31AM +0300, Jarkko Nikula wrote: > > On Sat, 30 Aug 2008 20:16:17 +0300 > > "ext Felipe Balbi" <[EMAIL PROTECTED]> wrote: > > > > > From: Felipe Balbi <[EMAIL PRO

Bug in twl4030backupbatt_voltage()

2008-09-03 Thread Adrian Hunter
In the omap tree, commit f91fd98be045bec9c18e13110f22dafd44cb84b5 fixed a bug with the use of function twl4030_madc_conversion() relating to the local variable 'struct twl4030_madc_request req' not being fully initialised. The same problem appears to be in the function twl4030backupbatt_voltage()

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Eduardo Valentin
Hi Tony, On Tue, Sep 2, 2008 at 6:15 PM, Tony Lindgren <[EMAIL PROTECTED]> wrote: > Hi, > > * David Brownell <[EMAIL PROTECTED]> [080831 14:47]: >> On Wednesday 27 August 2008, Russell King wrote: >> > I sent a similar patch to the one below against mainline to Tony today. >> >> And I'm glad to se

Re: [RESEND 2][Review Request] Adding csi2_fck declaration to clock34xx.h

2008-09-03 Thread Paul Walmsley
Hello Sergio, On Tue, 2 Sep 2008, Aguirre Rodriguez, Sergio Alberto wrote: > From: Sergio Aguirre <[EMAIL PROTECTED]> > > ARM: OMAP: Add CSI2 clock struct for handling it with clock API. > > Add CSI2 clock struct for handling it with clock API when TI PM is disabled. > + .clkdm =

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Eduardo Valentin
Hi, On Wed, Aug 27, 2008 at 6:08 PM, Russell King <[EMAIL PROTECTED]> wrote: > ... are bad when overused. Unexpected bugs can creap in. That one cast > which someone added to shut up a perfectly valid compiler warning now > hides a potential problem. > > It is possible to eliminate _lots_ of the

[RESEND 3][PATCH] Adding csi2_fck declaration to clock34xx.h

2008-09-03 Thread Aguirre Rodriguez, Sergio Alberto
From: Sergio Aguirre <[EMAIL PROTECTED]> ARM: OMAP: Add CSI2 clock struct for handling it with clock API. Add CSI2 clock struct for handling it with clock API when TI PM is disabled. Signed-off-by: Sergio Aguirre <[EMAIL PROTECTED]> --- arch/arm/mach-omap2/clock34xx.h | 12

RE: [RESEND 2][Review Request] Adding csi2_fck declaration to clock34xx.h

2008-09-03 Thread Aguirre Rodriguez, Sergio Alberto
Oh ok, I'll revert that then, no problem. So, shall I consider everything else as accepted in the patch? Thank you very much for your time and your comments. I appreciate it. Regards, Sergio -Original Message- From: Paul Walmsley [mailto:[EMAIL PROTECTED] Sent: Wednesday, September 03,

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Tony Lindgren
* Russell King - ARM Linux <[EMAIL PROTECTED]> [080903 00:56]: > On Tue, Sep 02, 2008 at 03:15:10PM -0700, Tony Lindgren wrote: > > Back online now. These changes look ggood to me in general, except > > I suggest that we use the following standard: > > > > - Keep OMAP1_IO_ADDRESS() and OMAP2_IO_AD

Re: Reset unused clocks during boot causing problem on 5912 OSK

2008-09-03 Thread Arun KS
Hi Balbi, That clk_enable() is for sys_clkout, which is the MCLK for the CODEC. I am following n810.c file itself. But I m talking about omap_mcbsp_request(0), which will only be called when we do an aplay, by the platform driver of the omap(sound/soc/omap/ompa-mcbsp.c). But before this the unus

Re: Reset unused clocks during boot causing problem on 5912 OSK

2008-09-03 Thread Yuri Jaeger Monti
My system hangs too when loading the old AIC23 codec module on the OSK5912, don' t know why. --- Yuri Jaeger Monti On Wed, Sep 3, 2008 at 2:27 PM, Arun KS <[EMAIL PROTECTED]> wrote: > > Hi Balbi, > > That clk_enable() is for sys_clkout, which is the MCLK for the CODEC. > > I am following n810.c

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Tony Lindgren
* Tony Lindgren <[EMAIL PROTECTED]> [080902 15:16]: > Hi, > > * David Brownell <[EMAIL PROTECTED]> [080831 14:47]: > > On Wednesday 27 August 2008, Russell King wrote: > > > > So I can try my patch for the omap_udc thing ... even though for > > OSK the mainline kernel ** DOES NOT BOOT ** and d

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Russell King
On Wed, Sep 03, 2008 at 11:33:51AM -0400, Eduardo Valentin wrote: > > @@ -159,7 +159,7 @@ static struct omap_mcbsp_platform_data > > omap730_mcbsp_pdata[] = { > > #ifdef CONFIG_ARCH_OMAP15XX > > static struct omap_mcbsp_platform_data omap15xx_mcbsp_pdata[] = { > >{ > > - .v

Re: Reset unused clocks during boot causing problem on 5912 OSK

2008-09-03 Thread Russell King - ARM Linux
On Wed, Sep 03, 2008 at 07:32:51PM +0530, Arun KS wrote: > I am trying to write an ASOC driver for tlv320aic23 on 5912osk. 5912osk is OMAP1, yes ? Uses arch/arm/mach-omap1/mcbsp.c ? Which is the file I pointed out passing physical addresses for mcbsp1 and mcbsp3 instead of virtual addresses, res

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Eduardo Valentin
Hi, On Wed, Sep 3, 2008 at 2:48 PM, Russell King <[EMAIL PROTECTED]> wrote: > On Wed, Sep 03, 2008 at 11:33:51AM -0400, Eduardo Valentin wrote: >> > @@ -159,7 +159,7 @@ static struct omap_mcbsp_platform_data >> > omap730_mcbsp_pdata[] = { >> > #ifdef CONFIG_ARCH_OMAP15XX >> > static struct omap

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Russell King - ARM Linux
On Wed, Sep 03, 2008 at 09:40:08AM -0700, Tony Lindgren wrote: > * Russell King - ARM Linux <[EMAIL PROTECTED]> [080903 00:56]: > > > - Use io_p2v() for initializing dynamic stuff as it can be a function > > > for non-optimized multiboot binaries. > > > > It can't become a function - it's used i

Doubts about patching

2008-09-03 Thread Ramón Finger Lilienthal
Fellows, I am new on generating and applying patchs. I am a little bit confused. Someone has a tutorial or a weblink that explains how to generate and apply a patch? Please, any help will be welcome. Regards, Eng. Ramón F. Lilienthal Project Enginneer Instramed Ltda. - www.instramed.com.br

Re: Doubts about patching

2008-09-03 Thread Diego Dompe
Quick google search for "tutorial patch diff" http://www.linuxjournal.com/article/1237 http://www.linuxtutorialblog.com/post/introduction-using-diff-and-patch-tutorial Regards, Diego On Sep 3, 2008, at 1:13 PM, Ramón Finger Lilienthal wrote: Fellows, I am new on generating and applying patc

Re: Doubts about patching

2008-09-03 Thread Felipe Balbi
On Wed, Sep 03, 2008 at 04:13:28PM -0300, Ram?n Finger Lilienthal wrote: > Fellows, > > I am new on generating and applying patchs. I am a little bit confused. > > Someone has a tutorial or a weblink that explains how to generate and > apply a patch? There's a series of git usage tips in my blo

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Russell King - ARM Linux
On Wed, Sep 03, 2008 at 03:33:55PM -0400, Eduardo Valentin wrote: > Hi, > > On Wed, Sep 3, 2008 at 2:48 PM, Russell King <[EMAIL PROTECTED]> wrote: > > On Wed, Sep 03, 2008 at 11:33:51AM -0400, Eduardo Valentin wrote: > >> > @@ -159,7 +159,7 @@ static struct omap_mcbsp_platform_data > >> > omap73

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Tony Lindgren
* Russell King - ARM Linux <[EMAIL PROTECTED]> [080903 12:34]: > On Wed, Sep 03, 2008 at 09:40:08AM -0700, Tony Lindgren wrote: > > * Russell King - ARM Linux <[EMAIL PROTECTED]> [080903 00:56]: > > > > - Use io_p2v() for initializing dynamic stuff as it can be a function > > > > for non-optimize

RE: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Woodruff, Richard
> From: [EMAIL PROTECTED] [mailto:linux-omap- > [EMAIL PROTECTED] On Behalf Of Russell King - ARM Linux > Sent: Wednesday, September 03, 2008 2:34 PM > The question is why do we need it? If the correct physical address > is passed, then things should work out just fine anyway, especially > if dr

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Eduardo Valentin
Hi Russell, On Wed, Sep 3, 2008 at 3:48 PM, Russell King - ARM Linux <[EMAIL PROTECTED]> wrote: > On Wed, Sep 03, 2008 at 03:33:55PM -0400, Eduardo Valentin wrote: >> Hi, >> >> On Wed, Sep 3, 2008 at 2:48 PM, Russell King <[EMAIL PROTECTED]> wrote: >> > On Wed, Sep 03, 2008 at 11:33:51AM -0400, Ed

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Russell King - ARM Linux
On Wed, Sep 03, 2008 at 02:58:10PM -0500, Woodruff, Richard wrote: > Fixed translations do have some benefits. You can ensure that you are > using section or super section descriptors to cover large areas. This > does result in better TLB usage. Along with freeing up TLB entries you > also gener

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Tony Lindgren
* Woodruff, Richard <[EMAIL PROTECTED]> [080903 12:58]: > > > From: [EMAIL PROTECTED] [mailto:linux-omap- > > [EMAIL PROTECTED] On Behalf Of Russell King - ARM Linux > > Sent: Wednesday, September 03, 2008 2:34 PM > > > The question is why do we need it? If the correct physical address > > is pa

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Tony Lindgren
* Russell King - ARM Linux <[EMAIL PROTECTED]> [080903 12:49]: > On Wed, Sep 03, 2008 at 03:33:55PM -0400, Eduardo Valentin wrote: > > Hi, > > > > On Wed, Sep 3, 2008 at 2:48 PM, Russell King <[EMAIL PROTECTED]> wrote: > > > On Wed, Sep 03, 2008 at 11:33:51AM -0400, Eduardo Valentin wrote: > > >>

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Russell King - ARM Linux
On Wed, Sep 03, 2008 at 04:04:11PM -0400, Eduardo Valentin wrote: > Hi Russell, > > On Wed, Sep 3, 2008 at 3:48 PM, Russell King - ARM Linux > <[EMAIL PROTECTED]> wrote: > > Yes, that will be virtual. But what does it mean to call: > > > >omap_set_dma_dest_params() > > > > specifying a vi

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Tony Lindgren
* Russell King - ARM Linux <[EMAIL PROTECTED]> [080903 13:46]: > On Wed, Sep 03, 2008 at 04:04:11PM -0400, Eduardo Valentin wrote: > > Hi Russell, > > > > On Wed, Sep 3, 2008 at 3:48 PM, Russell King - ARM Linux > > <[EMAIL PROTECTED]> wrote: > > > Yes, that will be virtual. But what does it mean

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Tony Lindgren
* Tony Lindgren <[EMAIL PROTECTED]> [080903 13:50]: > * Russell King - ARM Linux <[EMAIL PROTECTED]> [080903 13:46]: > > On Wed, Sep 03, 2008 at 04:04:11PM -0400, Eduardo Valentin wrote: > > > Hi Russell, > > > > > > On Wed, Sep 3, 2008 at 3:48 PM, Russell King - ARM Linux > > > <[EMAIL PROTECTED]

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Koen Kooi
Op 3 sep 2008, om 22:45 heeft Russell King - ARM Linux het volgende geschreven: I would look up this CDSA register in the OMAP manuals, if they were accessible... Are they not publically available? (Google isn't helping, neither is searching on ti.com...) Try http://www.ti.com/litv/pdf/sp

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Russell King - ARM Linux
On Wed, Sep 03, 2008 at 01:37:21PM -0700, Tony Lindgren wrote: > * Russell King - ARM Linux <[EMAIL PROTECTED]> [080903 12:49]: > > Yes, that will be virtual. But what does it mean to call: > > > > omap_set_dma_dest_params() > > > > specifying a virtual address? Can the DMA controller cope

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Russell King - ARM Linux
On Wed, Sep 03, 2008 at 01:56:18PM -0700, Tony Lindgren wrote: > * Tony Lindgren <[EMAIL PROTECTED]> [080903 13:50]: > > There is a 3450 public TRM somewhere AFAIK, anybody got a link for that? > > At least 5912 TRM is available somewhere. > > Sorry I meant 3550 public TRM. Here's the link to the

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Tony Lindgren
* Russell King - ARM Linux <[EMAIL PROTECTED]> [080903 14:07]: > On Wed, Sep 03, 2008 at 01:56:18PM -0700, Tony Lindgren wrote: > > * Tony Lindgren <[EMAIL PROTECTED]> [080903 13:50]: > > > There is a 3450 public TRM somewhere AFAIK, anybody got a link for that? > > > At least 5912 TRM is available

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread David Brownell
On Wednesday 03 September 2008, Tony Lindgren wrote: > > > > I still want to hear on the other build fix in the patch, and there's > > > > also a missing function for mmc stuff which I've not looked into yet. > > > > > > The mpuio build fix? That is a correct fix. > > > > No, see drivers/usb/host

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread David Brownell
On Wednesday 03 September 2008, Woodruff, Richard wrote: > Fixed translations do have some benefits.  You can ensure that you > are using section or super section descriptors to cover large areas. > This does result in better TLB usage.  Along with freeing up TLB > entries you also generally avoid

RE: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Woodruff, Richard
> From: Russell King - ARM Linux [mailto:[EMAIL PROTECTED] > Sent: Wednesday, September 03, 2008 3:30 PM > On Wed, Sep 03, 2008 at 02:58:10PM -0500, Woodruff, Richard wrote: > > Fixed translations do have some benefits. You can ensure that you are > > using section or super section descriptors

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Eduardo Valentin
Hi Russell, On Wed, Sep 3, 2008 at 5:04 PM, Russell King - ARM Linux <[EMAIL PROTECTED]> wrote: > On Wed, Sep 03, 2008 at 01:37:21PM -0700, Tony Lindgren wrote: >> * Russell King - ARM Linux <[EMAIL PROTECTED]> [080903 12:49]: >> > Yes, that will be virtual. But what does it mean to call: >> > >>

RE: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Woodruff, Richard
> From: Tony Lindgren [mailto:[EMAIL PROTECTED] > Sent: Wednesday, September 03, 2008 3:33 PM > > Fixed translations do have some benefits. You can ensure that you are > using section or super section descriptors to cover large areas. This > does result in better TLB usage. Along with freeing

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Tony Lindgren
* Woodruff, Richard <[EMAIL PROTECTED]> [080903 14:32]: > > > From: Tony Lindgren [mailto:[EMAIL PROTECTED] > > Sent: Wednesday, September 03, 2008 3:33 PM > > > > Fixed translations do have some benefits. You can ensure that you are > > using section or super section descriptors to cover large

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread David Brownell
On Wednesday 03 September 2008, Russell King - ARM Linux wrote: > - virtual addresses are pointer like. > - physical addresses are integer like. > > So, if it's a physical address, it should be stored in an integer type > large enough to contain it, and that means something like u32, or > unsigned

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Russell King - ARM Linux
On Wed, Sep 03, 2008 at 04:32:10PM -0500, Woodruff, Richard wrote: > > > From: Tony Lindgren [mailto:[EMAIL PROTECTED] > > Sent: Wednesday, September 03, 2008 3:33 PM > > > > Fixed translations do have some benefits. You can ensure that you are > > using section or super section descriptors to c

RE: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Woodruff, Richard
> From: David Brownell [mailto:[EMAIL PROTECTED] > On Wednesday 03 September 2008, Woodruff, Richard wrote: > > Fixed translations do have some benefits. You can ensure that you > > are using section or super section descriptors to cover large areas. > > This does result in better TLB usage. Al

Multi-Boot: Was RE: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Woodruff, Richard
> From: Russell King - ARM Linux [mailto:[EMAIL PROTECTED] > Can we _PLEASE_ take this discussion to a different thread. OFF TOPIC. > Thank you. Ok for me. Sorry if it defocused. Regards, Richard W. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a mes

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Tony Lindgren
* Eduardo Valentin <[EMAIL PROTECTED]> [080903 14:27]: > Hi Russell, > > On Wed, Sep 3, 2008 at 5:04 PM, Russell King - ARM Linux > <[EMAIL PROTECTED]> wrote: > > On Wed, Sep 03, 2008 at 01:37:21PM -0700, Tony Lindgren wrote: > >> * Russell King - ARM Linux <[EMAIL PROTECTED]> [080903 12:49]: > >>

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread David Brownell
On Wednesday 03 September 2008, Woodruff, Richard wrote: > > > ... [ big/inexpensive fixed mappings can be virtuous ... ] > > > > ... which is exactly why some linux/arch/... code makes sure ioremap() > > can return fixed mappings instead of always requiring dynamic ones. > > That does sound like

Git tree updated to v2.6.27-rc5, some USB diffs still lurking

2008-09-03 Thread Tony Lindgren
Hi all, I've updated our git tree to -rc5. Now drivers/usb is pretty close to being in sync with mainline except for the following diff. Dave, could you take a look at the following USB patch and see what should be merged, fixed or removed? The patch is mostly some ehci-omap.[ch] and musb includ

Re: Git tree updated to v2.6.27-rc5, some USB diffs still lurking

2008-09-03 Thread Felipe Balbi
On Wed, Sep 03, 2008 at 03:12:19PM -0700, Tony Lindgren wrote: > Hi all, > > I've updated our git tree to -rc5. Now drivers/usb is pretty close to > being in sync with mainline except for the following diff. > > Dave, could you take a look at the following USB patch and see what > should be merge

Re: Git tree updated to v2.6.27-rc5, some USB diffs still lurking

2008-09-03 Thread Tony Lindgren
* Felipe Balbi <[EMAIL PROTECTED]> [080903 15:18]: > On Wed, Sep 03, 2008 at 03:12:19PM -0700, Tony Lindgren wrote: > > Hi all, > > > > I've updated our git tree to -rc5. Now drivers/usb is pretty close to > > being in sync with mainline except for the following diff. > > > > Dave, could you take

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Russell King - ARM Linux
On Wed, Sep 03, 2008 at 03:05:59PM -0700, David Brownell wrote: > According to Mr. Grep, there are at least 10 ARMs that work > like that in mainline. Maybe Russell can recommend one of > them as a preferred model. As I've been trying to say, I see this as a separate issue for the near future. A

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Russell King - ARM Linux
On Wed, Sep 03, 2008 at 02:09:14PM -0700, David Brownell wrote: > On Wednesday 03 September 2008, Tony Lindgren wrote: > > > > > I still want to hear on the other build fix in the patch, and there's > > > > > also a missing function for mmc stuff which I've not looked into yet. > > > > > > > > The

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Russell King - ARM Linux
On Wed, Sep 03, 2008 at 02:35:52PM -0700, David Brownell wrote: > On Wednesday 03 September 2008, Russell King - ARM Linux wrote: > > - virtual addresses are pointer like. > > - physical addresses are integer like. > > > > So, if it's a physical address, it should be stored in an integer type > >

Re: Git tree updated to v2.6.27-rc5, some USB diffs still lurking

2008-09-03 Thread Russell King - ARM Linux
On Wed, Sep 03, 2008 at 03:12:19PM -0700, Tony Lindgren wrote: > @@ -2687,7 +2696,7 @@ > udc->gadget.name = driver_name; > > device_initialize(&udc->gadget.dev); > - dev_set_name(&udc->gadget.dev, "gadget"); > + strcpy (udc->gadget.dev.bus_id, "gadget"); FWIW, this one looks

Re: Git tree updated to v2.6.27-rc5, some USB diffs still lurking

2008-09-03 Thread Tony Lindgren
* Russell King - ARM Linux <[EMAIL PROTECTED]> [080903 16:28]: > On Wed, Sep 03, 2008 at 03:12:19PM -0700, Tony Lindgren wrote: > > @@ -2687,7 +2696,7 @@ > > udc->gadget.name = driver_name; > > > > device_initialize(&udc->gadget.dev); > > - dev_set_name(&udc->gadget.dev, "gadget"); > >

Re: Git tree updated to v2.6.27-rc5, some USB diffs still lurking

2008-09-03 Thread Felipe Balbi
On Wed, Sep 03, 2008 at 04:52:25PM -0700, Tony Lindgren wrote: > OK, fixed for l-o tree. Felipe has the include fixes patch in Greg's > queue, but looks like that needs updating too. Felipe, can you take > care of that? Yeah, I had a missing change in tusb6010_omap.c for which I have already sent

[ANNOUNCE] musb procfs

2008-09-03 Thread Felipe Balbi
Hi all, This mail is just to let you know musb procfs file has been removed from musb for several reasons being "drivers shouldn't add procfs files" the most outstanding one. For those used to change musb debugging level via "echo D3 > /proc/drivers/musb_hdrc" the same approach can be done by cha

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread David Brownell
On Wednesday 03 September 2008, Tony Lindgren wrote: > My OSK is booting just fine here with current mainline and the > omap_osk_5912_defconfig. This is the current head at > 2.6.27-rc5-00100-gec0c15a. Have you checked if yours boots with > the mainline omap_osk_5912_defconfig? Today's root seems

Re: Git tree updated to v2.6.27-rc5, some USB diffs still lurking

2008-09-03 Thread Måns Rullgård
Russell King - ARM Linux <[EMAIL PROTECTED]> writes: > On Wed, Sep 03, 2008 at 03:12:19PM -0700, Tony Lindgren wrote: > >> +#define OMAP_USBHOST_TLL_BASE (OMAP_USBHOST_BASE + 0x2000) >> +#define OMAP_USBTLL_REVISION(OMAP_USBHOST_TLL_BASE + 0x00) >> +#define OMAP_USBTLL_SYSCONFIG

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread Tony Lindgren
* Russell King - ARM Linux <[EMAIL PROTECTED]> [080903 15:57]: > On Wed, Sep 03, 2008 at 03:05:59PM -0700, David Brownell wrote: > > According to Mr. Grep, there are at least 10 ARMs that work > > like that in mainline. Maybe Russell can recommend one of > > them as a preferred model. > > As I've

Re: [PATCH 21/33] add OMAP HighSpeed mmc controller driver

2008-09-03 Thread Tony Lindgren
* Felipe Balbi <[EMAIL PROTECTED]> [080831 15:03]: > On Sun, Aug 31, 2008 at 02:51:18PM -0700, David Brownell wrote: > > Fixable, I'd expect. :) > > I'm sure it is :-) > > > > so even if we > > > have all the drivers there. Mainline has support up to omap2430, if i'm > > > not wrong. So looks l

Re: FOR COMMENT: void __iomem * and similar casts are Bad News

2008-09-03 Thread David Brownell
On Wednesday 03 September 2008, Russell King - ARM Linux wrote: > On Wed, Sep 03, 2008 at 03:05:59PM -0700, David Brownell wrote: > > According to Mr. Grep, there are at least 10 ARMs that work > > like that [__arch_ioremap] in mainline. Maybe Russell can > > recommend one of them as a preferred m

Re: [PATCH 05/33] add tlv320aic23 driver

2008-09-03 Thread Tony Lindgren
* Felipe Balbi <[EMAIL PROTECTED]> [080901 08:20]: > On Mon, Sep 01, 2008 at 12:02:44PM -0300, Yuri Jaeger Monti wrote: > > We're still using OSK here. The patch compiled fine, although I'm still > > having > > some troubles with the audio in the board. > > Unfortunately I still don't have the kno

Re: [PATCH] The clockdomain of clkout2_src_ck fixed

2008-09-03 Thread Tony Lindgren
* Tomi Valkeinen <[EMAIL PROTECTED]> [080829 07:50]: > The clockdomain of clkout2_src_ck was set to "core_clkdm", which does not > exist. Can you please reply with your Signed-off-by? Thanks, Tony > --- > arch/arm/mach-omap2/clock34xx.h |2 +- > 1 files changed, 1 insertions(+), 1 deletion

Errors on boot with 2.6.27-rc5 on omap3

2008-09-03 Thread Steve Sakoman
Anyone else seeing this (error output below)? I get similar results for both Beagle and Overo, so the error occurs for both nand and onenand. The boot eventually completes, of course without nand/onenand support. Steve OneNAND driver initializing omap2-onenand omap2-onenand: initializing on CS0

Re: Errors on boot with 2.6.27-rc5 on omap3

2008-09-03 Thread Kyungmin Park
Hi, It's already discussed at mtd mailing list and solved (sent to linux) Please check it at mtd. Thank you, Kyungmin Park On Thu, Sep 4, 2008 at 12:41 PM, Steve Sakoman <[EMAIL PROTECTED]> wrote: > Anyone else seeing this (error output below)? > > I get similar results for both Beagle and Over

[PATCH 0/4] ARM: OMAP2: Add support for Gumstix Overo (resubmission)

2008-09-03 Thread sakoman
This patch series adds support for the Gumstix Overo board. Since it includes an ASoC driver I am copying both linux-omap and alsa-devel This version incorporates feedback from both linux-omap and alsa-devel -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body o

[PATCH 1/5] ARM: OMAP2: Add support for the Gumstix Overo board

2008-09-03 Thread sakoman
From: Steve Sakoman <[EMAIL PROTECTED]> This patch adds support for basic features: onenand, uarts, i2c, mmc, rtc, and usb --- arch/arm/mach-omap2/Kconfig |4 + arch/arm/mach-omap2/Makefile |4 + arch/arm/mach-omap2/board-overo.c | 234 + 3

[PATCH 2/5] ARM: OMAP2: Add video driver the Gumstix Overo board

2008-09-03 Thread sakoman
From: Steve Sakoman <[EMAIL PROTECTED]> --- arch/arm/mach-omap2/board-overo.c | 11 +++ drivers/video/omap/Kconfig| 21 + drivers/video/omap/Makefile |1 + drivers/video/omap/lcd_overo.c| 179 + 4 files changed, 212 insertions(+),

[PATCH 3/5] SOUND: SOC: CODECS: Add support for the TWL4030 audio codec

2008-09-03 Thread sakoman
From: Steve Sakoman <[EMAIL PROTECTED]> --- sound/soc/codecs/Kconfig |5 + sound/soc/codecs/Makefile |2 + sound/soc/codecs/twl4030.c | 628 sound/soc/codecs/twl4030.h | 197 ++ 4 files changed, 832 insertions(+), 0 deletions(-

[PATCH 4/5] SOUND: SOC: OMAP: Add support for Gumstix Overo

2008-09-03 Thread sakoman
From: Steve Sakoman <[EMAIL PROTECTED]> --- sound/soc/omap/Kconfig |9 +++ sound/soc/omap/Makefile |3 + sound/soc/omap/overo.c | 149 +++ 3 files changed, 161 insertions(+), 0 deletions(-) create mode 100644 sound/soc/omap/overo.c diff --g

[PATCH 5/5] ARM: OMAP2: defconfig for the Gumstix Overo board

2008-09-03 Thread sakoman
From: Steve Sakoman <[EMAIL PROTECTED]> --- arch/arm/configs/overo_defconfig | 1973 ++ 1 files changed, 1973 insertions(+), 0 deletions(-) create mode 100644 arch/arm/configs/overo_defconfig diff --git a/arch/arm/configs/overo_defconfig b/arch/arm/configs/ov

Re: Errors on boot with 2.6.27-rc5 on omap3

2008-09-03 Thread David Brownell
On Wednesday 03 September 2008, Steve Sakoman wrote: > I get similar results for both Beagle and Overo, so the error occurs > for both nand and onenand.  The boot eventually completes, of course > without nand/onenand support. Does this help? http://marc.info/?l=linux-kernel&m=121999823616458&w

Re: Errors on boot with 2.6.27-rc5 on omap3

2008-09-03 Thread Steve Sakoman
On Wed, Sep 3, 2008 at 10:10 PM, David Brownell <[EMAIL PROTECTED]> wrote: > On Wednesday 03 September 2008, Steve Sakoman wrote: >> I get similar results for both Beagle and Overo, so the error occurs >> for both nand and onenand. The boot eventually completes, of course >> without nand/onenand s

Re: Errors on boot with 2.6.27-rc5 on omap3

2008-09-03 Thread Steve Sakoman
On Wed, Sep 3, 2008 at 10:27 PM, Steve Sakoman <[EMAIL PROTECTED]> wrote: > On Wed, Sep 3, 2008 at 10:10 PM, David Brownell <[EMAIL PROTECTED]> wrote: >> On Wednesday 03 September 2008, Steve Sakoman wrote: >>> I get similar results for both Beagle and Overo, so the error occurs >>> for both nand a