Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1449416
| From | Jonathan Cameron <jic23@jic23.retrosnub.co.uk> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH 5/5] mfd: sunxi-gpadc-mfd: probe sunxi-gpadc-ts driver |
| Date | 2016-07-25 12:10 +0200 |
| Message-ID | <rYKZH-3i7-7@gated-at.bofh.it> (permalink) |
| References | <rWVcR-7sY-3@gated-at.bofh.it> <rWVcS-7sY-37@gated-at.bofh.it> <rXflf-3Ad-3@gated-at.bofh.it> <rYpLz-7hd-7@gated-at.bofh.it> <rYKQ2-2ZK-9@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On 25 July 2016 10:51:13 BST, Maxime Ripard <maxime.ripard@free-electrons.com> wrote:
>Hi Jonathan,
>
>On Sun, Jul 24, 2016 at 12:26:56PM +0100, Jonathan Cameron wrote:
>> >> @@ -142,23 +159,45 @@ static int sunxi_gpadc_mfd_probe(struct
>platform_device *pdev)
>> >> }
>> >>
>> >> if (of_device_is_compatible(pdev->dev.of_node,
>> >> - "allwinner,sun4i-a10-ts"))
>> >> - ret = mfd_add_devices(sunxi_gpadc_mfd_dev->dev, 0,
>> >> - sun4i_gpadc_mfd_cells,
>> >> - ARRAY_SIZE(sun4i_gpadc_mfd_cells), NULL,
>> >> - 0, NULL);
>> >> - else if (of_device_is_compatible(pdev->dev.of_node,
>> >> - "allwinner,sun5i-a13-ts"))
>> >> - ret = mfd_add_devices(sunxi_gpadc_mfd_dev->dev, 0,
>> >> - sun5i_gpadc_mfd_cells,
>> >> - ARRAY_SIZE(sun5i_gpadc_mfd_cells), NULL,
>> >> - 0, NULL);
>> >> - else if (of_device_is_compatible(pdev->dev.of_node,
>> >> - "allwinner,sun6i-a31-ts"))
>> >> - ret = mfd_add_devices(sunxi_gpadc_mfd_dev->dev, 0,
>> >> - sun6i_gpadc_mfd_cells,
>> >> - ARRAY_SIZE(sun6i_gpadc_mfd_cells), NULL,
>> >> - 0, NULL);
>> >> + "allwinner,sun4i-a10-ts")) {
>> >> + if (of_property_read_bool(pdev->dev.of_node,
>> >> + "allwinner,ts-attached"))
>> >> + ret = mfd_add_devices(sunxi_gpadc_mfd_dev->dev, 0,
>> >> + sun4i_gpadc_mfd_cells_ts,
>> >> + ARRAY_SIZE(sun4i_gpadc_mfd_cells_ts),
>> >> + NULL, 0, NULL);
>> >> + else
>> >> + ret = mfd_add_devices(sunxi_gpadc_mfd_dev->dev, 0,
>> >> + sun4i_gpadc_mfd_cells,
>> >> + ARRAY_SIZE(sun4i_gpadc_mfd_cells),
>> >> + NULL, 0, NULL);
>> >> + } else if (of_device_is_compatible(pdev->dev.of_node,
>> >> + "allwinner,sun5i-a13-ts")) {
>> >> + if (of_property_read_bool(pdev->dev.of_node,
>> >> + "allwinner,ts-attached"))
>> >> + ret = mfd_add_devices(sunxi_gpadc_mfd_dev->dev, 0,
>> >> + sun5i_gpadc_mfd_cells_ts,
>> >> + ARRAY_SIZE(sun5i_gpadc_mfd_cells_ts),
>> >> + NULL, 0, NULL);
>> >> + else
>> >> + ret = mfd_add_devices(sunxi_gpadc_mfd_dev->dev, 0,
>> >> + sun5i_gpadc_mfd_cells,
>> >> + ARRAY_SIZE(sun5i_gpadc_mfd_cells),
>> >> + NULL, 0, NULL);
>> >> + } else if (of_device_is_compatible(pdev->dev.of_node,
>> >> + "allwinner,sun6i-a31-ts")) {
>> >> + if (of_property_read_bool(pdev->dev.of_node,
>> >> + "allwinner,ts-attached"))
>> >> + ret = mfd_add_devices(sunxi_gpadc_mfd_dev->dev, 0,
>> >> + sun6i_gpadc_mfd_cells_ts,
>> >> + ARRAY_SIZE(sun6i_gpadc_mfd_cells_ts),
>> >> + NULL, 0, NULL);
>> >> + else
>> >> + ret = mfd_add_devices(sunxi_gpadc_mfd_dev->dev, 0,
>> >> + sun6i_gpadc_mfd_cells,
>> >> + ARRAY_SIZE(sun6i_gpadc_mfd_cells),
>> >> + NULL, 0, NULL);
>> >> + }
>> >
>> > Please don't use any of_device_is_compatible.
>> Hi Maxime,
>>
>> Why? (Just curious...)
>
>This is completely redundant. The compatible has already been looked
>up once to match the driver, and you can associate a void * pointer to
>any compatible you register in the of_device_id array.
>
>So you can just retrieve the compatible that probed you in the first
>place, and use it's private data pointer to store whatever you want,
>without the numerous (and expensive) calls to of_device_is_compatible.
>
>It's also easier to maintain in the long term, since you can simply
>add a new field to the structure you would register there, instead of
>keeping adding more conditions.
Fair enough, now I get what you mean.
Thanks,
Jonathan
>
>Maxime
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH 0/5] add resistive touchscreen support for new Allwinner SoCs' GPADC's driver Quentin Schulz <quentin.schulz@free-electrons.com> - 2016-07-20 10:40 +0200
[PATCH 5/5] mfd: sunxi-gpadc-mfd: probe sunxi-gpadc-ts driver Quentin Schulz <quentin.schulz@free-electrons.com> - 2016-07-20 10:40 +0200
Re: [PATCH 5/5] mfd: sunxi-gpadc-mfd: probe sunxi-gpadc-ts driver Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-07-21 08:10 +0200
Re: [PATCH 5/5] mfd: sunxi-gpadc-mfd: probe sunxi-gpadc-ts driver Jonathan Cameron <jic23@kernel.org> - 2016-07-24 13:30 +0200
Re: [PATCH 5/5] mfd: sunxi-gpadc-mfd: probe sunxi-gpadc-ts driver Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-07-25 12:00 +0200
Re: [PATCH 5/5] mfd: sunxi-gpadc-mfd: probe sunxi-gpadc-ts driver Jonathan Cameron <jic23@jic23.retrosnub.co.uk> - 2016-07-25 12:10 +0200
Re: [PATCH 5/5] mfd: sunxi-gpadc-mfd: probe sunxi-gpadc-ts driver Lee Jones <lee.jones@linaro.org> - 2016-07-25 12:30 +0200
[PATCH 3/5] iio: adc: sunxi-gpadc-iio: enable iio_buffers Quentin Schulz <quentin.schulz@free-electrons.com> - 2016-07-20 10:40 +0200
Re: [PATCH 3/5] iio: adc: sunxi-gpadc-iio: enable iio_buffers Jonathan Cameron <jic23@kernel.org> - 2016-07-24 13:10 +0200
csiph-web