On paź 18, 2018 19:31, Lars-Peter Clausen wrote: > On 10/18/2018 01:49 PM, Slawomir Stepien wrote: > > The custom build function ad7280_crc8_build_table is not needed. The > > crc8_populate_msb function from linux/crc8.h will build the same crc > > table. > > > > Signed-off-by: Slawomir Stepien <sst@xxxxxxxxx> > > This looks good, thanks. > > Acked-by: Lars-Peter Clausen <lars@xxxxxxxxxx> Thank you. > I supposed the code could be improved a bit further in a follow up patch by > only having one global table since it will be the same for all instances. Yes that's a good idea. How should I proceed? v3 or a separate commit based on the changes in this commit? I think separate commit will be better, so I can make a good commit message...or maybe put it all in a patch series with your ack added to this change? Sorry I don't know the rules in such cases. -- Slawomir Stepien