Re: [PATCH] crypto: Allow drivers to build if COMPILE_TEST is enabled

From	Javier Martinez Canillas <javier@osg.samsung.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] crypto: Allow drivers to build if COMPILE_TEST is enabled
Date	2015-10-14 16:20 +0200
Message-ID	<qjv4m-Wf-27@gated-at.bofh.it> (permalink)
References	<qj87N-8dT-23@gated-at.bofh.it> <qjeZA-1zx-17@gated-at.bofh.it> <qjuUF-KS-9@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hello Herbert,

On 10/14/2015 04:08 PM, Herbert Xu wrote:
> On Wed, Oct 14, 2015 at 05:03:30AM +0800, kbuild test robot wrote:
>> Hi Javier,
>>
>> [auto build test WARNING on cryptodev/master -- if it's inappropriate base, please suggest rules for selecting the more suitable base]
> 
> Javier, please make sure these warnings are fixed before you enable
> COMPILE_TEST.
>

Yes, I will. I built tested on arm32 and x86 before posting the patch
but forgot about sparse.

Getting more build coverage for the drivers will likely lead to new
build warnings like the one reported by Fengguang so I wonder if should
just desist in enabling COMPILE_TEST for drivers since I would not be
able to test in every possible setup.

On the other hand, I guess the fact that there are issues found, shows
the usefulness of COMPILE_TEST.

> Thanks,
> 

Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH] crypto: Allow drivers to build if COMPILE_TEST is enabled Javier Martinez Canillas <javier@osg.samsung.com> - 2015-10-13 15:50 +0200
  Re: [PATCH] crypto: Allow drivers to build if COMPILE_TEST is enabled kbuild test robot <lkp@intel.com> - 2015-10-13 23:10 +0200
    Re: [PATCH] crypto: Allow drivers to build if COMPILE_TEST is enabled Herbert Xu <herbert@gondor.apana.org.au> - 2015-10-14 16:10 +0200
      Re: [PATCH] crypto: Allow drivers to build if COMPILE_TEST is enabled Javier Martinez Canillas <javier@osg.samsung.com> - 2015-10-14 16:20 +0200

csiph-web