Victor Rodriguez <[email protected]> writes: > On Tue, Sep 7, 2010 at 9:02 AM, Syed Mohammed, Khasim <[email protected]> wrote: >> >> >>> -----Original Message----- >>> From: Syed Mohammed, Khasim >>> Sent: Thursday, September 02, 2010 10:35 PM >>> To: 'Kevin Hilman'; Victor Rodriguez >>> Cc: [email protected] >>> Subject: RE: [PATCH] hawkboard v_2 >>> >>> Kevin, >>> >>> > -----Original Message----- >>> > From: davinci-linux-open-source-bounces+khasim=ti....@linux.davincidsp.com >>> > [mailto:davinci-linux-open-source- >>> [email protected]] >>> > On Behalf Of Kevin Hilman >>> > Sent: Thursday, September 02, 2010 10:30 PM >>> > To: Victor Rodriguez >>> > Cc: [email protected] >>> > Subject: Re: [PATCH] hawkboard v_2 >>> > >>> > Victor Rodriguez <[email protected]> writes: >>> > >>> > > Basic support for Hawkboard OMAPL138 just UART cominication is included >>> > > >>> > > Signed-off-by: Victor Rodriguez <[email protected]> >>> > > >>> > >>> > [...] >>> > >>> > > +static __init void omapl138_hawk_init(void) >>> > > +{ >>> > > + >>> > > + davinci_serial_init(&omapl138_hawk_uart_config); >>> > > + >>> > > + /* >>> > > + * shut down uart 0 and 1; they are not used on the board and >>> > > + * accessing them causes endless "too much work in irq53" messages >>> > > + * with arago fs >>> > > + */ >>> > > + __raw_writel(0, IO_ADDRESS(DA8XX_UART1_BASE) + 0x30); >>> > > + __raw_writel(0, IO_ADDRESS(DA8XX_UART0_BASE) + 0x30); >>> > >>> > This hack/workaround needs to be removed and/or better described so we >>> > know *why* such a thing is needed. >>> > >>> > Khasim, do you know what the root cause is here? >>> I will have to look into this, I will get back to you in a day or two. >>> >> These lines were directly copied from da850 board file, I am still trying to >> figure out if this is true for hawk as well. >> >> Looks like some thing is not handled properly in driver that is causing this >> irq 53, we had similar issue in OMAP3 with irq33. >> >> I will have to discuss this with Sekhar, he is on vacation this week. Will >> keep you updated as soon as I get more info into this. >> >> For now can we put a big disclaimer on this (TBD) and move forward? - this >> is just a request. >> >> Regards, >> Khasim >> > > I agree if Kevin is ok I could propose one more version of the patch > and keep working in the other things to rebase like USB, MMC and so on
Yes, that's OK. I hadn't noticed it came from the da850 EVM board file. What I'd like is for the da850 EVM board file to be cleaned up with a better description of the problem as well, but that can be done by TI (Sekhar?) as a separate patch. Kevin _______________________________________________ Davinci-linux-open-source mailing list [email protected] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
