The Linux Kernel Mailing List
 help / color / mirror / Atom feed
From: Jonathan Cameron <jic23@kernel.org>
To: Joshua Crofts <joshua.crofts1@gmail.com>
Cc: dylanliu050903 <dylanliu050903@protonmail.com>,
	"Andy Shevchenko" <andriy.shevchenko@intel.com>,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	"Dylan Liu" <pxyforeverlove1314@gmail.com>,
	"Nuno Sá" <nuno.sa@analog.com>,
	"Michael Hennerich" <Michael.Hennerich@analog.com>,
	"David Lechner" <dlechner@baylibre.com>,
	"Andy Shevchenko" <andy@kernel.org>,
	linux@analog.com, linux-iio@vger.kernel.org,
	linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org
Subject: Re: 回复: Re: 回复: Re: [PATCH] Signed-off-by: Dylan Liu <dylanliu050903@protonmail.com>
Date: Mon, 13 Jul 2026 01:59:54 +0100	[thread overview]
Message-ID: <20260713015954.3ea787b2@jic23-huawei> (raw)
In-Reply-To: <20260712184602.0c07aab0@systembl0wer>

On Sun, 12 Jul 2026 18:46:02 +0200
Joshua Crofts <joshua.crofts1@gmail.com> wrote:

> On Sun, 12 Jul 2026 15:41:54 +0000
> dylanliu050903 <dylanliu050903@protonmail.com> wrote:
> > Hi Andy,
> > 
> > First, I apologize for the HTML email earlier. I have fixed my email client settings.I was using the ProtonMail web client instead of `git send-email`, which unexpectedly defaulted to HTML.
> > 
> > Regarding the ABI change: I made a classic beginner mistake. I changed it purely because the `checkpatch.pl` script threw a warning suggesting it should be in snake_case. I blindly followed the script's advice without realizing that I was modifying a user/kernel ABI. 
> > 
> > I now understand that ABI stability (and aligning with IIO maintainers for any necessary staging transitions) absolutely takes precedence over resolving style warnings.
> > 
> > Please drop this patch. I will be much more careful with checkpatch warnings involving exposed interfaces in the future.I will also dive deeper into the codebase to ensure my future commits provide real value.
> > 
> > Thank you for your time and the guidance!
> > 
> > Best regards,
> > Dylan Liu
> >   
>

> Hi Dylan,
> 
> Please wrap your email body to 72 characters per line, as you would
> with a commit message. It's easier to read.
> 
> Thanks!

For any of the drivers in staging it is worth taking a look at lore.kernel.org
and seeing what has been proposed in the past. IIRC, this one has been the subject
of more than one attempt to clean it up. The userspace ABI is a long way with
compliant with IIO ABI. The challenge with sorting that out is that the changes
are invasive so you'd need to figure out how to test them.

Jonathan


      reply	other threads:[~2026-07-13  1:00 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-12 14:19 [PATCH] Signed-off-by: Dylan Liu <dylanliu050903@protonmail.com> Dylan Liu
2026-07-12 14:25 ` Greg Kroah-Hartman
2026-07-12 14:25 ` Greg Kroah-Hartman
2026-07-12 14:56   ` 回复: " dylanliu050903
2026-07-12 15:05     ` Andy Shevchenko
2026-07-12 15:41       ` 回复: " dylanliu050903
2026-07-12 16:46         ` Joshua Crofts
2026-07-13  0:59           ` Jonathan Cameron [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260713015954.3ea787b2@jic23-huawei \
    --to=jic23@kernel.org \
    --cc=Michael.Hennerich@analog.com \
    --cc=andriy.shevchenko@intel.com \
    --cc=andy@kernel.org \
    --cc=dlechner@baylibre.com \
    --cc=dylanliu050903@protonmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=joshua.crofts1@gmail.com \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-staging@lists.linux.dev \
    --cc=linux@analog.com \
    --cc=nuno.sa@analog.com \
    --cc=pxyforeverlove1314@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox