Hi Prabhakar, On Wed, Nov 25, 2020 at 6:23 PM Lad, Prabhakar <prabhakar.csengg@xxxxxxxxx> wrote: > On Wed, Nov 25, 2020 at 4:32 PM Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote: > > On Wed, Nov 25, 2020 at 5:26 PM Prabhakar Mahadev Lad > > <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> wrote: > > > > -----Original Message----- > > > > From: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> > > > > Sent: 25 November 2020 16:21 > > > > To: Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > > > > Cc: Magnus Damm <magnus.damm@xxxxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>; Linux-Renesas <linux- > > > > renesas-soc@xxxxxxxxxxxxxxx>; open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS > > > > <devicetree@xxxxxxxxxxxxxxx>; Linux Kernel Mailing List <linux-kernel@xxxxxxxxxxxxxxx>; Biju Das > > > > <biju.das.jz@xxxxxxxxxxxxxx>; Prabhakar <prabhakar.csengg@xxxxxxxxx> > > > > Subject: Re: [PATCH v2 0/2] r8a7742-iwg21d-q7-dbcm: Add support for ov7725 sensors > > > > > > > > On Wed, Nov 25, 2020 at 2:02 PM Lad Prabhakar > > > > <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> wrote: > > > > > This patch set enables to connect ov7725 sensors on iWave-RZ/G1H Qseven > > > > > board. > > > > > > > > Thanks for your series! > > > > > > > > Do you think it's a valid use case to mix and match ov5640 and ov7725 > > > > cameras? E.g. connect two of each? > > > > > > > Yes that is valid case to mix and match. Do you want me to make it configurable too ? > > > > If this is a valid use case, then please do so. > > > In that case do we still want to keep the ov7725/ov5640 nodes in > separate dtsi ? You can, if you define only a single camera in each include file, and #include four camera include files, with some #define/#undef-ery around each #include. Cfr. my suggestion in https://lore.kernel.org/linux-renesas-soc/CAMuHMdXAB-eUAMSeptptajr0eReHXHFuoR5HZkB-X+AKBUsyxA@xxxxxxxxxxxxxx/ Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds