On Tue, May 30, 2017 at 08:57:33PM +0300, Alex A. Mihaylov wrote:
> + if (!w1_reset_select_slave(sl)) {
> + w1_write_8(sl->master, W1_CMD_READ_DATA);
> + w1_write_8(sl->master, reg);
> + *val = w1_read_8(sl->master);
> + } else
> + ret = -ENODEV;
CodingStyle says { } on both sides of an if statement if they're on one.
Otherwise this looks good.
signature.asc
Description: PGP signature
