Messages in this thread | | | Date | Tue, 7 Feb 2023 21:15:21 +0100 | Subject | Re: [PATCH 1/2] media: samsung: exynos4-is: drop simple-bus from compatibles | From | Krzysztof Kozlowski <> |
| |
On 07/02/2023 20:29, Krzysztof Kozlowski wrote: > The FIMC camera node wrapper is not a bus, so using simple-bus fallback > compatible just to instantiate its children nodes was never correct. > Driver should explicitly populate all its children devices. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > drivers/media/platform/samsung/exynos4-is/media-dev.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/media/platform/samsung/exynos4-is/media-dev.c b/drivers/media/platform/samsung/exynos4-is/media-dev.c > index 98a60f01129d..70c4ca7844d3 100644 > --- a/drivers/media/platform/samsung/exynos4-is/media-dev.c > +++ b/drivers/media/platform/samsung/exynos4-is/media-dev.c > @@ -1440,6 +1440,10 @@ static int fimc_md_probe(struct platform_device *pdev) > if (!fmd) > return -ENOMEM; > > + ret = devm_of_platform_populate(dev); > + if (ret < 0) > + return -ENOMEM; > +
This cannot be devm-like, because on errors unpopulating devices breaks links and IOMMU. I will make it of_platform_populate() instead.
Best regards, Krzysztof
| |