From owner-svn-src-head@freebsd.org Fri May 18 17:31:10 2018 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 18E8BEAF589; Fri, 18 May 2018 17:31:10 +0000 (UTC) (envelope-from sbruno@freebsd.org) Received: from mail.ignoranthack.me (ignoranthack.me [199.102.79.106]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id A7567808D9; Fri, 18 May 2018 17:31:09 +0000 (UTC) (envelope-from sbruno@freebsd.org) Received: from [192.168.0.6] (67-0-228-75.albq.qwest.net [67.0.228.75]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: sbruno@ignoranthack.me) by mail.ignoranthack.me (Postfix) with ESMTPSA id A406A192882; Fri, 18 May 2018 09:35:29 +0000 (UTC) Subject: Re: svn commit: r333801 - head/sys/contrib/dev/ath/ath_hal/ar9300 To: Ian Lepore , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org References: <201805181723.w4IHNN6Y091949@repo.freebsd.org> <1526664543.32688.57.camel@freebsd.org> From: Sean Bruno Openpgp: preference=signencrypt Autocrypt: addr=sbruno@freebsd.org; prefer-encrypt=mutual; keydata= xsBNBFk+0UEBCADaf4bgxxKvMOhRV5NPoGWRCCGm49d6+1VFNlQ77WsY/+Zvf95TPULdRlnG w648KfxWt7+O3kdKhdRwnqlXWC7zA2Qt0dRE1yIqOGJ4jp4INvp/bcxWzgr0aoKOjrlnfxRV bh+s0rzdZt6TsNL3cVYxkC8oezjaUkHdW4mFJU249U1QJogkF8g0FeKNfEcjEkwJNX6lQJH+ EzCWT0NCk6J+Xyo+zOOljxPp1OUfdvZi3ulkU/qTZstGVWxFVsP8xQklV/y3AFcbIYx6iGJ4 5L7WuB0IWhO7Z4yHENr8wFaNYwpod9i4egX2BugbrM8pOfhN2/qqdeG1L5LMtXw3yyAhABEB AAHNN1NlYW4gQnJ1bm8gKEZyZWVCU0QgRGV2ZWxvcGVyIEtleSkgPHNicnVub0BmcmVlYnNk Lm9yZz7CwJQEEwEKAD4WIQToxOn4gDUE4eP0ujS95PX+ibX8tgUCWT7RQQIbAwUJBaOagAUL CQgHAwUVCgkICwUWAwIBAAIeAQIXgAAKCRC95PX+ibX8ttKTCACFKzRc56EBAlVotq02EjZP SfX+unlk6AuPBzShxqRxeK+bGYVCigrYd1M8nnskv0dEiZ5iYeND9HIxbpEyopqgpVTibA7w gBXaZ7SOEhNX1wXwg14JrralfSmPFMYni+sWegPMX/zwfAsn1z4mG1Nn44Xqo3o7CfpkMPy6 M5Bow2IDzIhEYISLR+urxs74/aHU35PLtBSDtu18914SEMDdva27MARN8mbeCDbuJVfGCPWy YHuy2t+9u2Zn5Dd+t3sBXLM9gpeaMm+4x6TNPpESygbVdh4tDdjVZ9DK/bWFg0kMgfZoaq6J l0jNsQXrZV3bzYNFbVw04pFcvA2GIJ7xzsBNBFk+0UEBCADIXBmQOaKMHGbc9vwjhV4Oj5aZ DdhNedn12FVeTdOXJvuTOusgxS29lla0RenHGDsgD08UiFpasBXWq/E+BhQ19d+iRbLLR17O KKc1ZGefoVbLARLXD68J5j4XAyK+6k2KqBLlqzAEpHTzsksM9naARkVXiEVcrt6ciw0FSm8n kuK3gDKKe93XfzfP+TQdbvvzJc7Fa+appLbXz61TM1aikaQlda8bWubDegwXbuoJdB34xU1m yjr/N4o+raL0x7QrzdH+wwgrTTo+H4S2c1972Skt5K5tbxLowfHicRl23V8itVQr3sBtlX4+ 66q+Apm7+R36bUS/k+G45Sp6iPpxABEBAAHCwHwEGAEKACYWIQToxOn4gDUE4eP0ujS95PX+ ibX8tgUCWT7RQQIbDAUJBaOagAAKCRC95PX+ibX8trrIB/9Pljqt/JGamD9tx4dOVmxSyFg9 z2xzgklTLuDgS73MM120mM7ao9AQUeWiSle/H0UCK7xPOzC/aeUC4oygDQKAfkkNbCNTo3+A qDjBRA8qx0e9a/QjDL+RFgD4L5kLT4tToY8T8HaBp8h03LBfk510IaI8oL/Jg7vpM3PDtJMW tUi2H+yNFmL3NfM2oBToWKLFsoP54f/eeeImrNnrlLjLHPzqS+/9apgYqX2Jwiv3tHBc4FTO GuY8VvF7BpixJs8Pc2RUuCfSyodrp1YG1kRGlXAH0cqwwr0Zmk4+7dZvtVQMCl6kS6q1+84q JwtItxS2eXSEA4NO0sQ3BXUywANh Message-ID: <7e76af2a-7229-ae4b-e060-7b9a355764be@freebsd.org> Date: Fri, 18 May 2018 11:31:03 -0600 User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: <1526664543.32688.57.camel@freebsd.org> Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="gqoBk2eMRFXUDwtKBH73RCL503CaqjrE7" X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 18 May 2018 17:31:10 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --gqoBk2eMRFXUDwtKBH73RCL503CaqjrE7 Content-Type: multipart/mixed; boundary="i4GJo3K1oHs27WcmgOIJ0IkPqpj8uFWk7"; protected-headers="v1" From: Sean Bruno To: Ian Lepore , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Message-ID: <7e76af2a-7229-ae4b-e060-7b9a355764be@freebsd.org> Subject: Re: svn commit: r333801 - head/sys/contrib/dev/ath/ath_hal/ar9300 References: <201805181723.w4IHNN6Y091949@repo.freebsd.org> <1526664543.32688.57.camel@freebsd.org> In-Reply-To: <1526664543.32688.57.camel@freebsd.org> --i4GJo3K1oHs27WcmgOIJ0IkPqpj8uFWk7 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: quoted-printable On 05/18/18 11:29, Ian Lepore wrote: > On Fri, 2018-05-18 at 17:23 +0000, Sean Bruno wrote: >> Author: sbruno >> Date: Fri May 18 17:23:23 2018 >> New Revision: 333801 >> URL: https://svnweb.freebsd.org/changeset/base/333801 >> >> Log: >> =C2=A0 Quiesce a couple pages of clang warnings with a cast.=C2=A0=C2=A0= Duplicates >> =C2=A0 linux maintainer commit: >> =C2=A0=C2=A0 >> =C2=A0 https://github.com/torvalds/linux/commit/627871b71c89a6ec12fbed= 75063f238e0c7127b2#diff-8c6ddb4c3ad69a6fb9f289475821db56 >> =C2=A0=C2=A0 >> =C2=A0 ar9300template_aphrodite.h:575:40: warning: implicit conversion= from 'int' >> =C2=A0=C2=A0=C2=A0=C2=A0to 'u_int8_t' (aka 'unsigned char') changes va= lue from 3495 to 167 >> =C2=A0=C2=A0=C2=A0=C2=A0[-Wconstant-conversion] >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0/* Data[8].ctl_edges[7].bChannel*/FREQ2FBIN(5795, 0)} >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0^~~~~~~= ~~~~~~~~~~~ >> =C2=A0 ar9300eep.h:142:41: note: expanded from macro 'FREQ2FBIN' >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0(((y) =3D=3D HAL_FREQ_BAND_2GHZ) ?= ((x) - 2300) : (((x) - 4800) / 5)) >> =C2=A0=C2=A0 >> =C2=A0 Reviewed by: imp >> =C2=A0 MFC after: 1 week >> =C2=A0 Differential Revision: https://reviews.freebsd.org/D15476 >> >> Modified: >> =C2=A0 head/sys/contrib/dev/ath/ath_hal/ar9300/ar9300eep.h >> >> Modified: head/sys/contrib/dev/ath/ath_hal/ar9300/ar9300eep.h >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D >> --- head/sys/contrib/dev/ath/ath_hal/ar9300/ar9300eep.h Fri May 18 17:= 07:59 2018 (r333800) >> +++ head/sys/contrib/dev/ath/ath_hal/ar9300/ar9300eep.h Fri May 18 17:= 23:23 2018 (r333801) >> @@ -139,7 +139,7 @@ enum Ar9300EepromTemplate >> =C2=A0#define OSPREY_CUSTOMER_DATA_SIZE=C2=A0=C2=A0=C2=A0=C2=A020 >> =C2=A0 >> =C2=A0#define FREQ2FBIN(x,y) \ >> -=C2=A0=C2=A0=C2=A0=C2=A0(((y) =3D=3D HAL_FREQ_BAND_2GHZ) ? ((x) - 230= 0) : (((x) - 4800) / 5)) >> +=C2=A0=C2=A0=C2=A0=C2=A0(u_int8_t)(((y) =3D=3D HAL_FREQ_BAND_2GHZ) ? = ((x) - 2300) : (((x) - 4800) / 5)) >> =C2=A0#define FBIN2FREQ(x,y) \ >> =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0(((y) =3D=3D HAL_FREQ_BAND_2GHZ) ? (2300= + x) : (4800 + 5 * x)) >> =C2=A0#define OSPREY_MAX_CHAINS=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A03 >> >=20 > How can this change possibly be correct? All the values involved are > compile-time constants, the compiler did the math and tells you the > result is=C2=A03495. Cast or not, that value is never going to fit into= a > uint8_t. Sure this supresses the warning, but what about actually > fixing the overflow? >=20 > -- Ian >=20 >=20 An excellent question all around. It appears that the maintainer at Qualcom acked the linux commit, so I'm not sure how to interpret all of this. sean --i4GJo3K1oHs27WcmgOIJ0IkPqpj8uFWk7-- --gqoBk2eMRFXUDwtKBH73RCL503CaqjrE7 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- iQGTBAEBCgB9FiEE6MTp+IA1BOHj9Lo0veT1/om1/LYFAlr/DddfFIAAAAAALgAo aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldEU4 QzRFOUY4ODAzNTA0RTFFM0Y0QkEzNEJERTRGNUZFODlCNUZDQjYACgkQveT1/om1 /LZi1ggA0qnZSZFdhY7nLXKfTAWYtbmQYTH+wtS7Ok5OWCuqBrTfzUoVgYPacJRL zAgXhCF3gm6/kBzQ/GB8zgiLFVD5z3nhNHlzOoMkZW65vyH6zRQdRtrzc8ezI/+U XvUE0VHpMmFfxGN9J0PkibzbF8PDByuHMY3reVKS7zj0PSJoCMfrYgTu0kqHfFtU JuHcnYYSRKNkUaSIoOQx4KMwjch/o6dMJ8UkNvM4c7xroFPeBI4Nx9yXggywxS9H MPFBOnfmlEWe3+Ff5jLO2/r0lhniKFFv8USTDaQfFwt0zynyOC4AA3EDDnbEr1yV LardcsrWCoI6a6Yb8czzquSXDY3Lzg== =9reQ -----END PGP SIGNATURE----- --gqoBk2eMRFXUDwtKBH73RCL503CaqjrE7--