[PATCH v2 11/12] arm64: defconfig: Enable FSD CSIS DMA driver

Inbaraj E posted 12 patches 1 month, 3 weeks ago
[PATCH v2 11/12] arm64: defconfig: Enable FSD CSIS DMA driver
Posted by Inbaraj E 1 month, 3 weeks ago
Enable CSIS DMA driver support for FSD based platforms.

Signed-off-by: Inbaraj E <inbaraj.e@samsung.com>
---
 arch/arm64/configs/defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 58f87d09366c..7f7e1fadb74f 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -878,6 +878,7 @@ CONFIG_VIDEO_RENESAS_VSP1=m
 CONFIG_VIDEO_RCAR_DRIF=m
 CONFIG_VIDEO_ROCKCHIP_RGA=m
 CONFIG_VIDEO_SAMSUNG_EXYNOS_GSC=m
+CONFIG_VIDEO_FSD_CSIS=m
 CONFIG_VIDEO_SAMSUNG_S5P_JPEG=m
 CONFIG_VIDEO_SAMSUNG_S5P_MFC=m
 CONFIG_VIDEO_SUN6I_CSI=m
-- 
2.49.0
Re: [PATCH v2 11/12] arm64: defconfig: Enable FSD CSIS DMA driver
Posted by Krzysztof Kozlowski 1 month, 2 weeks ago
On 14/08/2025 16:09, Inbaraj E wrote:
> Enable CSIS DMA driver support for FSD based platforms.

Tesla FSD

You are changing defconfig for all platforms, it's not your personal or
company defconfig.


Best regards,
Krzysztof
RE: [PATCH v2 11/12] arm64: defconfig: Enable FSD CSIS DMA driver
Posted by Inbaraj E 1 month, 1 week ago
Hi Krzysztof,

Thanks for the review.

> 
> Tesla FSD
> 
> You are changing defconfig for all platforms, it's not your personal or
> company defconfig.

I noticed that Exynos and NXP driver configs are added in the defconfig.
Could you please clarify why I shouldn't add my driver config in defconfig?

> 
> 

Regards,
Inbaraj E
Re: [PATCH v2 11/12] arm64: defconfig: Enable FSD CSIS DMA driver
Posted by Krzysztof Kozlowski 1 month, 1 week ago
On 23/08/2025 04:05, Inbaraj E wrote:
> Hi Krzysztof,
> 
> Thanks for the review.
> 
>>
>> Tesla FSD
>>
>> You are changing defconfig for all platforms, it's not your personal or
>> company defconfig.
> 
> I noticed that Exynos and NXP driver configs are added in the defconfig.
> Could you please clarify why I shouldn't add my driver config in defconfig?

No, I gave you rationale why your commit description is poor.

Best regards,
Krzysztof
RE: [PATCH v2 11/12] arm64: defconfig: Enable FSD CSIS DMA driver
Posted by Inbaraj E 1 month, 1 week ago
Hi Krzysztof,

.
> >
> > I noticed that Exynos and NXP driver configs are added in the defconfig.
> > Could you please clarify why I shouldn't add my driver config in defconfig?
> 
> No, I gave you rationale why your commit description is poor.

I'll update the commit description in next patchset.

> 
> Best regards,
> Krzysztof

Regards,
Inbaraj E