Linux IIO development
 help / color / mirror / Atom feed
From: Joshua Crofts <joshua.crofts1@gmail.com>
To: Krzysztof Kozlowski <krzk@kernel.org>
Cc: David Lechner <dlechner@baylibre.com>,
	linux-iio@vger.kernel.org, jic23@kernel.org, andy@kernel.org,
	nuno.sa@analog.com
Subject: Re: [RFC] Maintainer entry profile/contributor guide for IIO
Date: Tue, 22 Sep 2026 16:54:17 +0200	[thread overview]
Message-ID: <20260922165417.00007e51@gmail.com> (raw)
In-Reply-To: <4c87fa3e-170e-45e3-b72e-fb1a6ee1a7d5@kernel.org>

On Sat, 19 Sep 2026 20:53:36 +0200
Krzysztof Kozlowski <krzk@kernel.org> wrote:
> >> Maintainers file already defines git tree. Patchwork as well. Please
> >> read existing docs first, because it seems you propose to duplicate it
> >> (including submitting patches and other process documents).
> >>
> >> Subsystem profiles are expected to document things which are done here a
> >> bit differently or specific subsystem expectations, narrowing general
> >> kernel process docs.
> >>
> >>  
> >>> - code style - the TODO is fine for existing problems in the subsystem but doesn't
> >>>   point out idioms we have in IIO, i.e. not using (the awful) kernel.h, preferring
> >>>   devm_* functions, not failing on a mismatched ID to ensure fallback etc. This is
> >>>   stuff that appears a lot in patches.    
> >>
> >> You just described standard kernel practice. Don't create documents just
> >> for sake of creating them.  
> > 
> > I'd like you to point me to a doc that says that fallbacks are a thing
> > and you shouldn't do dev_err_probe() on a bad ID (to name one "quirk").
> > 
> > Otherwise I agree that some of it is repetitive.  
> 
> You wrote devm and kernel.h...

I was referring to the next paragraph. Yeah, perhaps it's not the greatest
idea to add new docs if people don't read them. Someone should instead come
up with a way to *force* newcomers to read documentation instead.

-- 
Kind regards,
Joshua Crofts

  reply	other threads:[~2026-09-22 14:54 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-17  9:18 [RFC] Maintainer entry profile/contributor guide for IIO Joshua Crofts
2026-08-17 18:50 ` Maxwell Doose
2026-08-18  7:22   ` Joshua Crofts
2026-08-18  0:31 ` David Lechner
2026-08-18  7:06   ` Joshua Crofts
2026-09-19  8:38     ` Krzysztof Kozlowski
2026-09-19  9:35       ` Joshua Crofts
2026-09-19 18:53         ` Krzysztof Kozlowski
2026-09-22 14:54           ` Joshua Crofts [this message]
2026-09-21  9:28     ` Nuno Sá
2026-09-22 14:51       ` Joshua Crofts
2026-09-22 15:03         ` Nuno Sá
2026-09-22 15:07           ` Joshua Crofts
2026-09-22 15:44             ` Nuno Sá
2026-09-22 15:48               ` Joshua Crofts

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=20260922165417.00007e51@gmail.com \
    --to=joshua.crofts1@gmail.com \
    --cc=andy@kernel.org \
    --cc=dlechner@baylibre.com \
    --cc=jic23@kernel.org \
    --cc=krzk@kernel.org \
    --cc=linux-iio@vger.kernel.org \
    --cc=nuno.sa@analog.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