linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: broonie@kernel.org (Mark Brown)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 3/4] Documentation: DT: Add DSPI driver
Date: Wed, 24 Jul 2013 11:20:34 +0100	[thread overview]
Message-ID: <20130724102034.GA9858@sirena.org.uk> (raw)
In-Reply-To: <1374643943-3324-4-git-send-email-b44548@freescale.com>

On Wed, Jul 24, 2013 at 01:32:22PM +0800, Chao Fu wrote:

Please make sure this document is word wrapped to less than 80 columns.

> +-dspi,ctar-dbr : Double baud rate
> +Doubles the effective baud rate of the Serial Communications Clock (SCK). This field is used only in

> +-dspi,ctar-pbr : Baud Rate Prescaler
> +	0: Baud Rate Prescaler value is 2.
> +	1: Baud Rate Prescaler value is 3.
> +	2: Baud Rate Prescaler value is 5.
> +	3: Baud Rate Prescaler value is 7.

Why do these properties need to be set in DT?  They appears to be just
setting the clock rate for the bus so I'd expect them to be configured
based on that rather than manually specified.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130724/6eda3df5/attachment.sig>

  reply	other threads:[~2013-07-24 10:20 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-24  5:32 [PATCH v2 0/4]Add Freescale DSPI driver for Vybrid VF610 platform Chao Fu
2013-07-24  5:32 ` [PATCH v2 1/4] ARM:dts: vf610: Add DSPI nodes Chao Fu
2013-07-24  5:32 ` [PATCH v2 2/4] ARM:dts:vf610-twr:Enable DSPI0 devices and Flash at26df081a Chao Fu
2013-07-24  5:32 ` [PATCH v2 3/4] Documentation: DT: Add DSPI driver Chao Fu
2013-07-24 10:20   ` Mark Brown [this message]
2013-07-24  5:32 ` [PATCH v2 4/4] spi:Add Freescale DSPI driver for Vybrid VF610 platform Chao Fu
2013-07-26  7:21   ` Sascha Hauer
2013-07-28 18:41     ` Mark Brown
2013-08-07  7:58       ` Fu Chao-B44548
2013-08-08  6:54     ` Fu Chao-B44548

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=20130724102034.GA9858@sirena.org.uk \
    --to=broonie@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.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).