All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@domain.hid>
To: Rodrigo Rosenfeld Rosas <lbocseg@domain.hid>
Cc: xenomai@xenomai.org, jlpamart@domain.hid
Subject: Re: [Xenomai-core] First version for PCI framegrabber skeleton
Date: Sun, 15 Apr 2007 14:55:59 +0200	[thread overview]
Message-ID: <462220DF.7050107@domain.hid> (raw)
In-Reply-To: <20070411170158.89634.qmail@domain.hid>

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

Rodrigo Rosenfeld Rosas wrote:
> First of all I would like to state I'm glad to write to this list once more :)
> 
> Secondly, I would like to apologize for the late that I'm sending you this first version of the
> old promised skeleton for the real-time PCI framegrabber I've developed last year...
> 
> Thirdly, I would like to apologize for it being not organized nor documented, but I'll explain the
> reasons for that.
> 
> Since I've finished my Master Thesis, I have been in a hurry and at this moment I'm working from
> 8h to 22h from monday to thursday and from 8h to 18h at friday in three different jobs... So, it
> is true... I do not have time at the moment for writing the skeleton the way I want. But since I
> received the message below yesterday (actually it was sent a week before but I didn't have time to
> check my other mail), I tried to help him.
> 
> I tried to get some free time at my job today for organizing a bit and mainly for removing the
> specific DataTranslation code from the skeleton because I do not have permission yet to send the
> full code. I cannot assure the attached code will even compile, since I work on a Windows box at
> work... Since I'm going to São Paulo tomorrow morning (6 am), in a business trip, I hope some of
> you could assist him for any doubts due to code desorganization and lack of documentation...
> 
> Hope I can help with this code. Sorry, but I cannot do more than that until monday, when I come
> back.

Your effort toward an RT framegrabbing driver interface is very welcome,
and I hope it can lay the ground for further work in this domain, also
via different media like FireWire or USB.

Just one issue remains as you already apologised for: this code needs a
/little/ bit cleanup. I tried to get an impression by browsing through
it, but it's fairly hard to read even for someone who thinks to know
what to look for. Too much experimental or dead code.

Maybe, once time permits, you can start the cleanup by reducing the
driver to a level that only leaves the removed HW access as markers,
killing everything that is not used right now in your real driver.

Jan


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 250 bytes --]

      reply	other threads:[~2007-04-15 12:55 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-11 17:01 [Xenomai-core] First version for PCI framegrabber skeleton Rodrigo Rosenfeld Rosas
2007-04-15 12:55 ` Jan Kiszka [this message]

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=462220DF.7050107@domain.hid \
    --to=jan.kiszka@domain.hid \
    --cc=jlpamart@domain.hid \
    --cc=lbocseg@domain.hid \
    --cc=xenomai@xenomai.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.