[PATCH] ARM: dts: stm32: Rework LAN8710Ai PHY reset on DHCOM SoM

[Marek Vasut]

On 6/28/21 3:38 PM, Greg KH wrote:
> On Mon, Jun 28, 2021 at 03:10:36PM +0200, Marek Vasut wrote:
>> On 6/28/21 3:02 PM, Greg KH wrote:
>>> On Mon, Jun 28, 2021 at 02:32:50PM +0200, Marek Vasut wrote:
>>>> On 6/28/21 2:29 PM, Greg KH wrote:
>>>>> On Mon, Jun 28, 2021 at 12:44:37PM +0200, Marek Vasut wrote:
>>>>>> On 4/9/21 1:00 AM, Marek Vasut wrote:
>>>>>>> The Microchip LAN8710Ai PHY requires XTAL1/CLKIN external clock to be
>>>>>>> enabled when the nRST is toggled according to datasheet Microchip
>>>>>>> LAN8710A/LAN8710Ai DS00002164B page 35 section 3.8.5.1 Hardware Reset:
>>>>>>
>>>>>> [...]
>>>>>>
>>>>>>> Fixes: 34e0c7847dcf ("ARM: dts: stm32: Add DH Electronics DHCOM STM32MP1 SoM and PDK2 board")
>>>>>>
>>>>>> Adding stable to CC.
>>>>>>
>>>>>> Patch is now part of Linux 5.13 as commit
>>>>>>
>>>>>> 1cebcf9932ab ("ARM: dts: stm32: Rework LAN8710Ai PHY reset on DHCOM SoM")
>>>>>
>>>>> $ git show 1cebcf9932ab
>>>>> fatal: ambiguous argument '1cebcf9932ab': unknown revision or path not in the working tree.
>>>>> Use '--' to separate paths from revisions, like this:
>>>>> 'git <command> [<revision>...] -- [<file>...]'
>>>>>
>>>>> Are you sure?
>>>>
>>>> This would seem to indicate so:
>>>>
>>>> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=1cebcf9932ab76102e8cfc555879574693ba8956
>>>>
>>>> linux-2.6$ git describe 1cebcf9932ab76102e8cfc555879574693ba8956
>>>> v5.13-rc1-1-g1cebcf9932ab
>>>>
>>>> Did the commit get abbreviated too much ?
>>>
>>> Something is really odd, as that commit _is_ in linux-next, but it is
>>> not in my local copy of Linus's tree.
>>>
>>> So how it is showing up in that link above is beyond me.  Can you see it
>>> locally on your machine?
>>
>> Yes, that's where the git describe came from. And I used a different repo
>> than the one from which I submitted the patch originally, so the commit
>> must've come from fetching origin (i.e. linus tree).
>>
>> Could it be this "ambiguous argument '1cebcf9932ab'" , which would indicate
>> the commit hash got abbreviated too much ?
> 
> The web site "lies" it has a shared backend.  Trust your local copy of
> the tree, that shows that this commit is NOT in Linus's tree just yet.
> Please let stable at vger know when it does hit Linus's tree and we will be
> glad to take it.

Doh, of course, it is in next and not linus tree, now it makes sense.
I'll wait a bit until it is there and revisit this, unless it gets 
picked automatically by the Fixes tag.

Sorry for the noise.