[v2] ptp_qoriq: add initialization message

[v2] ptp_qoriq: add initialization message

[Yangbo Lu]

Current ptp_qoriq driver prints only warning or error messages.
It may be loaded successfully without any messages.
Although this is fine, it would be convenient to have an oneline
initialization log showing success and PTP clock index.
The goods are,
- The ptp_qoriq driver users may know whether this driver is loaded
  successfully, or not, or not loaded from the booting log.
- The ptp_qoriq driver users don't have to install an ethtool to
  check the PTP clock index for using. Or don't have to check which
  /sys/class/ptp/ptpX is PTP QorIQ clock.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
---
Changes for v2:
	- Added more in commit message.
---
 drivers/ptp/ptp_qoriq.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/ptp/ptp_qoriq.c b/drivers/ptp/ptp_qoriq.c
index b27c46e..66e7d57 100644
--- a/drivers/ptp/ptp_qoriq.c
+++ b/drivers/ptp/ptp_qoriq.c
@@ -541,6 +541,8 @@ int ptp_qoriq_init(struct ptp_qoriq *ptp_qoriq, void __iomem *base,
 
 	ptp_qoriq->phc_index = ptp_clock_index(ptp_qoriq->clock);
 	ptp_qoriq_create_debugfs(ptp_qoriq);
+
+	pr_info("new PTP clock ptp%d\n", ptp_qoriq->phc_index);
 	return 0;
 }
 EXPORT_SYMBOL_GPL(ptp_qoriq_init);
-- 
2.7.4

Re: [v2] ptp_qoriq: add initialization message

[Richard Cochran]

On Wed, Feb 12, 2020 at 06:19:16PM +0800, Yangbo Lu wrote:
> - The ptp_qoriq driver users don't have to install an ethtool to
>   check the PTP clock index for using. Or don't have to check which
>   /sys/class/ptp/ptpX is PTP QorIQ clock.

You just stated the reason why your new pr_info() is more spam in the
kernel log.  So, NAK on this patch.

Sorry,
Richard

Re: [v2] ptp_qoriq: add initialization message

[David Miller]

From: Yangbo Lu <yangbo.lu@nxp.com>
Date: Wed, 12 Feb 2020 18:19:16 +0800

> Current ptp_qoriq driver prints only warning or error messages.
> It may be loaded successfully without any messages.
> Although this is fine, it would be convenient to have an oneline
> initialization log showing success and PTP clock index.
> The goods are,
> - The ptp_qoriq driver users may know whether this driver is loaded
>   successfully, or not, or not loaded from the booting log.
> - The ptp_qoriq driver users don't have to install an ethtool to
>   check the PTP clock index for using. Or don't have to check which
>   /sys/class/ptp/ptpX is PTP QorIQ clock.
> 
> Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
> ---
> Changes for v2:
> 	- Added more in commit message.

I explained yesterday why your change is inappropriate and why I won't
be applying your patch.

It is not appropriate to send the same patch again with a different
commit message.

What might be appropriate is replying to my email and explaining your
side of the argument and why you think it is appropriate.

You must engage with other developers and discuss your change here on
the list if you want us to consider it still.

Thank you.