Re: [PATCH] kernel/memremap.c: use ALIGN/ALIGN_DOWN to calculate align_start/end

2019-06-14 Thread Wei Yang
On Thu, Jun 13, 2019 at 11:06:13PM -0700, Dan Williams wrote: >On Thu, Jun 13, 2019 at 11:02 PM Wei Yang >wrote: >> >> The purpose of align_start/end is to expand to SECTION boundary. Use >> ALIGN/ALIGN_DOWN directly is more self-explain and clean. > >I'm actively trying to kill this code [1] so

Re: [PATCH] kernel/memremap.c: use ALIGN/ALIGN_DOWN to calculate align_start/end

2019-06-13 Thread Dan Williams
On Thu, Jun 13, 2019 at 11:02 PM Wei Yang wrote: > > The purpose of align_start/end is to expand to SECTION boundary. Use > ALIGN/ALIGN_DOWN directly is more self-explain and clean. I'm actively trying to kill this code [1] so I don't see the need for this patch. [1]: https://lore.kernel.org/lk

[PATCH] kernel/memremap.c: use ALIGN/ALIGN_DOWN to calculate align_start/end

2019-06-13 Thread Wei Yang
The purpose of align_start/end is to expand to SECTION boundary. Use ALIGN/ALIGN_DOWN directly is more self-explain and clean. Signed-off-by: Wei Yang --- kernel/memremap.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/kernel/memremap.c b/kernel/memremap.c index 1490