linux-kernel - Re: [RESEND][PATCH 5/7] nvmem: imx-ocotp: Add i.MX7D timing write clock setup support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <6b6d7832-692d-a729-3fa8-734e3f362694@nexus-software.ie>
Date:   Thu, 5 Oct 2017 14:00:22 +0100
From:   Bryan O'Donoghue <pure.logic@...us-software.ie>
To:     Philipp Zabel <p.zabel@...gutronix.de>,
        richard.leitner@...data.com, srinivas.kandagatla@...aro.org,
        axel.lin@...ics.com, ping.bai@....com, d.schultz@...tec.de,
        peng.fan@....com, van.freenix@...il.com
Cc:     linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org
Subject: Re: [RESEND][PATCH 5/7] nvmem: imx-ocotp: Add i.MX7D timing write
 clock setup support

On 05/10/17 11:52, Philipp Zabel wrote:
> Do we expect this to grow more different cases in the future?

Grow - I'm not so sure. Change definitely there's at least one other 
type of MX7 (which I don't have) that doesn't require a clock setup here 
at all.

> 
> Maybe it would be better to just store a .set_timing function pointer in
> the params.

Good idea.

Thanks for the reviews.

---
bod