From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from imap.sh.mvista.com (unknown [63.81.120.155]) by ozlabs.org (Postfix) with ESMTP id 6C333DDEFE for ; Thu, 17 May 2007 06:01:58 +1000 (EST) Message-ID: <464B638E.4070204@ru.mvista.com> Date: Thu, 17 May 2007 00:03:26 +0400 From: Sergei Shtylyov MIME-Version: 1.0 To: Segher Boessenkool Subject: Re: [PATCH 2/5] Add legacy devices to mpc8641_hpcn.dts References: <1179245829.8132.100.camel@rhino> <1179246470.8132.112.camel@rhino> <1179333421.7018.17.camel@ld0161-tx32> In-Reply-To: Content-Type: text/plain; charset=us-ascii; format=flowed Cc: linuxppc-dev List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Hello. Segher Boessenkool wrote: >>>+ 8042@60 { >>>+ device_type = "8042"; > Drop the device_type. A number as a name isn't > all that great, either. Perhaps it should be called "keyboard" (and device_type should also be set to it) since 8042 is in fact (mostly) a keyboard controller... although it also "drives" PS/2 mouse... >>>+ rtc@70 { >>>+ device_type = "rtc"; >>>+ reg = <1 70 2>; >>>+ }; >>Is "rtc" a standard OF device, > Yes, it is defined in the "Device Support Extensions" > recommended practice. And it also defines "keyboard". WBR, Sergei