Re: [PATCH] platform/x86: touchscreen_dmi: Add support for setting touchscreen properties from cmdline

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, May 25, 2024 at 8:41 PM Gregor Riepl <onitake@xxxxxxxxx> wrote:
>
> >> And the kind of refactoring of next_arg() you are asking for here
> >> is way out of scope,
> >
> > Of course, of course, but why introduce Yet Another Parser to begin with?
> >
> >> so sorry but I don't plan to change this part
> >> of the patch.
> >
> > This is not good.
> > But I have no power to stop it, while being very sad about this attitude.
>
> Andy, I do agree with you, but what would you suggest we can do about the situation?

Ask for more help. See below as well.

> Write a generic parser that everyone should use from now on, and then migrate all usage to this parser?

Yeah, that's, if you noticed, what I'm doing (see Git history of
lib/*cmdline*, for example and gpio-aggregator.c) and _that's_ why I'm
not happy about the situation. What I'm trying to say is that we need
to change people's mind about double efforts and duplication.

-- 
With Best Regards,
Andy Shevchenko





[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux