Commit 2caf08e7 authored Nov 13, 2020 by Jakub Kicinski

Merge branch 'net-ipa-two-fixes'

Alex Elder says:

====================
net: ipa: two fixes

This small series makes two fixes to the IPA code:
  - While reviewing something else I found that one of the resource
    limits on the SDM845 used the wrong value.  The first patch
    fixes this.  The correct value allocates more resources of this
    type for IPA to use, and otherwise does not change behavior.
  - When the IPA-resident microcontroller starts up it generates an
    event, which triggers an AP interrupt.  The event merely
    provides some information for logging, which we don't support.
    We already ignore the event, and that's harmless.  So this
    patch explicitly ignores it rather than issuing a warning when
    it occurs.
====================

Link: https://lore.kernel.org/r/20201112121157.19784-1-elder@linaro.org


Signed-off-by: Jakub Kicinski <kuba@kernel.org>

parents 992c75ae 0a5096ec

drivers/net/ipa/ipa_data-sdm845.c

+2 −2

Original line number	Diff line number	Diff line
		@@ -150,11 +150,11 @@ static const struct ipa_resource_src ipa_resource_src[] = {
		.type = IPA_RESOURCE_TYPE_SRC_PKT_CONTEXTS,
		.limits[0] = {
		.min = 1,
		.max = 63,
		.max = 255,
		},
		.limits[1] = {
		.min = 1,
		.max = 63,
		.max = 255,
		},
		},
		{

drivers/net/ipa/ipa_uc.c

+2 −1

Original line number	Diff line number	Diff line
		@@ -129,9 +129,10 @@ static void ipa_uc_event_handler(struct ipa *ipa, enum ipa_irq_id irq_id)

		if (shared->event == IPA_UC_EVENT_ERROR)
		dev_err(dev, "microcontroller error event\n");
		else
		else if (shared->event != IPA_UC_EVENT_LOG_INFO)
		dev_err(dev, "unsupported microcontroller event %hhu\n",
		shared->event);
		/* The LOG_INFO event can be safely ignored */
		}

		/* Microcontroller response IPA interrupt handler */

Admin message