Re: [PATCH] bus: fsl-mc: Remove deadcode

Hi Greg,

* Ioana Ciornei (ioana.ciornei@nxp.com) wrote:

On Thu, Mar 13, 2025 at 02:37:56PM +0100, Christophe Leroy wrote:

quoted

Le 13/03/2025 à 14:21, Dr. David Alan Gilbert a écrit :

quoted

[Vous ne recevez pas souvent de courriers de linux@treblig.org. Découvrez pourquoi ceci est important à https://aka.ms/LearnAboutSenderIdentification ]

* Christophe Leroy (christophe.leroy@csgroup.eu) wrote:

quoted

Le 15/11/2024 à 16:20, linux@treblig.org a écrit :

quoted

[Vous ne recevez pas souvent de courriers de linux@treblig.org. Découvrez pourquoi ceci est important à https://aka.ms/LearnAboutSenderIdentification ]

From: "Dr. David Alan Gilbert" <redacted>

fsl_mc_allocator_driver_exit() was added explicitly by
commit 1e8ac83b6caf ("bus: fsl-mc: add fsl_mc_allocator cleanup function")
but was never used.

Remove it.

fsl_mc_portal_reset() was added in 2015 by
commit 197f4d6a4a00 ("staging: fsl-mc: fsl-mc object allocator driver")
but was never used.

Remove it.

fsl_mc_portal_reset() was the only caller of dpmcp_reset().

Remove it.

Signed-off-by: Dr. David Alan Gilbert <redacted>

Acked-by: Christophe Leroy <redacted>

Hi,
   Can someone pick this old change up please?  I see the PPC patchwork says
   'handled elsewhere' but doesn't say where.

MAINTAINERS file says where:

QORIQ DPAA2 FSL-MC BUS DRIVER
M:	Stuart Yoder [off-list ref]
M:	Laurentiu Tudor [off-list ref]
L:	linux-kernel@vger.kernel.org
S:	Maintained
F:	Documentation/ABI/stable/sysfs-bus-fsl-mc
F:	Documentation/devicetree/bindings/misc/fsl,qoriq-mc.txt
F:
Documentation/networking/device_drivers/ethernet/freescale/dpaa2/overview.rst
F:	drivers/bus/fsl-mc/
F:	include/uapi/linux/fsl_mc.h

FREESCALE SOC DRIVERS
M:	Christophe Leroy [off-list ref]
L:	linuxppc-dev@lists.ozlabs.org
L:	linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
S:	Maintained
F:	Documentation/devicetree/bindings/misc/fsl,dpaa2-console.yaml
F:	Documentation/devicetree/bindings/soc/fsl/
F:	drivers/soc/fsl/
F:	include/linux/fsl/
F:	include/soc/fsl/

I acked the 2 line changes in include/linux/fsl/mc.h, the main changes being
in the C files which are not under my scope.

Stuart, Laurentiu, can you pick up the patch ?

Stuart and Laurentiu are no longer at NXP. Even when they handled the
fsl-mc bus driver, they didn't have a tree themselves to pick patches up
but rather, historically, patches on the fsl-mc bus were picked by Greg
KH.

OK, copied Greg in.

Dave

Ioana

-- 
 -----Open up your eyes, open up your mind, open up your code -------   
/ Dr. David Alan Gilbert    |       Running GNU/Linux       | Happy  \ 
\        dave @ treblig.org |                               | In Hex /
 \ _________________________|_____ http://www.treblig.org   |_______/