Re: [RFC v3 18/19] of: unittest: split out a couple of test cases from unittest

From: Frank Rowand
Date: Tue Dec 04 2018 - 05:58:44 EST


Hi Brendan,

On 11/28/18 11:36 AM, Brendan Higgins wrote:
> Split out a couple of test cases that these features in base.c from the
> unittest.c monolith. The intention is that we will eventually split out
> all test cases and group them together based on what portion of device
> tree they test.

Why does splitting this file apart improve the implementation?


>
> Signed-off-by: Brendan Higgins <brendanhiggins@xxxxxxxxxx>
> ---
> drivers/of/Makefile | 2 +-
> drivers/of/base-test.c | 214 ++++++++++++++++++++++++++
> drivers/of/test-common.c | 149 ++++++++++++++++++
> drivers/of/test-common.h | 16 ++
> drivers/of/unittest.c | 316 +--------------------------------------
> 5 files changed, 381 insertions(+), 316 deletions(-)
> create mode 100644 drivers/of/base-test.c
> create mode 100644 drivers/of/test-common.c
> create mode 100644 drivers/of/test-common.h
>
< snip >

-Frank