* Samsung-clk patches for 3.15
@ 2014-01-24 14:38 Tomasz Figa
2014-01-27 23:37 ` Tomasz Figa
0 siblings, 1 reply; 6+ messages in thread
From: Tomasz Figa @ 2014-01-24 14:38 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
Linux 3.14 is going to include Andrzej Hajda's patches converting
Samsung clock drivers to use clock ID defines in include/dt-bindings,
instead of local enums, but to avoid unnecessary merge conflicts we have
converted only the clock driver, leaving DTS files unchanged yet.
We intend to complete the conversion in 3.15, by replacing magic numbers
in DTS files with respective preprocessor macros, but to reduce
potential conflicts we need help of you, Samsung clock patches authors :).
I'd like to ask anybody who already has patches for DTS files adding any
clock-related contents still using numeric IDs, e.g. clock properties in
nodes or full nodes containing clock properties, to make sure that the
patches are merged before Andrzej sends the conversion patches. Then
Andrzej's script will generate patches updating all clock properties,
leaving no numeric IDs in DTS files.
If you are just starting your work on a patch that introduces changes as
mentioned above, please make sure to already use clock macros, not
numeric IDs. Otherwise you risk having needless rebases with a lot of
conflicts. You have been warned ;).
Best regards,
Tomasz
^ permalink raw reply [flat|nested] 6+ messages in thread
* Samsung-clk patches for 3.15
2014-01-24 14:38 Samsung-clk patches for 3.15 Tomasz Figa
@ 2014-01-27 23:37 ` Tomasz Figa
2014-01-28 0:09 ` Kukjin Kim
0 siblings, 1 reply; 6+ messages in thread
From: Tomasz Figa @ 2014-01-27 23:37 UTC (permalink / raw)
To: linux-arm-kernel
[Forgot to Cc Mike...]
On 24.01.2014 15:38, Tomasz Figa wrote:
> Hi,
>
> Linux 3.14 is going to include Andrzej Hajda's patches converting
> Samsung clock drivers to use clock ID defines in include/dt-bindings,
> instead of local enums, but to avoid unnecessary merge conflicts we have
> converted only the clock driver, leaving DTS files unchanged yet.
>
> We intend to complete the conversion in 3.15, by replacing magic numbers
> in DTS files with respective preprocessor macros, but to reduce
> potential conflicts we need help of you, Samsung clock patches authors :).
>
> I'd like to ask anybody who already has patches for DTS files adding any
> clock-related contents still using numeric IDs, e.g. clock properties in
> nodes or full nodes containing clock properties, to make sure that the
> patches are merged before Andrzej sends the conversion patches. Then
> Andrzej's script will generate patches updating all clock properties,
> leaving no numeric IDs in DTS files.
>
> If you are just starting your work on a patch that introduces changes as
> mentioned above, please make sure to already use clock macros, not
> numeric IDs. Otherwise you risk having needless rebases with a lot of
> conflicts. You have been warned ;).
>
> Best regards,
> Tomasz
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Samsung-clk patches for 3.15
2014-01-27 23:37 ` Tomasz Figa
@ 2014-01-28 0:09 ` Kukjin Kim
2014-01-28 0:17 ` Tomasz Figa
0 siblings, 1 reply; 6+ messages in thread
From: Kukjin Kim @ 2014-01-28 0:09 UTC (permalink / raw)
To: linux-arm-kernel
Tomasz Figa wrote:
>
> [Forgot to Cc Mike...]
>
> On 24.01.2014 15:38, Tomasz Figa wrote:
> > Hi,
> >
Hi,
> > Linux 3.14 is going to include Andrzej Hajda's patches converting
> > Samsung clock drivers to use clock ID defines in include/dt-bindings,
> > instead of local enums, but to avoid unnecessary merge conflicts we
> > have converted only the clock driver, leaving DTS files unchanged yet.
> >
> > We intend to complete the conversion in 3.15, by replacing magic
> > numbers in DTS files with respective preprocessor macros, but to
> > reduce potential conflicts we need help of you, Samsung clock patches
> authors :).
> >
> > I'd like to ask anybody who already has patches for DTS files adding
> > any clock-related contents still using numeric IDs, e.g. clock
> > properties in nodes or full nodes containing clock properties, to make
> > sure that the patches are merged before Andrzej sends the conversion
> > patches. Then Andrzej's script will generate patches updating all
> > clock properties, leaving no numeric IDs in DTS files.
> >
There are several DTS patches in v3.14-drop/soc-exynos-2 branch of my tree
for 3.15 and it will be merged after done of multiplatform, I need to rebase
them based on v3.14-rc1 though...So I think, would be better if we could
update DTS with using Andrzej's script after merging it into arm-soc...
> > If you are just starting your work on a patch that introduces changes
> > as mentioned above, please make sure to already use clock macros, not
> > numeric IDs. Otherwise you risk having needless rebases with a lot of
> > conflicts. You have been warned ;).
> >
Thanks,
Kukjin
^ permalink raw reply [flat|nested] 6+ messages in thread
* Samsung-clk patches for 3.15
2014-01-28 0:09 ` Kukjin Kim
@ 2014-01-28 0:17 ` Tomasz Figa
2014-02-14 0:05 ` Kukjin Kim
0 siblings, 1 reply; 6+ messages in thread
From: Tomasz Figa @ 2014-01-28 0:17 UTC (permalink / raw)
To: linux-arm-kernel
On 28.01.2014 01:09, Kukjin Kim wrote:
> Tomasz Figa wrote:
>>
>> [Forgot to Cc Mike...]
>>
>> On 24.01.2014 15:38, Tomasz Figa wrote:
>>> Hi,
>>>
>
> Hi,
>
>>> Linux 3.14 is going to include Andrzej Hajda's patches converting
>>> Samsung clock drivers to use clock ID defines in include/dt-bindings,
>>> instead of local enums, but to avoid unnecessary merge conflicts we
>>> have converted only the clock driver, leaving DTS files unchanged yet.
>>>
>>> We intend to complete the conversion in 3.15, by replacing magic
>>> numbers in DTS files with respective preprocessor macros, but to
>>> reduce potential conflicts we need help of you, Samsung clock patches
>> authors :).
>>>
>>> I'd like to ask anybody who already has patches for DTS files adding
>>> any clock-related contents still using numeric IDs, e.g. clock
>>> properties in nodes or full nodes containing clock properties, to make
>>> sure that the patches are merged before Andrzej sends the conversion
>>> patches. Then Andrzej's script will generate patches updating all
>>> clock properties, leaving no numeric IDs in DTS files.
>>>
>
> There are several DTS patches in v3.14-drop/soc-exynos-2 branch of my tree
> for 3.15 and it will be merged after done of multiplatform, I need to rebase
> them based on v3.14-rc1 though...So I think, would be better if we could
> update DTS with using Andrzej's script after merging it into arm-soc...
Basically my intention is to:
1) Have any existing patches using clock numbers merged in reasonable
period of time (to not miss the merge window with DTS conversion patches).
2) Stop accepting such patches anymore.
3) Rerun Andrzej's script and convert all device tree sources to use
clock macros.
Andrzej's patches for DTSes should go through your tree anyway (as any
Samsung DTS patches by default), so it shouldn't be a problem,
regardless of merging anything into arm-soc.
Best regards,
Tomasz
^ permalink raw reply [flat|nested] 6+ messages in thread
* Samsung-clk patches for 3.15
2014-01-28 0:17 ` Tomasz Figa
@ 2014-02-14 0:05 ` Kukjin Kim
2014-02-14 0:24 ` Kukjin Kim
0 siblings, 1 reply; 6+ messages in thread
From: Kukjin Kim @ 2014-02-14 0:05 UTC (permalink / raw)
To: linux-arm-kernel
On 01/28/14 09:17, Tomasz Figa wrote:
>
>
> On 28.01.2014 01:09, Kukjin Kim wrote:
>> Tomasz Figa wrote:
>>>
>>> [Forgot to Cc Mike...]
>>>
>>> On 24.01.2014 15:38, Tomasz Figa wrote:
>>>> Hi,
>>>>
>>
>> Hi,
>>
>>>> Linux 3.14 is going to include Andrzej Hajda's patches converting
>>>> Samsung clock drivers to use clock ID defines in include/dt-bindings,
>>>> instead of local enums, but to avoid unnecessary merge conflicts we
>>>> have converted only the clock driver, leaving DTS files unchanged yet.
>>>>
>>>> We intend to complete the conversion in 3.15, by replacing magic
>>>> numbers in DTS files with respective preprocessor macros, but to
>>>> reduce potential conflicts we need help of you, Samsung clock patches
>>> authors :).
>>>>
>>>> I'd like to ask anybody who already has patches for DTS files adding
>>>> any clock-related contents still using numeric IDs, e.g. clock
>>>> properties in nodes or full nodes containing clock properties, to make
>>>> sure that the patches are merged before Andrzej sends the conversion
>>>> patches. Then Andrzej's script will generate patches updating all
>>>> clock properties, leaving no numeric IDs in DTS files.
>>>>
>>
>> There are several DTS patches in v3.14-drop/soc-exynos-2 branch of my
>> tree
>> for 3.15 and it will be merged after done of multiplatform, I need to
>> rebase
>> them based on v3.14-rc1 though...So I think, would be better if we could
>> update DTS with using Andrzej's script after merging it into arm-soc...
>
> Basically my intention is to:
>
> 1) Have any existing patches using clock numbers merged in reasonable
> period of time (to not miss the merge window with DTS conversion patches).
>
> 2) Stop accepting such patches anymore.
>
> 3) Rerun Andrzej's script and convert all device tree sources to use
> clock macros.
>
> Andrzej's patches for DTSes should go through your tree anyway (as any
> Samsung DTS patches by default), so it shouldn't be a problem,
> regardless of merging anything into arm-soc.
>
Makes sense, OK. I agreed. I'll apply Andrzej's "[PATCH RESEND v2 00/12]
clk/exynos convert clock IDs to macros" and every exynos dt should be
posted based on that from now on.
Thanks,
- Kukjin
^ permalink raw reply [flat|nested] 6+ messages in thread
* Samsung-clk patches for 3.15
2014-02-14 0:05 ` Kukjin Kim
@ 2014-02-14 0:24 ` Kukjin Kim
0 siblings, 0 replies; 6+ messages in thread
From: Kukjin Kim @ 2014-02-14 0:24 UTC (permalink / raw)
To: linux-arm-kernel
On 02/14/14 09:05, Kukjin Kim wrote:
> On 01/28/14 09:17, Tomasz Figa wrote:
>>
>>
>> On 28.01.2014 01:09, Kukjin Kim wrote:
>>> Tomasz Figa wrote:
>>>>
>>>> [Forgot to Cc Mike...]
>>>>
>>>> On 24.01.2014 15:38, Tomasz Figa wrote:
>>>>> Hi,
>>>>>
>>>
>>> Hi,
>>>
>>>>> Linux 3.14 is going to include Andrzej Hajda's patches converting
>>>>> Samsung clock drivers to use clock ID defines in include/dt-bindings,
>>>>> instead of local enums, but to avoid unnecessary merge conflicts we
>>>>> have converted only the clock driver, leaving DTS files unchanged yet.
>>>>>
>>>>> We intend to complete the conversion in 3.15, by replacing magic
>>>>> numbers in DTS files with respective preprocessor macros, but to
>>>>> reduce potential conflicts we need help of you, Samsung clock patches
>>>> authors :).
>>>>>
>>>>> I'd like to ask anybody who already has patches for DTS files adding
>>>>> any clock-related contents still using numeric IDs, e.g. clock
>>>>> properties in nodes or full nodes containing clock properties, to make
>>>>> sure that the patches are merged before Andrzej sends the conversion
>>>>> patches. Then Andrzej's script will generate patches updating all
>>>>> clock properties, leaving no numeric IDs in DTS files.
>>>>>
>>>
>>> There are several DTS patches in v3.14-drop/soc-exynos-2 branch of my
>>> tree
>>> for 3.15 and it will be merged after done of multiplatform, I need to
>>> rebase
>>> them based on v3.14-rc1 though...So I think, would be better if we could
>>> update DTS with using Andrzej's script after merging it into arm-soc...
>>
>> Basically my intention is to:
>>
>> 1) Have any existing patches using clock numbers merged in reasonable
>> period of time (to not miss the merge window with DTS conversion
>> patches).
>>
>> 2) Stop accepting such patches anymore.
>>
>> 3) Rerun Andrzej's script and convert all device tree sources to use
>> clock macros.
>>
>> Andrzej's patches for DTSes should go through your tree anyway (as any
>> Samsung DTS patches by default), so it shouldn't be a problem,
>> regardless of merging anything into arm-soc.
>>
>
> Makes sense, OK. I agreed. I'll apply Andrzej's "[PATCH RESEND v2 00/12]
> clk/exynos convert clock IDs to macros" and every exynos dt should be
> posted based on that from now on.
>
Tomasz, I've applied Andrej's regarding dt patches into samsung tree.
Thanks,
Kukjin
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-02-14 0:24 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-01-24 14:38 Samsung-clk patches for 3.15 Tomasz Figa
2014-01-27 23:37 ` Tomasz Figa
2014-01-28 0:09 ` Kukjin Kim
2014-01-28 0:17 ` Tomasz Figa
2014-02-14 0:05 ` Kukjin Kim
2014-02-14 0:24 ` Kukjin Kim
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).