From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Subject: Re: [PATCH 1/4] spi: core: add spi_message_dump and spi_transfer_dump Date: Fri, 18 Dec 2015 21:28:20 +0200 Message-ID: References: <1450442668-2391-1-git-send-email-kernel@martin.sperl.org> <1450442668-2391-2-git-send-email-kernel@martin.sperl.org> <407EFA5A-72AE-4609-8081-887CCA3C2944@martin.sperl.org> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: Mark Brown , linux-spi To: Martin Sperl Return-path: In-Reply-To: <407EFA5A-72AE-4609-8081-887CCA3C2944-TqfNSX0MhmxHKSADF0wUEw@public.gmane.org> Sender: linux-spi-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: On Fri, Dec 18, 2015 at 6:53 PM, Martin Sperl wrote: >> On 18.12.2015, at 14:32, Andy Shevchenko wrote: >> On Fri, Dec 18, 2015 at 2:44 PM, wrote: >>> +static void __spi_transfer_dump_chunk(struct spi_device *spi, char *pre, >>> + const void *ptr, size_t start, >>> + size_t len) > ... > all comments fixed in next patch (mostly by rewriting code to use %*ph) > >> Documentation/printk-formats.txt is your friend. In particular a >> chapter about %*ph. > had to do a sprintf for the format itself as %*ph does not allow for > variable length - it still reduced the codesize... I didn't get this. What variable length you are talking about? You may supply any amount of bytes you would like to print (will be printed up to 64 bytes in a line). >> How many users do you have for the helpers? I suggest to add a helper >> whenever there are at least 2+ users present. >> > As these are mostly convenience methods for use when debugging things > there are not typically that many users that would be needing those. > So I would prefer to keep them. Up to Mark, though in my practice most of the maintainers do not like orphaned helpers. -- With Best Regards, Andy Shevchenko -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html