On Thu, Dec 14, 2017 at 10:02:15PM +0530, Vidya Sagar wrote: > On Tuesday 21 November 2017 02:51 AM, Bjorn Helgaas wrote: > >On Sun, Nov 12, 2017 at 06:47:52PM +0530, Vidya Sagar wrote: > >>Programs T_cmrt (Commmon Mode Restore Time) and T_pwr_on (Power On) > >Please use the exact names used in the spec. In this case, I think > >that would be "Port Common_Mode_Restore_Time" and "Port T_POWER_ON" > >(from the L1 PM Substates Capabilities Register, sec 7.33.2). > >Searching the spec for "T_cmrt" isn't going to find anything useful. > I'll take care of this in next patch > >These look like things that maybe should be in the DT because they > >depend on external component values, so you might need different > >values depending on what board the Tegra is on? Same for clk_m? > I'm afraid not. These are fixed for Tegra including clk_m. That's actually a *good* thing :) If they're fixed and can be embedded in the driver, that's better than having to extend the DT. Bjorn -- 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