Re: [PATCH v15 00/22] media: i2c: add Maxim GMSL2/3 serializer and deserializer drivers
"Artur Andrzejczak" <[email protected]>
| Newsgroups | org.kernel.vger.linux-devicetree,dev.linux.lists.linux-staging,org.kernel.vger.linux-gpio,org.kernel.vger.linux-kernel,org.kernel.vger.linux-media |
|---|---|
| Message-ID | <[email protected]> |
On Fri Aug 7, 2026 at 3:02 PM CEST, Dumitru Ceclan wrote: > [...] Thanks for the patches, good to see the GMSL support integrated into a common framework. Small note: patch 16 adds a second config VIDEO_MAX96717 and a second max96717.o. The old driver is only removed in patch 21. One symbol then builds two max96717.o objects, which scripts/modules-check.sh rejects. Won't allmodconfig bisection be broken for patches 16..20? Both drivers also match maxim,max96717f in that window. The same overlap exists for maxim,max96712 (staging driver vs the new max96724 one, patches 17..19) and maxim,max96714f (18..21), where probing becomes order-dependent. Can the removals be reordered so that these windows close? Kind Regards, Artur Andrzejczak