lkml.org 
[lkml]   [2017]   [Jul]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.9 109/125] staging: sm750fb: avoid conflicting vesafb
    Date
    4.9-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Teddy Wang <teddy.wang@siliconmotion.com>

    commit 740c433ec35187b45abe08bb6c45a321a791be8e upstream.

    If vesafb is enabled in the config then /dev/fb0 is created by vesa
    and this sm750 driver gets fb1, fb2. But we need to be fb0 and fb1 to
    effectively work with xorg.
    So if it has been alloted fb1, then try to remove the other fb0.

    In the previous send, why #ifdef is used was asked.
    https://lkml.org/lkml/2017/6/25/57

    Answered at: https://lkml.org/lkml/2017/6/25/69
    Also pasting here for reference.

    'Did a quick research into "why".
    The patch d8801e4df91e ("x86/PCI: Set IORESOURCE_ROM_SHADOW only for the
    default VGA device") has started setting IORESOURCE_ROM_SHADOW in flags
    for a default VGA device and that is being done only for x86.
    And so, we will need that #ifdef to check IORESOURCE_ROM_SHADOW as that
    needs to be checked only for a x86 and not for other arch.'

    Signed-off-by: Teddy Wang <teddy.wang@siliconmotion.com>
    Signed-off-by: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    drivers/staging/sm750fb/sm750.c | 24 ++++++++++++++++++++++++
    1 file changed, 24 insertions(+)

    --- a/drivers/staging/sm750fb/sm750.c
    +++ b/drivers/staging/sm750fb/sm750.c
    @@ -1049,6 +1049,26 @@ release_fb:
    return err;
    }

    +static int lynxfb_kick_out_firmware_fb(struct pci_dev *pdev)
    +{
    + struct apertures_struct *ap;
    + bool primary = false;
    +
    + ap = alloc_apertures(1);
    + if (!ap)
    + return -ENOMEM;
    +
    + ap->ranges[0].base = pci_resource_start(pdev, 0);
    + ap->ranges[0].size = pci_resource_len(pdev, 0);
    +#ifdef CONFIG_X86
    + primary = pdev->resource[PCI_ROM_RESOURCE].flags &
    + IORESOURCE_ROM_SHADOW;
    +#endif
    + remove_conflicting_framebuffers(ap, "sm750_fb1", primary);
    + kfree(ap);
    + return 0;
    +}
    +
    static int lynxfb_pci_probe(struct pci_dev *pdev,
    const struct pci_device_id *ent)
    {
    @@ -1057,6 +1077,10 @@ static int lynxfb_pci_probe(struct pci_d
    int fbidx;
    int err;

    + err = lynxfb_kick_out_firmware_fb(pdev);
    + if (err)
    + return err;
    +
    /* enable device */
    err = pcim_enable_device(pdev);
    if (err)

    \
     
     \ /
      Last update: 2017-07-25 22:19    [W:4.402 / U:0.064 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site