On Friday 04 November 2011 07:24 PM, Henning Heinold wrote:
> Hi Varun,
>
> thanks that you come up with an "official" patch for the aes-stuff.
>
> Against which tree you did test the patch?
I tested it against Linus's master branch but unfortunately, some other
changes crept inside this patch due to which you saw that compilation
errors.
> I tested it against
> git://git.kernel.org/pub/scm/linux/kernel/git/olof/tegra.git for-next
> choose to build it as module and got the following errors:
>
> drivers/crypto/tegra-aes.c: In function 'aes_start_crypt':
> drivers/crypto/tegra-aes.c:226:13: error: 'eng' undeclared (first use in this
> function)
> drivers/crypto/tegra-aes.c:226:13: note: each undeclared identifier is
> reported only once for each function it appears in
> drivers/crypto/tegra-aes.c: In function 'aes_irq':
> drivers/crypto/tegra-aes.c:614:18: error: 'intr_err_mask' undeclared (first
> use in this function)
> drivers/crypto/tegra-aes.c:620:1: warning: label 'done' defined but not used
>
> Which are problems inside the driver it self, which can be easy fixed:
>
> - aes_writel(eng, 0xFFFFFFFF, INTR_STATUS);
> + aes_writel(dd, 0xFFFFFFFF, INTR_STATUS);
>
> - aes_writel(dd, intr_err_mask, INTR_STATUS);
> + aes_writel(dd, INT_ERROR_MASK, INTR_STATUS);
>
> Second problem it don't build as modul
>
> first:
> typo in
> MODULE_LICENSE("GPLv2") it needs a space MODULE_LICENSE("GPL v2")
All the above changes will be present in my next patch.
> second:
> tegra_chip_uid function is not exported
Will submit a different patch to Olof and Stephen to get this change in
the mach-tegra tree. If you want to post the fuse changes to them
yourself, please go ahead.
> I have attched patch which fixes all the stuff besides the not used variable.
> I did not runtime test it for 3.x kernels, but works backported to
> the 2.6.38-chromeos tree.
Thanks for your help.
> Bye Henning
--
To unsubscribe from this list: send the line "unsubscribe linux-crypto" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
