MIME-version: 1.0
Content-type: text/plain; charset=UTF-8; format=flowed
Content-transfer-encoding: 8BIT
Message-id: <5649F040.6090002@samsung.com>
Date: Mon, 16 Nov 2015 16:03:28 +0100
From: Jacek Anaszewski <j.anaszewski@samsung.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130804
 Thunderbird/17.0.8
To: Simon Arlott <simon@fire.lp0.eu>
Cc: =?UTF-8?B?w4FsdmFybyBGZXJuw6FuZGV6IFJvamFz?= <noltari@gmail.com>,
        linux-leds@vger.kernel.org,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        devicetree@vger.kernel.org
Subject: Re: leds-bcm6328: add more init configuration options
References: <5648C090.9060904@simon.arlott.org.uk>
In-reply-to: <5648C090.9060904@simon.arlott.org.uk>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1564
Lines: 44

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/