Hi Kieran,
Thank you for the patch.
On Tue, Dec 04, 2018 at 03:51:40PM +0000, Kieran Bingham wrote:
> Provide an initial test which can run as part of the test suite.
> This test will report the platform and kernel version, along with
> the identified paths of required utilities.
>
> This will aid in ensuring that required tools are available on a
> running platform - and report the kernel and platform details in
> any test suite output for clarification of results.
>
> Signed-off-by: Kieran Bingham <kieran.bingham@xxxxxxxxxxxxxxxx>
> ---
> tests/vsp-unit-test-0000.sh | 20 ++++++++++++++++++++
> 1 file changed, 20 insertions(+)
> create mode 100755 tests/vsp-unit-test-0000.sh
>
> diff --git a/tests/vsp-unit-test-0000.sh b/tests/vsp-unit-test-0000.sh
> new file mode 100755
> index 000000000000..144cfc677b32
> --- /dev/null
> +++ b/tests/vsp-unit-test-0000.sh
> @@ -0,0 +1,20 @@
> +#!/bin/sh
> +
> +# Report testing conditions
> +
> +model=`cat /sys/firmware/devicetree/base/model`
Please use $(...) instead of `...`, it's more readable.
> +
> +echo "Test Conditions:"
> +
> +function check_all() {
> + echo " Platform: " "$model"
> + echo " Kernel release: " `uname -r`
> + echo " convert: " `which convert`
> + echo " compare: " `which compare`
> + echo " killall: " `which killall`
> + echo " raw2rgbpnm: " `which raw2rgbpnm`
> + echo " stress: " `which stress`
> + echo " yavta: " `which yavta`
> +}
> +
> +check_all | column -ts ":"
Could we remove the dependency on the column tool ? I don't have it in
my buildroot environment :-) I could of course add it, but in this case
I think it's easy enough to align the columns manually.
With these fixed,
Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
--
Regards,
Laurent Pinchart
