* [PATCH 0/8] enable endian checks for all sparse builds @ 2016-12-15 5:15 Michael S. Tsirkin 2016-12-15 5:15 ` [PATCH 5/8] linux: drop __bitwise__ everywhere Michael S. Tsirkin 2016-12-15 5:15 ` [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags Michael S. Tsirkin 0 siblings, 2 replies; 13+ messages in thread From: Michael S. Tsirkin @ 2016-12-15 5:15 UTC (permalink / raw) To: linux-arm-kernel This is just a reposting of the patch that enables endian checks, with addition of trivial patches that drop __bitwise__ and __CHECK_ENDIAN__ everywhere. I plan to include this in my pull request unless I hear otherwise. Michael S. Tsirkin (8): linux/types.h: enable endian checks for all sparse builds tools: enable endian checks for all sparse builds Documentation/sparse: drop __bitwise__ checkpatch: replace __bitwise__ with __bitwise linux: drop __bitwise__ everywhere Documentation/sparse: drop __CHECK_ENDIAN__ fs/logfs: drop __CHECK_ENDIAN__ Makefile: drop -D__CHECK_ENDIAN__ from cflags Documentation/translations/zh_CN/sparse.txt | 7 +------ 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 ++-- fs/logfs/logfs.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/types.h | 4 ---- include/uapi/linux/virtio_types.h | 6 +++--- net/ieee802154/6lowpan/6lowpan_i.h | 2 +- net/mac80211/ieee80211_i.h | 4 ++-- tools/include/linux/types.h | 4 ---- Documentation/dev-tools/sparse.rst | 14 +------------- 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 -- scripts/checkpatch.pl | 4 ++-- 56 files changed, 30 insertions(+), 120 deletions(-) -- MST ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 5/8] linux: drop __bitwise__ everywhere 2016-12-15 5:15 [PATCH 0/8] enable endian checks for all sparse builds Michael S. Tsirkin @ 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; 13+ messages in thread From: Michael S. Tsirkin @ 2016-12-15 5:15 UTC (permalink / raw) To: linux-arm-kernel __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] 13+ messages in thread
* [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; 13+ messages in thread From: Stefan Schmidt @ 2016-12-15 9:04 UTC (permalink / raw) To: linux-arm-kernel 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] 13+ messages in thread
* [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; 13+ messages in thread From: Greg Kroah-Hartman @ 2016-12-15 11:51 UTC (permalink / raw) To: linux-arm-kernel 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] 13+ messages in thread
* [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; 13+ messages in thread From: Krzysztof Kozlowski @ 2016-12-15 17:28 UTC (permalink / raw) To: linux-arm-kernel 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] 13+ messages in thread
* [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; 13+ messages in thread From: Lee Duncan @ 2016-12-15 19:44 UTC (permalink / raw) To: linux-arm-kernel 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] 13+ messages in thread
* [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; 13+ messages in thread From: Luca Coelho @ 2016-12-19 9:09 UTC (permalink / raw) To: linux-arm-kernel 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] 13+ messages in thread
* [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags 2016-12-15 5:15 [PATCH 0/8] enable endian checks for all sparse builds Michael S. Tsirkin 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; 13+ messages in thread From: Michael S. Tsirkin @ 2016-12-15 5:15 UTC (permalink / raw) To: linux-arm-kernel 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] 13+ messages in thread
* [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; 13+ messages in thread From: Marcel Holtmann @ 2016-12-15 7:00 UTC (permalink / raw) To: linux-arm-kernel 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] 13+ messages in thread
* [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; 13+ messages in thread From: Marc Kleine-Budde @ 2016-12-15 7:57 UTC (permalink / raw) To: linux-arm-kernel 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 | -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 488 bytes Desc: OpenPGP digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20161215/0b31e486/attachment.sig> ^ permalink raw reply [flat|nested] 13+ messages in thread
* [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; 13+ messages in thread From: Greg Kroah-Hartman @ 2016-12-15 11:50 UTC (permalink / raw) To: linux-arm-kernel 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] 13+ messages in thread
* [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; 13+ messages in thread From: Arend Van Spriel @ 2016-12-15 20:15 UTC (permalink / raw) To: linux-arm-kernel 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] 13+ messages in thread
* [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; 13+ messages in thread From: Luca Coelho @ 2016-12-19 9:08 UTC (permalink / raw) To: linux-arm-kernel 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] 13+ messages in thread
end of thread, other threads:[~2016-12-19 9:09 UTC | newest] Thread overview: 13+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-12-15 5:15 [PATCH 0/8] enable endian checks for all sparse builds Michael S. Tsirkin 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).