[PATCH] cramfs: probe(): fix cdev lookup
Marc Kleine-Budde
mkl at pengutronix.de
Tue Dec 15 15:40:10 EST 2009
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Peter Korsgaard wrote:
>>>>>> "Marc" == Marc Kleine-Budde <mkl at pengutronix.de> writes:
>
> Hi,
>
> >> @@ -431,7 +431,7 @@ static int cramfs_probe(struct device_d *dev)
> >> if (strncmp(fsdev->backingstore, "/dev/", 5))
> >> return -ENODEV;
> >>
> >> - priv->cdev = cdev_by_name(fsdev->backingstore);
> >> + priv->cdev = cdev_by_name(fsdev->backingstore + 5);
>
> Marc> IMHO sizeof("/dev/") (and perhaps a comment) make the code more readable.
>
> That would be sizeof("/dev/") - 1 to not count in the trailing 0. The
doh! I'm out of brown paper bags.
> same should presumably also be done in the strncmp above - I don't find
> the sizeof - 1 syntax particular more readable, but I can change it if
> preferred.
mkl
- --
Pengutronix e.K. | Marc Kleine-Budde |
Linux Solutions for Science and Industry | Phone: +49-231-2826-924 |
Vertretung West/Dortmund | Fax: +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de |
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iEYEARECAAYFAksn9CoACgkQjTAFq1RaXHOfqQCfam0hzM7PNw8+/udW/h4zPmCL
EI4AnRZq9aZiac1KT9NWohrU3QaUrzBC
=AH16
-----END PGP SIGNATURE-----
More information about the u-boot-v2
mailing list