From: Alexander Holler <holler@ahsoftware.de>
To: Nikolay Dimitrov <picmaster@mail.bg>
Cc: "meta-freescale@yoctoproject.org"
<meta-freescale@yoctoproject.org>,
Jon Nettleton <jon.nettleton@gmail.com>,
Otavio Salvador <otavio@ossystems.com.br>
Subject: Re: [PATCH 0/1] arm: imx: fsl_otp: make fuses (OTP memory) read-only
Date: Sun, 09 Nov 2014 11:14:39 +0100 [thread overview]
Message-ID: <545F3E8F.8040003@ahsoftware.de> (raw)
In-Reply-To: <545E65D3.5080003@ahsoftware.de>
Am 08.11.2014 19:49, schrieb Alexander Holler:
> I'm not confused, at least not in regard what you want to suggest. Of
> course, I'm totally confused about the fact that almost nobody else
> before has critized that write functionality of this driver, also I'm
> already used to the fact that I'm unable to understand many things which
> are happing in todays world. But nobody is perfect. ;)
Just to make it more clear what this thread is about, here is a relevant
sentence copied form the reference manual for the chip:
"In order to avoid "rogue" code performing erroneous writes to OTP, a
special unlocking sequence is required for writes to the fuse banks."
Now guess why the HW was designed this way. And then look again at the
driver which nullifies the careful implementation in the HW. It doesn't
have to be the fault of the author, e.g. he just might have written it
for internal use. The problem is that it went into kernels for public
use and nobody has seen a problem. Might be because of missing knowledge
about what the driver does at all or whatever. I don't know.
Alexander Holler
next prev parent reply other threads:[~2014-11-09 10:15 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-07 9:43 [PATCH 0/1] arm: imx: fsl_otp: make fuses (OTP memory) read-only Alexander Holler
2014-11-07 9:43 ` [PATCH 1/1] " Alexander Holler
2014-11-07 11:34 ` Otavio Salvador
2014-11-07 14:40 ` Alexander Holler
2014-11-07 15:06 ` Otavio Salvador
2014-11-07 15:26 ` Alexander Holler
2014-11-07 14:00 ` [PATCH 0/1] " Eric Bénard
2014-11-07 14:31 ` Jon Nettleton
2014-11-07 14:55 ` Alexander Holler
2014-11-07 15:04 ` Eric Bénard
2014-11-07 15:07 ` Otavio Salvador
2014-11-07 15:23 ` Alexander Holler
2014-11-07 16:00 ` Otavio Salvador
2014-11-07 16:38 ` Alexander Holler
2014-11-08 2:03 ` Nikolay Dimitrov
2014-11-08 8:58 ` Chris Tapp
2014-11-08 9:32 ` Jon Nettleton
2014-11-08 18:49 ` Alexander Holler
2014-11-09 10:14 ` Alexander Holler [this message]
2014-11-09 15:09 ` Eric Bénard
2014-11-09 17:03 ` Bob Cochran
2014-11-09 12:34 ` Nikolay Dimitrov
2014-11-09 18:09 ` Alexander Holler
2014-11-09 19:20 ` Nikolay Dimitrov
2014-11-07 16:03 ` Eric Bénard
2014-11-07 15:50 ` Eric Bénard
-- strict thread matches above, loose matches on Subject: below --
2014-11-08 21:54 Robin Findley
2014-11-10 14:11 ` Alexander Holler
2014-11-13 19:19 ` Alexander Holler
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=545F3E8F.8040003@ahsoftware.de \
--to=holler@ahsoftware.de \
--cc=jon.nettleton@gmail.com \
--cc=meta-freescale@yoctoproject.org \
--cc=otavio@ossystems.com.br \
--cc=picmaster@mail.bg \
/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.