[lkml]   [2021]   [Jun]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH v4 1/2] dt-bindings: gpio-mux-input: add documentation

On 1.6.2021 13.44, Linus Walleij wrote:
> On Sun, May 30, 2021 at 6:16 PM Mauri Sandberg <> wrote:
>> Add documentation for a general GPIO multiplexer.
>> Signed-off-by: Mauri Sandberg <>
>> Tested-by: Drew Fustini <>
>> Reviewed-by: Drew Fustini <>
> After some thinking I realized these bindings should not
> be restricted to just input. There exist electronic constructions
> such as open drain that would make it possible to mux also
> outputs.
>> .../bindings/gpio/gpio-mux-input.yaml | 75 +++++++++++++++++++
> Rename it just gpio-mux.yaml
>> +$id:
> Also here
>> +title: Generic GPIO input multiplexer
> Generic GPIO multiplexer
>> +description: |
>> + A generic GPIO based input multiplexer
> Not just input
>> + This driver uses a mux-controller to drive the multiplexer and has a single
>> + output pin for reading the inputs to the mux.
> Make this clearer and do not mention "driver".
> Here is a suggestion:
> This hardware construction multiplexes (cascades) several GPIO
> lines from one-to-many using a software controlled multiplexer.
> The most common use case is probably reading several inputs
> by switching the multiplexer over several input lines, which in
> practice works well since input lines has high impedance.
> Constructions with multiplexed outputs are also possible using
> open drain electronics.
>> + For GPIO consumer documentation see gpio.txt.
> No need to mention this I think, not your problem :D
>> + pin-gpios:
> I still want this renamed like in my previous mail.
> Hope all is clear!
> Yours,
> Linus Walleij

Hi and thanks the  comments.

Generally I agree with everything you noted above and elsewhere and will
make changes
accordingly. But there is a small detail that needs to be sorted out.
The name 'gpio-mux'
has already been taken by 'mux-gpio' driver [2] [3].

Should we look for another name for this driver and it's bindings or
refactor the mux-gpio's bindings
first? I would be inclined to do the latter as the config symbol for
mux-gpio is the same way around,

The bindings for mux-gpio need to be converted to .yaml anyhow and maybe
the issues with the schema
that Rob pointed out elsewhere would go away too. Otherwise I cannot
really say what's wrong as the
errors look unrelated to me.

-- Mauri


 \ /
  Last update: 2021-06-02 11:33    [W:0.074 / U:0.288 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site