linux-kernel - Re: leds-bcm6328: add more init configuration options

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 16 Nov 2015 16:03:28 +0100
From:	Jacek Anaszewski <j.anaszewski@...sung.com>
To:	Simon Arlott <simon@...e.lp0.eu>
Cc:	Álvaro Fernández Rojas <noltari@...il.com>,
	linux-leds@...r.kernel.org,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	devicetree@...r.kernel.org
Subject: Re: leds-bcm6328: add more init configuration options

Hi Simon,

On 11/15/2015 06:27 PM, Simon Arlott wrote:
> commit 1d3ff22d633d35048c35c3deeead7ab116cebc32
> Author: Álvaro Fernández Rojas <noltari@...il.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@...il.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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/