On Thu, Jan 2, 2014 at 3:32 PM, Peter De Schrijver <[email protected]> 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 >> <[email protected]> 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 <[email protected]> >> >> 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-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
