From: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
To: Laurent Vivier <laurent@vivier.eu>, qemu-devel@nongnu.org
Subject: Re: [PATCH 4/8] mac_via: add GPIO for A/UX mode
Date: Fri, 15 Oct 2021 20:59:30 +0100 [thread overview]
Message-ID: <c31a73dd-ef00-76be-85df-38d097696d97@ilande.co.uk> (raw)
In-Reply-To: <500bb7ea-6e59-a3d9-2da6-4efc192c894f@vivier.eu>
On 15/10/2021 08:17, Laurent Vivier wrote:
> Le 13/10/2021 à 23:21, Mark Cave-Ayland a écrit :
>> Add a new auxmode GPIO that is updated when port B bit 6 is changed indicating
>> whether the hardware is configured for A/UX mode.
>
> Stupid question: why do you use GPIO to pass the auxmode information between VIA and GLUE?
>
> Can't we use object_property_set_link() to set a pointer to the GLUE object?
For devices that are independent i.e. not contained within others I prefer to
restrict the interface to properties that are visible within "info qom-tree" which
are MRs and GPIOs. Otherwise GLUE requires knowledge of VIA internals which breaks
the device abstraction.
ATB,
Mark.
next prev parent reply other threads:[~2021-10-15 20:01 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-13 21:21 [PATCH 0/8] q800: GLUE updates for A/UX mode Mark Cave-Ayland
2021-10-13 21:21 ` [PATCH 1/8] mac_via: update comment for VIA1B_vMystery bit Mark Cave-Ayland
2021-10-15 6:14 ` Laurent Vivier
2021-10-15 19:30 ` Mark Cave-Ayland
2021-10-13 21:21 ` [PATCH 2/8] q800: move VIA1 IRQ from level 1 to level 6 Mark Cave-Ayland
2021-10-15 6:24 ` Laurent Vivier
2021-10-13 21:21 ` [PATCH 3/8] q800: use GLUE IRQ numbers instead of IRQ level for GLUE IRQs Mark Cave-Ayland
2021-10-15 6:31 ` Laurent Vivier
2021-10-15 8:51 ` BALATON Zoltan
2021-10-15 19:42 ` Mark Cave-Ayland
2021-10-17 9:40 ` Mark Cave-Ayland
2021-10-17 13:30 ` Laurent Vivier
2021-10-13 21:21 ` [PATCH 4/8] mac_via: add GPIO for A/UX mode Mark Cave-Ayland
2021-10-15 6:58 ` Laurent Vivier
2021-10-15 19:50 ` Mark Cave-Ayland
2021-10-16 17:04 ` Laurent Vivier
2021-10-15 7:17 ` Laurent Vivier
2021-10-15 19:59 ` Mark Cave-Ayland [this message]
2021-10-16 17:06 ` Laurent Vivier
2021-10-13 21:21 ` [PATCH 5/8] q800: wire up auxmode GPIO to GLUE Mark Cave-Ayland
2021-10-15 7:01 ` Laurent Vivier
2021-10-16 18:00 ` Laurent Vivier
2021-10-13 21:21 ` [PATCH 6/8] q800: route SONIC on-board Ethernet IRQ via nubus IRQ 9 in classic mode Mark Cave-Ayland
2021-10-16 18:08 ` Laurent Vivier
2021-10-17 10:07 ` Mark Cave-Ayland
2021-10-13 21:21 ` [PATCH 7/8] q800: wire up remaining IRQs " Mark Cave-Ayland
2021-10-16 18:09 ` Laurent Vivier
2021-10-13 21:21 ` [PATCH 8/8] q800: add NMI handler Mark Cave-Ayland
2021-10-15 8:40 ` Laurent Vivier
2021-10-15 20:12 ` Mark Cave-Ayland
2021-10-16 17:09 ` Laurent Vivier
2021-10-17 10:00 ` Mark Cave-Ayland
2021-10-17 16:56 ` Laurent Vivier
2021-10-20 13:32 ` Mark Cave-Ayland
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=c31a73dd-ef00-76be-85df-38d097696d97@ilande.co.uk \
--to=mark.cave-ayland@ilande.co.uk \
--cc=laurent@vivier.eu \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.