>> Well if there's no way to detect certain omaps during runtime, please patch
>> arch/arm/plat-omap/common.c to have something like struct omap_globals 
>> omap3503_globals.

There was no need to re-define these structures for the current series of the 
OMAP35x processors.
Hence, it was left as it.

>> BTW, why can't the dsp code detect if the DSP is there or not?

(Assuming you mean the code running on ARM that looks for the DSP/SGX/...)
The detection was based on the chip ID. This happens to be same for OMAP34XX 
and OMAP35X.

~sanjeev


-----Original Message-----
From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Tony Lindgren
Sent: Wednesday, August 06, 2008 2:37 PM
To: Kridner, Jason
Cc: '[EMAIL PROTECTED]'; 'linux-omap@vger.kernel.org'
Subject: Re: [PATCH] Added support for OMAP35x processor series

* Kridner, Jason <[EMAIL PROTECTED]> [080805 17:12]:
> No, ROM CRC is useful for detecting between some device revisions, but not 
> OMAp3430 vs OMAP3530.
>
> ----- Original Message -----
> From: Koen Kooi <[EMAIL PROTECTED]>
> To: linux-omap@vger.kernel.org <linux-omap@vger.kernel.org>
> Cc: Kridner, Jason
> Sent: Tue Aug 05 05:10:40 2008
> Subject: Re: [PATCH] Added support for OMAP35x processor series
>
>
> Op 5 aug 2008, om 11:50 heeft Igor Stoppa het volgende geschreven:
>
> > On Tue, 2008-08-05 at 15:09 +0530, ext Premi, Sanjeev wrote:
> >> This patch is needed to ensure that we can make decisions based on
> >> the processor capabilities.
> >> E.g. OMAP3503 does not contain a DSP. We shouldn't be trying to
> >> disable/ manage the clocks for DSP on this processor. Same for SGX.
> >
> > Why can't the detection happen at runtime?
>
> AIUI the ID bits are the same (linux misdetect the 3530 on my beagle
> for a 3430). I heard that the only definitive way to do it at runtime
> is to CRC check the ROM code.
>
> Jason, is that correct?

Well if there's no way to detect certain omaps during runtime, please patch 
arch/arm/plat-omap/common.c to have something like struct omap_globals 
omap3503_globals.

BTW, why can't the dsp code detect if the DSP is there or not?

Tony


To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to