Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Date:   Wed, 8 Jun 2022 11:51:03 +0200 (CEST)
From:   Jiri Kosina <jikos@kernel.org>
To:     =?ISO-8859-15?Q?Jos=E9_Exp=F3sito?= <jose.exposito89@gmail.com>
cc:     benjamin.tissoires@redhat.com, spbnick@gmail.com,
        linux-input@vger.kernel.org, linux-kernel@vger.kernel.org,
        stefanberzl@gmail.com, dlatypov@google.com,
        kunit-dev@googlegroups.com
Subject: Re: [PATCH v3 0/4] Add support for XP-PEN Deco L
In-Reply-To: <20220606101237.GA939277@elementary>
Message-ID: <nycvar.YFH.7.76.2206081150060.10851@cbobk.fhfr.pm>
References: <20220531181246.190729-1-jose.exposito89@gmail.com> <20220606101237.GA939277@elementary>
User-Agent: Alpine 2.21 (LSU 202 2017-01-01)
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-2
Content-Transfer-Encoding: 8BIT
Precedence: bulk

On Mon, 6 Jun 2022, Jos? Exp?sito wrote:

> > This patchset adds support for the UGEE XP-PEN Deco L tablet.
> > This device belongs to a family of tablets that must receive a chunk of
> > magic data in order to be enabled (named UGEE v2 internally).
> >
> > [...]
> >
> > ---
> > 
> > v1 - Initial version
> > https://lore.kernel.org/linux-input/20220518224702.1409327-1-jose.exposito89@gmail.com/
> > 
> > v2 - Fix some errors reported by the kernel test robot:
> > 
> >  - Add MODULE_DESCRIPTION/LICENSE/AUTHOR to hid-uclogic-rdesc-test.c
> >  - Remove unused variable in uclogic_params_ugee_v2_init
> >  - Add sparse __force anotation in the result of cpu_to_le16
> >  - Add DISABLE_STRUCTLEAK_PLUGIN for KUnit tests
> > 
> > https://lore.kernel.org/linux-input/20220518224702.1409327-1-jose.exposito89@gmail.com/T/
> > 
> > v3 - This version:
> > 
> >  - Improve Kconfig message
> >  - Add .kunitconfig. Thanks to Daniel Latypov for suggesting it in an
> >    unrelated patch:
> >    https://lore.kernel.org/dri-devel/CAGS_qxpV2SsihEdgXZ6+7N0dxLmdRANq+qE4iUZ2aNrf6vuLYg@mail.gmail.com/
> 
> As mentioned in the v3 changelog, I'm working on a DRM patch including
> a few KUnit tests [1].
> 
> Some interesting highlights from the comments there:
> 
> - It looks like the maintainers prefer to include a single symbol [2]
> to enable all tests in the subsystem.
> 
> Ji??, Benjamin, how would you like to handle it? Would you prefer to
> have a symbol for each test (HID_UCLOGIC_KUNIT_TEST)? Or a symbol for
> all tests (HID_KUNIT_TEST)?

I don't really see any benefit in breaking it down to per-driver tests, so 
yes, please hide it all under one unified HID_KUNIT_TEST as far as I am 
concerned.

Thanks,

-- 
Jiri Kosina
SUSE Labs