Hi Krzysztof,

Thanks for the review - all fair, and we'll fix them in v2:

Drop L: [email protected] (keeping only linux-crypto).
Drop the T: line - we don't maintain a tree; the driver will go through the 
crypto tree.

Yes, Joel and Thi reviewed and acknowledged with the statement of oversight.

Krishnan

________________________________________
From: Krzysztof Kozlowski <[email protected]>
Sent: Friday, June 26, 2026 3:57 AM
To: Krishnamoorthy, Saravanakrishnan
Cc: Albert Ou; Ousherovitch, Alex; Conor Dooley; David S. Miller; Herbert Xu; 
Jonathan Corbet; Krzysztof Kozlowski; Palmer Dabbelt; Paul Walmsley; Rob 
Herring; Shuah Khan; Alexandre Ghiti; [email protected]; Wittenauer, 
Joel; [email protected]; [email protected]; 
[email protected]; [email protected]; 
[email protected]; [email protected]; Shuah Khan; 
SIPSupport; Nguyen, Thi
Subject: Re: [PATCH 19/19] MAINTAINERS: add Rambus CryptoManager Hub (CMH)

[Some people who received this message don't often get email from 
[email protected]. Learn why this is important at 
https://aka.ms/LearnAboutSenderIdentification ]

Caution: < External Email >

On Thu, Jun 25, 2026 at 10:33:27AM -0700, Saravanakrishnan Krishnamoorthy wrote:
> From: Alex Ousherovitch <[email protected]>
>
> Add MAINTAINERS entry for the CRI CryptoManager Hub (CMH) hardware
> crypto accelerator driver under drivers/crypto/cmh/.
>
> Co-developed-by: Saravanakrishnan Krishnamoorthy <[email protected]>
> Signed-off-by: Saravanakrishnan Krishnamoorthy <[email protected]>
> Signed-off-by: Alex Ousherovitch <[email protected]>
> Reviewed-by: Joel Wittenauer <[email protected]>
> Reviewed-by: Thi Nguyen <[email protected]>

Are these people really provided you with Reviewer's statement of
oversight? Do they understand what does it mean?

> ---
>  MAINTAINERS | 19 +++++++++++++++++++
>  1 file changed, 19 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 90034eb7874e..ecb389795e3d 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -6797,6 +6797,25 @@ F:       kernel/cred.c
>  F:     rust/kernel/cred.rs
>  F:     Documentation/security/credentials.rst
>
> +CRI CRYPTOMANAGER HUB (CMH) HARDWARE CRYPTO ACCELERATOR
> +M:     Alex Ousherovitch <[email protected]>
> +M:     Saravanakrishnan Krishnamoorthy <[email protected]>
> +R:     Joel Wittenauer <[email protected]>
> +R:     Thi Nguyen <[email protected]>
> +L:     [email protected]
> +L:     [email protected] (moderated for non-subscribers)

NAK, drop. You are not allowed to add here internal moderated mailing
lists. We are not going to participate in your corporate dances.

> +S:     Maintained
> +T:     git 
> https://git.kernel.org/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git

Drop, you do not have commit rights there.

> +F:     Documentation/ABI/testing/cmh-mgmt
> +F:     Documentation/ABI/testing/debugfs-driver-cmh
> +F:     Documentation/ABI/testing/sysfs-driver-cmh
> +F:     Documentation/crypto/device_drivers/cmh.rst
> +F:     Documentation/devicetree/bindings/crypto/cri,cmh.yaml
> +F:     Documentation/userspace-api/ioctl/cmh_mgmt.rst
> +F:     drivers/crypto/cmh/
> +F:     include/uapi/linux/cmh_mgmt_ioctl.h
> +F:     tools/testing/selftests/drivers/crypto/cmh/
> +
>  INTEL CRPS COMMON REDUNDANT PSU DRIVER
>  M:     Ninad Palsule <[email protected]>
>  L:     [email protected]
> --
> 2.43.7
>
>
> ** This message and any attachments are for the sole use of the intended 
> recipient(s). It may contain information that is confidential and privileged. 
> If you are not the intended recipient of this message, you are prohibited 
> from printing, copying, forwarding or saving it. Please delete the message 
> and attachments and notify the sender immediately. **

Heh, I should have ignored your message...


Best regards,
Krzysztof


Reply via email to