From: Jean Delvare <khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org>
To: Byron Bradley <byron.bbradley-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: a.zummo-BfzFCNDTiLLj+vYz1yj4TQ@public.gmane.org,
repvik-ifZ3rLY3rVnQT0dZR+AlfA@public.gmane.org,
hvr-mXXj517/zsQ@public.gmane.org,
david-b-yBeKhBN/0LDR7s880joybQ@public.gmane.org,
i2c-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org,
timtimred-f/KTTADhmRsdnm+yROfE0A@public.gmane.org,
akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org
Subject: Re: [PATCH 0/2] Add support for the S-35390A RTC chip.
Date: Fri, 4 Jan 2008 19:36:21 +0100 [thread overview]
Message-ID: <20080104193621.1067b885@hyperion.delvare> (raw)
In-Reply-To: <1197751376-9767-1-git-send-email-byron.bbradley-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Hi Byron,
On Sat, 15 Dec 2007 20:42:54 +0000, Byron Bradley wrote:
> These patches provide support the S-35390A RTC chip as used in the
> QNAP TS-109/TS-209 NAS devices. Along with a change from Andrew Morton
> this has already been accepted into the -mm tree but it was suggested
> that I send it to this list for an I2C review.
>
> This chip is special in that the first 4-bits of the address are fixed
> to 0xc and the next 3-bits are used as a command so it takes up 8 addresses
> on the I2C bus.
What a weird design :(
> 0001 Add support for the S-35390A RTC chip.
> 0002 rtc-add-support-for-the-s-35390a-rtc-chip-fix
> Written by Andrew Morton this replaces my bit-reversing function
> with bitrev8().
Given that your original patch isn't merged in git yet, the preferred
approach is to merge the fixes in your patch before you resubmit it.
This makes further reviews easier.
--
Jean Delvare
next parent reply other threads:[~2008-01-04 18:36 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1197751376-9767-1-git-send-email-byron.bbradley@gmail.com>
[not found] ` <1197751376-9767-1-git-send-email-byron.bbradley-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2008-01-04 18:36 ` Jean Delvare [this message]
[not found] ` <1197751376-9767-2-git-send-email-byron.bbradley@gmail.com>
[not found] ` <1197751376-9767-2-git-send-email-byron.bbradley-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2008-01-04 20:06 ` [PATCH 1/2] Add support for the S-35390A RTC chip Jean Delvare
[not found] ` <20080104210605.05e1a0d4-ig7AzVSIIG7kN2dkZ6Wm7A@public.gmane.org>
2008-01-04 21:42 ` David Brownell
2008-01-04 22:46 ` Byron Bradley
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=20080104193621.1067b885@hyperion.delvare \
--to=khali-puyad+kwke1g9huczpvpmw@public.gmane.org \
--cc=a.zummo-BfzFCNDTiLLj+vYz1yj4TQ@public.gmane.org \
--cc=akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org \
--cc=byron.bbradley-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=david-b-yBeKhBN/0LDR7s880joybQ@public.gmane.org \
--cc=hvr-mXXj517/zsQ@public.gmane.org \
--cc=i2c-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org \
--cc=repvik-ifZ3rLY3rVnQT0dZR+AlfA@public.gmane.org \
--cc=timtimred-f/KTTADhmRsdnm+yROfE0A@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox