[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC V2 2/2] sdhci: Move sdhci.h to include/
From: |
Sai Pavan Boddu |
Subject: |
Re: [Qemu-devel] [RFC V2 2/2] sdhci: Move sdhci.h to include/ |
Date: |
Wed, 9 Sep 2015 09:43:36 +0000 |
Hi Stefan,
> -----Original Message-----
> From: Stefan Hajnoczi [mailto:address@hidden
> Sent: Wednesday, September 09, 2015 3:08 PM
> To: Sai Pavan Boddu
> Cc: address@hidden; address@hidden;
> address@hidden; address@hidden; Sai Pavan Boddu; Edgar
> Iglesias; Alistair Francis
> Subject: Re: [Qemu-devel] [RFC V2 2/2] sdhci: Move sdhci.h to include/
>
> On Mon, Sep 07, 2015 at 07:28:39PM +0530, Sai Pavan Boddu wrote:
> > Move sdhci.h to include/hw/sd/. Which makes easy creation of device
> > using object_initialize.
> >
> > Signed-off-by: Sai Pavan Boddu <address@hidden>
> > ---
> > Changes for V2:
> > Create new area in includes for sd. And move sdhci.h to same.
> > ---
> > hw/sd/sdhci.c | 2 +-
> > {hw => include/hw}/sd/sdhci.h | 0
> > 2 files changed, 1 insertion(+), 1 deletion(-)
> > rename {hw => include/hw}/sd/sdhci.h (100%)
>
> Please keep hardware constants and any other device emulation internals
> in hw/sd/. The header files in include/ should only contain that
> "public" APIs that other parts of QEMU consume.
>
> I think the declarations that need to be in include/ are:
> 1. typedef struct SDHCIState
> 2. TYPE_PCI_SDHCI and PCI_SDHCI()
> 3. TYPE_SYSBUS_SDHCI and SYSBUS_SDHCI()
[Sai Pavan ] Ok sounds fair, So I will try to send out patches splitting the
header into two. And moving the public content into include/.
Regards,
Sai Pavan
>
> The other declarations are internals that should be kept in hw/sd/.
>
> This way there is a clear public API and internals (used by SD emulation
> code).