linux-kernel - Re: [PATCH v2 1/6] phy: miphy365x: Use the generic phy type constants in dt-bindings/phy/phy.h

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]

Message-ID: <551AC37E.70503@st.com>
Date:	Tue, 31 Mar 2015 17:55:42 +0200
From:	Maxime Coquelin <maxime.coquelin@...com>
To:	Kishon Vijay Abraham I <kishon@...com>,
	Lee Jones <lee.jones@...aro.org>
CC:	Peter Griffin <peter.griffin@...aro.org>,
	<linux-arm-kernel@...ts.infradead.org>,
	<linux-kernel@...r.kernel.org>, <srinivas.kandagatla@...il.com>,
	<patrice.chotard@...com>, <balbi@...com>,
	<devicetree@...r.kernel.org>, <robherring2@...il.com>
Subject: Re: [PATCH v2 1/6] phy: miphy365x: Use the generic phy type constants
 in dt-bindings/phy/phy.h

Hi Kishon,

On 03/31/2015 05:30 PM, Kishon Vijay Abraham I wrote:
> Maxime,
>
> On Tuesday 31 March 2015 01:10 PM, Lee Jones wrote:
>> On Tue, 31 Mar 2015, Kishon Vijay Abraham I wrote:
>>
>>> Hi Patrice, Maxime,
>>>
>>> On Monday 30 March 2015 08:47 PM, Peter Griffin wrote:
>>>> Now there are generic phy type constants declared in phy.h, migrate 
>>>> over to
>>>> using them rather than defining our own. This change has been done 
>>>> as one
>>>> atomic commit to be bisectable.
>>>>
>>>> Note: The values of the defines are the same, so there is no ABI 
>>>> breakage
>>>> with this patch.
>>>>
>>>> Signed-off-by: Peter Griffin <peter.griffin@...aro.org>
>>>> Acked-by: Rob Herring <robh@...nel.org>
>>>> Acked-by: Lee Jones <lee.jones@...aro.org>
>>>
>>> Are you okay for this patch to go via PHY tree? It modifies 
>>> arch/arm/boot/dts
>>> /stih416.dtsi.
>>
>> These files need to be changed simultaneously in order to prevent
>> breakage during bisection and the like.  I think the best bet moving
>> forward is to pull this into its own branch, tag it and send Maxime a
>> pull-request to the immutable branch.  It's the best way to minimise
>> merge conflicts during the v4.1 merge window.
>
> okay. I still need an Ack from Maxime.

You have my Ack.

I haven't sent a pull request for STi DT files for v4.1 now, and it is 
too late to send one.
So creating an immutable tag is not even needed in my opinion, you can 
just take the DT patch.

Thanks,
Maxime

>
> Thanks
> Kishon

--
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/