<div dir="auto">Hi, <div dir="auto"><br></div><div dir="auto">While you're at it, I was wondering of calling those ioremap() functions would ever fail, and if you need to add any code to handle those cases?</div><div dir="auto"><br></div><div dir="auto">Christ van Willegen</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Op vr 6 sep. 2019 09:53 schreef H. Nikolaus Schaller <<a href="mailto:hns@goldelico.com">hns@goldelico.com</a>>:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
I am preparing the next PATCH version now. Incl. updating the commit<br>
messages to carry more documentation about the opp-supported-hw bit<br>
definitions.<br>
<br>
So please do not merge yet.<br>
<br>
> Am 05.09.2019 um 16:27 schrieb Tony Lindgren <<a href="mailto:tony@atomide.com" target="_blank" rel="noreferrer">tony@atomide.com</a>>:<br>
> <br>
> Hi,<br>
> <br>
> * H. Nikolaus Schaller <<a href="mailto:hns@goldelico.com" target="_blank" rel="noreferrer">hns@goldelico.com</a>> [190904 08:54]:<br>
>> According to omap.txt bindings documentation, matching the ti-cpufreq driver needs<br>
>> to specify explicitly if a board uses an omap3430 or omap36xx chip.<br>
>> <br>
>> This needs to add ti,omap3430 to most omap34xx boards and replace ti,omap3630<br>
>> by ti,omap36xx for some omap36xx boards (most others already have done it right).<br>
>> <br>
>> We also clean up some instances of missing ti,am3517 so that we can rely on<br>
>> seeing either one of:<br>
>> <br>
>> ti,am3517<br>
>> ti,omap34xx<br>
>> ti,omap36xx<br>
>> <br>
>> in addition to ti,omap3.<br>
> <br>
> Please set up things to use ti,omap3630 in addition to ti,omap36xx<br>
> for compatible to avoid churning the same files later.<br>
> <br>
>> diff --git a/arch/arm/boot/dts/logicpd-som-lv-37xx-devkit.dts b/arch/arm/boot/dts/logicpd-som-lv-37xx-devkit.dts<br>
>> --- a/arch/arm/boot/dts/logicpd-som-lv-37xx-devkit.dts<br>
>> +++ b/arch/arm/boot/dts/logicpd-som-lv-37xx-devkit.dts<br>
>> @@ -9,5 +9,5 @@<br>
>> <br>
>> / {<br>
>>      model = "LogicPD Zoom DM3730 SOM-LV Development Kit";<br>
>> -    compatible = "logicpd,dm3730-som-lv-devkit", "ti,omap3630", "ti,omap3";<br>
>> +    compatible = "logicpd,dm3730-som-lv-devkit", "ti,omap36xx", "ti,omap3";<br>
>> };<br>
> <br>
> So just make this:<br>
> <br>
> compatible = "logicpd,dm3730-som-lv-devkit", "ti,omap3630", "ti,omap36xx", "ti,omap3";<br>
<br>
Do we really need both? The clock driver already checks for both variants and the<br>
ti-cpufreq will as well. So it suffices to have either "ti,omap3630" or "ti,omap36xx".<br>
<br>
But yes, there may be user-space code that reads sysfs. So we should keep the<br>
"ti,omap3630" and add "ti,omap36xx".<br>
<br>
> <br>
> And so on. It's fine to use ti,omap3630 for 37xx too as they're the same.<br>
> <br>
>> diff --git a/arch/arm/boot/dts/omap3-ldp.dts b/arch/arm/boot/dts/omap3-ldp.dts<br>
>> index 9a5fde2d9bce..9947574bd0f8 100644<br>
>> --- a/arch/arm/boot/dts/omap3-ldp.dts<br>
>> +++ b/arch/arm/boot/dts/omap3-ldp.dts<br>
>> @@ -10,7 +10,7 @@<br>
>> <br>
>> / {<br>
>>      model = "TI OMAP3430 LDP (Zoom1 Labrador)";<br>
>> -    compatible = "ti,omap3-ldp", "ti,omap3";<br>
>> +    compatible = "ti,omap3-ldp", "ti,omap34xx, "ti,omap3";<br>
> <br>
> This fails to compile, it's missing a '"' for ti,omap34xx. And here too,<br>
> please update to use:<br>
<br>
Ah yes. I missed that (and my build script did not build all DTB).<br>
<br>
> <br>
> compatible = "ti,omap3-ldp", "ti,omap3430", "ti,omap34xx", "ti,omap3";<br>
<br>
After thinking a little about the whole topic the main rule of this change must be:<br>
<br>
* do not break any existing in-tree DTS<br>
        => only *add* to compatible what we need to distinguish between omap34 and omap36<br>
<br>
* additions shall only follow new scheme<br>
        => we only add "ti,omap34xx" or "ti,omap36xx"<br>
           but neither "ti,omap3630" nor "ti,omap3430"<br>
<br>
* cover some out-of-tree DTS<br>
        => make the ti-cpufreq driver still match "ti,omap3430" or "ti,omap3630"<br>
           even if this duplicates compatibility<br>
<br>
This would mean that the logicpd-som-lv-37xx-devkit.dts gets the additional "ti,omap36xx"<br>
while the omap3-ldp.dts would only get an "ti,omap34xx" but no "ti,omap3430" (since we<br>
do not use it anywhere).<br>
<br>
Could you agree on this approach?<br>
<br>
> <br>
> And again it's fine to add "ti,omap3430" for 3530 variants.<br>
<br>
Yes, sure.<br>
<br>
BR,<br>
Nikolaus<br>
<br>
_______________________________________________<br>
<a href="http://projects.goldelico.com/p/gta04-kernel/" rel="noreferrer noreferrer" target="_blank">http://projects.goldelico.com/p/gta04-kernel/</a><br>
Letux-kernel mailing list<br>
<a href="mailto:Letux-kernel@openphoenux.org" target="_blank" rel="noreferrer">Letux-kernel@openphoenux.org</a><br>
<a href="http://lists.goldelico.com/mailman/listinfo.cgi/letux-kernel" rel="noreferrer noreferrer" target="_blank">http://lists.goldelico.com/mailman/listinfo.cgi/letux-kernel</a><br>
</blockquote></div>