linux-kernel - RE: [PATCHv3 1/2] ARM: imx: Add core definitions for MX53

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <86A0E76937111F4C92FABEC0A20988510523D224@az33exm21>
Date:	Wed, 10 Nov 2010 08:51:19 -0700
From:	Nguyen Dinh-R00091 <R00091@...escale.com>
To:	Sascha Hauer <s.hauer@...gutronix.de>
CC:	<linux-kernel@...r.kernel.org>,
	<linux-arm-kernel@...ts.infradead.org>, <linux@....linux.org.uk>,
	<u.kleine-koenig@...gutronix.de>, <amit.kucheria@...onical.com>,
	<eric@...rea.com>
Subject: RE: [PATCHv3 1/2] ARM: imx: Add core definitions for MX53

Hi Sascha,

>-----Original Message-----
>From: Sascha Hauer [mailto:s.hauer@...gutronix.de]
>Sent: Tuesday, November 09, 2010 3:46 PM
>To: Nguyen Dinh-R00091
>Cc: linux-kernel@...r.kernel.org; linux-arm-kernel@...ts.infradead.org;
linux@....linux.org.uk;
>u.kleine-koenig@...gutronix.de; amit.kucheria@...onical.com;
eric@...rea.com
>Subject: Re: [PATCHv3 1/2] ARM: imx: Add core definitions for MX53
>
>Hi Dinh,
>
>On Fri, Nov 05, 2010 at 10:55:50AM -0500, Dinh.Nguyen@...escale.com
wrote:
>> From: Dinh Nguyen <Dinh.Nguyen@...escale.com>
>>
>> Add iomux, clocks,  and memory map for Freescale's MX53 SoC.
>> Add cpu_is_mx53 function to common.h.
>> Add 3 more banks of gpio's to mxc_gpio_ports.
>> Add MX53 phys offset address.
>>
>> Signed-off-by: Dinh Nguyen <Dinh.Nguyen@...escale.com>
>> ---
>>  arch/arm/mach-mx5/clock-mx53.c              |  857
+++++++++++++++++++++++++++
>>  arch/arm/mach-mx5/devices.c                 |   27 +
>>  arch/arm/plat-mxc/include/mach/common.h     |    4 +
>>  arch/arm/plat-mxc/include/mach/iomux-mx53.h |  303 ++++++++++
>>  arch/arm/plat-mxc/include/mach/memory.h     |    1 +
>>  arch/arm/plat-mxc/include/mach/mx53.h       |  406 +++++++++++++
>>  6 files changed, 1598 insertions(+), 0 deletions(-)
>>  create mode 100644 arch/arm/mach-mx5/clock-mx53.c
>>  create mode 100644 arch/arm/plat-mxc/include/mach/iomux-mx53.h
>>  create mode 100644 arch/arm/plat-mxc/include/mach/mx53.h
>>
>> diff --git a/arch/arm/mach-mx5/clock-mx53.c
b/arch/arm/mach-mx5/clock-mx53.c
>> new file mode 100644
>> index 0000000..d547cb8
>> --- /dev/null
>> +++ b/arch/arm/mach-mx5/clock-mx53.c
>
>Most of this code looks like it could be shared with clock-mx51.c

Agreed, but I was unsure how to proceed with adding clock code for mx53.
Should I merge mx53 clock code into clock-mx51 and rename clock-mx51? 

>
>> +extern int mx53_revision(void);
>> +#endif
>> +
>> +#endif	/* ifndef __MACH_H__ */
>
>__MACH_MX53_H__
>
>> --
>> 1.6.0.4
>>
>>
>>
>
>--
>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 for the review,

Dinh


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