Hi,
attached patch adds the compareboard utility, which compares a kbuildall build's variables with those of an abuild build.
The previous version which I used so far only looked for existance of variables (and had a bug for AMD boards, so some of them lack a bunch of crucial values, sorry about that). This version also tells you the values, but converts them from hex to integer first. While this is not very helpful for vendor/device id pairs and addresses, it helps because it eliminates "0xa" vs. "10" issues.
It hooks up to kbuildall, so just running it leads to abuild being called (warning: large set of large trees around), and compares their output. The results are dumped to stdout and stored as kbuildall.results/$boardname.variables.txt
Starting from these diffs, it should be easier to make board configurations in kconfig more correct.
Signed-off-by: Patrick Georgi patrick.georgi@coresystems.de
Patrick
Signed-off-by: Patrick Georgi patrick.georgi@coresystems.de
Can you make it check if the abuild tree is already there and skip the abuild?
Acked-by: Myles Watson mylesgw@gmail.com
Thanks, Myles
Am Montag, den 12.10.2009, 11:25 -0600 schrieb Myles Watson:
Signed-off-by: Patrick Georgi patrick.georgi@coresystems.de
Can you make it check if the abuild tree is already there and skip the abuild?
abuild does that itself already.
On Mon, Oct 12, 2009 at 12:28 PM, Patrick Georgi patrick@georgi-clan.de wrote:
Am Montag, den 12.10.2009, 11:25 -0600 schrieb Myles Watson:
Signed-off-by: Patrick Georgi patrick.georgi@coresystems.de
Can you make it check if the abuild tree is already there and skip the abuild?
abuild does that itself already.
Right. Sorry for the noise.
Myles
This version also tells you the values, but converts them from hex to integer first. While this is not very helpful for vendor/device id pairs and addresses, it helps because it eliminates "0xa" vs. "10" issues.
More of the tricky values are hex than ints, so maybe printing them all as hex would be better.
Thanks, Myles
On 12.10.2009 19:00, Patrick Georgi wrote:
attached patch adds the compareboard utility, which compares a kbuildall build's variables with those of an abuild build.
Signed-off-by: Patrick Georgi patrick.georgi@coresystems.de
Very nice! Acked-by: Carl-Daniel Hailfinger c-d.hailfinger.devel.2006@gmx.net
Regards, Carl-Daniel