| Message ID | 1462959859-6669-1-git-send-email-aar@pengutronix.de (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
Le 11/05/2016 11:44, Alexander Aring a écrit : > The PAD define should be above the experimental support. We don't care > about if we break userspace in experimental stuff but PAD is part of the > existing UAPI. > > Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com> > Signed-off-by: Alexander Aring <aar@pengutronix.de> Acked-by: Nicolas Dichtel <nicolas.dichtel@6wind.com> -- To unsubscribe from this list: send the line "unsubscribe linux-wpan" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hello. On 11/05/16 11:44, Alexander Aring wrote: > The PAD define should be above the experimental support. We don't care > about if we break userspace in experimental stuff but PAD is part of the > existing UAPI. > > Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com> > Signed-off-by: Alexander Aring <aar@pengutronix.de> > --- > include/net/nl802154.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/include/net/nl802154.h b/include/net/nl802154.h > index fcab4de..7aad2fd 100644 > --- a/include/net/nl802154.h > +++ b/include/net/nl802154.h > @@ -124,6 +124,8 @@ enum nl802154_attrs { > > NL802154_ATTR_ACKREQ_DEFAULT, > > + NL802154_ATTR_PAD, > + > /* add attributes here, update the policy in nl802154.c */ > > #ifdef CONFIG_IEEE802154_NL802154_EXPERIMENTAL > @@ -138,8 +140,6 @@ enum nl802154_attrs { > NL802154_ATTR_SEC_KEY, > #endif /* CONFIG_IEEE802154_NL802154_EXPERIMENTAL */ > > - NL802154_ATTR_PAD, > - > __NL802154_ATTR_AFTER_LAST, > NL802154_ATTR_MAX = __NL802154_ATTR_AFTER_LAST - 1 > }; Reviewed-by: Stefan Schmidt<stefan@osg.samsung.com> regards Stefan Schmidt -- To unsubscribe from this list: send the line "unsubscribe linux-wpan" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/include/net/nl802154.h b/include/net/nl802154.h index fcab4de..7aad2fd 100644 --- a/include/net/nl802154.h +++ b/include/net/nl802154.h @@ -124,6 +124,8 @@ enum nl802154_attrs { NL802154_ATTR_ACKREQ_DEFAULT, + NL802154_ATTR_PAD, + /* add attributes here, update the policy in nl802154.c */ #ifdef CONFIG_IEEE802154_NL802154_EXPERIMENTAL @@ -138,8 +140,6 @@ enum nl802154_attrs { NL802154_ATTR_SEC_KEY, #endif /* CONFIG_IEEE802154_NL802154_EXPERIMENTAL */ - NL802154_ATTR_PAD, - __NL802154_ATTR_AFTER_LAST, NL802154_ATTR_MAX = __NL802154_ATTR_AFTER_LAST - 1 };
The PAD define should be above the experimental support. We don't care about if we break userspace in experimental stuff but PAD is part of the existing UAPI. Cc: Nicolas Dichtel <nicolas.dichtel@6wind.com> Signed-off-by: Alexander Aring <aar@pengutronix.de> --- include/net/nl802154.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)