On 16/11/17 14:28, Peter De Schrijver wrote: > This patch series introduces the Memory Built-In Self Test (MBIST) > work around (WAR) needed when power ungating certain domains. More > details can be found in 'clk: tegra: MBIST WAR for Tegra210'. I choose to > implement the WAR in the Tegra210 clock driver, because most accesses are > to CAR registers and for the VENC domain, we need to make sure the CSI > clock source is not changed during the WAR execution. In general, I don't have any problems with the proposal, as there is no great way to implement the workaround AFAICT. My only thought was if we could expose the mbist WAR as a 'reset' via the clk driver and use the reset_control_xxx APIs to request and reset the mbist? Yes there is no reset assert/deassert here, but the reset framework does have a 'reset' hook to reset logic and if we are considering these WARs to reset the mbist logic maybe it is not a complete hack/abuse of the API? Feel free to tell me to get lost if it is a naff idea. Cheers Jon -- nvpublic -- 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
