Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d268f8dd authored by Fabrice Gasnier's avatar Fabrice Gasnier Committed by Greg Kroah-Hartman
Browse files

iio: adc: stm32-dfsdm: fix device used to request dma 



[ Upstream commit b455d06e6fb3c035711e8aab1ca18082ccb15d87 ]

DMA channel request should use device struct from platform device struct.
Currently it's using iio device struct. But at this stage when probing,
device struct isn't yet registered (e.g. device_register is done in
iio_device_register). Since commit 71723a96b8b1 ("dmaengine: Create
symlinks between DMA channels and slaves"), a warning message is printed
as the links in sysfs can't be created, due to device isn't yet registered:
- Cannot create DMA slave symlink
- Cannot create DMA dma:rx symlink

Fix this by using device struct from platform device to request dma chan.

Fixes: eca94980 ("IIO: ADC: add stm32 DFSDM support for PDM microphone")

Signed-off-by: default avatarFabrice Gasnier <fabrice.gasnier@st.com>
Cc: <Stable@vger.kernel.org>
Signed-off-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
parent ae862e35

drivers/iio/adc/stm32-dfsdm-adc.c

+8 −7
Original line number Diff line number Diff line
@@ -45,7 +45,7 @@ enum sd_converter_type { 


struct stm32_dfsdm_dev_data {

	int type;

	int (*init)(struct iio_dev *indio_dev);

	int (*init)(struct device *dev, struct iio_dev *indio_dev);

	unsigned int num_channels;

	const struct regmap_config *regmap_cfg;

};
@@ -923,7 +923,8 @@ static void stm32_dfsdm_dma_release(struct iio_dev *indio_dev) 
	}

}



static int stm32_dfsdm_dma_request(struct iio_dev *indio_dev)

static int stm32_dfsdm_dma_request(struct device *dev,

				   struct iio_dev *indio_dev)

{

	struct stm32_dfsdm_adc *adc = iio_priv(indio_dev);

	struct dma_slave_config config = {
@@ -933,7 +934,7 @@ static int stm32_dfsdm_dma_request(struct iio_dev *indio_dev) 
	};

	int ret;



	adc->dma_chan = dma_request_chan(&indio_dev->dev, "rx");

	adc->dma_chan = dma_request_chan(dev, "rx");

	if (IS_ERR(adc->dma_chan)) {

		int ret = PTR_ERR(adc->dma_chan);
@@ -997,7 +998,7 @@ static int stm32_dfsdm_adc_chan_init_one(struct iio_dev *indio_dev, 
					  &adc->dfsdm->ch_list[ch->channel]);

}



static int stm32_dfsdm_audio_init(struct iio_dev *indio_dev)

static int stm32_dfsdm_audio_init(struct device *dev, struct iio_dev *indio_dev)

{

	struct iio_chan_spec *ch;

	struct stm32_dfsdm_adc *adc = iio_priv(indio_dev);
@@ -1027,10 +1028,10 @@ static int stm32_dfsdm_audio_init(struct iio_dev *indio_dev) 
	indio_dev->num_channels = 1;

	indio_dev->channels = ch;



	return stm32_dfsdm_dma_request(indio_dev);

	return stm32_dfsdm_dma_request(dev, indio_dev);

}



static int stm32_dfsdm_adc_init(struct iio_dev *indio_dev)

static int stm32_dfsdm_adc_init(struct device *dev, struct iio_dev *indio_dev)

{

	struct iio_chan_spec *ch;

	struct stm32_dfsdm_adc *adc = iio_priv(indio_dev);
@@ -1174,7 +1175,7 @@ static int stm32_dfsdm_adc_probe(struct platform_device *pdev) 
		adc->dfsdm->fl_list[adc->fl_id].sync_mode = val;



	adc->dev_data = dev_data;

	ret = dev_data->init(iio);

	ret = dev_data->init(dev, iio);

	if (ret < 0)

		return ret;