From: Frank Rowand <frowand.list@gmail.com> To: Geert Uytterhoeven <geert@linux-m68k.org>, Matthias Brugger <matthias.bgg@gmail.com> Cc: Stephen Boyd <sboyd@kernel.org>, Michael Turquette <mturquette@baylibre.com>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Linux ARM <linux-arm-kernel@lists.infradead.org>, linux-clk <linux-clk@vger.kernel.org>, linux-mediatek@lists.infradead.org, "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@vger.kernel.org>, ryder.lee@mediatek.com, Rob Herring <robh+dt@kernel.org> Subject: Re: [PATCH 1/4] of/device: Add a way to probe drivers by match data Date: Fri, 9 Nov 2018 08:56:40 -0800 [thread overview] Message-ID: <07ec49cb-e5b7-72bc-f563-2d2d7c2e2c0d@gmail.com> (raw) In-Reply-To: <CAMuHMdWm2P8fa6d9vLhYtWHBe0uNPj9jfyL1rtmdLixPEi1rUg@mail.gmail.com> On 11/9/18 2:36 AM, Geert Uytterhoeven wrote: > Hi Matthias, > > On Fri, Nov 9, 2018 at 11:29 AM Matthias Brugger <matthias.bgg@gmail.com> wrote: >> On 08/11/2018 18:58, Stephen Boyd wrote: >>> Quoting Matthias Brugger (2018-11-08 00:29:46) >>>> On 06/11/2018 19:36, Stephen Boyd wrote: >>>>> +int platform_driver_probe_by_of_match_data(struct platform_device *pdev) >>>>> +{ >>>>> + int (*probe_func)(struct platform_device *pdev); >>>>> + >>>>> + probe_func = of_device_get_match_data(&pdev->dev); >>>> >>>> Shouldn't we check if probe_func is not NULL? >>> >>> Is the oops from the NULL pointer deref insufficient? >> >> Do you think we should crash the machine if someone uses the call wrongly? Or >> should we provide the possibility to error out on the caller side? > > I believe that would be a bug in the driver, to be discovered ASAP. > So yes, please do crash ;-) This is one of Linus' pet peeves. He does not think crashing the machine is the proper choice (as a general statement). -Frank > > Gr{oetje,eeting}s, > > Geert >
WARNING: multiple messages have this Message-ID (diff)
From: frowand.list@gmail.com (Frank Rowand) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 1/4] of/device: Add a way to probe drivers by match data Date: Fri, 9 Nov 2018 08:56:40 -0800 [thread overview] Message-ID: <07ec49cb-e5b7-72bc-f563-2d2d7c2e2c0d@gmail.com> (raw) In-Reply-To: <CAMuHMdWm2P8fa6d9vLhYtWHBe0uNPj9jfyL1rtmdLixPEi1rUg@mail.gmail.com> On 11/9/18 2:36 AM, Geert Uytterhoeven wrote: > Hi Matthias, > > On Fri, Nov 9, 2018 at 11:29 AM Matthias Brugger <matthias.bgg@gmail.com> wrote: >> On 08/11/2018 18:58, Stephen Boyd wrote: >>> Quoting Matthias Brugger (2018-11-08 00:29:46) >>>> On 06/11/2018 19:36, Stephen Boyd wrote: >>>>> +int platform_driver_probe_by_of_match_data(struct platform_device *pdev) >>>>> +{ >>>>> + int (*probe_func)(struct platform_device *pdev); >>>>> + >>>>> + probe_func = of_device_get_match_data(&pdev->dev); >>>> >>>> Shouldn't we check if probe_func is not NULL? >>> >>> Is the oops from the NULL pointer deref insufficient? >> >> Do you think we should crash the machine if someone uses the call wrongly? Or >> should we provide the possibility to error out on the caller side? > > I believe that would be a bug in the driver, to be discovered ASAP. > So yes, please do crash ;-) This is one of Linus' pet peeves. He does not think crashing the machine is the proper choice (as a general statement). -Frank > > Gr{oetje,eeting}s, > > Geert >
next prev parent reply other threads:[~2018-11-09 16:56 UTC|newest] Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-11-06 18:36 [PATCH 0/4] Simplify mediatek clk driver probes Stephen Boyd 2018-11-06 18:36 ` Stephen Boyd 2018-11-06 18:36 ` [PATCH 1/4] of/device: Add a way to probe drivers by match data Stephen Boyd 2018-11-06 18:36 ` Stephen Boyd 2018-11-06 20:44 ` Rob Herring 2018-11-06 20:44 ` Rob Herring 2018-11-06 20:44 ` Rob Herring 2018-11-07 18:37 ` Stephen Boyd 2018-11-07 18:37 ` Stephen Boyd 2018-11-07 18:37 ` Stephen Boyd 2018-11-09 9:56 ` Geert Uytterhoeven 2018-11-09 9:56 ` Geert Uytterhoeven 2018-11-09 9:56 ` Geert Uytterhoeven 2018-11-09 16:59 ` Stephen Boyd 2018-11-09 16:59 ` Stephen Boyd 2018-11-09 16:59 ` Stephen Boyd 2018-11-09 19:18 ` Geert Uytterhoeven 2018-11-09 19:18 ` Geert Uytterhoeven 2018-11-09 19:18 ` Geert Uytterhoeven 2018-11-30 0:28 ` Stephen Boyd 2018-11-30 0:28 ` Stephen Boyd 2018-11-30 0:28 ` Stephen Boyd 2018-11-30 1:01 ` Rob Herring 2018-11-30 1:01 ` Rob Herring 2018-11-30 1:01 ` Rob Herring 2018-11-30 7:03 ` Stephen Boyd 2018-11-30 7:03 ` Stephen Boyd 2018-11-30 7:03 ` Stephen Boyd 2018-11-08 8:29 ` Matthias Brugger 2018-11-08 8:29 ` Matthias Brugger 2018-11-08 17:58 ` Stephen Boyd 2018-11-08 17:58 ` Stephen Boyd 2018-11-09 10:29 ` Matthias Brugger 2018-11-09 10:29 ` Matthias Brugger 2018-11-09 10:36 ` Geert Uytterhoeven 2018-11-09 10:36 ` Geert Uytterhoeven 2018-11-09 10:36 ` Geert Uytterhoeven 2018-11-09 16:30 ` Rob Herring 2018-11-09 16:30 ` Rob Herring 2018-11-09 16:30 ` Rob Herring 2018-11-09 16:56 ` Frank Rowand [this message] 2018-11-09 16:56 ` Frank Rowand 2018-11-09 16:56 ` Frank Rowand 2018-11-06 18:36 ` [PATCH 2/4] clk: mediatek: Convert to platform_driver_probe_by_match_data() Stephen Boyd 2018-11-06 18:36 ` Stephen Boyd 2018-11-06 18:36 ` [PATCH 3/4] clk: mediatek: Drop THIS_MODULE from platform_driver Stephen Boyd 2018-11-06 18:36 ` Stephen Boyd 2018-11-06 18:36 ` [PATCH 4/4] clk: mediatek: Simplify single driver probes Stephen Boyd 2018-11-06 18:36 ` Stephen Boyd
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=07ec49cb-e5b7-72bc-f563-2d2d7c2e2c0d@gmail.com \ --to=frowand.list@gmail.com \ --cc=devicetree@vger.kernel.org \ --cc=geert@linux-m68k.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-clk@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mediatek@lists.infradead.org \ --cc=matthias.bgg@gmail.com \ --cc=mturquette@baylibre.com \ --cc=robh+dt@kernel.org \ --cc=ryder.lee@mediatek.com \ --cc=sboyd@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.