Re: [PATCH v2 07/12] memory: tegra: Parameterize interrupt handler

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jun 01, 2021 at 10:37:19PM +0300, Dmitry Osipenko wrote:
> 01.06.2021 21:53, Dmitry Osipenko пишет:
> > 01.06.2021 20:59, Thierry Reding пишет:
> >> --- a/drivers/memory/tegra/mc.h
> >> +++ b/drivers/memory/tegra/mc.h
> >> @@ -135,9 +135,13 @@ extern const struct tegra_mc_soc tegra210_mc_soc;
> >>      defined(CONFIG_ARCH_TEGRA_132_SOC) || \
> >>      defined(CONFIG_ARCH_TEGRA_210_SOC)
> >>  int tegra30_mc_probe(struct tegra_mc *mc);
> >> +irqreturn_t tegra30_mc_handle_irq(int irq, void *data);
> > 
> > tegra30_mc_handle_irq() doesn't need to be made global, it's defined and
> > used only within mc.h.
> > 
> 
> You could make this function static and annotate it with maybe_unused.

This doesn't even need to be __maybe_unused because it's declared in the
same #if/#endif block as tegra30_mc_ops. But yeah, I can make this
static to mc.c. Initially I had thought that this was going to be used
by tegra186_mc_ops as well, but we can always export it if necessary
when that work materializes.

Thanks,
Thierry

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux