From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marc Kleine-Budde Subject: Re: [PATCH 2/4] can: cc770: Fix likely misuse of | for & Date: Thu, 31 May 2012 22:54:43 +0200 Message-ID: <4FC7DA93.1080905@pengutronix.de> References: <6c251b03dc626215cf696e894ac1cdda530f38d9.1338408931.git.joe@perches.com> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="------------enig63DDF846AE003BF88EFAFA75" Cc: linux-kernel@vger.kernel.org, Wolfgang Grandegger , linux-can@vger.kernel.org, netdev@vger.kernel.org To: Joe Perches Return-path: In-Reply-To: <6c251b03dc626215cf696e894ac1cdda530f38d9.1338408931.git.joe@perches.com> Sender: linux-can-owner@vger.kernel.org List-Id: netdev.vger.kernel.org This is an OpenPGP/MIME signed message (RFC 2440 and 3156) --------------enig63DDF846AE003BF88EFAFA75 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On 05/30/2012 10:25 PM, Joe Perches wrote: > Using | with a constant is always true. > Likely this should have be &. >=20 > Signed-off-by: Joe Perches Sounds reasonable. And there are no in tree users of the platform driver that this fix could break. commited to linux-can, Marc > --- > drivers/net/can/cc770/cc770_platform.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) >=20 > diff --git a/drivers/net/can/cc770/cc770_platform.c b/drivers/net/can/c= c770/cc770_platform.c > index 53115ee..688371c 100644 > --- a/drivers/net/can/cc770/cc770_platform.c > +++ b/drivers/net/can/cc770/cc770_platform.c > @@ -154,7 +154,7 @@ static int __devinit cc770_get_platform_data(struct= platform_device *pdev, > struct cc770_platform_data *pdata =3D pdev->dev.platform_data; > =20 > priv->can.clock.freq =3D pdata->osc_freq; > - if (priv->cpu_interface | CPUIF_DSC) > + if (priv->cpu_interface & CPUIF_DSC) > priv->can.clock.freq /=3D 2; > priv->clkout =3D pdata->cor; > priv->bus_config =3D pdata->bcr; --=20 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 | --------------enig63DDF846AE003BF88EFAFA75 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAk/H2pgACgkQjTAFq1RaXHNAqwCeIgt27xgLy0AE5WyKEfeNK/EY HSEAnRX1XvklF/b1VuIeKLJ0dithQs8H =G9+R -----END PGP SIGNATURE----- --------------enig63DDF846AE003BF88EFAFA75--