Re: [PATCH v11 0/7] iio: add new backend framework

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Feb 10, 2024 at 10:57 PM Nuno Sa via B4 Relay
<devnull+nuno.sa.analog.com@xxxxxxxxxx> wrote:

> Changes in v11:
>  - Patch 6
>    * Directly use dev in devm_iio_backend_get();
>    * Move comment above the proper place.
>  - Patch 7
>    * Added blank line between includes (to logically separate them);
>    * Move back to 10 millisecond sleep;
>    * Constify expected_ver and removed unneeded cast.

Assuming that the fwnode vs. device reference count is fine (as we
expect the backends to call respective APIs and hence have backend
device pointer valid despote fwnode reference being dropped before
getting the device) and timeout thingy had been reverted to the
original value, FWIW,
Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>
for patches 4-7 (OF is not my area of interest at all :).

> Jonathan, the series is based on next-20240202 since it already includes
> the io-channels fix Rob applied in his tree. I guess it should land in rc3 so
> after you rebase, all patches should apply cleanly (if applying them of course
> :)). Let me know if anything fails...
>
> Keeping the block diagram  so we don't have to follow links
> to check one of the typical setups.
>
>                                            -------------------------------------------------------
>  ------------------                        | -----------         ------------      -------  FPGA |
>  |     ADC        |------------------------| | AXI ADC |---------| DMA CORE |------| RAM |       |
>  | (Frontend/IIO) | Serial Data (eg: LVDS) | |(backend)|---------|          |------|     |       |
>  |                |------------------------| -----------         ------------      -------       |
>  ------------------                        -------------------------------------------------------


-- 
With Best Regards,
Andy Shevchenko





[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux