linux-kernel - Re: [PATCH v3 2/4] mfd: sony-cronos-cpld: Add driver for Sony Cronos CPLD

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <C519436F-296F-4313-87ED-F665CF8EBDAF@sony.com>
Date: Wed, 19 Nov 2025 00:51:04 +0000
From: "Yakovlev, Georgy" <Georgy.Yakovlev@...y.com>
To: Lee Jones <lee@...nel.org>
CC: Timothy Pearson <tpearson@...torengineering.com>,
        devicetree
	<devicetree@...r.kernel.org>,
        linux-kernel <linux-kernel@...r.kernel.org>,
        Rob Herring <robh+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Shawn Anastasio
	<sanastasio@...torengineering.com>
Subject: Re: [PATCH v3 2/4] mfd: sony-cronos-cpld: Add driver for Sony Cronos
 CPLD



> On Oct 9, 2025, at 3:42 AM, Lee Jones <lee@...nel.org> wrote:
> 
> This doesn't need to / shouldn't reside in the kernel.
> 
> Why not use I2C and handle all of this in userspace?
We would prefer to keep the sysfs interfaces in-kernel due to I2C bus contention between the MFD sub-drivers and userspace applications.
Keeping everything in-kernel allows the kernel itself to arbitrate between the various MFD sub-drivers; punting to userspace complicates each userspace application as they each individually would need to handle I2C bus contention and associated EBUSY responses vs. a sysfs read.

> 
>> \ No newline at end of file
> 
> ???
> 
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 0e897f2ba9eb..a1052663fb27 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -23458,6 +23458,13 @@ S: Maintained
>> F: drivers/ssb/
>> F: include/linux/ssb/
>> 
>> +SONY CRONOS CPLD DRIVER
>> +M: Georgy Yakovlev <Georgy.Yakovlev@...y.com>
> 
> This person is going to have to Ack the fact that you're making them the
> person responsible for maintaining this.
Yes, I acknowledge.
Not new to open source in general, but new to kernel submissions.
Sorry for delayed response, was away.
> 
>> +S: Maintained
>> +F: Documentation/devicetree/bindings/mfd/sony,cronos-cpld.yaml
>> +F: drivers/mfd/sony-cronos-cpld.c
>> +F: include/linux/mfd/sony/cronos/
>> +
>> SONY IMX208 SENSOR DRIVER
>> M: Sakari Ailus <sakari.ailus@...ux.intel.com>
>> L: linux-media@...r.kernel.org
>> diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
>> index 425c5fba6cb1..cd0c624022f6 100644
>> --- a/drivers/mfd/Kconfig
>> +++ b/drivers/mfd/Kconfig
>> @@ -2341,6 +2341,17 @@ config MFD_QCOM_PM8008
>>   under it in the device tree. Additional drivers must be enabled in
>>   order to use the functionality of the device.
>> 
>> +config MFD_SONY_CRONOS_CPLD
>> + tristate "Sony Cronos CPLD Support"
>