Re: [cbootimage PATCH v1 1/2] Add support for Tegra210

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/20/2015 05:41 PM, Jimmy Zhang wrote:
Missed your email. Will resubmit v2 patch shortly.

That change can easily be a separate patch to avoid reposting the current patches.


Jimmy

-----Original Message-----
From: linux-tegra-owner@xxxxxxxxxxxxxxx [mailto:linux-tegra-
owner@xxxxxxxxxxxxxxx] On Behalf Of Stephen Warren
Sent: Friday, March 20, 2015 3:49 PM
To: Jimmy Zhang
Cc: Allen Martin; Stephen Warren; linux-tegra@xxxxxxxxxxxxxxx
Subject: Re: [cbootimage PATCH v1 1/2] Add support for Tegra210

On 03/20/2015 04:27 PM, Jimmy Zhang wrote:
Stephen Warren wrote at Wednesday, March 18, 2015 11:03 AM:
On 03/16/2015 04:51 PM, Jimmy Zhang wrote:
use option -t 210 or -s tegra210 to specify t210.

diff --git a/src/cbootimage.c b/src/cbootimage.c

@@ -71,11 +71,11 @@ usage(void)
    	printf("    -gbct                 Generate the new bct file.\n");
    	printf("    -o<ODM_DATA>          Specify the odm_data(in hex).\n");
    	printf("    -t|--tegra NN         Select target device. Must be one of:\n");
-	printf("                          20, 30, 114, 124, 132.\n");
+	printf("                          20, 30, 114, 124, 132, 210.\n");

Since the option is deprecated, I think we should only support --soc
for Tegra210. I won't mention this in any other places that would
need to be updated to fix that.

Ok, it can be removed here. But, the place that actually interprets -t
option may not worth being changed because both -t and -s run the same
code:

Ah I see. I wonder if we shouldn't remove the text "This option is
deprecated" from the -t option's help text then. If it's that easy to support,
there's no really need to deprecate it.
--
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


--
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




[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux