On Mon, 9 Feb 2026 20:20:07 +0200 Erikas Bitovtas <[email protected]> wrote:
> On 2/9/26 5:49 PM, Andy Shevchenko wrote: > > On Mon, Feb 09, 2026 at 05:48:22PM +0200, Andy Shevchenko wrote: > >> On Mon, Feb 09, 2026 at 04:23:40PM +0200, Erikas Bitovtas wrote: > >>> This patch series adds support for Capella cm36686 and cm36672p ambient > >>> light and proximity sensors. > >>> > >>> Capella cm36686 is a combined ambient light and proximity sensor with > >>> adjustable integration time, interrupt and hysteresis support. It has > >>> the slave address of 0x60. cm36672p is fully compatible with cm36686, > >>> except that it is a proximity-only sensor. > >> > >> So, can we share the code (assuming there is a driver and it already > >> uses or may be converted to regmap APIs)? > > > > Ah, it's being added with the same driver... So, have you checked if there > > is > > any similar HW with the available driver in upstream? > > > > I checked only for other Capella sensors, I forgot that Vishay acquired > Capella > Microsystems. Now that I checked, vcnl4040 uses the same registers and > regfields > as cm36686 and can be used instead... > I am sorry for wasting your time. Should I add a fallback to vcnl4040's > devicetree schema and add cm36672p support by excluding the light channels > instead? > Sounds good to me. Jonathan >
