[PATCH v6 0/4] memory: tegra: Add MC error logging support for Tegra264 SoC

Ketan Patil posted 4 patches 1 week ago
drivers/memory/tegra/mc.c       | 112 ++++++---
drivers/memory/tegra/mc.h       | 162 ++++++++----
drivers/memory/tegra/tegra114.c |  16 +-
drivers/memory/tegra/tegra124.c |  36 ++-
drivers/memory/tegra/tegra186.c |  30 ++-
drivers/memory/tegra/tegra194.c |  20 +-
drivers/memory/tegra/tegra20.c  |  33 ++-
drivers/memory/tegra/tegra210.c |  19 +-
drivers/memory/tegra/tegra234.c |  20 +-
drivers/memory/tegra/tegra264.c | 428 +++++++++++++++++++++++++++++++-
drivers/memory/tegra/tegra30.c  |  16 +-
include/soc/tegra/mc.h          |  38 ++-
12 files changed, 785 insertions(+), 145 deletions(-)
[PATCH v6 0/4] memory: tegra: Add MC error logging support for Tegra264 SoC
Posted by Ketan Patil 1 week ago
In Tegra264, different components from memory subsystems like Memory
Controller Fabric (MCF), HUB, HUB Common (HUBC), Side Band Shim (SBS)
and MC Channels have different interrupt lines for receiving memory
controller error interrupts. This patch set includes changes to add
support and enable Memory Controller error logging for Tegra264.

Ketan Patil (4):
  memory: tegra: Group error handling related registers
  memory: tegra: Group register and fields
  memory: tegra: Add support for multiple IRQs
  memory: tegra: Add MC error logging support for Tegra264

---
Changes in v6:
- Updated copyright year and commit message.
- Removed redundant mc_ prefix from few structs.
- Grouped the masks and bits next to the associated registers which
were missed in previous revision.
- Separated out Tegra264 specific registers from previous SoC
registers.
- Reduced variables in mcf_log_fault, hub_log_fault functions.

Changes in v5:
- Updated commit message to clarify the need of grouping of mc-err
registers and support for multiple interrupts, so that it can support
Tegra264.

Changes in v4:
- Fixed typo in variable name
- Added MC instance aperture mapping for hubc registers and corrected
irq handler function to read from respective mc aperture.

Changes in v3:
- Fixed compilation error with W=1 build

Changes in v2:
- Fixed warning due to use of uninitialized variable by returning back
from the function
- Removed unnecessary extern declaration of tegra264_mc_ops
- Updated tegra20_mc_irq_handlers, tegra264_mc_irq_handlers and
tegra264_mc_ops to static

 drivers/memory/tegra/mc.c       | 112 ++++++---
 drivers/memory/tegra/mc.h       | 162 ++++++++----
 drivers/memory/tegra/tegra114.c |  16 +-
 drivers/memory/tegra/tegra124.c |  36 ++-
 drivers/memory/tegra/tegra186.c |  30 ++-
 drivers/memory/tegra/tegra194.c |  20 +-
 drivers/memory/tegra/tegra20.c  |  33 ++-
 drivers/memory/tegra/tegra210.c |  19 +-
 drivers/memory/tegra/tegra234.c |  20 +-
 drivers/memory/tegra/tegra264.c | 428 +++++++++++++++++++++++++++++++-
 drivers/memory/tegra/tegra30.c  |  16 +-
 include/soc/tegra/mc.h          |  38 ++-
 12 files changed, 785 insertions(+), 145 deletions(-)

-- 
2.17.1
Re: [PATCH v6 0/4] memory: tegra: Add MC error logging support for Tegra264 SoC
Posted by Jon Hunter 4 days, 11 hours ago
On 30/01/2026 17:30, Ketan Patil wrote:
> In Tegra264, different components from memory subsystems like Memory
> Controller Fabric (MCF), HUB, HUB Common (HUBC), Side Band Shim (SBS)
> and MC Channels have different interrupt lines for receiving memory
> controller error interrupts. This patch set includes changes to add
> support and enable Memory Controller error logging for Tegra264.
> 
> Ketan Patil (4):
>    memory: tegra: Group error handling related registers
>    memory: tegra: Group register and fields
>    memory: tegra: Add support for multiple IRQs
>    memory: tegra: Add MC error logging support for Tegra264
> 


For the series ...

Reviewed-by: Jon Hunter <jonathanh@nvidia.com>
Tested-by: Jon Hunter <jonathanh@nvidia.com>

Thanks
Jon

-- 
nvpublic