> >> +  {
> >> +          .compatible = "marvell,mv78230-a0-i2c",
> >> +          .data = &mv64xxx_i2c_regs_mv64xxx
> >> +  },
> > 
> > I think a oneliner entry like the entries above is easier to read, but
> > that is very minor...
> 
> By using one line we would break the 80 character rule,
> hat why I did in this way.

This rule is there to keep code readable. If it doesn't help
readability, it may/can/should be broken IMO. It's a mileage, though.

Attachment: signature.asc
Description: Digital signature

Reply via email to