Re: [PATCH 2/6] virt: bao: Add Bao IPC shared memory driver

João Peixoto <[email protected]>
Newsgroups org.infradead.lists.linux-riscv,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-doc,org.kernel.vger.linux-kernel
Message-ID <[email protected]>
On 1/7/26 18:54, Randy Dunlap wrote:
> On 1/7/26 8:28 AM,[email protected] wrote:
>> diff --git a/drivers/virt/bao/ipcshmem/Kconfig b/drivers/virt/bao/ipcshmem/Kconfig
>> new file mode 100644
>> index 000000000000..966bb75aa495
>> --- /dev/null
>> +++ b/drivers/virt/bao/ipcshmem/Kconfig
>> @@ -0,0 +1,8 @@
>> +# SPDX-License-Identifier: GPL-2.0
>> +config BAO_SHMEM
>> +	tristate "Bao hypervisor shared memory support"
>> +	help
>> +	This enables support for Bao shared memory communication.
>> +	It allows the kernel to interface with guests running under
>> +	the Bao hypervisor, providing a character device interface
>> +	for exchanging data through dedicated shared-memory regions.
> Please follow Documentation/process/coding-style.rst:
>
> 10) Kconfig configuration files
> -------------------------------
>
> For all of the Kconfig* configuration files throughout the source tree,
> the indentation is somewhat different.  Lines under a ``config`` definition
> are indented with one tab, while help text is indented an additional two
> spaces.  Example::
>
>    config AUDIT
> 	bool "Auditing support"
> 	depends on NET
> 	help
> 	  Enable auditing infrastructure that can be used with another
> 	  kernel subsystem, such as SELinux (which requires this for
> 	  logging of avc messages output).  Does not do system-call
> 	  auditing without CONFIG_AUDITSYSCALL.
>
Fixed in v3, the help text is now indented one tab plus two spaces:

   config BAO_SHMEM
       tristate "Bao hypervisor shared memory support"
       help
         This enables support for Bao shared memory communication.
         [...]
         If unsure, say N.

Thanks for catching it.

_______________________________________________
linux-riscv mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/linux-riscv
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.