On Thu, Jan 2, 2014 at 3:32 PM, Peter De Schrijver <pdeschrijver@xxxxxxxxxx> wrote: > On Thu, Jan 02, 2014 at 01:37:43PM +0100, Linus Walleij wrote: >> On Thu, Dec 19, 2013 at 4:54 PM, Peter De Schrijver >> <pdeschrijver@xxxxxxxxxx> wrote: >> >> > Reduce fuse.c to the minimum functionality required for the early bootstages. >> > Also export tegra_read_straps() for use by the fuse driver. >> > >> > Signed-off-by: Peter De Schrijver <pdeschrijver@xxxxxxxxxx> >> >> Aha, but why? >> > > The efuse driver in patch 2 of this series, adds the randomness, so we don't > need to add it here then? Aha I wasn't CC:ed on patch 2 so I didn't see that... > Yes, it's done in the efuse driver now, do you think that's too late? Don't know but I don't think so. The important part is that it's done before drivers needing good randomness (as opposed to good entropy). The only Tegra driver I'm aware of that needs this is the crypto driver (which I patched a bit) and I think that will only come into play when you do something like set up a VPN tunnel or similar which will be waaaaay later, so no problem AFAICT. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
