<p dir="ltr">Yes. Didn't think of it till I saw this..</p>
<p dir="ltr">Will resubmit this..</p>
<p dir="ltr">Michael</p>
<div class="gmail_quote">On Oct 12, 2013 3:20 PM, "Kok, Auke-jan H" <<a href="mailto:auke-jan.h.kok@intel.com">auke-jan.h.kok@intel.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Fri, Oct 11, 2013 at 4:47 PM, Michael Demeter<br>
<<a href="mailto:michael.demeter@intel.com">michael.demeter@intel.com</a>> wrote:<br>
> - Add AC_DEFINE for HAVE_SMACK to <a href="http://configure.ac" target="_blank">configure.ac</a><br>
> - Add Check for smack in Makefile.am to include smack default rules<br>
> - Add smack default rules to label /dev/xxx correctly for access<br>
><br>
> Change-Id: Iab07eb632b487b9ac4567cd08d0da6879709d44f<br>
> Signed-off-by: Michael Demeter <<a href="mailto:michael.demeter@intel.com">michael.demeter@intel.com</a>><br>
> ---<br>
>  Makefile.am                       |  5 +++++<br>
>  <a href="http://configure.ac" target="_blank">configure.ac</a>                      |  1 +<br>
>  rules/55-udev-smack-default.rules | 18 ++++++++++++++++++<br>
>  3 files changed, 24 insertions(+)<br>
>  create mode 100644 rules/55-udev-smack-default.rules<br>
><br>
> diff --git a/Makefile.am b/Makefile.am<br>
> index e8be76b..bd397c4 100644<br>
> --- a/Makefile.am<br>
> +++ b/Makefile.am<br>
> @@ -1943,6 +1943,11 @@ dist_udevrules_DATA += \<br>
>         rules/80-net-name-slot.rules \<br>
>         rules/95-udev-late.rules<br>
><br>
> +if HAVE_SMACK<br>
> +dist_udevrules_DATA += \<br>
> +       rules/55-udev-smack-default.rules<br>
> +endif<br>
> +<br>
>  dist_udevhwdb_DATA = \<br>
>         hwdb/20-pci-vendor-model.hwdb \<br>
>         hwdb/20-pci-classes.hwdb \<br>
> diff --git a/<a href="http://configure.ac" target="_blank">configure.ac</a> b/<a href="http://configure.ac" target="_blank">configure.ac</a><br>
> index 091fe20..facd931 100644<br>
> --- a/<a href="http://configure.ac" target="_blank">configure.ac</a><br>
> +++ b/<a href="http://configure.ac" target="_blank">configure.ac</a><br>
> @@ -472,6 +472,7 @@ AS_HELP_STRING([--with-smack-run-label=STRING],<br>
><br>
>  if test "x${have_smack}" = xyes ; then<br>
>          AC_DEFINE(HAVE_SMACK, 1, [Define if SMACK is available])<br>
> +        AM_CONDITIONAL([HAVE_SMACK], [true])<br>
>  fi<br>
><br>
>  # ------------------------------------------------------------------------------<br>
> diff --git a/rules/55-udev-smack-default.rules b/rules/55-udev-smack-default.rules<br>
> new file mode 100644<br>
> index 0000000..6df90bb<br>
> --- /dev/null<br>
> +++ b/rules/55-udev-smack-default.rules<br>
> @@ -0,0 +1,18 @@<br>
> +# do not edit this file, it will be overwritten on update<br>
> +<br>
> +KERNEL=="null",SMACK="*"<br>
> +KERNEL=="zero",SMACK="*"<br>
> +KERNEL=="console",SMACK="*"<br>
> +KERNEL=="kmsg",SMACK="*"<br>
> +KERNEL=="video0",SMACK="*"<br>
> +KERNEL=="card0",SMACK="*"<br>
<br>
shouldn't these be "card*" and "video*" ?<br>
<br>
Auke<br>
</blockquote></div>