Messages in this thread | | | From | David Howells <> | Subject | Re: [PATCH V7 00/41] Intel(R) Resource Director Technology Cache Pseudo-Locking enabling | Date | Sun, 24 Jun 2018 00:14:54 +0100 |
| |
Thomas Gleixner <tglx@linutronix.de> wrote:
> - Stick the struct into the local header file and not at some random place in > the source.
Why? It's only used in that one file. There doesn't seem to be any particular need to share it around.
> - Get rid of the fugly camel case
It's not camel case. It's the name of the option it's encoding. But if it makes you happy...
> - Move the enablement into a separate function
Okay.
David
| |