| Message ID | 20210308153052.2353885-1-arnd@kernel.org |
|---|---|
| State | New |
| Headers | show |
| Series | leds: lgm: fix gpiolib dependency | expand |
Hi! > > From: Arnd Bergmann <arnd@arndb.de> > > > > Without gpiolib, the driver fails to build: > > > > drivers/leds/blink/leds-lgm-sso.c:123:19: error: field has incomplete type 'struct gpio_chip' > > struct gpio_chip chip; > > ^ > > Thanks, applied. > > I'd like people from Intel to contact me. There's more to fix there, > and AFAICT original author went away. The following message to <mallikarjunax.reddy@linux.intel.com> was undeliverable. The reason for the problem: 5.1.0 - Unknown address error 550-'5.1.1 <mallikarjunax.reddy@linux.intel.com>: Recipient +address rejected: User unknown in virtual mailbox table' commit c3987cd2bca34ddfec69027acedb2fae5ffcf7a0 Author: Amireddy Mallikarjuna reddy <mallikarjunax.reddy@linux.intel.com> Signed-off-by: Amireddy Mallikarjuna reddy <mallikarjunax.reddy@linux.intel.com> If someone knows how to contact the author, that would be welcome. Best regards, Pavel -- http://www.livejournal.com/~pavelmachek
Hi! > > > I'd like people from Intel to contact me. There's more to fix there, > > > and AFAICT original author went away. > > > > The following message to <mallikarjunax.reddy@linux.intel.com> was > > undeliverable. > > > <mallikarjunax.reddy@linux.intel.com>: Recipient > > +address rejected: User unknown in virtual mailbox table' > > > commit c3987cd2bca34ddfec69027acedb2fae5ffcf7a0 > > Author: Amireddy Mallikarjuna reddy <mallikarjunax.reddy@linux.intel.com> > > I asked around, and got told Mallikarjuna has been "sold" to MaxLinear, > together with the rest of the Connected Home Division. So he most likely > still works on this stuff, just under a different banner. > > > If someone knows how to contact the author, that would be welcome. > > Alas, no idea about his MaxLinear address. Thanks for the effort. Anyway, I suspect I'll just do this: BR, Pavel diff --git a/drivers/leds/blink/Kconfig b/drivers/leds/blink/Kconfig index 6dedc58c47b3..79493f21d365 100644 --- a/drivers/leds/blink/Kconfig +++ b/drivers/leds/blink/Kconfig @@ -1,14 +1,6 @@ -menuconfig LEDS_BLINK - bool "LED Blink support" - depends on LEDS_CLASS - help - This option enables blink support for the leds class. - If unsure, say Y. - -if LEDS_BLINK - config LEDS_BLINK_LGM tristate "LED support for Intel LGM SoC series" + depends on BROKEN depends on GPIOLIB depends on LEDS_CLASS depends on MFD_SYSCON @@ -17,5 +9,3 @@ config LEDS_BLINK_LGM Parallel to serial conversion, which is also called SSO controller, can drive external shift register for LED outputs. This enables LED support for Serial Shift Output controller(SSO). - -endif # LEDS_BLINK -- http://www.livejournal.com/~pavelmachek
On Wed, Mar 10, 2021 at 8:30 AM Pavel Machek <pavel@ucw.cz> wrote: > > Hi! > > > > > I'd like people from Intel to contact me. There's more to fix there, > > > > and AFAICT original author went away. > > > > > > The following message to <mallikarjunax.reddy@linux.intel.com> was > > > undeliverable. > > > > > <mallikarjunax.reddy@linux.intel.com>: Recipient > > > +address rejected: User unknown in virtual mailbox table' > > > > > commit c3987cd2bca34ddfec69027acedb2fae5ffcf7a0 > > > Author: Amireddy Mallikarjuna reddy <mallikarjunax.reddy@linux.intel.com> > > > > I asked around, and got told Mallikarjuna has been "sold" to MaxLinear, > > together with the rest of the Connected Home Division. So he most likely > > still works on this stuff, just under a different banner. > > > > > If someone knows how to contact the author, that would be welcome. > > > > Alas, no idea about his MaxLinear address. > > Thanks for the effort. Anyway, I suspect I'll just do this: Maybe Hauke or John (added both to cc) know who at MaxLinear is responsible for maintaining the Lightning Mountain drivers now. Arnd > diff --git a/drivers/leds/blink/Kconfig b/drivers/leds/blink/Kconfig > index 6dedc58c47b3..79493f21d365 100644 > --- a/drivers/leds/blink/Kconfig > +++ b/drivers/leds/blink/Kconfig > @@ -1,14 +1,6 @@ > -menuconfig LEDS_BLINK > - bool "LED Blink support" > - depends on LEDS_CLASS > - help > - This option enables blink support for the leds class. > - If unsure, say Y. > - > -if LEDS_BLINK > - > config LEDS_BLINK_LGM > tristate "LED support for Intel LGM SoC series" > + depends on BROKEN > depends on GPIOLIB > depends on LEDS_CLASS > depends on MFD_SYSCON > @@ -17,5 +9,3 @@ config LEDS_BLINK_LGM > Parallel to serial conversion, which is also called SSO controller, > can drive external shift register for LED outputs. > This enables LED support for Serial Shift Output controller(SSO). > - > -endif # LEDS_BLINK > > > -- > http://www.livejournal.com/~pavelmachek
On 3/10/21 5:57 PM, Arnd Bergmann wrote: > On Wed, Mar 10, 2021 at 8:30 AM Pavel Machek <pavel@ucw.cz> wrote: >> >> Hi! >> >>>>> I'd like people from Intel to contact me. There's more to fix there, >>>>> and AFAICT original author went away. >>>> >>>> The following message to <mallikarjunax.reddy@linux.intel.com> was >>>> undeliverable. >>> >>>> <mallikarjunax.reddy@linux.intel.com>: Recipient >>>> +address rejected: User unknown in virtual mailbox table' >>> >>>> commit c3987cd2bca34ddfec69027acedb2fae5ffcf7a0 >>>> Author: Amireddy Mallikarjuna reddy <mallikarjunax.reddy@linux.intel.com> >>> >>> I asked around, and got told Mallikarjuna has been "sold" to MaxLinear, >>> together with the rest of the Connected Home Division. So he most likely >>> still works on this stuff, just under a different banner. >>> >>>> If someone knows how to contact the author, that would be welcome. >>> >>> Alas, no idea about his MaxLinear address. >> >> Thanks for the effort. Anyway, I suspect I'll just do this: > > Maybe Hauke or John (added both to cc) know who at MaxLinear is > responsible for maintaining the Lightning Mountain drivers now. > > Arnd Hi Qiming, It looks like there are some problems and potential fixes for the led driver Mallikarjuna added to the Linux kernel for LGM some time ago. Could you please take care of this request from Pavel and Arnd. Hauke > >> diff --git a/drivers/leds/blink/Kconfig b/drivers/leds/blink/Kconfig >> index 6dedc58c47b3..79493f21d365 100644 >> --- a/drivers/leds/blink/Kconfig >> +++ b/drivers/leds/blink/Kconfig >> @@ -1,14 +1,6 @@ >> -menuconfig LEDS_BLINK >> - bool "LED Blink support" >> - depends on LEDS_CLASS >> - help >> - This option enables blink support for the leds class. >> - If unsure, say Y. >> - >> -if LEDS_BLINK >> - >> config LEDS_BLINK_LGM >> tristate "LED support for Intel LGM SoC series" >> + depends on BROKEN >> depends on GPIOLIB >> depends on LEDS_CLASS >> depends on MFD_SYSCON >> @@ -17,5 +9,3 @@ config LEDS_BLINK_LGM >> Parallel to serial conversion, which is also called SSO controller, >> can drive external shift register for LED outputs. >> This enables LED support for Serial Shift Output controller(SSO). >> - >> -endif # LEDS_BLINK >> >> >> -- >> http://www.livejournal.com/~pavelmachek
On Thu, Mar 11, 2021 at 6:48 AM Rahul Tanwar <rtanwar@maxlinear.com> wrote: > Hi Arnd, Pavel, > > Sorry for the hiccup due to missing email address in the email chain during the ownership transition. > > Henceforth, I will be the maintainer for all kernel drivers/code related to “formerly Intel’s now MaxLinear’s” Lightning Mountain SoC. > > Please send any Lightning Mountain SoC related issues email to Rahul Tanwar (rtanwar@maxlinear.com) and I will ensure that I address the issues in a timely manner. Thank you for the reply and for stepping up as maintainer. I tend to merge updates to the MAINTAINERS file as bugfixes the file contains the correct addresses at all times. If you sent an update for this to soc@kernel.org, I'll get that merged. Since I think this is an x86 platform, you can alternatively send the same patch to the x86 maintainers. Are you also planning to maintain or add drivers for some of the older SoC generations from Ti/Lantiq/Infineon/Intel that are now owned by MaxLinear? It would be good to be explicit about which ones of these you are working with. From what I can tell, the arch/mips/lantiq/ platform is only for fairly old designs (xrx200 and older), while support for the slightly later mips and x86 based chips was submitted a few years ago but never merged. > I will wait for more details on your fix request for LGM LED driver. Thanks. From my side, only an Ack on the original bugfix I sent [1] is needed, but Pavel had other concerns about the driver. I expect he will follow up on those with you. Arnd [1] https://lore.kernel.org/lkml/20210308153052.2353885-1-arnd@kernel.org/
Hi! > Sorry for the hiccup due to missing email address in the email chain during the ownership transition. > > Henceforth, I will be the maintainer for all kernel drivers/code related to “formerly Intel’s now MaxLinear’s” Lightning Mountain SoC. > > Please send any Lightning Mountain SoC related issues email to Rahul Tanwar (rtanwar@maxlinear.com<mailto:rtanwar@maxlinear.com>) and I will ensure that I > address the issues in a timely manner. > > I will wait for more details on your fix request for LGM LED > driver. Thanks. Please review/Ack arnd's change. Plus I'd like you to 1) remove LEDS_BLINK Kconfig symbol 2) provide useful Kconfig help text for your driver. Thank you, Pavel -- http://www.livejournal.com/~pavelmachek
Hi Arnd, On 11/3/2021 6:01 pm, Arnd Bergmann wrote: > This email was sent from outside of MaxLinear. > > On Thu, Mar 11, 2021 at 6:48 AM Rahul Tanwar <rtanwar@maxlinear.com> wrote: > > Hi Arnd, Pavel, > > > > Sorry for the hiccup due to missing email address in the email chain > during the ownership transition. > > > > Henceforth, I will be the maintainer for all kernel drivers/code > related to “formerly Intel’s now MaxLinear’s” Lightning Mountain SoC. > > > > Please send any Lightning Mountain SoC related issues email to Rahul > Tanwar (rtanwar@maxlinear.com) and I will ensure that I address the > issues in a timely manner. > > Thank you for the reply and for stepping up as maintainer. > > I tend to merge updates to the MAINTAINERS file as bugfixes the file > contains > the correct addresses at all times. If you sent an update for this to > soc@kernel.org, > I'll get that merged. > > Since I think this is an x86 platform, you can alternatively send the > same patch > to the x86 maintainers. > Haven't sent any patch to update MAINTAINERS file yet. Thanks for your suggestion. I will take it up soon.. > Are you also planning to maintain or add drivers for some of the older SoC > generations from Ti/Lantiq/Infineon/Intel that are now owned by MaxLinear? > It would be good to be explicit about which ones of these you are working > with. From what I can tell, the arch/mips/lantiq/ platform is only for > fairly > old designs (xrx200 and older), while support for the slightly later mips > and x86 based chips was submitted a few years ago but never merged. > Thank you for asking. At this point of time, our focus for upstreaming and maintaining is on LGM(Lightning Mountain) SoC. Regards, Rahul > > I will wait for more details on your fix request for LGM LED driver. > Thanks. > > From my side, only an Ack on the original bugfix I sent [1] is needed, but > Pavel had other concerns about the driver. I expect he will follow up on > those > with you. > > Arnd > > [1] > https://lore.kernel.org/lkml/20210308153052.2353885-1-arnd@kernel.org/ > <https://lore.kernel.org/lkml/20210308153052.2353885-1-arnd@kernel.org>
diff --git a/drivers/leds/blink/Kconfig b/drivers/leds/blink/Kconfig index 265b53476a80..6dedc58c47b3 100644 --- a/drivers/leds/blink/Kconfig +++ b/drivers/leds/blink/Kconfig @@ -9,6 +9,7 @@ if LEDS_BLINK config LEDS_BLINK_LGM tristate "LED support for Intel LGM SoC series" + depends on GPIOLIB depends on LEDS_CLASS depends on MFD_SYSCON depends on OF