All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paulo Miguel Almeida <paulo.miguel.almeida.rodenas@gmail.com>
To: kernelnewbies@kernelnewbies.org
Cc: paulo.miguel.almeida.rodenas@gmail.com
Subject: ioctl number change / backwards compatibility doubt
Date: Mon, 17 Jan 2022 20:01:25 +1300	[thread overview]
Message-ID: <20220117070125.GA17186@mail.google.com> (raw)

Hi everyone,

Context:

I've been working on a driver called pi433 in the staging area and it 
basically exposes a char device so the user can read/write stuff to
it while obtaining tx/rx configuration via ioctl calls.

Tx/Rx configurations are both structs that (ideally) would be exposed
to the userspace to let the developer to #include it on their code.

Info that *might* be relevant about this driver:

- This driver went straight to the staging area due to a few TODOs
  listed by the original author. 
- The ioctl Code and Seq numbers are conflicting with the ones listed
  in the ioctl-numbers.rst

Problem:

I realized that one of the structs used to pass/retrieve info needs
to have some of its members changed (data type and etc)

Questions:

1: Given the driver's history and ioctl number conflit, is the backwards
compatibility something to be kept or not to be taken into consideration
as ioctl numbering rules weren't followed?

2: The original author lists on the TODO file of the driver that 'he is
afraid that using ioctl wasn't a good idea'. I pondered the alternatives
and, *in case I can get rid of ioctl*, sysfs || configfs could be used. Does
anyone suggests a different approach?

Best regards,

Paulo Almeida


_______________________________________________
Kernelnewbies mailing list
Kernelnewbies@kernelnewbies.org
https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies

             reply	other threads:[~2022-01-17  7:03 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-17  7:01 Paulo Miguel Almeida [this message]
2022-01-17 12:58 ` ioctl number change / backwards compatibility doubt Greg KH
2022-01-23  7:55   ` Paulo Miguel Almeida
2022-01-23 11:04     ` Greg KH
2022-01-24  4:49       ` Paulo Miguel Almeida
2022-01-24  6:20         ` Greg KH
2022-03-12  0:05           ` Paulo Miguel Almeida
2022-03-14 12:30             ` Rogério Valentim Feitoza da Silva
2022-03-16 14:07             ` Greg KH

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=20220117070125.GA17186@mail.google.com \
    --to=paulo.miguel.almeida.rodenas@gmail.com \
    --cc=kernelnewbies@kernelnewbies.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.