Messages in this thread | | | From | Andy Shevchenko <> | Date | Tue, 20 Jul 2021 17:07:31 +0300 | Subject | Re: [PATCH v6 1/2] iio: frequency: adrf6780: add support for ADRF6780 |
| |
On Tue, Jul 20, 2021 at 4:17 PM Miclaus, Antoniu <Antoniu.Miclaus@analog.com> wrote: > > From: Andy Shevchenko <andy.shevchenko@gmail.com> > > Sent: Friday, July 16, 2021 5:53 PM > > On Fri, Jul 16, 2021 at 2:43 PM Antoniu Miclaus > > <antoniu.miclaus@analog.com> wrote:
...
> > First question is why not to use the regmap API (I have heard it has > > gained support of 17 bit)? > > Initially that was the plan, but after this patch: > https://github.com/torvalds/linux/commit/4191f19792bf91267835eb090d970e9cd6277a65 > the custom write formats for regmap allow the read only via cached registers. > > Therefore, I preferred using spi transfers for write/read to/from the device.
Not sure I follow you. That patch is upstream. Does it prevent you from switching to regmap SPI API?
...
> > > + depends on COMMON_CLK > > > > Is it mandatory for any function inside the device? > > Yes. It will serve as LO input to the device.
But can the device work without it (with limited functionality)?
-- With Best Regards, Andy Shevchenko
| |