Hi Loic,
On Wed, Apr 02, 2014, Poulain, Loic wrote:
> +static int bcm43xx(int fd, struct uart_t *u, struct termios *ti)
> +{
> + return bcm43xx_init(fd, u->speed, ti, u->bdaddr);
> +}
What did you do to your indentation settings? You seem to have switched
indentation style for the entire patch since v4. BlueZ uses only tabs
for indentation but now your entire patch uses 4 spaces.
> + struct timespec tm_mode = {0, 50000};
> + struct timespec tm_ready = {0, 2000000};
Still missing spaces after { and before }.
> + fprintf (stderr, "Cannot open directory '%s': %s\n",
> + dir_name, strerror (errno));
No space after the function name here. Applies both for fprintf as well
as strerror.
> + /* Recursively look for a BCM43XX*.hcd */
> + while (1) {
> + entry = readdir(dir);
Try to declare variables in the smallest scope possible. Seems like
entry could be declared in the beginning of this while-loop instead of
the beginning of the function.
Johan
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
