* [PATCH 5/8] linux: drop __bitwise__ everywhere [not found] <1481778865-27667-1-git-send-email-mst@redhat.com> @ 2016-12-15 5:15 ` Michael S. Tsirkin 2016-12-15 9:04 ` Stefan Schmidt ` (4 more replies) 2016-12-15 5:15 ` [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags Michael S. Tsirkin 1 sibling, 5 replies; 12+ messages in thread From: Michael S. Tsirkin @ 2016-12-15 5:15 UTC (permalink / raw) To: linux-kernel Cc: Kukjin Kim, Krzysztof Kozlowski, Javier Martinez Canillas, Russell King, Alasdair Kergon, Mike Snitzer, dm-devel, Shaohua Li, Johannes Berg, Emmanuel Grumbach, Luca Coelho, Intel Linux Wireless, Kalle Valo, Greg Kroah-Hartman, Jiri Slaby, Lee Duncan, Chris Leech, James E.J. Bottomley, Martin K. Petersen, Nicholas A. Bellinger, Jason Wang, Alexander Aring, Stefan Schmidt, David S. Miller, linux-arm-kernel, linux-samsung-soc, linux-raid, netdev, linux-wireless, linux-mm, open-iscsi, linux-scsi, target-devel, virtualization, linux-wpan __bitwise__ used to mean "yes, please enable sparse checks unconditionally", but now that we dropped __CHECK_ENDIAN__ __bitwise is exactly the same. There aren't many users, replace it by __bitwise everywhere. Signed-off-by: Michael S. Tsirkin <mst@redhat.com> --- arch/arm/plat-samsung/include/plat/gpio-cfg.h | 2 +- drivers/md/dm-cache-block-types.h | 6 +++--- drivers/net/ethernet/sun/sunhme.h | 2 +- drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h | 4 ++-- include/linux/mmzone.h | 2 +- include/linux/serial_core.h | 4 ++-- include/linux/types.h | 4 ++-- include/scsi/iscsi_proto.h | 2 +- include/target/target_core_base.h | 2 +- include/uapi/linux/virtio_types.h | 6 +++--- net/ieee802154/6lowpan/6lowpan_i.h | 2 +- net/mac80211/ieee80211_i.h | 4 ++-- 12 files changed, 20 insertions(+), 20 deletions(-) diff --git a/arch/arm/plat-samsung/include/plat/gpio-cfg.h b/arch/arm/plat-samsung/include/plat/gpio-cfg.h index 21391fa..e55d1f5 100644 --- a/arch/arm/plat-samsung/include/plat/gpio-cfg.h +++ b/arch/arm/plat-samsung/include/plat/gpio-cfg.h @@ -26,7 +26,7 @@ #include <linux/types.h> -typedef unsigned int __bitwise__ samsung_gpio_pull_t; +typedef unsigned int __bitwise samsung_gpio_pull_t; /* forward declaration if gpio-core.h hasn't been included */ struct samsung_gpio_chip; diff --git a/drivers/md/dm-cache-block-types.h b/drivers/md/dm-cache-block-types.h index bed4ad4..389c9e8 100644 --- a/drivers/md/dm-cache-block-types.h +++ b/drivers/md/dm-cache-block-types.h @@ -17,9 +17,9 @@ * discard bitset. */ -typedef dm_block_t __bitwise__ dm_oblock_t; -typedef uint32_t __bitwise__ dm_cblock_t; -typedef dm_block_t __bitwise__ dm_dblock_t; +typedef dm_block_t __bitwise dm_oblock_t; +typedef uint32_t __bitwise dm_cblock_t; +typedef dm_block_t __bitwise dm_dblock_t; static inline dm_oblock_t to_oblock(dm_block_t b) { diff --git a/drivers/net/ethernet/sun/sunhme.h b/drivers/net/ethernet/sun/sunhme.h index f430765..4a8d5b1 100644 --- a/drivers/net/ethernet/sun/sunhme.h +++ b/drivers/net/ethernet/sun/sunhme.h @@ -302,7 +302,7 @@ * Always write the address first before setting the ownership * bits to avoid races with the hardware scanning the ring. */ -typedef u32 __bitwise__ hme32; +typedef u32 __bitwise hme32; struct happy_meal_rxd { hme32 rx_flags; diff --git a/drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h b/drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h index 1ad0ec1..84813b5 100644 --- a/drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h +++ b/drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h @@ -228,7 +228,7 @@ enum iwl_ucode_tlv_flag { IWL_UCODE_TLV_FLAGS_BCAST_FILTERING = BIT(29), }; -typedef unsigned int __bitwise__ iwl_ucode_tlv_api_t; +typedef unsigned int __bitwise iwl_ucode_tlv_api_t; /** * enum iwl_ucode_tlv_api - ucode api @@ -258,7 +258,7 @@ enum iwl_ucode_tlv_api { #endif }; -typedef unsigned int __bitwise__ iwl_ucode_tlv_capa_t; +typedef unsigned int __bitwise iwl_ucode_tlv_capa_t; /** * enum iwl_ucode_tlv_capa - ucode capabilities diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h index 0f088f3..36d9896 100644 --- a/include/linux/mmzone.h +++ b/include/linux/mmzone.h @@ -246,7 +246,7 @@ struct lruvec { #define ISOLATE_UNEVICTABLE ((__force isolate_mode_t)0x8) /* LRU Isolation modes. */ -typedef unsigned __bitwise__ isolate_mode_t; +typedef unsigned __bitwise isolate_mode_t; enum zone_watermarks { WMARK_MIN, diff --git a/include/linux/serial_core.h b/include/linux/serial_core.h index 5d49488..5def8e8 100644 --- a/include/linux/serial_core.h +++ b/include/linux/serial_core.h @@ -111,8 +111,8 @@ struct uart_icount { __u32 buf_overrun; }; -typedef unsigned int __bitwise__ upf_t; -typedef unsigned int __bitwise__ upstat_t; +typedef unsigned int __bitwise upf_t; +typedef unsigned int __bitwise upstat_t; struct uart_port { spinlock_t lock; /* port lock */ diff --git a/include/linux/types.h b/include/linux/types.h index baf7183..d501ad3 100644 --- a/include/linux/types.h +++ b/include/linux/types.h @@ -154,8 +154,8 @@ typedef u64 dma_addr_t; typedef u32 dma_addr_t; #endif -typedef unsigned __bitwise__ gfp_t; -typedef unsigned __bitwise__ fmode_t; +typedef unsigned __bitwise gfp_t; +typedef unsigned __bitwise fmode_t; #ifdef CONFIG_PHYS_ADDR_T_64BIT typedef u64 phys_addr_t; diff --git a/include/scsi/iscsi_proto.h b/include/scsi/iscsi_proto.h index c1260d8..df156f1 100644 --- a/include/scsi/iscsi_proto.h +++ b/include/scsi/iscsi_proto.h @@ -74,7 +74,7 @@ static inline int iscsi_sna_gte(u32 n1, u32 n2) #define zero_data(p) {p[0]=0;p[1]=0;p[2]=0;} /* initiator tags; opaque for target */ -typedef uint32_t __bitwise__ itt_t; +typedef uint32_t __bitwise itt_t; /* below makes sense only for initiator that created this tag */ #define build_itt(itt, age) ((__force itt_t)\ ((itt) | ((age) << ISCSI_AGE_SHIFT))) diff --git a/include/target/target_core_base.h b/include/target/target_core_base.h index c211900..0055828 100644 --- a/include/target/target_core_base.h +++ b/include/target/target_core_base.h @@ -149,7 +149,7 @@ enum se_cmd_flags_table { * Used by transport_send_check_condition_and_sense() * to signal which ASC/ASCQ sense payload should be built. */ -typedef unsigned __bitwise__ sense_reason_t; +typedef unsigned __bitwise sense_reason_t; enum tcm_sense_reason_table { #define R(x) (__force sense_reason_t )(x) diff --git a/include/uapi/linux/virtio_types.h b/include/uapi/linux/virtio_types.h index e845e8c..55c3b73 100644 --- a/include/uapi/linux/virtio_types.h +++ b/include/uapi/linux/virtio_types.h @@ -39,8 +39,8 @@ * - __le{16,32,64} for standard-compliant virtio devices */ -typedef __u16 __bitwise__ __virtio16; -typedef __u32 __bitwise__ __virtio32; -typedef __u64 __bitwise__ __virtio64; +typedef __u16 __bitwise __virtio16; +typedef __u32 __bitwise __virtio32; +typedef __u64 __bitwise __virtio64; #endif /* _UAPI_LINUX_VIRTIO_TYPES_H */ diff --git a/net/ieee802154/6lowpan/6lowpan_i.h b/net/ieee802154/6lowpan/6lowpan_i.h index 5ac7789..ac7c96b 100644 --- a/net/ieee802154/6lowpan/6lowpan_i.h +++ b/net/ieee802154/6lowpan/6lowpan_i.h @@ -7,7 +7,7 @@ #include <net/inet_frag.h> #include <net/6lowpan.h> -typedef unsigned __bitwise__ lowpan_rx_result; +typedef unsigned __bitwise lowpan_rx_result; #define RX_CONTINUE ((__force lowpan_rx_result) 0u) #define RX_DROP_UNUSABLE ((__force lowpan_rx_result) 1u) #define RX_DROP ((__force lowpan_rx_result) 2u) diff --git a/net/mac80211/ieee80211_i.h b/net/mac80211/ieee80211_i.h index d37a577..b2069fb 100644 --- a/net/mac80211/ieee80211_i.h +++ b/net/mac80211/ieee80211_i.h @@ -159,7 +159,7 @@ enum ieee80211_bss_valid_data_flags { IEEE80211_BSS_VALID_ERP = BIT(3) }; -typedef unsigned __bitwise__ ieee80211_tx_result; +typedef unsigned __bitwise ieee80211_tx_result; #define TX_CONTINUE ((__force ieee80211_tx_result) 0u) #define TX_DROP ((__force ieee80211_tx_result) 1u) #define TX_QUEUED ((__force ieee80211_tx_result) 2u) @@ -180,7 +180,7 @@ struct ieee80211_tx_data { }; -typedef unsigned __bitwise__ ieee80211_rx_result; +typedef unsigned __bitwise ieee80211_rx_result; #define RX_CONTINUE ((__force ieee80211_rx_result) 0u) #define RX_DROP_UNUSABLE ((__force ieee80211_rx_result) 1u) #define RX_DROP_MONITOR ((__force ieee80211_rx_result) 2u) -- MST ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 5/8] linux: drop __bitwise__ everywhere 2016-12-15 5:15 ` [PATCH 5/8] linux: drop __bitwise__ everywhere Michael S. Tsirkin @ 2016-12-15 9:04 ` Stefan Schmidt 2016-12-15 11:51 ` Greg Kroah-Hartman ` (3 subsequent siblings) 4 siblings, 0 replies; 12+ messages in thread From: Stefan Schmidt @ 2016-12-15 9:04 UTC (permalink / raw) To: Michael S. Tsirkin, linux-kernel Cc: Kukjin Kim, Krzysztof Kozlowski, Javier Martinez Canillas, Russell King, Alasdair Kergon, Mike Snitzer, dm-devel, Shaohua Li, Johannes Berg, Emmanuel Grumbach, Luca Coelho, Intel Linux Wireless, Kalle Valo, Greg Kroah-Hartman, Jiri Slaby, Lee Duncan, Chris Leech, James E.J. Bottomley, Martin K. Petersen, Nicholas A. Bellinger, Jason Wang, Alexander Aring, David S. Miller, linux-arm-kernel, linux-samsung-soc, linux-raid, netdev, linux-wireless, linux-mm, open-iscsi, linux-scsi, target-devel, virtualization, linux-wpan Hello. On 15/12/16 06:15, Michael S. Tsirkin wrote: > __bitwise__ used to mean "yes, please enable sparse checks > unconditionally", but now that we dropped __CHECK_ENDIAN__ > __bitwise is exactly the same. > There aren't many users, replace it by __bitwise everywhere. > > Signed-off-by: Michael S. Tsirkin <mst@redhat.com> > --- > arch/arm/plat-samsung/include/plat/gpio-cfg.h | 2 +- > drivers/md/dm-cache-block-types.h | 6 +++--- > drivers/net/ethernet/sun/sunhme.h | 2 +- > drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h | 4 ++-- > include/linux/mmzone.h | 2 +- > include/linux/serial_core.h | 4 ++-- > include/linux/types.h | 4 ++-- > include/scsi/iscsi_proto.h | 2 +- > include/target/target_core_base.h | 2 +- > include/uapi/linux/virtio_types.h | 6 +++--- > net/ieee802154/6lowpan/6lowpan_i.h | 2 +- > net/mac80211/ieee80211_i.h | 4 ++-- > 12 files changed, 20 insertions(+), 20 deletions(-) > > diff --git a/arch/arm/plat-samsung/include/plat/gpio-cfg.h b/arch/arm/plat-samsung/include/plat/gpio-cfg.h > index 21391fa..e55d1f5 100644 > --- a/arch/arm/plat-samsung/include/plat/gpio-cfg.h > +++ b/arch/arm/plat-samsung/include/plat/gpio-cfg.h > @@ -26,7 +26,7 @@ > > #include <linux/types.h> > > -typedef unsigned int __bitwise__ samsung_gpio_pull_t; > +typedef unsigned int __bitwise samsung_gpio_pull_t; > > /* forward declaration if gpio-core.h hasn't been included */ > struct samsung_gpio_chip; > diff --git a/drivers/md/dm-cache-block-types.h b/drivers/md/dm-cache-block-types.h > index bed4ad4..389c9e8 100644 > --- a/drivers/md/dm-cache-block-types.h > +++ b/drivers/md/dm-cache-block-types.h > @@ -17,9 +17,9 @@ > * discard bitset. > */ > > -typedef dm_block_t __bitwise__ dm_oblock_t; > -typedef uint32_t __bitwise__ dm_cblock_t; > -typedef dm_block_t __bitwise__ dm_dblock_t; > +typedef dm_block_t __bitwise dm_oblock_t; > +typedef uint32_t __bitwise dm_cblock_t; > +typedef dm_block_t __bitwise dm_dblock_t; > > static inline dm_oblock_t to_oblock(dm_block_t b) > { > diff --git a/drivers/net/ethernet/sun/sunhme.h b/drivers/net/ethernet/sun/sunhme.h > index f430765..4a8d5b1 100644 > --- a/drivers/net/ethernet/sun/sunhme.h > +++ b/drivers/net/ethernet/sun/sunhme.h > @@ -302,7 +302,7 @@ > * Always write the address first before setting the ownership > * bits to avoid races with the hardware scanning the ring. > */ > -typedef u32 __bitwise__ hme32; > +typedef u32 __bitwise hme32; > > struct happy_meal_rxd { > hme32 rx_flags; > diff --git a/drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h b/drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h > index 1ad0ec1..84813b5 100644 > --- a/drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h > +++ b/drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h > @@ -228,7 +228,7 @@ enum iwl_ucode_tlv_flag { > IWL_UCODE_TLV_FLAGS_BCAST_FILTERING = BIT(29), > }; > > -typedef unsigned int __bitwise__ iwl_ucode_tlv_api_t; > +typedef unsigned int __bitwise iwl_ucode_tlv_api_t; > > /** > * enum iwl_ucode_tlv_api - ucode api > @@ -258,7 +258,7 @@ enum iwl_ucode_tlv_api { > #endif > }; > > -typedef unsigned int __bitwise__ iwl_ucode_tlv_capa_t; > +typedef unsigned int __bitwise iwl_ucode_tlv_capa_t; > > /** > * enum iwl_ucode_tlv_capa - ucode capabilities > diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h > index 0f088f3..36d9896 100644 > --- a/include/linux/mmzone.h > +++ b/include/linux/mmzone.h > @@ -246,7 +246,7 @@ struct lruvec { > #define ISOLATE_UNEVICTABLE ((__force isolate_mode_t)0x8) > > /* LRU Isolation modes. */ > -typedef unsigned __bitwise__ isolate_mode_t; > +typedef unsigned __bitwise isolate_mode_t; > > enum zone_watermarks { > WMARK_MIN, > diff --git a/include/linux/serial_core.h b/include/linux/serial_core.h > index 5d49488..5def8e8 100644 > --- a/include/linux/serial_core.h > +++ b/include/linux/serial_core.h > @@ -111,8 +111,8 @@ struct uart_icount { > __u32 buf_overrun; > }; > > -typedef unsigned int __bitwise__ upf_t; > -typedef unsigned int __bitwise__ upstat_t; > +typedef unsigned int __bitwise upf_t; > +typedef unsigned int __bitwise upstat_t; > > struct uart_port { > spinlock_t lock; /* port lock */ > diff --git a/include/linux/types.h b/include/linux/types.h > index baf7183..d501ad3 100644 > --- a/include/linux/types.h > +++ b/include/linux/types.h > @@ -154,8 +154,8 @@ typedef u64 dma_addr_t; > typedef u32 dma_addr_t; > #endif > > -typedef unsigned __bitwise__ gfp_t; > -typedef unsigned __bitwise__ fmode_t; > +typedef unsigned __bitwise gfp_t; > +typedef unsigned __bitwise fmode_t; > > #ifdef CONFIG_PHYS_ADDR_T_64BIT > typedef u64 phys_addr_t; > diff --git a/include/scsi/iscsi_proto.h b/include/scsi/iscsi_proto.h > index c1260d8..df156f1 100644 > --- a/include/scsi/iscsi_proto.h > +++ b/include/scsi/iscsi_proto.h > @@ -74,7 +74,7 @@ static inline int iscsi_sna_gte(u32 n1, u32 n2) > #define zero_data(p) {p[0]=0;p[1]=0;p[2]=0;} > > /* initiator tags; opaque for target */ > -typedef uint32_t __bitwise__ itt_t; > +typedef uint32_t __bitwise itt_t; > /* below makes sense only for initiator that created this tag */ > #define build_itt(itt, age) ((__force itt_t)\ > ((itt) | ((age) << ISCSI_AGE_SHIFT))) > diff --git a/include/target/target_core_base.h b/include/target/target_core_base.h > index c211900..0055828 100644 > --- a/include/target/target_core_base.h > +++ b/include/target/target_core_base.h > @@ -149,7 +149,7 @@ enum se_cmd_flags_table { > * Used by transport_send_check_condition_and_sense() > * to signal which ASC/ASCQ sense payload should be built. > */ > -typedef unsigned __bitwise__ sense_reason_t; > +typedef unsigned __bitwise sense_reason_t; > > enum tcm_sense_reason_table { > #define R(x) (__force sense_reason_t )(x) > diff --git a/include/uapi/linux/virtio_types.h b/include/uapi/linux/virtio_types.h > index e845e8c..55c3b73 100644 > --- a/include/uapi/linux/virtio_types.h > +++ b/include/uapi/linux/virtio_types.h > @@ -39,8 +39,8 @@ > * - __le{16,32,64} for standard-compliant virtio devices > */ > > -typedef __u16 __bitwise__ __virtio16; > -typedef __u32 __bitwise__ __virtio32; > -typedef __u64 __bitwise__ __virtio64; > +typedef __u16 __bitwise __virtio16; > +typedef __u32 __bitwise __virtio32; > +typedef __u64 __bitwise __virtio64; > > #endif /* _UAPI_LINUX_VIRTIO_TYPES_H */ > diff --git a/net/ieee802154/6lowpan/6lowpan_i.h b/net/ieee802154/6lowpan/6lowpan_i.h > index 5ac7789..ac7c96b 100644 > --- a/net/ieee802154/6lowpan/6lowpan_i.h > +++ b/net/ieee802154/6lowpan/6lowpan_i.h > @@ -7,7 +7,7 @@ > #include <net/inet_frag.h> > #include <net/6lowpan.h> > > -typedef unsigned __bitwise__ lowpan_rx_result; > +typedef unsigned __bitwise lowpan_rx_result; > #define RX_CONTINUE ((__force lowpan_rx_result) 0u) > #define RX_DROP_UNUSABLE ((__force lowpan_rx_result) 1u) > #define RX_DROP ((__force lowpan_rx_result) 2u) > diff --git a/net/mac80211/ieee80211_i.h b/net/mac80211/ieee80211_i.h > index d37a577..b2069fb 100644 > --- a/net/mac80211/ieee80211_i.h > +++ b/net/mac80211/ieee80211_i.h > @@ -159,7 +159,7 @@ enum ieee80211_bss_valid_data_flags { > IEEE80211_BSS_VALID_ERP = BIT(3) > }; > > -typedef unsigned __bitwise__ ieee80211_tx_result; > +typedef unsigned __bitwise ieee80211_tx_result; > #define TX_CONTINUE ((__force ieee80211_tx_result) 0u) > #define TX_DROP ((__force ieee80211_tx_result) 1u) > #define TX_QUEUED ((__force ieee80211_tx_result) 2u) > @@ -180,7 +180,7 @@ struct ieee80211_tx_data { > }; > > > -typedef unsigned __bitwise__ ieee80211_rx_result; > +typedef unsigned __bitwise ieee80211_rx_result; > #define RX_CONTINUE ((__force ieee80211_rx_result) 0u) > #define RX_DROP_UNUSABLE ((__force ieee80211_rx_result) 1u) > #define RX_DROP_MONITOR ((__force ieee80211_rx_result) 2u) > For net/ieee802154/6lowpan/6lowpan_i.h Acked-by: Stefan Schmidt <stefan@osg.samsung.com> regards Stefan Schmidt ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 5/8] linux: drop __bitwise__ everywhere 2016-12-15 5:15 ` [PATCH 5/8] linux: drop __bitwise__ everywhere Michael S. Tsirkin 2016-12-15 9:04 ` Stefan Schmidt @ 2016-12-15 11:51 ` Greg Kroah-Hartman 2016-12-15 17:28 ` Krzysztof Kozlowski ` (2 subsequent siblings) 4 siblings, 0 replies; 12+ messages in thread From: Greg Kroah-Hartman @ 2016-12-15 11:51 UTC (permalink / raw) To: Michael S. Tsirkin Cc: linux-kernel, Kukjin Kim, Krzysztof Kozlowski, Javier Martinez Canillas, Russell King, Alasdair Kergon, Mike Snitzer, dm-devel, Shaohua Li, Johannes Berg, Emmanuel Grumbach, Luca Coelho, Intel Linux Wireless, Kalle Valo, Jiri Slaby, Lee Duncan, Chris Leech, James E.J. Bottomley, Martin K. Petersen, Nicholas A. Bellinger, Jason Wang, Alexander Aring, Stefan Schmidt, David S. Miller, linux-arm-kernel, linux-samsung-soc, linux-raid, netdev, linux-wireless, linux-mm, open-iscsi, linux-scsi, target-devel, virtualization, linux-wpan On Thu, Dec 15, 2016 at 07:15:20AM +0200, Michael S. Tsirkin wrote: > __bitwise__ used to mean "yes, please enable sparse checks > unconditionally", but now that we dropped __CHECK_ENDIAN__ > __bitwise is exactly the same. > There aren't many users, replace it by __bitwise everywhere. > > Signed-off-by: Michael S. Tsirkin <mst@redhat.com> > --- > arch/arm/plat-samsung/include/plat/gpio-cfg.h | 2 +- > drivers/md/dm-cache-block-types.h | 6 +++--- > drivers/net/ethernet/sun/sunhme.h | 2 +- > drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h | 4 ++-- > include/linux/mmzone.h | 2 +- > include/linux/serial_core.h | 4 ++-- > include/linux/types.h | 4 ++-- > include/scsi/iscsi_proto.h | 2 +- > include/target/target_core_base.h | 2 +- > include/uapi/linux/virtio_types.h | 6 +++--- > net/ieee802154/6lowpan/6lowpan_i.h | 2 +- > net/mac80211/ieee80211_i.h | 4 ++-- > 12 files changed, 20 insertions(+), 20 deletions(-) for include/linux/serial_core.h: Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 5/8] linux: drop __bitwise__ everywhere 2016-12-15 5:15 ` [PATCH 5/8] linux: drop __bitwise__ everywhere Michael S. Tsirkin 2016-12-15 9:04 ` Stefan Schmidt 2016-12-15 11:51 ` Greg Kroah-Hartman @ 2016-12-15 17:28 ` Krzysztof Kozlowski 2016-12-15 19:44 ` Lee Duncan 2016-12-19 9:09 ` Luca Coelho 4 siblings, 0 replies; 12+ messages in thread From: Krzysztof Kozlowski @ 2016-12-15 17:28 UTC (permalink / raw) To: Michael S. Tsirkin Cc: linux-kernel, Kukjin Kim, Krzysztof Kozlowski, Javier Martinez Canillas, Russell King, Alasdair Kergon, Mike Snitzer, dm-devel, Shaohua Li, Johannes Berg, Emmanuel Grumbach, Luca Coelho, Intel Linux Wireless, Kalle Valo, Greg Kroah-Hartman, Jiri Slaby, Lee Duncan, Chris Leech, James E.J. Bottomley, Martin K. Petersen, Nicholas A. Bellinger, Jason Wang, Alexander Aring, Stefan Schmidt, David S. Miller, linux-arm-kernel, linux-samsung-soc, linux-raid, netdev, linux-wireless, linux-mm, open-iscsi, linux-scsi, target-devel, virtualization, linux-wpan On Thu, Dec 15, 2016 at 07:15:20AM +0200, Michael S. Tsirkin wrote: > __bitwise__ used to mean "yes, please enable sparse checks > unconditionally", but now that we dropped __CHECK_ENDIAN__ > __bitwise is exactly the same. > There aren't many users, replace it by __bitwise everywhere. > > Signed-off-by: Michael S. Tsirkin <mst@redhat.com> > --- > arch/arm/plat-samsung/include/plat/gpio-cfg.h | 2 +- > drivers/md/dm-cache-block-types.h | 6 +++--- > drivers/net/ethernet/sun/sunhme.h | 2 +- > drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h | 4 ++-- > include/linux/mmzone.h | 2 +- > include/linux/serial_core.h | 4 ++-- > include/linux/types.h | 4 ++-- > include/scsi/iscsi_proto.h | 2 +- > include/target/target_core_base.h | 2 +- > include/uapi/linux/virtio_types.h | 6 +++--- > net/ieee802154/6lowpan/6lowpan_i.h | 2 +- > net/mac80211/ieee80211_i.h | 4 ++-- > 12 files changed, 20 insertions(+), 20 deletions(-) > > diff --git a/arch/arm/plat-samsung/include/plat/gpio-cfg.h b/arch/arm/plat-samsung/include/plat/gpio-cfg.h > index 21391fa..e55d1f5 100644 > --- a/arch/arm/plat-samsung/include/plat/gpio-cfg.h > +++ b/arch/arm/plat-samsung/include/plat/gpio-cfg.h > @@ -26,7 +26,7 @@ > > #include <linux/types.h> > > -typedef unsigned int __bitwise__ samsung_gpio_pull_t; > +typedef unsigned int __bitwise samsung_gpio_pull_t; > > /* forward declaration if gpio-core.h hasn't been included */ > struct samsung_gpio_chip; For plat-samsung: Acked-by: Krzysztof Kozlowski <krzk@kernel.org> Best regards, Krzysztof ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 5/8] linux: drop __bitwise__ everywhere 2016-12-15 5:15 ` [PATCH 5/8] linux: drop __bitwise__ everywhere Michael S. Tsirkin ` (2 preceding siblings ...) 2016-12-15 17:28 ` Krzysztof Kozlowski @ 2016-12-15 19:44 ` Lee Duncan 2016-12-19 9:09 ` Luca Coelho 4 siblings, 0 replies; 12+ messages in thread From: Lee Duncan @ 2016-12-15 19:44 UTC (permalink / raw) To: Michael S. Tsirkin, linux-kernel Cc: Kukjin Kim, Krzysztof Kozlowski, Javier Martinez Canillas, Russell King, Alasdair Kergon, Mike Snitzer, dm-devel, Shaohua Li, Johannes Berg, Emmanuel Grumbach, Luca Coelho, Intel Linux Wireless, Kalle Valo, Greg Kroah-Hartman, Jiri Slaby, Chris Leech, James E.J. Bottomley, Martin K. Petersen, Nicholas A. Bellinger, Jason Wang, Alexander Aring, Stefan Schmidt, David S. Miller, linux-arm-kernel, linux-samsung-soc, linux-raid, netdev, linux-wireless, linux-mm, open-iscsi, linux-scsi, target-devel, virtualization, linux-wpan On 12/14/2016 09:15 PM, Michael S. Tsirkin wrote: > __bitwise__ used to mean "yes, please enable sparse checks > unconditionally", but now that we dropped __CHECK_ENDIAN__ > __bitwise is exactly the same. > There aren't many users, replace it by __bitwise everywhere. > > Signed-off-by: Michael S. Tsirkin <mst@redhat.com> > --- > arch/arm/plat-samsung/include/plat/gpio-cfg.h | 2 +- > drivers/md/dm-cache-block-types.h | 6 +++--- > drivers/net/ethernet/sun/sunhme.h | 2 +- > drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h | 4 ++-- > include/linux/mmzone.h | 2 +- > include/linux/serial_core.h | 4 ++-- > include/linux/types.h | 4 ++-- > include/scsi/iscsi_proto.h | 2 +- > include/target/target_core_base.h | 2 +- > include/uapi/linux/virtio_types.h | 6 +++--- > net/ieee802154/6lowpan/6lowpan_i.h | 2 +- > net/mac80211/ieee80211_i.h | 4 ++-- > 12 files changed, 20 insertions(+), 20 deletions(-) > > diff --git a/arch/arm/plat-samsung/include/plat/gpio-cfg.h b/arch/arm/plat-samsung/include/plat/gpio-cfg.h > index 21391fa..e55d1f5 100644 > --- a/arch/arm/plat-samsung/include/plat/gpio-cfg.h > +++ b/arch/arm/plat-samsung/include/plat/gpio-cfg.h > @@ -26,7 +26,7 @@ > > #include <linux/types.h> > > -typedef unsigned int __bitwise__ samsung_gpio_pull_t; > +typedef unsigned int __bitwise samsung_gpio_pull_t; > > /* forward declaration if gpio-core.h hasn't been included */ > struct samsung_gpio_chip; > diff --git a/drivers/md/dm-cache-block-types.h b/drivers/md/dm-cache-block-types.h > index bed4ad4..389c9e8 100644 > --- a/drivers/md/dm-cache-block-types.h > +++ b/drivers/md/dm-cache-block-types.h > @@ -17,9 +17,9 @@ > * discard bitset. > */ > > -typedef dm_block_t __bitwise__ dm_oblock_t; > -typedef uint32_t __bitwise__ dm_cblock_t; > -typedef dm_block_t __bitwise__ dm_dblock_t; > +typedef dm_block_t __bitwise dm_oblock_t; > +typedef uint32_t __bitwise dm_cblock_t; > +typedef dm_block_t __bitwise dm_dblock_t; > > static inline dm_oblock_t to_oblock(dm_block_t b) > { > diff --git a/drivers/net/ethernet/sun/sunhme.h b/drivers/net/ethernet/sun/sunhme.h > index f430765..4a8d5b1 100644 > --- a/drivers/net/ethernet/sun/sunhme.h > +++ b/drivers/net/ethernet/sun/sunhme.h > @@ -302,7 +302,7 @@ > * Always write the address first before setting the ownership > * bits to avoid races with the hardware scanning the ring. > */ > -typedef u32 __bitwise__ hme32; > +typedef u32 __bitwise hme32; > > struct happy_meal_rxd { > hme32 rx_flags; > diff --git a/drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h b/drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h > index 1ad0ec1..84813b5 100644 > --- a/drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h > +++ b/drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h > @@ -228,7 +228,7 @@ enum iwl_ucode_tlv_flag { > IWL_UCODE_TLV_FLAGS_BCAST_FILTERING = BIT(29), > }; > > -typedef unsigned int __bitwise__ iwl_ucode_tlv_api_t; > +typedef unsigned int __bitwise iwl_ucode_tlv_api_t; > > /** > * enum iwl_ucode_tlv_api - ucode api > @@ -258,7 +258,7 @@ enum iwl_ucode_tlv_api { > #endif > }; > > -typedef unsigned int __bitwise__ iwl_ucode_tlv_capa_t; > +typedef unsigned int __bitwise iwl_ucode_tlv_capa_t; > > /** > * enum iwl_ucode_tlv_capa - ucode capabilities > diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h > index 0f088f3..36d9896 100644 > --- a/include/linux/mmzone.h > +++ b/include/linux/mmzone.h > @@ -246,7 +246,7 @@ struct lruvec { > #define ISOLATE_UNEVICTABLE ((__force isolate_mode_t)0x8) > > /* LRU Isolation modes. */ > -typedef unsigned __bitwise__ isolate_mode_t; > +typedef unsigned __bitwise isolate_mode_t; > > enum zone_watermarks { > WMARK_MIN, > diff --git a/include/linux/serial_core.h b/include/linux/serial_core.h > index 5d49488..5def8e8 100644 > --- a/include/linux/serial_core.h > +++ b/include/linux/serial_core.h > @@ -111,8 +111,8 @@ struct uart_icount { > __u32 buf_overrun; > }; > > -typedef unsigned int __bitwise__ upf_t; > -typedef unsigned int __bitwise__ upstat_t; > +typedef unsigned int __bitwise upf_t; > +typedef unsigned int __bitwise upstat_t; > > struct uart_port { > spinlock_t lock; /* port lock */ > diff --git a/include/linux/types.h b/include/linux/types.h > index baf7183..d501ad3 100644 > --- a/include/linux/types.h > +++ b/include/linux/types.h > @@ -154,8 +154,8 @@ typedef u64 dma_addr_t; > typedef u32 dma_addr_t; > #endif > > -typedef unsigned __bitwise__ gfp_t; > -typedef unsigned __bitwise__ fmode_t; > +typedef unsigned __bitwise gfp_t; > +typedef unsigned __bitwise fmode_t; > > #ifdef CONFIG_PHYS_ADDR_T_64BIT > typedef u64 phys_addr_t; > diff --git a/include/scsi/iscsi_proto.h b/include/scsi/iscsi_proto.h > index c1260d8..df156f1 100644 > --- a/include/scsi/iscsi_proto.h > +++ b/include/scsi/iscsi_proto.h > @@ -74,7 +74,7 @@ static inline int iscsi_sna_gte(u32 n1, u32 n2) > #define zero_data(p) {p[0]=0;p[1]=0;p[2]=0;} > > /* initiator tags; opaque for target */ > -typedef uint32_t __bitwise__ itt_t; > +typedef uint32_t __bitwise itt_t; > /* below makes sense only for initiator that created this tag */ > #define build_itt(itt, age) ((__force itt_t)\ > ((itt) | ((age) << ISCSI_AGE_SHIFT))) > diff --git a/include/target/target_core_base.h b/include/target/target_core_base.h > index c211900..0055828 100644 > --- a/include/target/target_core_base.h > +++ b/include/target/target_core_base.h > @@ -149,7 +149,7 @@ enum se_cmd_flags_table { > * Used by transport_send_check_condition_and_sense() > * to signal which ASC/ASCQ sense payload should be built. > */ > -typedef unsigned __bitwise__ sense_reason_t; > +typedef unsigned __bitwise sense_reason_t; > > enum tcm_sense_reason_table { > #define R(x) (__force sense_reason_t )(x) > diff --git a/include/uapi/linux/virtio_types.h b/include/uapi/linux/virtio_types.h > index e845e8c..55c3b73 100644 > --- a/include/uapi/linux/virtio_types.h > +++ b/include/uapi/linux/virtio_types.h > @@ -39,8 +39,8 @@ > * - __le{16,32,64} for standard-compliant virtio devices > */ > > -typedef __u16 __bitwise__ __virtio16; > -typedef __u32 __bitwise__ __virtio32; > -typedef __u64 __bitwise__ __virtio64; > +typedef __u16 __bitwise __virtio16; > +typedef __u32 __bitwise __virtio32; > +typedef __u64 __bitwise __virtio64; > > #endif /* _UAPI_LINUX_VIRTIO_TYPES_H */ > diff --git a/net/ieee802154/6lowpan/6lowpan_i.h b/net/ieee802154/6lowpan/6lowpan_i.h > index 5ac7789..ac7c96b 100644 > --- a/net/ieee802154/6lowpan/6lowpan_i.h > +++ b/net/ieee802154/6lowpan/6lowpan_i.h > @@ -7,7 +7,7 @@ > #include <net/inet_frag.h> > #include <net/6lowpan.h> > > -typedef unsigned __bitwise__ lowpan_rx_result; > +typedef unsigned __bitwise lowpan_rx_result; > #define RX_CONTINUE ((__force lowpan_rx_result) 0u) > #define RX_DROP_UNUSABLE ((__force lowpan_rx_result) 1u) > #define RX_DROP ((__force lowpan_rx_result) 2u) > diff --git a/net/mac80211/ieee80211_i.h b/net/mac80211/ieee80211_i.h > index d37a577..b2069fb 100644 > --- a/net/mac80211/ieee80211_i.h > +++ b/net/mac80211/ieee80211_i.h > @@ -159,7 +159,7 @@ enum ieee80211_bss_valid_data_flags { > IEEE80211_BSS_VALID_ERP = BIT(3) > }; > > -typedef unsigned __bitwise__ ieee80211_tx_result; > +typedef unsigned __bitwise ieee80211_tx_result; > #define TX_CONTINUE ((__force ieee80211_tx_result) 0u) > #define TX_DROP ((__force ieee80211_tx_result) 1u) > #define TX_QUEUED ((__force ieee80211_tx_result) 2u) > @@ -180,7 +180,7 @@ struct ieee80211_tx_data { > }; > > > -typedef unsigned __bitwise__ ieee80211_rx_result; > +typedef unsigned __bitwise ieee80211_rx_result; > #define RX_CONTINUE ((__force ieee80211_rx_result) 0u) > #define RX_DROP_UNUSABLE ((__force ieee80211_rx_result) 1u) > #define RX_DROP_MONITOR ((__force ieee80211_rx_result) 2u) > For iscsi initiator, looks good. Akced-by: Lee Duncan <lduncan@suse.com> -- Lee Duncan ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 5/8] linux: drop __bitwise__ everywhere 2016-12-15 5:15 ` [PATCH 5/8] linux: drop __bitwise__ everywhere Michael S. Tsirkin ` (3 preceding siblings ...) 2016-12-15 19:44 ` Lee Duncan @ 2016-12-19 9:09 ` Luca Coelho 4 siblings, 0 replies; 12+ messages in thread From: Luca Coelho @ 2016-12-19 9:09 UTC (permalink / raw) To: Michael S. Tsirkin, linux-kernel Cc: Kukjin Kim, Krzysztof Kozlowski, Javier Martinez Canillas, Russell King, Alasdair Kergon, Mike Snitzer, dm-devel, Shaohua Li, Johannes Berg, Emmanuel Grumbach, Intel Linux Wireless, Kalle Valo, Greg Kroah-Hartman, Jiri Slaby, Lee Duncan, Chris Leech, James E.J. Bottomley, Martin K. Petersen, Nicholas A. Bellinger, Jason Wang, Alexander Aring, Stefan Schmidt, David S. Miller, linux-arm-kernel, linux-samsung-soc, linux-raid, netdev, linux-wireless, linux-mm, open-iscsi, linux-scsi, target-devel, virtualization, linux-wpan On Thu, 2016-12-15 at 07:15 +0200, Michael S. Tsirkin wrote: > __bitwise__ used to mean "yes, please enable sparse checks > unconditionally", but now that we dropped __CHECK_ENDIAN__ > __bitwise is exactly the same. > There aren't many users, replace it by __bitwise everywhere. > > Signed-off-by: Michael S. Tsirkin <mst@redhat.com> > --- > arch/arm/plat-samsung/include/plat/gpio-cfg.h | 2 +- > drivers/md/dm-cache-block-types.h | 6 +++--- > drivers/net/ethernet/sun/sunhme.h | 2 +- > drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h | 4 ++-- For drivers/net/wireless/intel/iwlwifi/iwl-fw-file.h: Acked-by: Luca Coelho <luciano.coelho@intel.com> -- Luca. ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags [not found] <1481778865-27667-1-git-send-email-mst@redhat.com> 2016-12-15 5:15 ` [PATCH 5/8] linux: drop __bitwise__ everywhere Michael S. Tsirkin @ 2016-12-15 5:15 ` Michael S. Tsirkin 2016-12-15 7:00 ` Marcel Holtmann ` (4 more replies) 1 sibling, 5 replies; 12+ messages in thread From: Michael S. Tsirkin @ 2016-12-15 5:15 UTC (permalink / raw) To: linux-kernel Cc: Marcel Holtmann, Gustavo Padovan, Johan Hedberg, Wolfgang Grandegger, Marc Kleine-Budde, Vince Bridgers, Jay Cliburn, Chris Snook, Luis R. Rodriguez, Kalle Valo, Maya Erez, Arend van Spriel, Franky Lin, Hante Meuleman, Stanislaw Gruszka, Johannes Berg, Emmanuel Grumbach, Luca Coelho, Intel Linux Wireless, Jakub Kicinski, Larry Finger, Chaoming Li, Greg Kroah-Hartman, David S. Miller, Alexander Aring, Stefan Schmidt, Matthias Brugger, linux-bluetooth, linux-can, netdev, nios2-dev, linux-wireless, wil6210, brcm80211-dev-list.pdl, devel, linux-wpan, linux-arm-kernel, linux-mediatek That's the default now, no need for makefiles to set it. Signed-off-by: Michael S. Tsirkin <mst@redhat.com> --- drivers/bluetooth/Makefile | 2 -- drivers/net/can/Makefile | 1 - drivers/net/ethernet/altera/Makefile | 1 - drivers/net/ethernet/atheros/alx/Makefile | 1 - drivers/net/ethernet/freescale/Makefile | 2 -- drivers/net/wireless/ath/Makefile | 2 -- drivers/net/wireless/ath/wil6210/Makefile | 2 -- drivers/net/wireless/broadcom/brcm80211/brcmfmac/Makefile | 2 -- drivers/net/wireless/broadcom/brcm80211/brcmsmac/Makefile | 1 - drivers/net/wireless/intel/iwlegacy/Makefile | 2 -- drivers/net/wireless/intel/iwlwifi/Makefile | 2 +- drivers/net/wireless/intel/iwlwifi/dvm/Makefile | 2 +- drivers/net/wireless/intel/iwlwifi/mvm/Makefile | 2 +- drivers/net/wireless/intersil/orinoco/Makefile | 3 --- drivers/net/wireless/mediatek/mt7601u/Makefile | 2 -- drivers/net/wireless/realtek/rtlwifi/Makefile | 2 -- drivers/net/wireless/realtek/rtlwifi/btcoexist/Makefile | 2 -- drivers/net/wireless/realtek/rtlwifi/rtl8188ee/Makefile | 2 -- drivers/net/wireless/realtek/rtlwifi/rtl8192c/Makefile | 2 -- drivers/net/wireless/realtek/rtlwifi/rtl8192ce/Makefile | 2 -- drivers/net/wireless/realtek/rtlwifi/rtl8192cu/Makefile | 2 -- drivers/net/wireless/realtek/rtlwifi/rtl8192de/Makefile | 2 -- drivers/net/wireless/realtek/rtlwifi/rtl8192ee/Makefile | 2 -- drivers/net/wireless/realtek/rtlwifi/rtl8192se/Makefile | 2 -- drivers/net/wireless/realtek/rtlwifi/rtl8723ae/Makefile | 2 -- drivers/net/wireless/realtek/rtlwifi/rtl8723be/Makefile | 2 -- drivers/net/wireless/realtek/rtlwifi/rtl8723com/Makefile | 2 -- drivers/net/wireless/realtek/rtlwifi/rtl8821ae/Makefile | 2 -- drivers/net/wireless/ti/wl1251/Makefile | 2 -- drivers/net/wireless/ti/wlcore/Makefile | 2 -- drivers/staging/rtl8188eu/Makefile | 2 +- drivers/staging/rtl8192e/Makefile | 2 -- drivers/staging/rtl8192e/rtl8192e/Makefile | 2 -- net/bluetooth/Makefile | 2 -- net/ieee802154/Makefile | 2 -- net/mac80211/Makefile | 2 +- net/mac802154/Makefile | 2 -- net/wireless/Makefile | 2 -- 38 files changed, 5 insertions(+), 68 deletions(-) diff --git a/drivers/bluetooth/Makefile b/drivers/bluetooth/Makefile index b1fc29a..8062718 100644 --- a/drivers/bluetooth/Makefile +++ b/drivers/bluetooth/Makefile @@ -40,5 +40,3 @@ hci_uart-$(CONFIG_BT_HCIUART_QCA) += hci_qca.o hci_uart-$(CONFIG_BT_HCIUART_AG6XX) += hci_ag6xx.o hci_uart-$(CONFIG_BT_HCIUART_MRVL) += hci_mrvl.o hci_uart-objs := $(hci_uart-y) - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/can/Makefile b/drivers/net/can/Makefile index 26ba4b7..7a85495 100644 --- a/drivers/net/can/Makefile +++ b/drivers/net/can/Makefile @@ -31,5 +31,4 @@ obj-$(CONFIG_CAN_TI_HECC) += ti_hecc.o obj-$(CONFIG_CAN_XILINXCAN) += xilinx_can.o obj-$(CONFIG_PCH_CAN) += pch_can.o -subdir-ccflags-y += -D__CHECK_ENDIAN__ subdir-ccflags-$(CONFIG_CAN_DEBUG_DEVICES) += -DDEBUG diff --git a/drivers/net/ethernet/altera/Makefile b/drivers/net/ethernet/altera/Makefile index 3eff2fd..d4a187e 100644 --- a/drivers/net/ethernet/altera/Makefile +++ b/drivers/net/ethernet/altera/Makefile @@ -5,4 +5,3 @@ obj-$(CONFIG_ALTERA_TSE) += altera_tse.o altera_tse-objs := altera_tse_main.o altera_tse_ethtool.o \ altera_msgdma.o altera_sgdma.o altera_utils.o -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/ethernet/atheros/alx/Makefile b/drivers/net/ethernet/atheros/alx/Makefile index 5901fa4..ed4a605 100644 --- a/drivers/net/ethernet/atheros/alx/Makefile +++ b/drivers/net/ethernet/atheros/alx/Makefile @@ -1,3 +1,2 @@ obj-$(CONFIG_ALX) += alx.o alx-objs := main.o ethtool.o hw.o -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/ethernet/freescale/Makefile b/drivers/net/ethernet/freescale/Makefile index 4a13115..c46df5c 100644 --- a/drivers/net/ethernet/freescale/Makefile +++ b/drivers/net/ethernet/freescale/Makefile @@ -4,8 +4,6 @@ obj-$(CONFIG_FEC) += fec.o fec-objs :=fec_main.o fec_ptp.o -CFLAGS_fec_main.o := -D__CHECK_ENDIAN__ -CFLAGS_fec_ptp.o := -D__CHECK_ENDIAN__ obj-$(CONFIG_FEC_MPC52xx) += fec_mpc52xx.o ifeq ($(CONFIG_FEC_MPC52xx_MDIO),y) diff --git a/drivers/net/wireless/ath/Makefile b/drivers/net/wireless/ath/Makefile index 89f8d59..4cdebc7 100644 --- a/drivers/net/wireless/ath/Makefile +++ b/drivers/net/wireless/ath/Makefile @@ -19,6 +19,4 @@ ath-objs := main.o \ ath-$(CONFIG_ATH_DEBUG) += debug.o ath-$(CONFIG_ATH_TRACEPOINTS) += trace.o -ccflags-y += -D__CHECK_ENDIAN__ - CFLAGS_trace.o := -I$(src) diff --git a/drivers/net/wireless/ath/wil6210/Makefile b/drivers/net/wireless/ath/wil6210/Makefile index 11b544b..89bf2f9 100644 --- a/drivers/net/wireless/ath/wil6210/Makefile +++ b/drivers/net/wireless/ath/wil6210/Makefile @@ -22,5 +22,3 @@ wil6210-y += p2p.o # for tracing framework to find trace.h CFLAGS_trace.o := -I$(src) - -subdir-ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/Makefile b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/Makefile index d1568be..0383ba5 100644 --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/Makefile +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/Makefile @@ -19,8 +19,6 @@ ccflags-y += \ -Idrivers/net/wireless/broadcom/brcm80211/brcmfmac \ -Idrivers/net/wireless/broadcom/brcm80211/include -ccflags-y += -D__CHECK_ENDIAN__ - obj-$(CONFIG_BRCMFMAC) += brcmfmac.o brcmfmac-objs += \ cfg80211.o \ diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmsmac/Makefile b/drivers/net/wireless/broadcom/brcm80211/brcmsmac/Makefile index 960e6b8..ed83f33 100644 --- a/drivers/net/wireless/broadcom/brcm80211/brcmsmac/Makefile +++ b/drivers/net/wireless/broadcom/brcm80211/brcmsmac/Makefile @@ -16,7 +16,6 @@ # CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. ccflags-y := \ - -D__CHECK_ENDIAN__ \ -Idrivers/net/wireless/broadcom/brcm80211/brcmsmac \ -Idrivers/net/wireless/broadcom/brcm80211/brcmsmac/phy \ -Idrivers/net/wireless/broadcom/brcm80211/include diff --git a/drivers/net/wireless/intel/iwlegacy/Makefile b/drivers/net/wireless/intel/iwlegacy/Makefile index c985a01..c826a6b 100644 --- a/drivers/net/wireless/intel/iwlegacy/Makefile +++ b/drivers/net/wireless/intel/iwlegacy/Makefile @@ -13,5 +13,3 @@ iwl4965-$(CONFIG_IWLEGACY_DEBUGFS) += 4965-debug.o obj-$(CONFIG_IWL3945) += iwl3945.o iwl3945-objs := 3945-mac.o 3945.o 3945-rs.o iwl3945-$(CONFIG_IWLEGACY_DEBUGFS) += 3945-debug.o - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/wireless/intel/iwlwifi/Makefile b/drivers/net/wireless/intel/iwlwifi/Makefile index 6e7ed90..92e6118 100644 --- a/drivers/net/wireless/intel/iwlwifi/Makefile +++ b/drivers/net/wireless/intel/iwlwifi/Makefile @@ -15,7 +15,7 @@ iwlwifi-objs += $(iwlwifi-m) iwlwifi-$(CONFIG_IWLWIFI_DEVICE_TRACING) += iwl-devtrace.o -ccflags-y += -D__CHECK_ENDIAN__ -I$(src) +ccflags-y += -I$(src) obj-$(CONFIG_IWLDVM) += dvm/ obj-$(CONFIG_IWLMVM) += mvm/ diff --git a/drivers/net/wireless/intel/iwlwifi/dvm/Makefile b/drivers/net/wireless/intel/iwlwifi/dvm/Makefile index 4d19685..b256a354 100644 --- a/drivers/net/wireless/intel/iwlwifi/dvm/Makefile +++ b/drivers/net/wireless/intel/iwlwifi/dvm/Makefile @@ -10,4 +10,4 @@ iwldvm-objs += rxon.o devices.o iwldvm-$(CONFIG_IWLWIFI_LEDS) += led.o iwldvm-$(CONFIG_IWLWIFI_DEBUGFS) += debugfs.o -ccflags-y += -D__CHECK_ENDIAN__ -I$(src)/../ +ccflags-y += -I$(src)/../ diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/Makefile b/drivers/net/wireless/intel/iwlwifi/mvm/Makefile index 2e06dfc..83ac807 100644 --- a/drivers/net/wireless/intel/iwlwifi/mvm/Makefile +++ b/drivers/net/wireless/intel/iwlwifi/mvm/Makefile @@ -9,4 +9,4 @@ iwlmvm-$(CONFIG_IWLWIFI_LEDS) += led.o iwlmvm-y += tof.o fw-dbg.o iwlmvm-$(CONFIG_PM) += d3.o -ccflags-y += -D__CHECK_ENDIAN__ -I$(src)/../ +ccflags-y += -I$(src)/../ diff --git a/drivers/net/wireless/intersil/orinoco/Makefile b/drivers/net/wireless/intersil/orinoco/Makefile index bfdefb8..b7ecef8 100644 --- a/drivers/net/wireless/intersil/orinoco/Makefile +++ b/drivers/net/wireless/intersil/orinoco/Makefile @@ -12,6 +12,3 @@ obj-$(CONFIG_TMD_HERMES) += orinoco_tmd.o obj-$(CONFIG_NORTEL_HERMES) += orinoco_nortel.o obj-$(CONFIG_PCMCIA_SPECTRUM) += spectrum_cs.o obj-$(CONFIG_ORINOCO_USB) += orinoco_usb.o - -# Orinoco should be endian clean. -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/wireless/mediatek/mt7601u/Makefile b/drivers/net/wireless/mediatek/mt7601u/Makefile index ea9ed8a..08fc802 100644 --- a/drivers/net/wireless/mediatek/mt7601u/Makefile +++ b/drivers/net/wireless/mediatek/mt7601u/Makefile @@ -1,5 +1,3 @@ -ccflags-y += -D__CHECK_ENDIAN__ - obj-$(CONFIG_MT7601U) += mt7601u.o mt7601u-objs = \ diff --git a/drivers/net/wireless/realtek/rtlwifi/Makefile b/drivers/net/wireless/realtek/rtlwifi/Makefile index ad6d3c5..84c2e82 100644 --- a/drivers/net/wireless/realtek/rtlwifi/Makefile +++ b/drivers/net/wireless/realtek/rtlwifi/Makefile @@ -30,5 +30,3 @@ obj-$(CONFIG_RTLBTCOEXIST) += btcoexist/ obj-$(CONFIG_RTL8723_COMMON) += rtl8723com/ obj-$(CONFIG_RTL8821AE) += rtl8821ae/ obj-$(CONFIG_RTL8192EE) += rtl8192ee/ - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/wireless/realtek/rtlwifi/btcoexist/Makefile b/drivers/net/wireless/realtek/rtlwifi/btcoexist/Makefile index 47ceecf..d1454d4 100644 --- a/drivers/net/wireless/realtek/rtlwifi/btcoexist/Makefile +++ b/drivers/net/wireless/realtek/rtlwifi/btcoexist/Makefile @@ -3,5 +3,3 @@ btcoexist-objs := halbtc8723b2ant.o \ rtl_btc.o obj-$(CONFIG_RTLBTCOEXIST) += btcoexist.o - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/Makefile b/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/Makefile index 676e7de..dae4f0f 100644 --- a/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/Makefile +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/Makefile @@ -11,5 +11,3 @@ rtl8188ee-objs := \ trx.o obj-$(CONFIG_RTL8188EE) += rtl8188ee.o - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192c/Makefile b/drivers/net/wireless/realtek/rtlwifi/rtl8192c/Makefile index aee42d7..0546b75 100644 --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192c/Makefile +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192c/Makefile @@ -5,5 +5,3 @@ rtl8192c-common-objs := \ phy_common.o obj-$(CONFIG_RTL8192C_COMMON) += rtl8192c-common.o - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/Makefile b/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/Makefile index c0cb0cf..577c7ad 100644 --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/Makefile +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/Makefile @@ -9,5 +9,3 @@ rtl8192ce-objs := \ trx.o obj-$(CONFIG_RTL8192CE) += rtl8192ce.o - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/Makefile b/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/Makefile index ad2de6b..97437da 100644 --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/Makefile +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192cu/Makefile @@ -10,5 +10,3 @@ rtl8192cu-objs := \ trx.o obj-$(CONFIG_RTL8192CU) += rtl8192cu.o - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192de/Makefile b/drivers/net/wireless/realtek/rtlwifi/rtl8192de/Makefile index e3213c8..d0703f2 100644 --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192de/Makefile +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192de/Makefile @@ -10,5 +10,3 @@ rtl8192de-objs := \ trx.o obj-$(CONFIG_RTL8192DE) += rtl8192de.o - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192ee/Makefile b/drivers/net/wireless/realtek/rtlwifi/rtl8192ee/Makefile index 0315eed..f254b9f 100644 --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192ee/Makefile +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192ee/Makefile @@ -12,5 +12,3 @@ rtl8192ee-objs := \ obj-$(CONFIG_RTL8192EE) += rtl8192ee.o - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192se/Makefile b/drivers/net/wireless/realtek/rtlwifi/rtl8192se/Makefile index b7eb138..dfa9dbb 100644 --- a/drivers/net/wireless/realtek/rtlwifi/rtl8192se/Makefile +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192se/Makefile @@ -11,5 +11,3 @@ rtl8192se-objs := \ obj-$(CONFIG_RTL8192SE) += rtl8192se.o -ccflags-y += -D__CHECK_ENDIAN__ - diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/Makefile b/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/Makefile index 6220672..e7607d2 100644 --- a/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/Makefile +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/Makefile @@ -14,5 +14,3 @@ rtl8723ae-objs := \ obj-$(CONFIG_RTL8723AE) += rtl8723ae.o - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8723be/Makefile b/drivers/net/wireless/realtek/rtlwifi/rtl8723be/Makefile index a77c341..a841cbd 100644 --- a/drivers/net/wireless/realtek/rtlwifi/rtl8723be/Makefile +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8723be/Makefile @@ -12,5 +12,3 @@ rtl8723be-objs := \ obj-$(CONFIG_RTL8723BE) += rtl8723be.o - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8723com/Makefile b/drivers/net/wireless/realtek/rtlwifi/rtl8723com/Makefile index 345a68a..73da755 100644 --- a/drivers/net/wireless/realtek/rtlwifi/rtl8723com/Makefile +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8723com/Makefile @@ -5,5 +5,3 @@ rtl8723-common-objs := \ phy_common.o obj-$(CONFIG_RTL8723_COMMON) += rtl8723-common.o - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8821ae/Makefile b/drivers/net/wireless/realtek/rtlwifi/rtl8821ae/Makefile index f7a26f7..8ca406b 100644 --- a/drivers/net/wireless/realtek/rtlwifi/rtl8821ae/Makefile +++ b/drivers/net/wireless/realtek/rtlwifi/rtl8821ae/Makefile @@ -12,5 +12,3 @@ rtl8821ae-objs := \ obj-$(CONFIG_RTL8821AE) += rtl8821ae.o - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/wireless/ti/wl1251/Makefile b/drivers/net/wireless/ti/wl1251/Makefile index a5c6328..58b4f93 100644 --- a/drivers/net/wireless/ti/wl1251/Makefile +++ b/drivers/net/wireless/ti/wl1251/Makefile @@ -6,5 +6,3 @@ wl1251_sdio-objs += sdio.o obj-$(CONFIG_WL1251) += wl1251.o obj-$(CONFIG_WL1251_SPI) += wl1251_spi.o obj-$(CONFIG_WL1251_SDIO) += wl1251_sdio.o - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/net/wireless/ti/wlcore/Makefile b/drivers/net/wireless/ti/wlcore/Makefile index 0a69c13..e286713 100644 --- a/drivers/net/wireless/ti/wlcore/Makefile +++ b/drivers/net/wireless/ti/wlcore/Makefile @@ -8,5 +8,3 @@ wlcore-$(CONFIG_NL80211_TESTMODE) += testmode.o obj-$(CONFIG_WLCORE) += wlcore.o obj-$(CONFIG_WLCORE_SPI) += wlcore_spi.o obj-$(CONFIG_WLCORE_SDIO) += wlcore_sdio.o - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/staging/rtl8188eu/Makefile b/drivers/staging/rtl8188eu/Makefile index 29b9834..27af86e 100644 --- a/drivers/staging/rtl8188eu/Makefile +++ b/drivers/staging/rtl8188eu/Makefile @@ -53,4 +53,4 @@ r8188eu-y := \ obj-$(CONFIG_R8188EU) := r8188eu.o -ccflags-y += -D__CHECK_ENDIAN__ -I$(srctree)/$(src)/include +ccflags-y += -I$(srctree)/$(src)/include diff --git a/drivers/staging/rtl8192e/Makefile b/drivers/staging/rtl8192e/Makefile index cb18db7..7101fcc 100644 --- a/drivers/staging/rtl8192e/Makefile +++ b/drivers/staging/rtl8192e/Makefile @@ -17,5 +17,3 @@ obj-$(CONFIG_RTLLIB_CRYPTO_TKIP) += rtllib_crypt_tkip.o obj-$(CONFIG_RTLLIB_CRYPTO_WEP) += rtllib_crypt_wep.o obj-$(CONFIG_RTL8192E) += rtl8192e/ - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/drivers/staging/rtl8192e/rtl8192e/Makefile b/drivers/staging/rtl8192e/rtl8192e/Makefile index a2c4fb4..176a4a2 100644 --- a/drivers/staging/rtl8192e/rtl8192e/Makefile +++ b/drivers/staging/rtl8192e/rtl8192e/Makefile @@ -16,5 +16,3 @@ r8192e_pci-objs := \ rtl_wx.o \ obj-$(CONFIG_RTL8192E) += r8192e_pci.o - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/net/bluetooth/Makefile b/net/bluetooth/Makefile index b3ff12e..4bfaa19 100644 --- a/net/bluetooth/Makefile +++ b/net/bluetooth/Makefile @@ -20,5 +20,3 @@ bluetooth-$(CONFIG_BT_HS) += a2mp.o amp.o bluetooth-$(CONFIG_BT_LEDS) += leds.o bluetooth-$(CONFIG_BT_DEBUGFS) += hci_debugfs.o bluetooth-$(CONFIG_BT_SELFTEST) += selftest.o - -subdir-ccflags-y += -D__CHECK_ENDIAN__ diff --git a/net/ieee802154/Makefile b/net/ieee802154/Makefile index 4adfd4d..9b92ade 100644 --- a/net/ieee802154/Makefile +++ b/net/ieee802154/Makefile @@ -7,5 +7,3 @@ ieee802154-y := netlink.o nl-mac.o nl-phy.o nl_policy.o core.o \ ieee802154_socket-y := socket.o CFLAGS_trace.o := -I$(src) - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/net/mac80211/Makefile b/net/mac80211/Makefile index 0b202b3..2829122 100644 --- a/net/mac80211/Makefile +++ b/net/mac80211/Makefile @@ -61,4 +61,4 @@ rc80211_minstrel_ht-$(CONFIG_MAC80211_DEBUGFS) += rc80211_minstrel_ht_debugfs.o mac80211-$(CONFIG_MAC80211_RC_MINSTREL) += $(rc80211_minstrel-y) mac80211-$(CONFIG_MAC80211_RC_MINSTREL_HT) += $(rc80211_minstrel_ht-y) -ccflags-y += -D__CHECK_ENDIAN__ -DDEBUG +ccflags-y += -DDEBUG diff --git a/net/mac802154/Makefile b/net/mac802154/Makefile index 17a51e8..5857bb1 100644 --- a/net/mac802154/Makefile +++ b/net/mac802154/Makefile @@ -3,5 +3,3 @@ mac802154-objs := main.o rx.o tx.o mac_cmd.o mib.o \ iface.o llsec.o util.o cfg.o trace.o CFLAGS_trace.o := -I$(src) - -ccflags-y += -D__CHECK_ENDIAN__ diff --git a/net/wireless/Makefile b/net/wireless/Makefile index 4c9e39f..816c933 100644 --- a/net/wireless/Makefile +++ b/net/wireless/Makefile @@ -17,8 +17,6 @@ cfg80211-$(CONFIG_CFG80211_INTERNAL_REGDB) += regdb.o CFLAGS_trace.o := -I$(src) -ccflags-y += -D__CHECK_ENDIAN__ - $(obj)/regdb.c: $(src)/db.txt $(src)/genregdb.awk @$(AWK) -f $(srctree)/$(src)/genregdb.awk < $< > $@ -- MST ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags 2016-12-15 5:15 ` [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags Michael S. Tsirkin @ 2016-12-15 7:00 ` Marcel Holtmann 2016-12-15 7:57 ` Marc Kleine-Budde ` (3 subsequent siblings) 4 siblings, 0 replies; 12+ messages in thread From: Marcel Holtmann @ 2016-12-15 7:00 UTC (permalink / raw) To: Michael S. Tsirkin Cc: LKML, Gustavo F. Padovan, Johan Hedberg, Wolfgang Grandegger, Marc Kleine-Budde, Vince Bridgers, Jay Cliburn, Chris Snook, Luis R. Rodriguez, Kalle Valo, Maya Erez, Arend van Spriel, Franky Lin, Hante Meuleman, Stanislaw Gruszka, Johannes Berg, Emmanuel Grumbach, Luca Coelho, Intel Linux Wireless, Jakub Kicinski, Larry Finger, Chaoming Li, Greg Kroah-Hartman, David S. Miller, Alexander Aring, Stefan Schmidt, Matthias Brugger, linux-bluetooth, linux-can, netdev, nios2-dev, linux-wireless, wil6210, brcm80211-dev-list.pdl, devel, linux-wpan, linux-arm-kernel, linux-mediatek Hi Michael, > That's the default now, no need for makefiles to set it. > > Signed-off-by: Michael S. Tsirkin <mst@redhat.com> > --- > drivers/bluetooth/Makefile | 2 -- > drivers/net/can/Makefile | 1 - > drivers/net/ethernet/altera/Makefile | 1 - > drivers/net/ethernet/atheros/alx/Makefile | 1 - > drivers/net/ethernet/freescale/Makefile | 2 -- > drivers/net/wireless/ath/Makefile | 2 -- > drivers/net/wireless/ath/wil6210/Makefile | 2 -- > drivers/net/wireless/broadcom/brcm80211/brcmfmac/Makefile | 2 -- > drivers/net/wireless/broadcom/brcm80211/brcmsmac/Makefile | 1 - > drivers/net/wireless/intel/iwlegacy/Makefile | 2 -- > drivers/net/wireless/intel/iwlwifi/Makefile | 2 +- > drivers/net/wireless/intel/iwlwifi/dvm/Makefile | 2 +- > drivers/net/wireless/intel/iwlwifi/mvm/Makefile | 2 +- > drivers/net/wireless/intersil/orinoco/Makefile | 3 --- > drivers/net/wireless/mediatek/mt7601u/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/btcoexist/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8188ee/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8192c/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8192ce/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8192cu/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8192de/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8192ee/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8192se/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8723ae/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8723be/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8723com/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8821ae/Makefile | 2 -- > drivers/net/wireless/ti/wl1251/Makefile | 2 -- > drivers/net/wireless/ti/wlcore/Makefile | 2 -- > drivers/staging/rtl8188eu/Makefile | 2 +- > drivers/staging/rtl8192e/Makefile | 2 -- > drivers/staging/rtl8192e/rtl8192e/Makefile | 2 -- > net/bluetooth/Makefile | 2 -- > net/ieee802154/Makefile | 2 -- > net/mac80211/Makefile | 2 +- > net/mac802154/Makefile | 2 -- > net/wireless/Makefile | 2 -- > 38 files changed, 5 insertions(+), 68 deletions(-) for drivers/bluetooth, net/bluetooth, net/ieee802154 and net/mac802154 Acked-by: Marcel Holtmann <marcel@holtmann.org> Regards Marcel ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags 2016-12-15 5:15 ` [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags Michael S. Tsirkin 2016-12-15 7:00 ` Marcel Holtmann @ 2016-12-15 7:57 ` Marc Kleine-Budde 2016-12-15 11:50 ` Greg Kroah-Hartman ` (2 subsequent siblings) 4 siblings, 0 replies; 12+ messages in thread From: Marc Kleine-Budde @ 2016-12-15 7:57 UTC (permalink / raw) To: Michael S. Tsirkin, linux-kernel Cc: David S. Miller, Alexander Aring, Stefan Schmidt, Matthias Brugger, linux-bluetooth, linux-can, netdev, nios2-dev, linux-wireless, wil6210, brcm80211-dev-list.pdl, devel, linux-wpan, linux-arm-kernel, linux-mediatek [-- Attachment #1.1: Type: text/plain, Size: 631 bytes --] On 12/15/2016 06:15 AM, Michael S. Tsirkin wrote: > That's the default now, no need for makefiles to set it. > > Signed-off-by: Michael S. Tsirkin <mst@redhat.com> > --- [...] > drivers/net/can/Makefile | 1 - For drivers/net/can/Makefile: Acked-by: Marc Kleine-Budde <mkl@pengutronix.de> regards, Marc -- Pengutronix e.K. | Marc Kleine-Budde | Industrial Linux Solutions | Phone: +49-231-2826-924 | Vertretung West/Dortmund | Fax: +49-5121-206917-5555 | Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de | [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags 2016-12-15 5:15 ` [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags Michael S. Tsirkin 2016-12-15 7:00 ` Marcel Holtmann 2016-12-15 7:57 ` Marc Kleine-Budde @ 2016-12-15 11:50 ` Greg Kroah-Hartman 2016-12-15 20:15 ` Arend Van Spriel 2016-12-19 9:08 ` Luca Coelho 4 siblings, 0 replies; 12+ messages in thread From: Greg Kroah-Hartman @ 2016-12-15 11:50 UTC (permalink / raw) To: Michael S. Tsirkin Cc: linux-kernel, Emmanuel Grumbach, Stanislaw Gruszka, Gustavo Padovan, Arend van Spriel, Luca Coelho, devel, Jakub Kicinski, Stefan Schmidt, linux-mediatek, wil6210, linux-arm-kernel, Chris Snook, Wolfgang Grandegger, Jay Cliburn, linux-wpan, Johan Hedberg, Johannes Berg, Intel Linux Wireless, Alexander Aring, Marcel Holtmann, Hante Meuleman, linux-can, Marc Kleine-Budde, Matthias Brugger, Kalle Valo, Franky Lin, brcm80211-dev-list.pdl, Luis R. Rodriguez, linux-wireless, Maya Erez, linux-bluetooth, Chaoming Li, netdev, nios2-dev, Vince Bridgers, David S. Miller, Larry Finger On Thu, Dec 15, 2016 at 07:15:30AM +0200, Michael S. Tsirkin wrote: > That's the default now, no need for makefiles to set it. > > Signed-off-by: Michael S. Tsirkin <mst@redhat.com> > --- > drivers/bluetooth/Makefile | 2 -- > drivers/net/can/Makefile | 1 - > drivers/net/ethernet/altera/Makefile | 1 - > drivers/net/ethernet/atheros/alx/Makefile | 1 - > drivers/net/ethernet/freescale/Makefile | 2 -- > drivers/net/wireless/ath/Makefile | 2 -- > drivers/net/wireless/ath/wil6210/Makefile | 2 -- > drivers/net/wireless/broadcom/brcm80211/brcmfmac/Makefile | 2 -- > drivers/net/wireless/broadcom/brcm80211/brcmsmac/Makefile | 1 - > drivers/net/wireless/intel/iwlegacy/Makefile | 2 -- > drivers/net/wireless/intel/iwlwifi/Makefile | 2 +- > drivers/net/wireless/intel/iwlwifi/dvm/Makefile | 2 +- > drivers/net/wireless/intel/iwlwifi/mvm/Makefile | 2 +- > drivers/net/wireless/intersil/orinoco/Makefile | 3 --- > drivers/net/wireless/mediatek/mt7601u/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/btcoexist/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8188ee/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8192c/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8192ce/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8192cu/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8192de/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8192ee/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8192se/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8723ae/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8723be/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8723com/Makefile | 2 -- > drivers/net/wireless/realtek/rtlwifi/rtl8821ae/Makefile | 2 -- > drivers/net/wireless/ti/wl1251/Makefile | 2 -- > drivers/net/wireless/ti/wlcore/Makefile | 2 -- > drivers/staging/rtl8188eu/Makefile | 2 +- > drivers/staging/rtl8192e/Makefile | 2 -- > drivers/staging/rtl8192e/rtl8192e/Makefile | 2 -- > net/bluetooth/Makefile | 2 -- > net/ieee802154/Makefile | 2 -- > net/mac80211/Makefile | 2 +- > net/mac802154/Makefile | 2 -- > net/wireless/Makefile | 2 -- > 38 files changed, 5 insertions(+), 68 deletions(-) For drivers/staging: Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags 2016-12-15 5:15 ` [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags Michael S. Tsirkin ` (2 preceding siblings ...) 2016-12-15 11:50 ` Greg Kroah-Hartman @ 2016-12-15 20:15 ` Arend Van Spriel 2016-12-19 9:08 ` Luca Coelho 4 siblings, 0 replies; 12+ messages in thread From: Arend Van Spriel @ 2016-12-15 20:15 UTC (permalink / raw) To: Michael S. Tsirkin, linux-kernel Cc: Marcel Holtmann, Gustavo Padovan, Johan Hedberg, Wolfgang Grandegger, Marc Kleine-Budde, Vince Bridgers, Jay Cliburn, Chris Snook, Luis R. Rodriguez, Kalle Valo, Maya Erez, Franky Lin, Hante Meuleman, Stanislaw Gruszka, Johannes Berg, Emmanuel Grumbach, Luca Coelho, Intel Linux Wireless, Jakub Kicinski, Larry Finger, Chaoming Li, Greg Kroah-Hartman, David S. Miller, Alexander Aring, Stefan Schmidt, Matthias Brugger, linux-bluetooth, linux-can, netdev, nios2-dev, linux-wireless, wil6210, brcm80211-dev-list.pdl, devel, linux-wpan, linux-arm-kernel, linux-mediatek On 15-12-2016 6:15, Michael S. Tsirkin wrote: > That's the default now, no need for makefiles to set it. > > Signed-off-by: Michael S. Tsirkin <mst@redhat.com> > --- > drivers/bluetooth/Makefile | 2 -- > drivers/net/can/Makefile | 1 - > drivers/net/ethernet/altera/Makefile | 1 - > drivers/net/ethernet/atheros/alx/Makefile | 1 - > drivers/net/ethernet/freescale/Makefile | 2 -- > drivers/net/wireless/ath/Makefile | 2 -- > drivers/net/wireless/ath/wil6210/Makefile | 2 -- > drivers/net/wireless/broadcom/brcm80211/brcmfmac/Makefile | 2 -- > drivers/net/wireless/broadcom/brcm80211/brcmsmac/Makefile | 1 - For brcm80211 drivers: Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com> Regards, Arend ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags 2016-12-15 5:15 ` [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags Michael S. Tsirkin ` (3 preceding siblings ...) 2016-12-15 20:15 ` Arend Van Spriel @ 2016-12-19 9:08 ` Luca Coelho 4 siblings, 0 replies; 12+ messages in thread From: Luca Coelho @ 2016-12-19 9:08 UTC (permalink / raw) To: Michael S. Tsirkin, linux-kernel Cc: Marcel Holtmann, Gustavo Padovan, Johan Hedberg, Wolfgang Grandegger, Marc Kleine-Budde, Vince Bridgers, Jay Cliburn, Chris Snook, Luis R. Rodriguez, Kalle Valo, Maya Erez, Arend van Spriel, Franky Lin, Hante Meuleman, Stanislaw Gruszka, Johannes Berg, Emmanuel Grumbach, Intel Linux Wireless, Jakub Kicinski, Larry Finger, Chaoming Li, Greg Kroah-Hartman, David S. Miller, Alexander Aring, Stefan Schmidt, Matthias Brugger, linux-bluetooth, linux-can, netdev, nios2-dev, linux-wireless, wil6210, brcm80211-dev-list.pdl, devel, linux-wpan, linux-arm-kernel, linux-mediatek On Thu, 2016-12-15 at 07:15 +0200, Michael S. Tsirkin wrote: > That's the default now, no need for makefiles to set it. > > Signed-off-by: Michael S. Tsirkin <mst@redhat.com> > --- > drivers/bluetooth/Makefile | 2 -- > drivers/net/can/Makefile | 1 - > drivers/net/ethernet/altera/Makefile | 1 - > drivers/net/ethernet/atheros/alx/Makefile | 1 - > drivers/net/ethernet/freescale/Makefile | 2 -- > drivers/net/wireless/ath/Makefile | 2 -- > drivers/net/wireless/ath/wil6210/Makefile | 2 -- > drivers/net/wireless/broadcom/brcm80211/brcmfmac/Makefile | 2 -- > drivers/net/wireless/broadcom/brcm80211/brcmsmac/Makefile | 1 - > drivers/net/wireless/intel/iwlegacy/Makefile | 2 -- > drivers/net/wireless/intel/iwlwifi/Makefile | 2 +- > drivers/net/wireless/intel/iwlwifi/dvm/Makefile | 2 +- > drivers/net/wireless/intel/iwlwifi/mvm/Makefile | 2 +- For the drivers/net/wireless/intel/iwlwifi/ part: Acked-by: Luca Coelho <luciano.coelho@intel.com> -- Luca. ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2016-12-19 9:10 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1481778865-27667-1-git-send-email-mst@redhat.com>
2016-12-15 5:15 ` [PATCH 5/8] linux: drop __bitwise__ everywhere Michael S. Tsirkin
2016-12-15 9:04 ` Stefan Schmidt
2016-12-15 11:51 ` Greg Kroah-Hartman
2016-12-15 17:28 ` Krzysztof Kozlowski
2016-12-15 19:44 ` Lee Duncan
2016-12-19 9:09 ` Luca Coelho
2016-12-15 5:15 ` [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags Michael S. Tsirkin
2016-12-15 7:00 ` Marcel Holtmann
2016-12-15 7:57 ` Marc Kleine-Budde
2016-12-15 11:50 ` Greg Kroah-Hartman
2016-12-15 20:15 ` Arend Van Spriel
2016-12-19 9:08 ` Luca Coelho
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).