Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1558370
| From | Sekhar Nori <nsekhar@ti.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH v2 3/5] ARM: davinci_all_defconfig: enable iio and ADS7950 |
| Date | 2017-01-13 13:30 +0100 |
| Message-ID | <sZ99w-Oh-11@gated-at.bofh.it> (permalink) |
| References | <sWutQ-4H4-1@gated-at.bofh.it> <sWutQ-4H4-9@gated-at.bofh.it> <sXHp0-4by-11@gated-at.bofh.it> <sY6Qq-39z-21@gated-at.bofh.it> <sYmsa-4A7-21@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Wednesday 11 January 2017 01:53 PM, Sekhar Nori wrote: > On Tuesday 10 January 2017 09:13 PM, David Lechner wrote: >> On 01/09/2017 06:29 AM, Sekhar Nori wrote: >>> On Friday 06 January 2017 10:03 AM, David Lechner wrote: >>>> This enables the iio subsystem and the TI ADS7950 driver. This is >>>> used by >>>> LEGO MINDSTORMS EV3, which has an ADS7957 chip. >>> >>> Can you add your sign-off? >>> >>>> --- >>>> >>>> The CONFIG_TI_ADS7950 driver is currently in iio/testing, so some >>>> coordination >>>> may be needed before picking up this patch. >>>> >>>> arch/arm/configs/davinci_all_defconfig | 7 +++++++ >>>> 1 file changed, 7 insertions(+) >>>> >>>> diff --git a/arch/arm/configs/davinci_all_defconfig >>>> b/arch/arm/configs/davinci_all_defconfig >>>> index 2b1967a..a899876 100644 >>>> --- a/arch/arm/configs/davinci_all_defconfig >>>> +++ b/arch/arm/configs/davinci_all_defconfig >>>> @@ -200,6 +200,13 @@ CONFIG_TI_EDMA=y >>>> CONFIG_MEMORY=y >>>> CONFIG_TI_AEMIF=m >>>> CONFIG_DA8XX_DDRCTL=y >>>> +CONFIG_IIO=m >>>> +CONFIG_IIO_BUFFER_CB=m >>>> +CONFIG_IIO_SW_DEVICE=m >>>> +CONFIG_IIO_SW_TRIGGER=m >>> >>>> +CONFIG_TI_ADS7950=m >>> >>> Can you separate this from rest of the patch. I would like to enable >>> this option only after I can find the symbol in linux-next. >> >> Will resend without CONFIG_TI_ADS7950 >> >>> >>>> +CONFIG_IIO_HRTIMER_TRIGGER=m >>>> +CONFIG_IIO_SYSFS_TRIGGER=m >>> >>> Need CONFIG_IIO_TRIGGER=y also for these two options to take effect. >> >> CONFIG_IIO_TRIGGER is selected by IIO_TRIGGERED_BUFFER [=m] && IIO [=m] >> && IIO_BUFFER [=y], so save_defconfig does not pick it up. > > I do remember I did not see these two modules did not get enabled in > .config after 'make davinci_all_defconfig'. Will check what I may have > missed. So IIO_TRIGGERED_BUFFER is not selected in my tree because I dont have the ADS7950 driver enabled. Same thing with IIO_BUFFER. Can you try this patch over my tree? Thanks, Sekhar
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH v2 3/5] ARM: davinci_all_defconfig: enable iio and ADS7950 David Lechner <david@lechnology.com> - 2017-01-06 05:40 +0100
Re: [PATCH v2 3/5] ARM: davinci_all_defconfig: enable iio and ADS7950 Sekhar Nori <nsekhar@ti.com> - 2017-01-09 13:40 +0100
Re: [PATCH v2 3/5] ARM: davinci_all_defconfig: enable iio and ADS7950 David Lechner <david@lechnology.com> - 2017-01-10 16:50 +0100
Re: [PATCH v2 3/5] ARM: davinci_all_defconfig: enable iio and ADS7950 Sekhar Nori <nsekhar@ti.com> - 2017-01-11 09:30 +0100
Re: [PATCH v2 3/5] ARM: davinci_all_defconfig: enable iio and ADS7950 Sekhar Nori <nsekhar@ti.com> - 2017-01-13 13:30 +0100
Re: [PATCH v2 3/5] ARM: davinci_all_defconfig: enable iio and ADS7950 David Lechner <david@lechnology.com> - 2017-01-13 21:10 +0100
csiph-web