From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1VvykE-0000wK-7S for mharc-qemu-trivial@gnu.org; Wed, 25 Dec 2013 19:24:10 -0500 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35717) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vvyk8-0000oZ-8A for qemu-trivial@nongnu.org; Wed, 25 Dec 2013 19:24:08 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Vvyk3-0004Wi-Mk for qemu-trivial@nongnu.org; Wed, 25 Dec 2013 19:24:04 -0500 Received: from [222.73.24.84] (port=40177 helo=song.cn.fujitsu.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vvyju-0004W0-44; Wed, 25 Dec 2013 19:23:50 -0500 X-IronPort-AV: E=Sophos;i="4.95,551,1384272000"; d="scan'208";a="9327453" Received: from unknown (HELO tang.cn.fujitsu.com) ([10.167.250.3]) by song.cn.fujitsu.com with ESMTP; 26 Dec 2013 08:20:14 +0800 Received: from fnstmail02.fnst.cn.fujitsu.com (tang.cn.fujitsu.com [127.0.0.1]) by tang.cn.fujitsu.com (8.14.3/8.13.1) with ESMTP id rBQ0Nlw8020407; Thu, 26 Dec 2013 08:23:47 +0800 Received: from [10.167.226.22] ([10.167.226.22]) by fnstmail02.fnst.cn.fujitsu.com (Lotus Domino Release 8.5.3) with ESMTP id 2013122608230684-716012 ; Thu, 26 Dec 2013 08:23:06 +0800 Message-ID: <52BB76AB.3060502@cn.fujitsu.com> Date: Thu, 26 Dec 2013 08:22:03 +0800 From: Li Guang User-Agent: Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.9.1.16) Gecko/20120421 Iceape/2.0.11 MIME-Version: 1.0 To: Peter Maydell References: <1387956078-10651-1-git-send-email-lig.fnst@cn.fujitsu.com> In-Reply-To: X-MIMETrack: Itemize by SMTP Server on mailserver/fnst(Release 8.5.3|September 15, 2011) at 2013/12/26 08:23:06, Serialize by Router on mailserver/fnst(Release 8.5.3|September 15, 2011) at 2013/12/26 08:23:07, Serialize complete at 2013/12/26 08:23:07 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=UTF-8; format=flowed X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 222.73.24.84 Cc: Peter Crosthwaite , QEMU Trivial , QEMU Developers , Igor Mitsyanko , Stefan Hajnoczi , Paolo Bonzini Subject: Re: [Qemu-trivial] [Qemu-devel] [PATCH] hw/sd: move sdhci.h to include/hw X-BeenThere: qemu-trivial@nongnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 26 Dec 2013 00:24:09 -0000 Peter Maydell wrote: > On 25 December 2013 07:21, liguang wrote: > > This is where your patch should have had an explanation > for why you're making this change. What is the user outside > of hw/sd/ that needs this header that means we should > move it into include/ ? > > I don't mean someone will include it outside of hw/sd, just in the sense of "header files be better in a directory called include", A10's SDHC(really an odd controller, without public datasheet) emulation will use some of definitions in this file, and I think many other standard SDHC will also be happy to include this file. From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35706) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vvyjz-0000jq-2Q for qemu-devel@nongnu.org; Wed, 25 Dec 2013 19:23:59 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Vvyju-0004W7-Fa for qemu-devel@nongnu.org; Wed, 25 Dec 2013 19:23:54 -0500 Message-ID: <52BB76AB.3060502@cn.fujitsu.com> Date: Thu, 26 Dec 2013 08:22:03 +0800 From: Li Guang MIME-Version: 1.0 References: <1387956078-10651-1-git-send-email-lig.fnst@cn.fujitsu.com> In-Reply-To: Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=UTF-8; format=flowed Subject: Re: [Qemu-devel] [PATCH] hw/sd: move sdhci.h to include/hw List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: Peter Crosthwaite , QEMU Trivial , QEMU Developers , Igor Mitsyanko , Stefan Hajnoczi , Paolo Bonzini Peter Maydell wrote: > On 25 December 2013 07:21, liguang wrote: > > This is where your patch should have had an explanation > for why you're making this change. What is the user outside > of hw/sd/ that needs this header that means we should > move it into include/ ? > > I don't mean someone will include it outside of hw/sd, just in the sense of "header files be better in a directory called include", A10's SDHC(really an odd controller, without public datasheet) emulation will use some of definitions in this file, and I think many other standard SDHC will also be happy to include this file.