lkml.org 
[lkml]   [2010]   [Oct]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRE: [PATCH] ARM: imx: Add some core definitions for MX53
Date
From
Hi,

>-----Original Message-----
>From: Sascha Hauer [mailto:s.hauer@pengutronix.de]
>Sent: Wednesday, October 27, 2010 8:57 AM
>To: Eric Bénard
>Cc: Grant Likely; amit.kucheria@canonical.com; linux@arm.linux.org.uk; Nguyen Dinh-R00091; linux-
>kernel@vger.kernel.org; linux-arm-kernel@lists.infradead.org; daniel@caiaq.de; Uwe Kleine-König;
>valentin.longchamp@epfl.ch
>Subject: Re: [PATCH] ARM: imx: Add some core definitions for MX53
>
>On Wed, Oct 27, 2010 at 03:33:07PM +0200, Eric Bénard wrote:
>> Hi,
>>
>> Le 27/10/2010 15:22, Sascha Hauer a écrit :
>> > That's a way we can go but only if the common name is not MX5 or MX5x.
>> > We've done it with the mx21/27 and it failed when the mx25 came out
>> > which was completely incompatible. Since then everybody looking at
>> > the code must know that mx2x is mx21/27 but not mx25. I suggest
>> > something like MX513 instead.
>> >
>> MX513 is the partnumber of a member of the i.MX51 family so that's maybe
>> not the best choice.
>
>Yes, I know. Is MX5_1_3 better? Any other suggestions?

Not sure why we can't do something like this?

In mx5x.h,
MX51_PERIP_1 /* Address specific to mx51 */
MX53_PERIP_1 /* Address specific to mx53 */
MX5_PERIP_2 /* Address that is shared to mx51, mx53, and any later mx5X chips */
Then if something like mx57 comes, you can add to mx5x.h the differences with MX57_PERIP_1.

>
>--
>Pengutronix e.K. | |
>Industrial Linux Solutions | http://www.pengutronix.de/ |
>Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
>Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |

Thanks,
Dinh

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2010-10-27 16:59    [W:0.065 / U:0.340 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site