[U-Boot] [PATCH v6 4/7] net: phy: ti: Allow the driver to be more configurable

Dan Murphy
2016-04-28T12:26:25Z

Michal

On 04/28/2016 01:26 AM, Michal Simek wrote:
> Hi Joe,
>
> On 28.4.2016 06:52, Joe Hershberger wrote:
>> On Wed, Apr 27, 2016 at 10:46 AM, Dan Murphy <dmurphy@ti.com> wrote:
>>> Joe
>>>
>>> On 04/26/2016 04:44 PM, Joe Hershberger wrote:
>>>> On Mon, Apr 25, 2016 at 4:35 PM, Joe Hershberger
>>>> <joe.hershberger@gmail.com> wrote:
>>>>> On Fri, Apr 15, 2016 at 7:27 AM, Dan Murphy <dmurphy@ti.com> wrote:
>>>>>> Not all devices use the same internal delay or fifo depth.
>>>>>> Add the ability to set the internal delay for rx or tx and the
>>>>>> fifo depth via the devicetree. If the value is not set in the
>>>>>> devicetree then set the delay to the default.
>>>>>>
>>>>>> If devicetree is not used then use the default defines within the
>>>>>> driver.
>>>>>>
>>>>>> Signed-off-by: Dan Murphy <dmurphy@ti.com>
>>>>> Acked-by: Joe Hershberger <joe.hershberger@ni.com>
>>>> This patch is not checkpatch.pl clean. Please resubmit.
>>>>
>>>>
>>>> 610950.mbox:140: WARNING: line over 80 characters
>>>> 610950.mbox:153: WARNING: line over 80 characters
>>>> 610950.mbox:154: WARNING: line over 80 characters
>>>> 610950.mbox:165: WARNING: line over 80 characters
>>>> total: 0 errors, 4 warnings, 0 checks, 136 lines checked

Joe fixing these will break readability of the code.
I will document this in the next patch submission

>>> How do you want me to rebase these patches on the SGMII work from Michal on the ti.c
>>> or off of master? The patch I submitted was based off of the SGMII patchset.
>> I'm pulling in some of Michal's patches this release. Please reference
>> which patches this depends on (patchwork links).
> NOTE: A lot of my patches (maybe all of them) were already merged to
> mainline via Xilinx tree.

OK latest pull has the SGMII patches on the ti.c.

> Cheers,
> Michal

--
------------------
Dan Murphy