Skip to content

Commit

Permalink
ata: pata_of_platform: using of_property_read_u32() helper
Browse files Browse the repository at this point in the history
Using better of_property_read_u32() than generic of_get_property().

Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Cc: Hans de Goede <hdegoede@redhat.com>
Cc: Tejun Heo <tj@kernel.org>
Signed-off-by: Kefeng Wang <wangkefeng.wang@huawei.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
  • Loading branch information
Kefeng Wang authored and Tejun Heo committed Jan 24, 2017
1 parent 5946fda commit 73b2951
Showing 1 changed file with 2 additions and 7 deletions.
9 changes: 2 additions & 7 deletions drivers/ata/pata_of_platform.c
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@ static int pata_of_platform_probe(struct platform_device *ofdev)
unsigned int reg_shift = 0;
int pio_mode = 0;
int pio_mask;
const u32 *prop;

ret = of_address_to_resource(dn, 0, &io_res);
if (ret) {
Expand All @@ -50,13 +49,9 @@ static int pata_of_platform_probe(struct platform_device *ofdev)

irq_res = platform_get_resource(ofdev, IORESOURCE_IRQ, 0);

prop = of_get_property(dn, "reg-shift", NULL);
if (prop)
reg_shift = be32_to_cpup(prop);
of_property_read_u32(dn, "reg-shift", &reg_shift);

prop = of_get_property(dn, "pio-mode", NULL);
if (prop) {
pio_mode = be32_to_cpup(prop);
if (!of_property_read_u32(dn, "pio-mode", &pio_mode)) {
if (pio_mode > 6) {
dev_err(&ofdev->dev, "invalid pio-mode\n");
return -EINVAL;
Expand Down

0 comments on commit 73b2951

Please sign in to comment.