From: Mark Jackson <mpfj-kZtEnBLzDKq1Qrn1Bg8BZw@public.gmane.org>
To: David Brownell <david-b-yBeKhBN/0LDR7s880joybQ@public.gmane.org>
Cc: spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org,
Alessandro Zummo
<alessandro.zummo-BfzFCNDTiLLj+vYz1yj4TQ@public.gmane.org>,
lkml <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
rtc-linux-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
Subject: Re: [PATCH] Add Dallas DS1390 RTC chip
Date: Fri, 17 Oct 2008 19:57:29 +0100 [thread overview]
Message-ID: <48F8E019.20600@mimc.co.uk> (raw)
In-Reply-To: <200810170829.06699.david-b-yBeKhBN/0LDR7s880joybQ@public.gmane.org>
David Brownell wrote:
> On Friday 17 October 2008, Mark Jackson wrote:
>>>> + /* Can tx_buf and rx_buf be equal? The doc in spi.h is not sure... */
>>>> + xfer.tx_buf = chip->tx_buf;
>>>> + xfer.rx_buf = chip->rx_buf;
>>> you use the same buffer a few functions below. either
>>> one way or the other. please investigate with the spi subsystem maintainer.
>> David,
>>
>> Just to double check (as per Alessandro's suggestion), is this okay use of the
>> spi buffers ?
>
> Yes. That comment deserves to be updated.
>
> SPI does bi-directional I/O, and the underlying controller
> should be able to just replace the contents of a buffer
> with a spi_transfer that uses the same rx and tx buffers.
Excellent. Thanks for the confirmation.
Regards
Mark
-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
WARNING: multiple messages have this Message-ID (diff)
From: Mark Jackson <mpfj@mimc.co.uk>
To: David Brownell <david-b@pacbell.net>
Cc: Alessandro Zummo <alessandro.zummo@towertech.it>,
rtc-linux@googlegroups.com, lkml <linux-kernel@vger.kernel.org>,
spi-devel-general@lists.sourceforge.net
Subject: Re: [PATCH] Add Dallas DS1390 RTC chip
Date: Fri, 17 Oct 2008 19:57:29 +0100 [thread overview]
Message-ID: <48F8E019.20600@mimc.co.uk> (raw)
In-Reply-To: <200810170829.06699.david-b@pacbell.net>
David Brownell wrote:
> On Friday 17 October 2008, Mark Jackson wrote:
>>>> + /* Can tx_buf and rx_buf be equal? The doc in spi.h is not sure... */
>>>> + xfer.tx_buf = chip->tx_buf;
>>>> + xfer.rx_buf = chip->rx_buf;
>>> you use the same buffer a few functions below. either
>>> one way or the other. please investigate with the spi subsystem maintainer.
>> David,
>>
>> Just to double check (as per Alessandro's suggestion), is this okay use of the
>> spi buffers ?
>
> Yes. That comment deserves to be updated.
>
> SPI does bi-directional I/O, and the underlying controller
> should be able to just replace the contents of a buffer
> with a spi_transfer that uses the same rx and tx buffers.
Excellent. Thanks for the confirmation.
Regards
Mark
next prev parent reply other threads:[~2008-10-17 18:57 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <48F452BE.2030901@mimc.co.uk>
2008-10-14 8:17 ` [PATCH] Add Dallas DS1390 RTC chip Alessandro Zummo
2008-10-17 8:02 ` Mark Jackson
[not found] ` <48F8467C.5010902-kZtEnBLzDKq1Qrn1Bg8BZw@public.gmane.org>
2008-10-17 15:29 ` David Brownell
2008-10-17 15:29 ` David Brownell
[not found] ` <200810170829.06699.david-b-yBeKhBN/0LDR7s880joybQ@public.gmane.org>
2008-10-17 18:57 ` Mark Jackson [this message]
2008-10-17 18:57 ` Mark Jackson
2008-10-07 13:44 Mark Jackson
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=48F8E019.20600@mimc.co.uk \
--to=mpfj-kztenblzdkq1qrn1bg8bzw@public.gmane.org \
--cc=alessandro.zummo-BfzFCNDTiLLj+vYz1yj4TQ@public.gmane.org \
--cc=david-b-yBeKhBN/0LDR7s880joybQ@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=rtc-linux-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org \
--cc=spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.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.