>> 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: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap-owner@xxxxxxxxxxxxxxx] On Behalf Of Tony Lindgren Sent: Wednesday, August 06, 2008 2:37 PM To: Kridner, Jason Cc: 'k.kooi@xxxxxxxxxxxxxxxxxx'; 'linux-omap@xxxxxxxxxxxxxxx' Subject: Re: [PATCH] Added support for OMAP35x processor series * Kridner, Jason <jdk@xxxxxx> [080805 17:12]: > No, ROM CRC is useful for detecting between some device revisions, but not OMAp3430 vs OMAP3530. > > ----- Original Message ----- > From: Koen Kooi <k.kooi@xxxxxxxxxxxxxxxxxx> > To: linux-omap@xxxxxxxxxxxxxxx <linux-omap@xxxxxxxxxxxxxxx> > 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 majordomo@xxxxxxxxxxxxxxx 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
