qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Alberto Garcia <agarcia@igalia.com>
To: qemu-devel@nongnu.org
Cc: "Anthony Liguori" <aliguori@us.ibm.com>,
	"Alberto Garcia" <agarcia@igalia.com>,
	"Blue Swirl" <blauwirbel@gmail.com>,
	"Avi Kivity" <avi@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Andreas Färber" <afaerber@suse.de>
Subject: [Qemu-devel] [PATCH v2 0/2] Add TPCI200 and IP-Octal 232 IndustryPack emulation
Date: Wed,  5 Dec 2012 15:16:46 +0200	[thread overview]
Message-ID: <cover.1354713113.git.agarcia@igalia.com> (raw)
In-Reply-To: <cover.1345730024.git.agarcia@igalia.com>

[-- Attachment #1: Type: text/plain, Size: 2325 bytes --]


Hello again,

I'm sending the new version of my TPCI200 and IP-Octal 232 patches,
here's the original submission for reference:

https://lists.gnu.org/archive/html/qemu-devel/2012-08/msg04173.html

It's been a while since I posted the previous patches. I was on
holidays for quite some time, plus the trip to the KVM Forum where I
could have the chance to meet some of you in person :)

Then came the v1.3 code freeze, so I was making a few changes to both
devices since then. These are the changes from the previous version:

   * Rebased against the current master.  ipoctal renamed to
   * ipoctal232.  Big-endian mode can now be disabled (tpci200).
   * Implemented the RxFIFO (ipoctal232).  Simplified the Tx code
   * (ipoctal232).  Removed the timer (ipoctal232).  Rewrote the IRQ
   * code (both devices).  A few minor bugfixes.  Coding style fixes.

The Linux driver is available in the latest mainline kernel (tested
with v3.7-rc6).

I also devoted quite some time to testing the code (and fixing a few
bugs in the kernel driver along the way). The device has 8 serial
ports, I tried them with minicom, pppd, etc and everything seems to
work fine.

I also tried attaching 4 instances of ipoctal232 to the tpci200 bridge
-32 serial ports in total- and using all of them at the same time and
there was nothing unexpected.

As a last thing, I also tried to reorganize the memory addressing
using Avi's suggestions but I couldn't come up with a solution to the
problems that I described back then. Here's the original thread for
reference:

https://lists.gnu.org/archive/html/qemu-devel/2012-10/msg01117.html

I think that's all, if there's any questions please let me know.

Thanks,

Alberto Garcia (2):
  Add TEWS TPCI200 IndustryPack emulation
  Add GE IP-Octal 232 IndustryPack emulation

 default-configs/pci.mak |    1 +
 hw/Makefile.objs        |    3 +
 hw/ipack.c              |  106 ++++++++
 hw/ipack.h              |   75 ++++++
 hw/ipoctal.c            |  613 +++++++++++++++++++++++++++++++++++++++++++
 hw/pci_ids.h            |    3 +
 hw/tpci200.c            |  667 +++++++++++++++++++++++++++++++++++++++++++++++
 7 ficheiros modificados, 1468 adições(+)
 create mode 100644 hw/ipack.c
 create mode 100644 hw/ipack.h
 create mode 100644 hw/ipoctal.c
 create mode 100644 hw/tpci200.c

-- 
1.7.10.4


  parent reply	other threads:[~2012-12-05 13:17 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-23 13:59 [Qemu-devel] [PATCH 0/2] Add TPCI200 and IP-Octal 232 IndustryPack emulation Alberto Garcia
2012-08-23 13:59 ` [Qemu-devel] [PATCH 1/2] Add TEWS TPCI200 " Alberto Garcia
2012-08-23 13:59 ` [Qemu-devel] [PATCH 2/2] Add IP-Octal 232 " Alberto Garcia
2012-08-31 14:12 ` [Qemu-devel] Ping [PATCH 0/2] Add TPCI200 and " Alberto Garcia
2012-08-31 16:09   ` Andreas Färber
2012-09-10 13:10     ` Alberto Garcia
2012-10-05 13:20     ` Alberto Garcia
2012-10-05 14:28       ` Paolo Bonzini
2012-10-05 14:52         ` Anthony Liguori
2012-10-05 16:24         ` Blue Swirl
2012-10-06 11:29           ` Alberto Garcia
2012-10-07 10:13           ` Avi Kivity
2012-10-07 10:19             ` Avi Kivity
2012-10-08  8:02               ` Alberto Garcia
2012-10-10 10:24             ` Alberto Garcia
2012-10-10 11:35               ` Avi Kivity
2012-10-10 17:59                 ` Alberto Garcia
2012-10-05 14:54       ` Andreas Färber
2012-12-05 13:16 ` Alberto Garcia [this message]
2012-12-05 14:03   ` [Qemu-devel] [PATCH v2 " Andreas Färber
2012-12-05 13:16 ` [Qemu-devel] [PATCH v2 1/2] Add TEWS TPCI200 " Alberto Garcia
2012-12-05 13:16 ` [Qemu-devel] [PATCH v2 2/2] Add GE IP-Octal 232 " Alberto Garcia
2012-12-05 15:24 ` [Qemu-devel] [PATCH v3 0/2] Add TPCI200 and " Alberto Garcia
2012-12-17 15:27   ` [Qemu-devel] Ping " Alberto Garcia
2013-01-07 20:32     ` Anthony Liguori
2012-12-05 15:24 ` [Qemu-devel] [PATCH v3 1/2] Add TEWS TPCI200 " Alberto Garcia
2012-12-05 15:24 ` [Qemu-devel] [PATCH v3 2/2] Add GE IP-Octal 232 " Alberto Garcia

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=cover.1354713113.git.agarcia@igalia.com \
    --to=agarcia@igalia.com \
    --cc=afaerber@suse.de \
    --cc=aliguori@us.ibm.com \
    --cc=avi@redhat.com \
    --cc=blauwirbel@gmail.com \
    --cc=pbonzini@redhat.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).