[PATCH v10 4/7] cdx: add MCDI protocol interface for firmware interaction

[Gupta, Nipun]

> -----Original Message-----
> From: linux-arm-kernel <linux-arm-kernel-bounces at lists.infradead.org> On
> Behalf Of Geert Uytterhoeven
> Sent: Friday, April 28, 2023 1:44 PM
> To: Gupta, Nipun <Nipun.Gupta at amd.com>
> Cc: robh+dt at kernel.org; krzysztof.kozlowski+dt at linaro.org;
> gregkh at linuxfoundation.org; rafael at kernel.org; eric.auger at redhat.com;
> alex.williamson at redhat.com; cohuck at redhat.com;
> song.bao.hua at hisilicon.com; mchehab+huawei at kernel.org; maz at kernel.org;
> f.fainelli at gmail.com; jeffrey.l.hugo at gmail.com; saravanak at google.com;
> Michael.Srba at seznam.cz; mani at kernel.org; yishaih at nvidia.com;
> jgg at ziepe.ca; jgg at nvidia.com; robin.murphy at arm.com; will at kernel.org;
> joro at 8bytes.org; masahiroy at kernel.org; ndesaulniers at google.com;
> rdunlap at infradead.org; baolu.lu at linux.intel.com; linux-arm-
> kernel at lists.infradead.org; linux-kbuild at vger.kernel.org; linux-
> kernel at vger.kernel.org; devicetree at vger.kernel.org; okaya at kernel.org;
> Anand, Harpreet <harpreet.anand at amd.com>; Agarwal, Nikhil
> <nikhil.agarwal at amd.com>; Simek, Michal <michal.simek at amd.com>; Jansen
> Van Vuuren, Pieter <pieter.jansen-van-vuuren at amd.com>; Cascon, Pablo
> <pablo.cascon at amd.com>; git (AMD-Xilinx) <git at amd.com>
> Subject: Re: [PATCH v10 4/7] cdx: add MCDI protocol interface for firmware
> interaction
> 
> 
> Hi Nipun,
> 
> On Mon, Mar 13, 2023 at 2:28 PM Nipun Gupta <nipun.gupta at amd.com>
> wrote:
> > The MCDI (Management CPU Driver Interface) is used as a
> > protocol to communicate with the RPU firmware. It has
> > pre-defined set of messages for different message exchanges
> > between APU and RPU.
> >
> > Signed-off-by: Puneet Gupta <puneet.gupta at amd.com>
> > Signed-off-by: Nipun Gupta <nipun.gupta at amd.com>
> > Signed-off-by: Tarak Reddy <tarak.reddy at amd.com>
> > Reviewed-by: Pieter Jansen van Vuuren <pieter.jansen-van-vuuren at amd.com>
> > Tested-by: Nikhil Agarwal <nikhil.agarwal at amd.com>
> 
> Thanks for your patch, which is now commit eb96b740192b2a09 ("cdx:
> add MCDI protocol interface for firmware interaction") upstream.
> 
> > --- /dev/null
> > +++ b/drivers/cdx/controller/Kconfig
> > @@ -0,0 +1,20 @@
> > +# SPDX-License-Identifier: GPL-2.0-only
> > +#
> > +# CDX controller configuration
> > +#
> > +# Copyright (C) 2022-2023, Advanced Micro Devices, Inc.
> > +#
> > +
> > +if CDX_BUS
> > +
> > +config MCDI_LOGGING
> > +       bool "MCDI Logging for the CDX controller"
> > +       depends on CDX_CONTROLLER
> > +       help
> > +         Enable MCDI Logging for
> > +         the CDX Controller for debug
> > +         purpose.
> > +
> > +         If unsure, say N.
> > +
> > +endif
> 
> As there also exist config symbols SFC_MCDI_LOGGING and
> SFC_SIENA_MCDI_LOGGING, perhaps MCDI_LOGGING is too generic, and
> should
> be renamed to CDX_MCDI_LOGGING?

Sure, we will add a patch to rename this.

Thanks,
Nipun

> 
> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-
> m68k.org
> 
> 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
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel