Hi Prabhakar, On Wed, Nov 25, 2020 at 6:23 PM Lad, Prabhakar <[email protected]> wrote: > On Wed, Nov 25, 2020 at 4:32 PM Geert Uytterhoeven <[email protected]> > wrote: > > On Wed, Nov 25, 2020 at 5:26 PM Prabhakar Mahadev Lad > > <[email protected]> wrote: > > > > -----Original Message----- > > > > From: Geert Uytterhoeven <[email protected]> > > > > Sent: 25 November 2020 16:21 > > > > To: Prabhakar Mahadev Lad <[email protected]> > > > > Cc: Magnus Damm <[email protected]>; Rob Herring > > > > <[email protected]>; Linux-Renesas <linux- > > > > [email protected]>; open list:OPEN FIRMWARE AND FLATTENED > > > > DEVICE TREE BINDINGS > > > > <[email protected]>; Linux Kernel Mailing List > > > > <[email protected]>; Biju Das > > > > <[email protected]>; Prabhakar <[email protected]> > > > > 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 > > > > <[email protected]> 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+akbus...@mail.gmail.com/ Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected] 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

