lkml.org 
[lkml]   [2021]   [Jun]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v10] i2c: virtio: add a virtio i2c frontend driver
    On 30-06-21, 16:38, Wolfram Sang wrote:
    > > While we are at it, this has been replaced by a Rust counterpart [1]
    > > (as that makes it hypervisor agnostic, which is the goal of my work
    > > here) and I need someone with I2C knowledge to help review it. It
    > > should be okay even if you don't understand Rust a lot, just review
    > > this file[2] which is where most of i2c specific stuff lies.
    >
    > From the high level review I can provide, it looks good to me. Block
    > transfers are missing, but I think you said that already. Mising Rust
    > experience, I might miss details, of course. But the general approach
    > seems fine to me. smbus_prepare() will get a bit more messy when you add
    > block transfers, but it still looks bearable, I think.

    Thanks for having a look.

    --
    viresh

    \
     
     \ /
      Last update: 2021-06-30 17:10    [W:4.268 / U:2.540 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site