[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-ppc] [Qemu-devel] [PATCH 00/13] cuda: misc fixes and cleanups

From: Hervé Poussineau
Subject: Re: [Qemu-ppc] [Qemu-devel] [PATCH 00/13] cuda: misc fixes and cleanups
Date: Sun, 7 Feb 2016 21:38:17 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Icedove/38.5.0

Le 07/02/2016 20:50, Mark Cave-Ayland a écrit :
On 06/02/16 14:30, Hervé Poussineau wrote:

Hi Mark,

Le 06/02/2016 14:32, Mark Cave-Ayland a écrit :
On 23/01/16 20:39, Hervé Poussineau wrote:


This patchset cleans up a little bit the Apple CUDA emulation:
- correctly reject unknown commands
- correctly reject commands with wrong parameters
- support changing the frequency of auto-polling
- support changing device list probed in auto-poll
- remove unused commands (GET/SET_6805_ADDR)
- remove unimplemented GET_SET_IIC/COMBINED_FORMAT_IIC

GET_SET_IIC/COMBINED_FORMAT_IIC commands should be added again once
we implement the I2C bus provided by CUDA.

This also fixes MacBugs hanging at startup in the absence of
ADB mouse input.


Hi Hervé,

I've just tried to rebase your git remote onto master to run some tests
and it looks like due to recent changes then I get a number of
conflicts. Any chance that you could rebase this patchset onto master
and update your remote accordingly?

I've rebased this patchset. This is available at
http://repo.or.cz/qemu/hpoussin.git branch cuda

As a bonus, you have an implementation of CUDA_GET_SET_IIC command and
the I2C bus.
Unfortunately, I was unable to test if it really works.

I've posted a couple of minor comments on a couple of patches, but in
general I think this is a good improvement, especially given that there
is active work fixing up more CUDA functionality.

Other than the comments mentioned above, there is also the compile fix
for patch 4 posted to the list plus I still see some IIC debugging
enabled from the git repository which outputs to stdout, e.g.

CUDA: XXX receive data from I2C [addr 48] => ff
CUDA: XXX send data to I2C [addr 40]: 05 c0

Perhaps for the moment it is best to leave out the IIC functionality
simply to get the majority of the patchset included in preparation for
more detailed work later?

I2C bus is not meant to be committed right now. That's only a proof a concept 
of how things can work.
You can note that commit message was starting with "[RFC]".

With all these issues addressed in a revised v2 patchset, I'm fairly
confident I can give the Tested-by requested by David before inclusion
in ppc-next since I didn't observe any regressions on my test images
with the patchset in its current form.

Patchset v2 has just been sent a few minutes ago. Thanks for you review!


reply via email to

[Prev in Thread] Current Thread [Next in Thread]