* leds-bcm6328: add more init configuration options
@ 2015-11-15 17:27 Simon Arlott
2015-11-16 15:03 ` Jacek Anaszewski
0 siblings, 1 reply; 3+ messages in thread
From: Simon Arlott @ 2015-11-15 17:27 UTC (permalink / raw)
To: Álvaro Fernández Rojas, Jacek Anaszewski, linux-leds
Cc: Linux Kernel Mailing List, devicetree
commit 1d3ff22d633d35048c35c3deeead7ab116cebc32
Author: Álvaro Fernández Rojas <noltari@gmail.com>
Date: Thu Oct 8 12:35:55 2015 +0200
Documentation: leds: update DT bindings for leds-bcm6328
This patch adds latest DT bindings for leds-bcm6328 init register
configuration.
commit 9f82c778ce17789a038351a411ff65cd41033c7a
Author: Álvaro Fernández Rojas <noltari@gmail.com>
Date: Thu Oct 8 12:35:54 2015 +0200
leds-bcm6328: add more init configuration options
This patch adds more init register configuration options:
- Serial LEDs multiplexing.
- Serial LEDs clock signal low/high polarity.
- Serial LEDs data signal low/high polarity.
- Serial LEDs shift direction inverted/normal.
This was not a backwards-compatible change, because the driver is now
turning off all of the serial LED options by default when previously
it only turned off brcm,serial-leds and left the other options
unchanged.
There are currently no in-tree users of this binding, so any existing
users will be broken by this change.
--
Simon Arlott
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: leds-bcm6328: add more init configuration options
2015-11-15 17:27 leds-bcm6328: add more init configuration options Simon Arlott
@ 2015-11-16 15:03 ` Jacek Anaszewski
2015-11-16 21:35 ` Álvaro Fernández Rojas
0 siblings, 1 reply; 3+ messages in thread
From: Jacek Anaszewski @ 2015-11-16 15:03 UTC (permalink / raw)
To: Simon Arlott
Cc: Álvaro Fernández Rojas, linux-leds,
Linux Kernel Mailing List, devicetree
Hi Simon,
On 11/15/2015 06:27 PM, Simon Arlott wrote:
> commit 1d3ff22d633d35048c35c3deeead7ab116cebc32
> Author: Álvaro Fernández Rojas <noltari@gmail.com>
> Date: Thu Oct 8 12:35:55 2015 +0200
>
> Documentation: leds: update DT bindings for leds-bcm6328
>
> This patch adds latest DT bindings for leds-bcm6328 init register
> configuration.
>
> commit 9f82c778ce17789a038351a411ff65cd41033c7a
> Author: Álvaro Fernández Rojas <noltari@gmail.com>
> Date: Thu Oct 8 12:35:54 2015 +0200
>
> leds-bcm6328: add more init configuration options
>
> This patch adds more init register configuration options:
> - Serial LEDs multiplexing.
> - Serial LEDs clock signal low/high polarity.
> - Serial LEDs data signal low/high polarity.
> - Serial LEDs shift direction inverted/normal.
>
> This was not a backwards-compatible change, because the driver is now
> turning off all of the serial LED options by default when previously
> it only turned off brcm,serial-leds and left the other options
> unchanged.
>
> There are currently no in-tree users of this binding, so any existing
> users will be broken by this change.
>
Please submit a patch fixing the issue in such cases, so that we could
discuss possible solutions to the problem.
--
Best Regards,
Jacek Anaszewski
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: leds-bcm6328: add more init configuration options
2015-11-16 15:03 ` Jacek Anaszewski
@ 2015-11-16 21:35 ` Álvaro Fernández Rojas
0 siblings, 0 replies; 3+ messages in thread
From: Álvaro Fernández Rojas @ 2015-11-16 21:35 UTC (permalink / raw)
To: Jacek Anaszewski, Simon Arlott
Cc: linux-leds, Linux Kernel Mailing List, devicetree
Hi Simon and Jacek,
This is done on purpose because not every BCM63268 device has serial
LEDs and we should be able to enable/disable those options in order to
avoid depending on the bootloader setting the registers correctly.
Regards,
Álvaro.
El 16/11/2015 a las 16:03, Jacek Anaszewski escribió:
> Hi Simon,
>
> On 11/15/2015 06:27 PM, Simon Arlott wrote:
>> commit 1d3ff22d633d35048c35c3deeead7ab116cebc32
>> Author: Álvaro Fernández Rojas <noltari@gmail.com>
>> Date: Thu Oct 8 12:35:55 2015 +0200
>>
>> Documentation: leds: update DT bindings for leds-bcm6328
>>
>> This patch adds latest DT bindings for leds-bcm6328 init register
>> configuration.
>>
>> commit 9f82c778ce17789a038351a411ff65cd41033c7a
>> Author: Álvaro Fernández Rojas <noltari@gmail.com>
>> Date: Thu Oct 8 12:35:54 2015 +0200
>>
>> leds-bcm6328: add more init configuration options
>>
>> This patch adds more init register configuration options:
>> - Serial LEDs multiplexing.
>> - Serial LEDs clock signal low/high polarity.
>> - Serial LEDs data signal low/high polarity.
>> - Serial LEDs shift direction inverted/normal.
>>
>> This was not a backwards-compatible change, because the driver is now
>> turning off all of the serial LED options by default when previously
>> it only turned off brcm,serial-leds and left the other options
>> unchanged.
>>
>> There are currently no in-tree users of this binding, so any existing
>> users will be broken by this change.
>>
>
> Please submit a patch fixing the issue in such cases, so that we could
> discuss possible solutions to the problem.
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-11-16 21:36 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-11-15 17:27 leds-bcm6328: add more init configuration options Simon Arlott
2015-11-16 15:03 ` Jacek Anaszewski
2015-11-16 21:35 ` Álvaro Fernández Rojas
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).