lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <f3552c8a-d8c2-4c0d-9286-20ad95c5937a@linaro.org>
Date: Fri, 27 Jun 2025 10:18:57 +0100
From: James Clark <james.clark@...aro.org>
To: Suzuki K Poulose <suzuki.poulose@....com>,
 linux-arm-kernel@...ts.infradead.org, coresight@...ts.linaro.org
Cc: Russell King <linux@...linux.org.uk>, linux-kernel@...r.kernel.org,
 Linus Walleij <linus.walleij@...aro.org>
Subject: Re: [PATCH] arm: coresight: Finish removal of Coresight support in
 arch/arm/kernel



On 18/03/2025 10:48 am, Suzuki K Poulose wrote:
> On 10/02/2025 11:29, James Clark wrote:
>> Commit 184901a06a36 ("ARM: removing support for etb/etm in
>> "arch/arm/kernel/"") removed asm/hardware/coresight.h which is included
>> by this file. Therefore this is dead code so delete it.
>>
>> Signed-off-by: James Clark <james.clark@...aro.org>
> 
> Acked-by: Suzuki K Poulose <suzuki.poulose@....com>

Gentle ping. Or maybe you can take this in the Coresight tree Suzuki?

> 
>> ---
>>   arch/arm/include/asm/cti.h | 160 -------------------------------------
>>   1 file changed, 160 deletions(-)
>>   delete mode 100644 arch/arm/include/asm/cti.h
>>
>> diff --git a/arch/arm/include/asm/cti.h b/arch/arm/include/asm/cti.h
>> deleted file mode 100644
>> index f8500e5d6ea8..000000000000
>> --- a/arch/arm/include/asm/cti.h
>> +++ /dev/null
>> @@ -1,160 +0,0 @@
>> -/* SPDX-License-Identifier: GPL-2.0 */
>> -#ifndef __ASMARM_CTI_H
>> -#define __ASMARM_CTI_H
>> -
>> -#include    <asm/io.h>
>> -#include    <asm/hardware/coresight.h>
>> -
>> -/* The registers' definition is from section 3.2 of
>> - * Embedded Cross Trigger Revision: r0p0
>> - */
>> -#define        CTICONTROL        0x000
>> -#define        CTISTATUS        0x004
>> -#define        CTILOCK            0x008
>> -#define        CTIPROTECTION        0x00C
>> -#define        CTIINTACK        0x010
>> -#define        CTIAPPSET        0x014
>> -#define        CTIAPPCLEAR        0x018
>> -#define        CTIAPPPULSE        0x01c
>> -#define        CTIINEN            0x020
>> -#define        CTIOUTEN        0x0A0
>> -#define        CTITRIGINSTATUS        0x130
>> -#define        CTITRIGOUTSTATUS    0x134
>> -#define        CTICHINSTATUS        0x138
>> -#define        CTICHOUTSTATUS        0x13c
>> -#define        CTIPERIPHID0        0xFE0
>> -#define        CTIPERIPHID1        0xFE4
>> -#define        CTIPERIPHID2        0xFE8
>> -#define        CTIPERIPHID3        0xFEC
>> -#define        CTIPCELLID0        0xFF0
>> -#define        CTIPCELLID1        0xFF4
>> -#define        CTIPCELLID2        0xFF8
>> -#define        CTIPCELLID3        0xFFC
>> -
>> -/* The below are from section 3.6.4 of
>> - * CoreSight v1.0 Architecture Specification
>> - */
>> -#define        LOCKACCESS        0xFB0
>> -#define        LOCKSTATUS        0xFB4
>> -
>> -/**
>> - * struct cti - cross trigger interface struct
>> - * @base: mapped virtual address for the cti base
>> - * @irq: irq number for the cti
>> - * @trig_out_for_irq: triger out number which will cause
>> - *    the @irq happen
>> - *
>> - * cti struct used to operate cti registers.
>> - */
>> -struct cti {
>> -    void __iomem *base;
>> -    int irq;
>> -    int trig_out_for_irq;
>> -};
>> -
>> -/**
>> - * cti_init - initialize the cti instance
>> - * @cti: cti instance
>> - * @base: mapped virtual address for the cti base
>> - * @irq: irq number for the cti
>> - * @trig_out: triger out number which will cause
>> - *    the @irq happen
>> - *
>> - * called by machine code to pass the board dependent
>> - * @base, @irq and @trig_out to cti.
>> - */
>> -static inline void cti_init(struct cti *cti,
>> -    void __iomem *base, int irq, int trig_out)
>> -{
>> -    cti->base = base;
>> -    cti->irq  = irq;
>> -    cti->trig_out_for_irq = trig_out;
>> -}
>> -
>> -/**
>> - * cti_map_trigger - use the @chan to map @trig_in to @trig_out
>> - * @cti: cti instance
>> - * @trig_in: trigger in number
>> - * @trig_out: trigger out number
>> - * @channel: channel number
>> - *
>> - * This function maps one trigger in of @trig_in to one trigger
>> - * out of @trig_out using the channel @chan.
>> - */
>> -static inline void cti_map_trigger(struct cti *cti,
>> -    int trig_in, int trig_out, int chan)
>> -{
>> -    void __iomem *base = cti->base;
>> -    unsigned long val;
>> -
>> -    val = __raw_readl(base + CTIINEN + trig_in * 4);
>> -    val |= BIT(chan);
>> -    __raw_writel(val, base + CTIINEN + trig_in * 4);
>> -
>> -    val = __raw_readl(base + CTIOUTEN + trig_out * 4);
>> -    val |= BIT(chan);
>> -    __raw_writel(val, base + CTIOUTEN + trig_out * 4);
>> -}
>> -
>> -/**
>> - * cti_enable - enable the cti module
>> - * @cti: cti instance
>> - *
>> - * enable the cti module
>> - */
>> -static inline void cti_enable(struct cti *cti)
>> -{
>> -    __raw_writel(0x1, cti->base + CTICONTROL);
>> -}
>> -
>> -/**
>> - * cti_disable - disable the cti module
>> - * @cti: cti instance
>> - *
>> - * enable the cti module
>> - */
>> -static inline void cti_disable(struct cti *cti)
>> -{
>> -    __raw_writel(0, cti->base + CTICONTROL);
>> -}
>> -
>> -/**
>> - * cti_irq_ack - clear the cti irq
>> - * @cti: cti instance
>> - *
>> - * clear the cti irq
>> - */
>> -static inline void cti_irq_ack(struct cti *cti)
>> -{
>> -    void __iomem *base = cti->base;
>> -    unsigned long val;
>> -
>> -    val = __raw_readl(base + CTIINTACK);
>> -    val |= BIT(cti->trig_out_for_irq);
>> -    __raw_writel(val, base + CTIINTACK);
>> -}
>> -
>> -/**
>> - * cti_unlock - unlock cti module
>> - * @cti: cti instance
>> - *
>> - * unlock the cti module, or else any writes to the cti
>> - * module is not allowed.
>> - */
>> -static inline void cti_unlock(struct cti *cti)
>> -{
>> -    __raw_writel(CS_LAR_KEY, cti->base + LOCKACCESS);
>> -}
>> -
>> -/**
>> - * cti_lock - lock cti module
>> - * @cti: cti instance
>> - *
>> - * lock the cti module, so any writes to the cti
>> - * module will be not allowed.
>> - */
>> -static inline void cti_lock(struct cti *cti)
>> -{
>> -    __raw_writel(~CS_LAR_KEY, cti->base + LOCKACCESS);
>> -}
>> -#endif
> 


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ