LKML Archive on lore.kernel.org help / color / mirror / Atom feed
From: Robert Marko <robert.marko@sartura.hr> To: Andrew Lunn <andrew@lunn.ch> Cc: Linus Walleij <linus.walleij@linaro.org>, Rob Herring <robh@kernel.org>, Bartosz Golaszewski <bgolaszewski@baylibre.com>, Lee Jones <lee.jones@linaro.org>, Philipp Zabel <p.zabel@pengutronix.de>, "open list:GPIO SUBSYSTEM" <linux-gpio@vger.kernel.org>, devicetree <devicetree@vger.kernel.org>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Luka Perkov <luka.perkov@sartura.hr>, "Jonathan M. Polom" <jmp@epiphyte.org>, Paul Menzel <pmenzel@molgen.mpg.de>, Donald Buczek <buczek@molgen.mpg.de> Subject: Re: [PATCH v6 5/6] dt-bindings: mfd: Add Delta TN48M CPLD drivers bindings Date: Tue, 19 Oct 2021 12:49:05 +0200 [thread overview] Message-ID: <CA+HBbNH+4US9U_ZfGZHS2J-pBou2k1LCOAZwuestQHK1GmH0Eg@mail.gmail.com> (raw) In-Reply-To: <YW4iIP7jpRj4qcNN@lunn.ch> On Tue, Oct 19, 2021 at 3:40 AM Andrew Lunn <andrew@lunn.ch> wrote: > > > The SFP driver requires GPIO-s, it only knows how to use GPIO-s, and > > its a generic driver, > > it covers any device that has an SFP port that is implemented per spec. > > So, I cannot just extend it to suit my devices needs and I don't see > > how can I extend it in > > a generic manner so that it controls the pins directly via a regmap > > for example, especially since > > each switch has a different number of SFP ports and thus pins and a > > different register layout. > > > > I have added Andrew Lunn as he is one of the maintainers of PHYLIB > > under which the SFP driver > > is, he may have some input on how to proceed with this. > > > > I honestly think that we have some kind of misunderstanding here and > > look forward to resolving it. > > Hi Robert > > Can you describe your hardware and regmap in a bit more detail. What > do these registers look like? How do they map to the SFP cage pins? Hi Andrew, This board is simple as it only has 4 SFP ports so they have split the respective pins into individual registers per their purpose. So TX disable pins have their own 8bit register and they map pins using individual bits. This is how the register looks: +-----+---------------+-----+-------------------+---------------+ | Bit | Name | R/W | Description | Default value | +-----+---------------+-----+-------------------+---------------+ | 7:4 | Reserved | R/W | Not used | 0 | | 3 | TX_Disable_52 | R/W | Enable/disable | 0 | | 2 | TX_Disable_51 | R/W | SFP TX transmiter | 0 | | 1 | TX_Disable_50 | R/W | 1 = TX off | 0 | | 0 | TX_Disable_49 | R/W | 0 = TX on | 0 | +-----+---------------+-----+-------------------+---------------+ Presence and LOS pins also have their respective registers in the same format. So you can see that the register bits map directly to the SFP cage pins. Regards, Robert > > Andrew -- Robert Marko Staff Embedded Linux Engineer Sartura Ltd. Lendavska ulica 16a 10000 Zagreb, Croatia Email: robert.marko@sartura.hr Web: www.sartura.hr
next prev parent reply other threads:[~2021-10-19 10:49 UTC|newest] Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-06-07 12:33 [PATCH v6 1/6] mfd: simple-mfd-i2c: Add Delta TN48M CPLD support Robert Marko 2021-06-07 12:33 ` [PATCH v6 2/6] gpio: Add Delta TN48M CPLD GPIO driver Robert Marko 2021-06-07 12:33 ` [PATCH v6 3/6] dt-bindings: reset: Add Delta TN48M Robert Marko 2021-06-07 12:33 ` [PATCH v6 4/6] reset: Add Delta TN48M CPLD reset controller Robert Marko 2021-06-07 12:33 ` [PATCH v6 5/6] dt-bindings: mfd: Add Delta TN48M CPLD drivers bindings Robert Marko 2021-06-25 11:46 ` Robert Marko 2021-07-13 22:25 ` Rob Herring 2021-07-18 9:15 ` Robert Marko 2021-07-19 10:46 ` Lee Jones 2021-07-19 22:59 ` Rob Herring 2021-07-21 14:16 ` Linus Walleij 2021-08-03 19:22 ` Robert Marko 2021-08-11 12:17 ` Linus Walleij 2021-08-24 8:03 ` Robert Marko 2021-09-07 21:02 ` Robert Marko 2021-09-25 14:47 ` Robert Marko 2021-10-03 22:48 ` Linus Walleij 2021-10-12 16:31 ` Robert Marko 2021-10-19 1:40 ` Andrew Lunn 2021-10-19 10:49 ` Robert Marko [this message] 2021-10-19 2:05 ` Andrew Lunn 2021-10-19 10:54 ` Robert Marko 2021-06-07 12:33 ` [PATCH v6 6/6] MAINTAINERS: Add Delta Networks TN48M CPLD drivers Robert Marko
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=CA+HBbNH+4US9U_ZfGZHS2J-pBou2k1LCOAZwuestQHK1GmH0Eg@mail.gmail.com \ --to=robert.marko@sartura.hr \ --cc=andrew@lunn.ch \ --cc=bgolaszewski@baylibre.com \ --cc=buczek@molgen.mpg.de \ --cc=devicetree@vger.kernel.org \ --cc=jmp@epiphyte.org \ --cc=lee.jones@linaro.org \ --cc=linus.walleij@linaro.org \ --cc=linux-gpio@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=luka.perkov@sartura.hr \ --cc=p.zabel@pengutronix.de \ --cc=pmenzel@molgen.mpg.de \ --cc=robh@kernel.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: linkBe 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; as well as URLs for NNTP newsgroup(s).