From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wolfgang Grandegger Subject: Re: [PATCH v3 0/1] can: ucan: add driver for Theobroma Systems UCAN devices Date: Fri, 23 Mar 2018 11:04:08 +0100 Message-ID: References: <20180322135338.60923-1-jakob.unterwurzacher@theobroma-systems.com> <7e818d10-3410-b144-9cfb-c58a9945a8b1@theobroma-systems.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: <7e818d10-3410-b144-9cfb-c58a9945a8b1@theobroma-systems.com> Content-Language: en-GB Sender: linux-kernel-owner@vger.kernel.org To: Jakob Unterwurzacher Cc: Martin Elshuber , Philipp Tomsich , Marc Kleine-Budde , linux-can@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: linux-can.vger.kernel.org Am 23.03.2018 um 10:40 schrieb Jakob Unterwurzacher: > On 23.03.18 09:32, Wolfgang Grandegger wrote: >>> * add __func__ to all errors and warnings, and to info where it made >>> sense >> >> The final output messages in the driver should especially be useful for >> the end user... and not the developer! This is also true for the >> function names. You already use more "__func__" than all other CAN >> drivers together. Just my opinion! > > The idea was to make it clear which driver printed the message. In my > opinion, this is a problem: > >> drivers/net/can/usb$ git grep "No memory left for USB buffer" >> ems_usb.c:                      netdev_err(netdev, "No memory left for >> USB buffer\n"); >> ems_usb.c:              netdev_err(netdev, "No memory left for USB >> buffer\n"); >> esd_usb2.c:                              "No memory left for USB >> buffer\n"); >> esd_usb2.c:             netdev_err(netdev, "No memory left for USB >> buffer\n"); >> gs_usb.c:               netdev_err(netdev, "No memory left for USB >> buffer\n"); >> gs_usb.c:                                          "No memory left for >> USB buffer\n"); >> kvaser_usb.c:                            "No memory left for USB >> buffer\n"); >> mcba_usb.c:                     netdev_err(netdev, "No memory left for >> USB buffer\n"); >> usb_8dev.c:             netdev_err(netdev, "No memory left for USB >> buffer\n"); >> usb_8dev.c:                     netdev_err(netdev, "No memory left for >> USB buffer\n"); > But I'm open to other suggestions (use a fixed "ucan: " prefix?) or to > drop it entirely if you think it is not worth it. But there is already a device prefix, e.g.: peak_usb 1-6:1.0: PEAK-System PCAN-USB adapter hwrev 28 serial FFFFFFFF (1 channel) peak_usb 1-6:1.0 can0: attached to PCAN-USB channel 0 (device 255) ^^^^^^^^ No need to add another one! Wolfgang.