Re: [PATCH 3/7] qemu: piix: Introduce functions to get pin number from irq and vice versa

[Amit Shah <amit.shah@redhat.com>]

* On Tuesday 23 Sep 2008 21:43:26 Anthony Liguori wrote:
> Amit Shah wrote:
> > Signed-off-by: Amit Shah <amit.shah@redhat.com>
> > ---
> >  qemu/hw/piix_pci.c |   19 +++++++++++++++++++
> >  1 files changed, 19 insertions(+), 0 deletions(-)
> >
> > diff --git a/qemu/hw/piix_pci.c b/qemu/hw/piix_pci.c
> > index 6fbf47b..dc12c8a 100644
> > --- a/qemu/hw/piix_pci.c
> > +++ b/qemu/hw/piix_pci.c
> > @@ -243,6 +243,25 @@ static void piix3_set_irq(qemu_irq *pic, int
> > irq_num, int level) }
> >  }
> >
> > +int piix3_get_pin(int pic_irq)
> > +{
> > +    int i;
> > +    for (i = 0; i < 4; i++)
> > +        if (piix3_dev->config[0x60+i] == pic_irq)
> > +            return i;
> > +    return -1;
> > +}
> > +
> > +int piix_get_irq(int pin)
> > +{
> > +    if (piix3_dev)
> > +        return piix3_dev->config[0x60+pin];
> > +    if (piix4_dev)
> > +        return piix4_dev->config[0x60+pin];
> > +
> > +    return 0;
> > +}
> > +
>
> If these are being exported, don't they need to be declared in a header?

I'll put then in a header.