All of lore.kernel.org
 help / color / mirror / Atom feed
From: Joel Stanley <joel@jms.id.au>
To: Christopher Bostic <cbostic@linux.vnet.ibm.com>
Cc: OpenBMC Maillist <openbmc@lists.ozlabs.org>
Subject: Re: [PATCH linux dev-4.10 6/7] drivers/fsi: GPIO stability changes for Cronus/Hostboot
Date: Thu, 11 May 2017 21:37:43 +0930	[thread overview]
Message-ID: <CACPK8XcTJ+rRTjU1oHtQ8WPHc3SvHsMqZevjjdihc82saps4cA@mail.gmail.com> (raw)
In-Reply-To: <20170509213902.37939-7-cbostic@linux.vnet.ibm.com>

On Wed, May 10, 2017 at 7:09 AM, Christopher Bostic
<cbostic@linux.vnet.ibm.com> wrote:
> Improve stability in the FSI protocol by re-ordering the clocking
> delays. Increase clocking delays to 3 microseconds.  Remove
> delay prior to sampling input - now redundant given new clocking
> delay order.
>
> New effective delay per clock toggle is now 6 microseconds.
> Previous clocking summed up to 3 nS but real time measurements
> of each 1 nS delay actually was approx 1.8 microseconds.
> Previous total delay was then 1.8 * 3 = 5.4 microseconds.
>
> Signed-off-by: Christopher Bostic <cbostic@linux.vnet.ibm.com>

Applied to dev-4.10 with a note about this being a temporary fix until
we find the root cause.

Keep us updated with the investigation.

Cheers,

Joel

  reply	other threads:[~2017-05-11 12:08 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-09 21:38 [PATCH linux dev-4.10 0/7] OpenFSI Updates for dev-4.10 kernel Christopher Bostic
2017-05-09 21:38 ` [PATCH linux dev-4.10 1/7] drivers/fsi: Add slave interrupt polling Christopher Bostic
2017-05-10 22:26   ` Eddie James
2017-05-09 21:38 ` [PATCH linux dev-4.10 2/7] drivers/fsi: Add Client IRQ Enable / Disable Christopher Bostic
2017-05-09 21:38 ` [PATCH linux dev-4.10 3/7] drivers/fsi: Scan for hub sourced IRQ's Christopher Bostic
2017-05-09 21:38 ` [PATCH linux dev-4.10 4/7] drivers/fsi: Set IRQ masks along hub path Christopher Bostic
2017-05-09 21:39 ` [PATCH linux dev-4.10 5/7] drivers/fsi: Unscan on master unregister Christopher Bostic
2017-05-11 12:06   ` Joel Stanley
2017-05-09 21:39 ` [PATCH linux dev-4.10 6/7] drivers/fsi: GPIO stability changes for Cronus/Hostboot Christopher Bostic
2017-05-11 12:07   ` Joel Stanley [this message]
2017-05-09 21:39 ` [PATCH linux dev-4.10 7/7] drivers/fsi: Fix one and two byte bus reads/writes Christopher Bostic
2017-05-11 12:06   ` Joel Stanley
2017-05-10  2:25 ` [PATCH linux dev-4.10 0/7] OpenFSI Updates for dev-4.10 kernel Jeremy Kerr
2017-05-10 15:46   ` Christopher Bostic
2017-05-11 12:09 ` Joel Stanley

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=CACPK8XcTJ+rRTjU1oHtQ8WPHc3SvHsMqZevjjdihc82saps4cA@mail.gmail.com \
    --to=joel@jms.id.au \
    --cc=cbostic@linux.vnet.ibm.com \
    --cc=openbmc@lists.ozlabs.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.