Re: [PATCH 3/3] iio: adc: Add support for PMIC7 ADC

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

 



Hi Andy,

On 3/24/2020 10:03 PM, Andy Shevchenko wrote:
On Tue, Mar 24, 2020 at 5:46 PM Jishnu Prakash <jprakash@xxxxxxxxxxxxxx> wrote:
The ADC architecture on PMIC7 is changed as compared to PMIC5. The
major change from PMIC5 is that all SW communication to ADC goes through
PMK8350, which communicates with other PMICs through PBS when the ADC
on PMK8350 works in master mode. The SID register is used to identify the
PMICs with which the PBS needs to communicate. Add support for the same.

In addition, add definitions for ADC channels and virtual channel
definitions per PMIC, to be used by ADC clients for PMIC7.
...

+#define ADC_CHANNEL_OFFSET                     0x8
+#define ADC_CHANNEL_MASK                       0xff
GENMASK()
I'll fix this and the other simple changes in the next post.

...

+#define ADC_APP_SID                            0x40
+#define ADC_APP_SID_MASK                       0xf
GENMASK()

+#define ADC7_CONV_TIMEOUT                      msecs_to_jiffies(10)
Useless.
I'm not sure what you mean by this. It is used in the API adc7_do_conversion.
...

+ if (of_device_is_compatible(node, "qcom,spmi-adc7")) {
+               indio_dev->info = &adc7_info;
+               adc->is_pmic7 = true;
+       } else {
+               indio_dev->info = &adc5_info;
+       }
Hmm... I would rather put this as driver_data in ID structure(s).
I'll remove the check for the compatible string, using driver data, in the next post.

...

+static int adc5_exit(struct platform_device *pdev)
+{
+       struct adc5_chip *adc = platform_get_drvdata(pdev);
+
+       mutex_destroy(&adc->lock);
Are you sure you will have no race conditions? Does this driver use IRQs?
The driver does use an IRQ. Will fix this in the next post.





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux