On Wed, Apr 02, 2014 at 10:06:54AM -0700, Stephen Warren wrote: > On 04/01/2014 07:57 PM, Penny Chiu wrote: > > tegrarcm should work if the operating mode is ODM production > > secure mode. Add the operating mode definition and description. > > > diff --git a/src/main.c b/src/main.c > > > if (info.op_mode != RCM_OP_MODE_DEVEL && > > info.op_mode != RCM_OP_MODE_ODM_OPEN && > > + info.op_mode != RCM_OP_MODE_ODM_SECURE && > > info.op_mode != RCM_OP_MODE_PRE_PRODUCTION) > > Are there even more modes beyond those? I wonder if that check is even > necessary any more, or whether tegrarcm should just try to communicate > no matter what. There are other modes, but they are all secure modes, and tegrarcm has no support for encrypting RCM payloads today, so the proper thing to do is fail if its not one of these modes. > Still, any changes here can be addressed in a separate patch. > > Acked-by: > Stephen Warren <swarren@xxxxxxxxxx> > > Allen, will you apply this or do you want me to? Acked-by: Allen Martin <amartin@xxxxxxxxxx> I will apply. -Allen 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
