Remove PTP_PF_EXTTS support for non-PCI11x1x devices since they do not support
the PTP-IO Input event triggered timestamping mechanisms added
Fixes: 60942c397af6 ("Add support for PTP-IO Event Input External Timestamp
(extts)")
Reviewed-by: Jakub Kicinski <[email protected]>
Signed-off-by: Raju Lakkaraju <[email protected]>
---
Changes:
========
V2 -> V3:
- Correct the Fixes tag
V1 -> V2:
- Repost against net with a Fixes tag
drivers/net/ethernet/microchip/lan743x_ptp.c | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/drivers/net/ethernet/microchip/lan743x_ptp.c b/drivers/net/ethernet/microchip/lan743x_ptp.c
index 6a11e2ceb013..da3ea905adbb 100644
--- a/drivers/net/ethernet/microchip/lan743x_ptp.c
+++ b/drivers/net/ethernet/microchip/lan743x_ptp.c
@@ -1049,6 +1049,10 @@ static int lan743x_ptpci_verify_pin_config(struct ptp_clock_info *ptp,
enum ptp_pin_function func,
unsigned int chan)
{
+ struct lan743x_ptp *lan_ptp =
+ container_of(ptp, struct lan743x_ptp, ptp_clock_info);
+ struct lan743x_adapter *adapter =
+ container_of(lan_ptp, struct lan743x_adapter, ptp);
int result = 0;
/* Confirm the requested function is supported. Parameter
@@ -1057,7 +1061,10 @@ static int lan743x_ptpci_verify_pin_config(struct ptp_clock_info *ptp,
switch (func) {
case PTP_PF_NONE:
case PTP_PF_PEROUT:
+ break;
case PTP_PF_EXTTS:
+ if (!adapter->is_pci11x1x)
+ result = -1;
break;
case PTP_PF_PHYSYNC:
default:
--
2.25.1
Hi Horatiu,
Thank you for review comments.
In vim editor, set the textwidth as 75. Due to this, after 75 charecters of
the line, string placed in next line.
Please find the screenshot of column marker with textwidth.
I will fix it and repost the patch again.
The 09/28/2022 09:27, Horatiu Vultur wrote:
> The 09/28/2022 12:38, Raju Lakkaraju wrote:
>
> Hi Raju,
>
> > Remove PTP_PF_EXTTS support for non-PCI11x1x devices since they do not support
> > the PTP-IO Input event triggered timestamping mechanisms added
> >
> > Fixes: 60942c397af6 ("Add support for PTP-IO Event Input External Timestamp
> > (extts)")
>
> The fixes tag should not spread over multiple lines. Also you have extra
> spaces between External and Timestamp which doesn't appear in the actual
> commit.
>
> Also you have an empty line between Fixes and Reviewed-by tags.
> >
> > Reviewed-by: Jakub Kicinski <[email protected]>
>
> I am not sure that Jakub gave his Reviewed-by, but maybe I have missed
> that.
>
Fixes tag correction was suggested by Jakub in V2 patch.
> > Signed-off-by: Raju Lakkaraju <[email protected]>
> > ---
> > Changes:
> > ========
> > V2 -> V3:
> > - Correct the Fixes tag
> >
> > V1 -> V2:
> > - Repost against net with a Fixes tag
> >
> > drivers/net/ethernet/microchip/lan743x_ptp.c | 7 +++++++
> > 1 file changed, 7 insertions(+)
> >
> > diff --git a/drivers/net/ethernet/microchip/lan743x_ptp.c b/drivers/net/ethernet/microchip/lan743x_ptp.c
> > index 6a11e2ceb013..da3ea905adbb 100644
> > --- a/drivers/net/ethernet/microchip/lan743x_ptp.c
> > +++ b/drivers/net/ethernet/microchip/lan743x_ptp.c
> > @@ -1049,6 +1049,10 @@ static int lan743x_ptpci_verify_pin_config(struct ptp_clock_info *ptp,
> > enum ptp_pin_function func,
> > unsigned int chan)
> > {
> > + struct lan743x_ptp *lan_ptp =
> > + container_of(ptp, struct lan743x_ptp, ptp_clock_info);
> > + struct lan743x_adapter *adapter =
> > + container_of(lan_ptp, struct lan743x_adapter, ptp);
> > int result = 0;
> >
> > /* Confirm the requested function is supported. Parameter
> > @@ -1057,7 +1061,10 @@ static int lan743x_ptpci_verify_pin_config(struct ptp_clock_info *ptp,
> > switch (func) {
> > case PTP_PF_NONE:
> > case PTP_PF_PEROUT:
> > + break;
> > case PTP_PF_EXTTS:
> > + if (!adapter->is_pci11x1x)
> > + result = -1;
> > break;
> > case PTP_PF_PHYSYNC:
> > default:
> > --
> > 2.25.1
> >
>
> --
> /Horatiu
--
--------
Thanks,
Raju
The 09/28/2022 12:38, Raju Lakkaraju wrote:
Hi Raju,
> Remove PTP_PF_EXTTS support for non-PCI11x1x devices since they do not support
> the PTP-IO Input event triggered timestamping mechanisms added
>
> Fixes: 60942c397af6 ("Add support for PTP-IO Event Input External Timestamp
> (extts)")
The fixes tag should not spread over multiple lines. Also you have extra
spaces between External and Timestamp which doesn't appear in the actual
commit.
Also you have an empty line between Fixes and Reviewed-by tags.
>
> Reviewed-by: Jakub Kicinski <[email protected]>
I am not sure that Jakub gave his Reviewed-by, but maybe I have missed
that.
> Signed-off-by: Raju Lakkaraju <[email protected]>
> ---
> Changes:
> ========
> V2 -> V3:
> - Correct the Fixes tag
>
> V1 -> V2:
> - Repost against net with a Fixes tag
>
> drivers/net/ethernet/microchip/lan743x_ptp.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/net/ethernet/microchip/lan743x_ptp.c b/drivers/net/ethernet/microchip/lan743x_ptp.c
> index 6a11e2ceb013..da3ea905adbb 100644
> --- a/drivers/net/ethernet/microchip/lan743x_ptp.c
> +++ b/drivers/net/ethernet/microchip/lan743x_ptp.c
> @@ -1049,6 +1049,10 @@ static int lan743x_ptpci_verify_pin_config(struct ptp_clock_info *ptp,
> enum ptp_pin_function func,
> unsigned int chan)
> {
> + struct lan743x_ptp *lan_ptp =
> + container_of(ptp, struct lan743x_ptp, ptp_clock_info);
> + struct lan743x_adapter *adapter =
> + container_of(lan_ptp, struct lan743x_adapter, ptp);
> int result = 0;
>
> /* Confirm the requested function is supported. Parameter
> @@ -1057,7 +1061,10 @@ static int lan743x_ptpci_verify_pin_config(struct ptp_clock_info *ptp,
> switch (func) {
> case PTP_PF_NONE:
> case PTP_PF_PEROUT:
> + break;
> case PTP_PF_EXTTS:
> + if (!adapter->is_pci11x1x)
> + result = -1;
> break;
> case PTP_PF_PHYSYNC:
> default:
> --
> 2.25.1
>
--
/Horatiu
On Wed, 28 Sep 2022 13:50:50 +0530 Raju Lakkaraju wrote:
> > > Reviewed-by: Jakub Kicinski <[email protected]>
> >
> > I am not sure that Jakub gave his Reviewed-by, but maybe I have missed
> > that.
>
> Fixes tag correction was suggested by Jakub in V2 patch.
Does not mean I reviewed the patch, only add review tags if someone
explicitly sent them to you.