| Message ID | 20200309085806.155823-3-ravi.bangoria@linux.ibm.com (mailing list archive) |
|---|---|
| State | Changes Requested |
| Headers | show |
| Series | powerpc/watchpoint: Preparation for more than one watchpoint | expand |
| Context | Check | Description |
|---|---|---|
| snowpatch_ozlabs/apply_patch | success | Successfully applied on branch powerpc/merge (ab326587bb5fb91cc97df9b9f48e9e1469f04621) |
| snowpatch_ozlabs/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 14 lines checked |
| snowpatch_ozlabs/needsstable | success | Patch has no Fixes tags |
Le 09/03/2020 à 09:57, Ravi Bangoria a écrit : > Future Power architecture is introducing second DAWR. Add SPRN_ macros > for the same. I'm not sure this is called 'macros'. For me a macro is something more complex. For me those are 'constants'. Christophe > > Signed-off-by: Ravi Bangoria <ravi.bangoria@linux.ibm.com> > --- > arch/powerpc/include/asm/reg.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/arch/powerpc/include/asm/reg.h b/arch/powerpc/include/asm/reg.h > index 156ee89fa9be..062e74cf41fd 100644 > --- a/arch/powerpc/include/asm/reg.h > +++ b/arch/powerpc/include/asm/reg.h > @@ -284,6 +284,7 @@ > #define CTRL_TE 0x00c00000 /* thread enable */ > #define CTRL_RUNLATCH 0x1 > #define SPRN_DAWR0 0xB4 > +#define SPRN_DAWR1 0xB5 > #define SPRN_RPR 0xBA /* Relative Priority Register */ > #define SPRN_CIABR 0xBB > #define CIABR_PRIV 0x3 > @@ -291,6 +292,7 @@ > #define CIABR_PRIV_SUPER 2 > #define CIABR_PRIV_HYPER 3 > #define SPRN_DAWRX0 0xBC > +#define SPRN_DAWRX1 0xBD > #define DAWRX_USER __MASK(0) > #define DAWRX_KERNEL __MASK(1) > #define DAWRX_HYP __MASK(2) >
On Tue, Mar 17, 2020 at 11:16:34AM +0100, Christophe Leroy wrote: > > > Le 09/03/2020 à 09:57, Ravi Bangoria a écrit : > >Future Power architecture is introducing second DAWR. Add SPRN_ macros > >for the same. > > I'm not sure this is called 'macros'. For me a macro is something more > complex. It is called "macros" in the C standard, and in common usage as well. "Object-like macros", as opposed to "function-like macros": there are no arguments. > For me those are 'constants'. That would be more like "static const" in C since 1990 ;-) Segher
diff --git a/arch/powerpc/include/asm/reg.h b/arch/powerpc/include/asm/reg.h index 156ee89fa9be..062e74cf41fd 100644 --- a/arch/powerpc/include/asm/reg.h +++ b/arch/powerpc/include/asm/reg.h @@ -284,6 +284,7 @@ #define CTRL_TE 0x00c00000 /* thread enable */ #define CTRL_RUNLATCH 0x1 #define SPRN_DAWR0 0xB4 +#define SPRN_DAWR1 0xB5 #define SPRN_RPR 0xBA /* Relative Priority Register */ #define SPRN_CIABR 0xBB #define CIABR_PRIV 0x3 @@ -291,6 +292,7 @@ #define CIABR_PRIV_SUPER 2 #define CIABR_PRIV_HYPER 3 #define SPRN_DAWRX0 0xBC +#define SPRN_DAWRX1 0xBD #define DAWRX_USER __MASK(0) #define DAWRX_KERNEL __MASK(1) #define DAWRX_HYP __MASK(2)
Future Power architecture is introducing second DAWR. Add SPRN_ macros for the same. Signed-off-by: Ravi Bangoria <ravi.bangoria@linux.ibm.com> --- arch/powerpc/include/asm/reg.h | 2 ++ 1 file changed, 2 insertions(+)