[PATCH] net: dwc_eth_qos: cosmetic: remove unused define EQOS_DESCRIPTOR_ALIGN

[PATCH] net: dwc_eth_qos: cosmetic: remove unused define EQOS_DESCRIPTOR_ALIGN

[Patrick Delaunay]

Remove the define EQOS_DESCRIPTOR_ALIGN unused since the
commit 6f1e668d964e ("net: dwc_eth_qos: Pad descriptors to cacheline size")

Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
---

 drivers/net/dwc_eth_qos.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/net/dwc_eth_qos.c b/drivers/net/dwc_eth_qos.c
index e8242ca4e1..0b1e95e98b 100644
--- a/drivers/net/dwc_eth_qos.c
+++ b/drivers/net/dwc_eth_qos.c
@@ -240,8 +240,6 @@ struct eqos_tegra186_regs {
 #define EQOS_AUTO_CAL_STATUS_ACTIVE			BIT(31)
 
 /* Descriptors */
-/* We assume ARCH_DMA_MINALIGN >= 16; 16 is the EQOS HW minimum */
-#define EQOS_DESCRIPTOR_ALIGN	ARCH_DMA_MINALIGN
 #define EQOS_DESCRIPTORS_TX	4
 #define EQOS_DESCRIPTORS_RX	4
 #define EQOS_DESCRIPTORS_NUM	(EQOS_DESCRIPTORS_TX + EQOS_DESCRIPTORS_RX)
-- 
2.17.1

[PATCH] net: dwc_eth_qos: cosmetic: remove unused define EQOS_DESCRIPTOR_ALIGN

[Marek Vasut]

On 3/30/21 2:34 PM, Patrick Delaunay wrote:
> Remove the define EQOS_DESCRIPTOR_ALIGN unused since the
> commit 6f1e668d964e ("net: dwc_eth_qos: Pad descriptors to cacheline size")
> 
> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>

Acked-by: Marek Vasut <marex@denx.de>

Thanks

[PATCH] net: dwc_eth_qos: cosmetic: remove unused define EQOS_DESCRIPTOR_ALIGN

[Ramon Fried]

On Tue, Mar 30, 2021 at 5:12 PM Marek Vasut <marex@denx.de> wrote:
>
> On 3/30/21 2:34 PM, Patrick Delaunay wrote:
> > Remove the define EQOS_DESCRIPTOR_ALIGN unused since the
> > commit 6f1e668d964e ("net: dwc_eth_qos: Pad descriptors to cacheline size")
> >
> > Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>
> Acked-by: Marek Vasut <marex@denx.de>
>
> Thanks
Reviewed-by: Ramon Fried <rfried.dev@gmail.com>

Re: [PATCH] net: dwc_eth_qos: cosmetic: remove unused define EQOS_DESCRIPTOR_ALIGN

[Patrick DELAUNAY]

Hi Ramon,

On 3/31/21 4:43 PM, Ramon Fried wrote:
> On Tue, Mar 30, 2021 at 5:12 PM Marek Vasut <marex@denx.de> wrote:
>> On 3/30/21 2:34 PM, Patrick Delaunay wrote:
>>> Remove the define EQOS_DESCRIPTOR_ALIGN unused since the
>>> commit 6f1e668d964e ("net: dwc_eth_qos: Pad descriptors to cacheline size")
>>>
>>> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
>> Acked-by: Marek Vasut <marex@denx.de>
>>
>> Thanks
> Reviewed-by: Ramon Fried <rfried.dev@gmail.com>
It seens this patch is not applied on u-boot-net/network_master or on 
u-boot/next

even if it is accepted in patchwork.

http://patchwork.ozlabs.org/project/uboot/patch/20210330143429.1.I654d7aeb07f9cd0602752861d06f4c17e9a0ee17@changeid/


Any reason ?

or it is just a error in patchwork and the patch is forgotten...


best regards.

Patrick

Re: [PATCH] net: dwc_eth_qos: cosmetic: remove unused define EQOS_DESCRIPTOR_ALIGN

[Ramon Fried]

On Mon, Jul 5, 2021 at 8:50 PM Patrick DELAUNAY
<patrick.delaunay@foss.st.com> wrote:
>
> Hi Ramon,
>
> On 3/31/21 4:43 PM, Ramon Fried wrote:
> > On Tue, Mar 30, 2021 at 5:12 PM Marek Vasut <marex@denx.de> wrote:
> >> On 3/30/21 2:34 PM, Patrick Delaunay wrote:
> >>> Remove the define EQOS_DESCRIPTOR_ALIGN unused since the
> >>> commit 6f1e668d964e ("net: dwc_eth_qos: Pad descriptors to cacheline size")
> >>>
> >>> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
> >> Acked-by: Marek Vasut <marex@denx.de>
> >>
> >> Thanks
> > Reviewed-by: Ramon Fried <rfried.dev@gmail.com>
> It seens this patch is not applied on u-boot-net/network_master or on
> u-boot/next
>
> even if it is accepted in patchwork.
>
> http://patchwork.ozlabs.org/project/uboot/patch/20210330143429.1.I654d7aeb07f9cd0602752861d06f4c17e9a0ee17@changeid/
>
>
> Any reason ?
>
> or it is just a error in patchwork and the patch is forgotten...
>
>
> best regards.
>
> Patrick
>
>
>
>
Hi Patrick.
The patch was introduced just a bit before I started maintaining a
tree, up until then the patches were picked by Tom.
I applied it to my tree.
Thanks,
Ramon.