Message ID | 20201117193124.9789-4-ceggers@arri.de |
---|---|
State | Superseded |
Headers | show |
Series | [net-next,1/4] net: ptp: introduce enum ptp_msg_type | expand |
Hi Christian, I love your patch! Yet something to improve: [auto build test ERROR on net-next/master] url: https://github.com/0day-ci/linux/commits/Christian-Eggers/net-ptp-introduce-enum-ptp_msg_type/20201118-033828 base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git 72308ecbf33b145641aba61071be31a85ebfd92c config: sh-allmodconfig (attached as .config) compiler: sh4-linux-gcc (GCC) 9.3.0 reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://github.com/0day-ci/linux/commit/c4e4cfcabe3201e2ece90cc9025894e4ed08f069 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Christian-Eggers/net-ptp-introduce-enum-ptp_msg_type/20201118-033828 git checkout c4e4cfcabe3201e2ece90cc9025894e4ed08f069 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=sh If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com> All error/warnings (new ones prefixed by >>): >> drivers/ptp/ptp_ines.c:690:26: error: conflicting types for 'tag_to_msgtype' 690 | static enum ptp_msg_type tag_to_msgtype(u8 tag) | ^~~~~~~~~~~~~~ drivers/ptp/ptp_ines.c:178:11: note: previous declaration of 'tag_to_msgtype' was here 178 | static u8 tag_to_msgtype(u8 tag); | ^~~~~~~~~~~~~~ >> drivers/ptp/ptp_ines.c:178:11: warning: 'tag_to_msgtype' used but never defined drivers/ptp/ptp_ines.c:690:26: warning: 'tag_to_msgtype' defined but not used [-Wunused-function] 690 | static enum ptp_msg_type tag_to_msgtype(u8 tag) | ^~~~~~~~~~~~~~ vim +/tag_to_msgtype +690 drivers/ptp/ptp_ines.c 689 > 690 static enum ptp_msg_type tag_to_msgtype(u8 tag) 691 { 692 switch (tag) { 693 case MESSAGE_TYPE_SYNC: 694 return PTP_MSGTYPE_SYNC; 695 case MESSAGE_TYPE_P_DELAY_REQ: 696 return PTP_MSGTYPE_PDELAY_REQ; 697 case MESSAGE_TYPE_P_DELAY_RESP: 698 return PTP_MSGTYPE_PDELAY_RESP; 699 case MESSAGE_TYPE_DELAY_REQ: 700 return PTP_MSGTYPE_DELAY_REQ; 701 } 702 return 0xf; 703 } 704 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
On Wed, Nov 18, 2020 at 07:17:41AM +0800, kernel test robot wrote: > >> drivers/ptp/ptp_ines.c:690:26: error: conflicting types for 'tag_to_msgtype' > 690 | static enum ptp_msg_type tag_to_msgtype(u8 tag) > | ^~~~~~~~~~~~~~ > drivers/ptp/ptp_ines.c:178:11: note: previous declaration of 'tag_to_msgtype' was here > 178 | static u8 tag_to_msgtype(u8 tag); > | ^~~~~~~~~~~~~~ Wait for the patches to simmer a little bit more before resending. And please make sure to create a cover letter when doing so.
diff --git a/drivers/ptp/ptp_ines.c b/drivers/ptp/ptp_ines.c index 4700ffbdfced..4836c9c35604 100644 --- a/drivers/ptp/ptp_ines.c +++ b/drivers/ptp/ptp_ines.c @@ -108,11 +108,6 @@ MODULE_LICENSE("GPL"); #define MESSAGE_TYPE_P_DELAY_RESP 3 #define MESSAGE_TYPE_DELAY_REQ 4 -#define SYNC 0x0 -#define DELAY_REQ 0x1 -#define PDELAY_REQ 0x2 -#define PDELAY_RESP 0x3 - static LIST_HEAD(ines_clocks); static DEFINE_MUTEX(ines_clocks_lock); @@ -442,7 +437,7 @@ static bool ines_match(struct sk_buff *skb, unsigned int ptp_class, { struct ptp_header *hdr; u16 portn, seqid; - u8 msgtype; + enum ptp_msg_type msgtype; u64 clkid; if (unlikely(ptp_class & PTP_CLASS_V1)) @@ -675,7 +670,7 @@ static void ines_txtstamp_work(struct work_struct *work) static bool is_sync_pdelay_resp(struct sk_buff *skb, int type) { struct ptp_header *hdr; - u8 msgtype; + enum ptp_msg_type msgtype; hdr = ptp_parse_header(skb, type); if (!hdr) @@ -683,26 +678,26 @@ static bool is_sync_pdelay_resp(struct sk_buff *skb, int type) msgtype = ptp_get_msgtype(hdr, type); - switch ((msgtype & 0xf)) { - case SYNC: - case PDELAY_RESP: + switch (msgtype) { + case PTP_MSGTYPE_SYNC: + case PTP_MSGTYPE_PDELAY_RESP: return true; default: return false; } } -static u8 tag_to_msgtype(u8 tag) +static enum ptp_msg_type tag_to_msgtype(u8 tag) { switch (tag) { case MESSAGE_TYPE_SYNC: - return SYNC; + return PTP_MSGTYPE_SYNC; case MESSAGE_TYPE_P_DELAY_REQ: - return PDELAY_REQ; + return PTP_MSGTYPE_PDELAY_REQ; case MESSAGE_TYPE_P_DELAY_RESP: - return PDELAY_RESP; + return PTP_MSGTYPE_PDELAY_RESP; case MESSAGE_TYPE_DELAY_REQ: - return DELAY_REQ; + return PTP_MSGTYPE_DELAY_REQ; } return 0xf; }
Use new return type of ptp_get_msgtype(). Remove driver internal defines for this. Signed-off-by: Christian Eggers <ceggers@arri.de> Cc: Richard Cochran <richardcochran@gmail.com> Cc: Kurt Kanzenbach <kurt@linutronix.de> --- drivers/ptp/ptp_ines.c | 25 ++++++++++--------------- 1 file changed, 10 insertions(+), 15 deletions(-)